On Fri, Nov 12, 2021 at 5:40 AM Tim Harvey <tharvey@xxxxxxxxxxxxx> wrote: > > On Thu, Nov 11, 2021 at 2:15 AM Jagan Teki <jagan@xxxxxxxxxxxxxxxxxxxx> 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@xxxxxxxxxxxxxxxxxxxx/ > > 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@xxxxxxxxxxxxxxxxxxxx/ [2] https://patchwork.kernel.org/project/linux-arm-kernel/cover/20210621072424.111733-1-jagan@xxxxxxxxxxxxxxxxxxxx/ [3] https://patchwork.kernel.org/project/linux-arm-kernel/patch/20211106155427.753197-1-aford173@xxxxxxxxx/ Jagan.
