| Message ID | 20211111101456.584061-1-jagan@amarulasolutions.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | arm64: imx8mm: Add MIPI DSI support | expand |
On Thu, Nov 11, 2021 at 2:15 AM Jagan Teki <jagan@amarulasolutions.com> wrote: > > This series support MIPI DSI on i.MX8MM. > > The DSIM bridge still need to work to make it compatible for > exynos drm dsi hardware block. > > This series work directly on to of linux-next with recent > dispmix-blk-ctrl changes. > Jagan, Thanks - I was able to get this series working using the set of exynos/drm patches from Michael submitted back in 2020-09-11: https://patchwork.kernel.org/project/dri-devel/list/?series=347439&archive=both&state=* > Tested on i.Core MX8M Mini SoM with EDIMM2.2 and CTOUCH2 > Carrier boards. > > Required changes: > 1. DSIM driver > https://patchwork.kernel.org/project/linux-arm-kernel/cover/20210704090230.26489-1-jagan@amarulasolutions.com/ This exynos/drm RFC series you posted back in July was where I recalled the discussion about if the exynos driver could be split up vs duplicating parts of it in a separate driver. There were also some comments about this series. Can you address those comments, rebase and resend? I have not been able to get my hardware to work with this series yet and am still debugging that (currently crashing in samsung_dsim_host_attach) > 2. DPHY change > https://www.spinics.net/lists/devicetree/msg381691.html This was originally from Marek submitted on Oct 3 2020: [PATCH] phy: exynos-mipi-video: Add support for NXP i.MX8MM This one seems to have been acked but never got picked up for some reason. Marek, can you add the tags and re-submit? > 3. Bus format fix > https://github.com/openedev/linux/commit/6ca9781ed53ea75e26341dd57250e63794638b20 > Jagan, can you submit this? Best regards, Tim > Complete repo: > https://github.com/openedev/linux/commits/111021-imx8mm-dsim > > Any inputs? > Jagan. > > Jagan Teki (3): > arm64: dts: imx8mm: Add eLCDIF node support > arm64: dts: imx8mm: Add MIPI DSI pipeline > arm64: dts: imx8mm-icore: Enable LVDS panel for EDIMM2.2 > > .../freescale/imx8mm-icore-mx8mm-edimm2.2.dts | 85 +++++++++++++++++++ > arch/arm64/boot/dts/freescale/imx8mm.dtsi | 74 ++++++++++++++++ > 2 files changed, 159 insertions(+) > > -- > 2.25.1 >
On Fri, Nov 12, 2021 at 5:40 AM Tim Harvey <tharvey@gateworks.com> wrote: > > On Thu, Nov 11, 2021 at 2:15 AM Jagan Teki <jagan@amarulasolutions.com> wrote: > > > > This series support MIPI DSI on i.MX8MM. > > > > The DSIM bridge still need to work to make it compatible for > > exynos drm dsi hardware block. > > > > This series work directly on to of linux-next with recent > > dispmix-blk-ctrl changes. > > > > Jagan, > > Thanks - I was able to get this series working using the set of > exynos/drm patches from Michael submitted back in 2020-09-11: > https://patchwork.kernel.org/project/dri-devel/list/?series=347439&archive=both&state=* > > > Tested on i.Core MX8M Mini SoM with EDIMM2.2 and CTOUCH2 > > Carrier boards. > > > > Required changes: > > 1. DSIM driver > > https://patchwork.kernel.org/project/linux-arm-kernel/cover/20210704090230.26489-1-jagan@amarulasolutions.com/ > > This exynos/drm RFC series you posted back in July was where I > recalled the discussion about if the exynos driver could be split up > vs duplicating parts of it in a separate driver. Not sure Laurent, Inki has some discussion about this [1], looks like they are still looking for a common driver. > > There were also some comments about this series. Can you address those > comments, rebase and resend? > > I have not been able to get my hardware to work with this series yet > and am still debugging that (currently crashing in > samsung_dsim_host_attach) I've initially tried a separate driver instead of exynos.[2] > > > 2. DPHY change > > https://www.spinics.net/lists/devicetree/msg381691.html > > This was originally from Marek submitted on Oct 3 2020: [PATCH] phy: > exynos-mipi-video: Add support for NXP i.MX8MM I'm thinking this may not be required, as dphy reset can now handle via blk-ctrl like this [3]. I have tested the reset handling via blk-ctrl and it works for me. > > This one seems to have been acked but never got picked up for some reason. > > Marek, can you add the tags and re-submit? > > > 3. Bus format fix > > https://github.com/openedev/linux/commit/6ca9781ed53ea75e26341dd57250e63794638b20 > > > > Jagan, can you submit this? This is indeed not required, drm handles the bridge state via atomic API's. I did check that as well. I will link my latest series soon. [1] https://patchwork.kernel.org/project/linux-arm-kernel/patch/20210704090230.26489-7-jagan@amarulasolutions.com/ [2] https://patchwork.kernel.org/project/linux-arm-kernel/cover/20210621072424.111733-1-jagan@amarulasolutions.com/ [3] https://patchwork.kernel.org/project/linux-arm-kernel/patch/20211106155427.753197-1-aford173@gmail.com/ Jagan.