* [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL @ 2020-10-03 22:45 Marek Vasut 2020-10-03 22:45 ` [PATCH 2/5] dt-bindings: clock: imx8mm: Add media blk_ctl clock IDs Marek Vasut ` (4 more replies) 0 siblings, 5 replies; 17+ messages in thread From: Marek Vasut @ 2020-10-03 22:45 UTC (permalink / raw) To: linux-arm-kernel Cc: Marek Vasut, Abel Vesa, Dong Aisheng, Fabio Estevam, Guido Günther, Lucas Stach, Rob Herring, Shawn Guo, NXP Linux Team, devicetree Add the i.MX8MM BLK_CTL compatible string to the list. Signed-off-by: Marek Vasut <marex@denx.de> Cc: Abel Vesa <abel.vesa@nxp.com> Cc: Dong Aisheng <aisheng.dong@nxp.com> Cc: Fabio Estevam <festevam@gmail.com> Cc: Guido Günther <agx@sigxcpu.org> Cc: Lucas Stach <l.stach@pengutronix.de> Cc: Rob Herring <robh+dt@kernel.org> Cc: Shawn Guo <shawnguo@kernel.org> Cc: NXP Linux Team <linux-imx@nxp.com> Cc: devicetree@vger.kernel.org --- Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml b/Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml index 5e9eb402b9b6..346429f49093 100644 --- a/Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml +++ b/Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml @@ -20,6 +20,7 @@ properties: compatible: items: - enum: + - fsl,imx8mm-dispmix-blk-ctl - fsl,imx8mp-audio-blk-ctl - fsl,imx8mp-hdmi-blk-ctl - fsl,imx8mp-media-blk-ctl -- 2.28.0 ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 2/5] dt-bindings: clock: imx8mm: Add media blk_ctl clock IDs 2020-10-03 22:45 [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL Marek Vasut @ 2020-10-03 22:45 ` Marek Vasut 2020-10-06 21:12 ` Rob Herring 2020-10-03 22:45 ` [PATCH 3/5] dt-bindings: reset: imx8mm: Add media blk_ctl reset IDs Marek Vasut ` (3 subsequent siblings) 4 siblings, 1 reply; 17+ messages in thread From: Marek Vasut @ 2020-10-03 22:45 UTC (permalink / raw) To: linux-arm-kernel Cc: Marek Vasut, Abel Vesa, Dong Aisheng, Fabio Estevam, Guido Günther, Lucas Stach, Rob Herring, Shawn Guo, NXP Linux Team, devicetree These will be used by the imx8mm for blk_ctl driver. Signed-off-by: Marek Vasut <marex@denx.de> Cc: Abel Vesa <abel.vesa@nxp.com> Cc: Dong Aisheng <aisheng.dong@nxp.com> Cc: Fabio Estevam <festevam@gmail.com> Cc: Guido Günther <agx@sigxcpu.org> Cc: Lucas Stach <l.stach@pengutronix.de> Cc: Rob Herring <robh+dt@kernel.org> Cc: Shawn Guo <shawnguo@kernel.org> Cc: NXP Linux Team <linux-imx@nxp.com> Cc: devicetree@vger.kernel.org --- include/dt-bindings/clock/imx8mm-clock.h | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/include/dt-bindings/clock/imx8mm-clock.h b/include/dt-bindings/clock/imx8mm-clock.h index e63a5530aed7..576b229a72f2 100644 --- a/include/dt-bindings/clock/imx8mm-clock.h +++ b/include/dt-bindings/clock/imx8mm-clock.h @@ -276,4 +276,11 @@ #define IMX8MM_CLK_END 252 +#define IMX8MM_CLK_MEDIA_BLK_CTL_LCDIF_APB 0 +#define IMX8MM_CLK_MEDIA_BLK_CTL_LCDIF_PIXEL 1 +#define IMX8MM_CLK_MEDIA_BLK_CTL_MIPI_DSI_PCLK 2 +#define IMX8MM_CLK_MEDIA_BLK_CTL_MIPI_DSI_CLKREF 3 + +#define IMX8MM_CLK_MEDIA_BLK_CTL_END 4 + #endif -- 2.28.0 ^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [PATCH 2/5] dt-bindings: clock: imx8mm: Add media blk_ctl clock IDs 2020-10-03 22:45 ` [PATCH 2/5] dt-bindings: clock: imx8mm: Add media blk_ctl clock IDs Marek Vasut @ 2020-10-06 21:12 ` Rob Herring 0 siblings, 0 replies; 17+ messages in thread From: Rob Herring @ 2020-10-06 21:12 UTC (permalink / raw) To: Marek Vasut Cc: NXP Linux Team, Dong Aisheng, Fabio Estevam, Lucas Stach, Guido Günther, linux-arm-kernel, Rob Herring, devicetree, Shawn Guo, Abel Vesa On Sun, 04 Oct 2020 00:45:52 +0200, Marek Vasut wrote: > These will be used by the imx8mm for blk_ctl driver. > > Signed-off-by: Marek Vasut <marex@denx.de> > Cc: Abel Vesa <abel.vesa@nxp.com> > Cc: Dong Aisheng <aisheng.dong@nxp.com> > Cc: Fabio Estevam <festevam@gmail.com> > Cc: Guido Günther <agx@sigxcpu.org> > Cc: Lucas Stach <l.stach@pengutronix.de> > Cc: Rob Herring <robh+dt@kernel.org> > Cc: Shawn Guo <shawnguo@kernel.org> > Cc: NXP Linux Team <linux-imx@nxp.com> > Cc: devicetree@vger.kernel.org > --- > include/dt-bindings/clock/imx8mm-clock.h | 7 +++++++ > 1 file changed, 7 insertions(+) > Acked-by: Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 3/5] dt-bindings: reset: imx8mm: Add media blk_ctl reset IDs 2020-10-03 22:45 [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL Marek Vasut 2020-10-03 22:45 ` [PATCH 2/5] dt-bindings: clock: imx8mm: Add media blk_ctl clock IDs Marek Vasut @ 2020-10-03 22:45 ` Marek Vasut 2020-10-06 21:12 ` Rob Herring 2020-10-06 21:12 ` [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL Rob Herring ` (2 subsequent siblings) 4 siblings, 1 reply; 17+ messages in thread From: Marek Vasut @ 2020-10-03 22:45 UTC (permalink / raw) To: linux-arm-kernel Cc: Marek Vasut, Abel Vesa, Dong Aisheng, Fabio Estevam, Guido Günther, Lucas Stach, Rob Herring, Shawn Guo, NXP Linux Team, devicetree These will be used by the imx8mm for blk_ctl driver. Signed-off-by: Marek Vasut <marex@denx.de> Cc: Abel Vesa <abel.vesa@nxp.com> Cc: Dong Aisheng <aisheng.dong@nxp.com> Cc: Fabio Estevam <festevam@gmail.com> Cc: Guido Günther <agx@sigxcpu.org> Cc: Lucas Stach <l.stach@pengutronix.de> Cc: Rob Herring <robh+dt@kernel.org> Cc: Shawn Guo <shawnguo@kernel.org> Cc: NXP Linux Team <linux-imx@nxp.com> Cc: devicetree@vger.kernel.org --- include/dt-bindings/reset/imx8mm-reset.h | 14 ++++++++++++++ 1 file changed, 14 insertions(+) create mode 100644 include/dt-bindings/reset/imx8mm-reset.h diff --git a/include/dt-bindings/reset/imx8mm-reset.h b/include/dt-bindings/reset/imx8mm-reset.h new file mode 100644 index 000000000000..1dda59d30c06 --- /dev/null +++ b/include/dt-bindings/reset/imx8mm-reset.h @@ -0,0 +1,14 @@ +/* SPDX-License-Identifier: GPL-2.0-only */ +/* + * Copyright (C) 2020 Marek Vasut <marex@denx.de> + */ + +#ifndef DT_BINDING_RESET_IMX8MM_H +#define DT_BINDING_RESET_IMX8MM_H + +#define IMX8MM_MEDIA_BLK_CTL_RESET_MIPI_DSI_I_PRESET 0 +#define IMX8MM_MEDIA_BLK_CTL_RESET_MIPI_M_RESET 1 + +#define IMX8MM_MEDIA_BLK_CTL_RESET_NUM 2 + +#endif -- 2.28.0 ^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [PATCH 3/5] dt-bindings: reset: imx8mm: Add media blk_ctl reset IDs 2020-10-03 22:45 ` [PATCH 3/5] dt-bindings: reset: imx8mm: Add media blk_ctl reset IDs Marek Vasut @ 2020-10-06 21:12 ` Rob Herring 0 siblings, 0 replies; 17+ messages in thread From: Rob Herring @ 2020-10-06 21:12 UTC (permalink / raw) To: Marek Vasut Cc: Dong Aisheng, Rob Herring, Guido Günther, Lucas Stach, Fabio Estevam, devicetree, Abel Vesa, Shawn Guo, NXP Linux Team, linux-arm-kernel On Sun, 04 Oct 2020 00:45:53 +0200, Marek Vasut wrote: > These will be used by the imx8mm for blk_ctl driver. > > Signed-off-by: Marek Vasut <marex@denx.de> > Cc: Abel Vesa <abel.vesa@nxp.com> > Cc: Dong Aisheng <aisheng.dong@nxp.com> > Cc: Fabio Estevam <festevam@gmail.com> > Cc: Guido Günther <agx@sigxcpu.org> > Cc: Lucas Stach <l.stach@pengutronix.de> > Cc: Rob Herring <robh+dt@kernel.org> > Cc: Shawn Guo <shawnguo@kernel.org> > Cc: NXP Linux Team <linux-imx@nxp.com> > Cc: devicetree@vger.kernel.org > --- > include/dt-bindings/reset/imx8mm-reset.h | 14 ++++++++++++++ > 1 file changed, 14 insertions(+) > create mode 100644 include/dt-bindings/reset/imx8mm-reset.h > Acked-by: Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL 2020-10-03 22:45 [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL Marek Vasut 2020-10-03 22:45 ` [PATCH 2/5] dt-bindings: clock: imx8mm: Add media blk_ctl clock IDs Marek Vasut 2020-10-03 22:45 ` [PATCH 3/5] dt-bindings: reset: imx8mm: Add media blk_ctl reset IDs Marek Vasut @ 2020-10-06 21:12 ` Rob Herring 2020-10-07 19:52 ` Adam Ford 2021-02-04 12:46 ` Adam Ford 4 siblings, 0 replies; 17+ messages in thread From: Rob Herring @ 2020-10-06 21:12 UTC (permalink / raw) To: Marek Vasut Cc: Lucas Stach, NXP Linux Team, Guido Günther, Fabio Estevam, Abel Vesa, devicetree, Dong Aisheng, Rob Herring, linux-arm-kernel, Shawn Guo On Sun, 04 Oct 2020 00:45:51 +0200, Marek Vasut wrote: > Add the i.MX8MM BLK_CTL compatible string to the list. > > Signed-off-by: Marek Vasut <marex@denx.de> > Cc: Abel Vesa <abel.vesa@nxp.com> > Cc: Dong Aisheng <aisheng.dong@nxp.com> > Cc: Fabio Estevam <festevam@gmail.com> > Cc: Guido Günther <agx@sigxcpu.org> > Cc: Lucas Stach <l.stach@pengutronix.de> > Cc: Rob Herring <robh+dt@kernel.org> > Cc: Shawn Guo <shawnguo@kernel.org> > Cc: NXP Linux Team <linux-imx@nxp.com> > Cc: devicetree@vger.kernel.org > --- > Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml | 1 + > 1 file changed, 1 insertion(+) > Acked-by: Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL 2020-10-03 22:45 [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL Marek Vasut ` (2 preceding siblings ...) 2020-10-06 21:12 ` [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL Rob Herring @ 2020-10-07 19:52 ` Adam Ford 2020-10-07 20:01 ` Marek Vasut 2021-02-04 12:46 ` Adam Ford 4 siblings, 1 reply; 17+ messages in thread From: Adam Ford @ 2020-10-07 19:52 UTC (permalink / raw) To: Marek Vasut Cc: arm-soc, Dong Aisheng, Abel Vesa, devicetree, Shawn Guo, Guido Günther, Rob Herring, NXP Linux Team, Fabio Estevam, Lucas Stach On Sun, Oct 4, 2020 at 12:53 AM Marek Vasut <marex@denx.de> wrote: > > Add the i.MX8MM BLK_CTL compatible string to the list. > > Signed-off-by: Marek Vasut <marex@denx.de> > Cc: Abel Vesa <abel.vesa@nxp.com> > Cc: Dong Aisheng <aisheng.dong@nxp.com> > Cc: Fabio Estevam <festevam@gmail.com> > Cc: Guido Günther <agx@sigxcpu.org> > Cc: Lucas Stach <l.stach@pengutronix.de> > Cc: Rob Herring <robh+dt@kernel.org> > Cc: Shawn Guo <shawnguo@kernel.org> > Cc: NXP Linux Team <linux-imx@nxp.com> > Cc: devicetree@vger.kernel.org > --- > Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml | 1 + > 1 file changed, 1 insertion(+) > Is there a DTSI change part of this patch? I was going to try to test it, but I am not seeing a change to the imx8mm.dtsi, and I am not sure where to put the node. adam > diff --git a/Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml b/Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml > index 5e9eb402b9b6..346429f49093 100644 > --- a/Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml > +++ b/Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml > @@ -20,6 +20,7 @@ properties: > compatible: > items: > - enum: > + - fsl,imx8mm-dispmix-blk-ctl > - fsl,imx8mp-audio-blk-ctl > - fsl,imx8mp-hdmi-blk-ctl > - fsl,imx8mp-media-blk-ctl > -- > 2.28.0 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL 2020-10-07 19:52 ` Adam Ford @ 2020-10-07 20:01 ` Marek Vasut 2020-10-07 20:08 ` Adam Ford 0 siblings, 1 reply; 17+ messages in thread From: Marek Vasut @ 2020-10-07 20:01 UTC (permalink / raw) To: Adam Ford Cc: arm-soc, Dong Aisheng, Abel Vesa, devicetree, Shawn Guo, Guido Günther, Rob Herring, NXP Linux Team, Fabio Estevam, Lucas Stach On 10/7/20 9:52 PM, Adam Ford wrote: > On Sun, Oct 4, 2020 at 12:53 AM Marek Vasut <marex@denx.de> wrote: >> >> Add the i.MX8MM BLK_CTL compatible string to the list. [...] >> --- >> Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml | 1 + >> 1 file changed, 1 insertion(+) >> > > Is there a DTSI change part of this patch? I was going to try to test > it, but I am not seeing a change to the imx8mm.dtsi, and I am not > sure where to put the node. There are in fact quite a few other pieces you need to have in place, this patchset in itself is not particularly useful, it is just infra for the LCDIF and MIPI DSIM block control. You might want to wait until they all land in next and test that result. ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL 2020-10-07 20:01 ` Marek Vasut @ 2020-10-07 20:08 ` Adam Ford 2020-10-07 20:17 ` Adam Ford 0 siblings, 1 reply; 17+ messages in thread From: Adam Ford @ 2020-10-07 20:08 UTC (permalink / raw) To: Marek Vasut Cc: arm-soc, Dong Aisheng, Abel Vesa, devicetree, Shawn Guo, Guido Günther, Rob Herring, NXP Linux Team, Fabio Estevam, Lucas Stach On Wed, Oct 7, 2020 at 3:03 PM Marek Vasut <marex@denx.de> wrote: > > On 10/7/20 9:52 PM, Adam Ford wrote: > > On Sun, Oct 4, 2020 at 12:53 AM Marek Vasut <marex@denx.de> wrote: > >> > >> Add the i.MX8MM BLK_CTL compatible string to the list. > [...] > >> --- > >> Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml | 1 + > >> 1 file changed, 1 insertion(+) > >> > > > > Is there a DTSI change part of this patch? I was going to try to test > > it, but I am not seeing a change to the imx8mm.dtsi, and I am not > > sure where to put the node. > > There are in fact quite a few other pieces you need to have in place, > this patchset in itself is not particularly useful, it is just infra for > the LCDIF and MIPI DSIM block control. You might want to wait until they > all land in next and test that result. I have several patches in place, the GPCv2, this block driver, enabling GPU DT node, I'm also working on the DSIM patch you posted. I was hoping to test them all together and reply to the various threads with tested-by. I also want to get my device tree stuff ready on the beacon boards so when everything lands, I can post DTS updates to enable the LCDIF, DSI, and the HDMI bridge. If you have a repo somewhere that has all these combined, I can just work on the final layer to enable the device tree plumbing on my board. I just need the imx8mm.dtsi changes for this, DSIM, and the LCDIF so I can finish the task. adam ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL 2020-10-07 20:08 ` Adam Ford @ 2020-10-07 20:17 ` Adam Ford 2020-10-07 20:50 ` Marek Vasut 0 siblings, 1 reply; 17+ messages in thread From: Adam Ford @ 2020-10-07 20:17 UTC (permalink / raw) To: Marek Vasut Cc: arm-soc, Dong Aisheng, Abel Vesa, devicetree, Shawn Guo, Guido Günther, Rob Herring, NXP Linux Team, Fabio Estevam, Lucas Stach On Wed, Oct 7, 2020 at 3:08 PM Adam Ford <aford173@gmail.com> wrote: > > On Wed, Oct 7, 2020 at 3:03 PM Marek Vasut <marex@denx.de> wrote: > > > > On 10/7/20 9:52 PM, Adam Ford wrote: > > > On Sun, Oct 4, 2020 at 12:53 AM Marek Vasut <marex@denx.de> wrote: > > >> > > >> Add the i.MX8MM BLK_CTL compatible string to the list. > > [...] > > >> --- > > >> Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml | 1 + > > >> 1 file changed, 1 insertion(+) > > >> > > > > > > Is there a DTSI change part of this patch? I was going to try to test > > > it, but I am not seeing a change to the imx8mm.dtsi, and I am not > > > sure where to put the node. > > > > There are in fact quite a few other pieces you need to have in place, > > this patchset in itself is not particularly useful, it is just infra for > > the LCDIF and MIPI DSIM block control. You might want to wait until they > > all land in next and test that result. > > I have several patches in place, the GPCv2, this block driver, > enabling GPU DT node, I'm also working on the DSIM patch you posted. > I was hoping to test them all together and reply to the various > threads with tested-by. I also want to get my device tree stuff ready > on the beacon boards so when everything lands, I can post DTS updates > to enable the LCDIF, DSI, and the HDMI bridge. > > If you have a repo somewhere that has all these combined, I can just > work on the final layer to enable the device tree plumbing on my > board. I just need the imx8mm.dtsi changes for this, DSIM, and the > LCDIF so I can finish the task. On that note, I also have a i.MX8M Nano board which is similar to my 8MM. If I understood the 8MM clock block driver better, I hope to adapt your changes for the Nano too. Once the GPCv2 driver is accepted, I was also going to look at updating it to support the Nano as well which also has the same DSIM and LCDIF as the 8MM as well and a better GPU than the Mini but lacking the VPU. adam > > adam ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL 2020-10-07 20:17 ` Adam Ford @ 2020-10-07 20:50 ` Marek Vasut 2020-11-30 11:47 ` Frieder Schrempf 0 siblings, 1 reply; 17+ messages in thread From: Marek Vasut @ 2020-10-07 20:50 UTC (permalink / raw) To: Adam Ford Cc: arm-soc, Dong Aisheng, Abel Vesa, devicetree, Shawn Guo, Guido Günther, Rob Herring, NXP Linux Team, Fabio Estevam, Lucas Stach On 10/7/20 10:17 PM, Adam Ford wrote: > On Wed, Oct 7, 2020 at 3:08 PM Adam Ford <aford173@gmail.com> wrote: >> >> On Wed, Oct 7, 2020 at 3:03 PM Marek Vasut <marex@denx.de> wrote: >>> >>> On 10/7/20 9:52 PM, Adam Ford wrote: >>>> On Sun, Oct 4, 2020 at 12:53 AM Marek Vasut <marex@denx.de> wrote: >>>>> >>>>> Add the i.MX8MM BLK_CTL compatible string to the list. >>> [...] >>>>> --- >>>>> Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml | 1 + >>>>> 1 file changed, 1 insertion(+) >>>>> >>>> >>>> Is there a DTSI change part of this patch? I was going to try to test >>>> it, but I am not seeing a change to the imx8mm.dtsi, and I am not >>>> sure where to put the node. >>> >>> There are in fact quite a few other pieces you need to have in place, >>> this patchset in itself is not particularly useful, it is just infra for >>> the LCDIF and MIPI DSIM block control. You might want to wait until they >>> all land in next and test that result. >> >> I have several patches in place, the GPCv2, this block driver, >> enabling GPU DT node, I'm also working on the DSIM patch you posted. >> I was hoping to test them all together and reply to the various >> threads with tested-by. I also want to get my device tree stuff ready >> on the beacon boards so when everything lands, I can post DTS updates >> to enable the LCDIF, DSI, and the HDMI bridge. >> >> If you have a repo somewhere that has all these combined, I can just >> work on the final layer to enable the device tree plumbing on my >> board. I just need the imx8mm.dtsi changes for this, DSIM, and the >> LCDIF so I can finish the task. > > On that note, I also have a i.MX8M Nano board which is similar to my > 8MM. If I understood the 8MM clock block driver better, I hope to > adapt your changes for the Nano too. Once the GPCv2 driver is > accepted, I was also going to look at updating it to support the Nano > as well which also has the same DSIM and LCDIF as the 8MM as well and > a better GPU than the Mini but lacking the VPU. I don't have a branch, but I sent you the collected patches off-list. ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL 2020-10-07 20:50 ` Marek Vasut @ 2020-11-30 11:47 ` Frieder Schrempf 2020-11-30 15:43 ` Adam Ford 0 siblings, 1 reply; 17+ messages in thread From: Frieder Schrempf @ 2020-11-30 11:47 UTC (permalink / raw) To: Marek Vasut, Adam Ford Cc: Dong Aisheng, devicetree, Abel Vesa, Fabio Estevam, Guido Günther, Rob Herring, NXP Linux Team, Shawn Guo, arm-soc, Lucas Stach Hi, On 07.10.20 22:50, Marek Vasut wrote: > On 10/7/20 10:17 PM, Adam Ford wrote: >> On Wed, Oct 7, 2020 at 3:08 PM Adam Ford <aford173@gmail.com> wrote: >>> >>> On Wed, Oct 7, 2020 at 3:03 PM Marek Vasut <marex@denx.de> wrote: >>>> >>>> On 10/7/20 9:52 PM, Adam Ford wrote: >>>>> On Sun, Oct 4, 2020 at 12:53 AM Marek Vasut <marex@denx.de> wrote: >>>>>> >>>>>> Add the i.MX8MM BLK_CTL compatible string to the list. >>>> [...] >>>>>> --- >>>>>> Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml | 1 + >>>>>> 1 file changed, 1 insertion(+) >>>>>> >>>>> >>>>> Is there a DTSI change part of this patch? I was going to try to test >>>>> it, but I am not seeing a change to the imx8mm.dtsi, and I am not >>>>> sure where to put the node. >>>> >>>> There are in fact quite a few other pieces you need to have in place, >>>> this patchset in itself is not particularly useful, it is just infra for >>>> the LCDIF and MIPI DSIM block control. You might want to wait until they >>>> all land in next and test that result. >>> >>> I have several patches in place, the GPCv2, this block driver, >>> enabling GPU DT node, I'm also working on the DSIM patch you posted. >>> I was hoping to test them all together and reply to the various >>> threads with tested-by. I also want to get my device tree stuff ready >>> on the beacon boards so when everything lands, I can post DTS updates >>> to enable the LCDIF, DSI, and the HDMI bridge. >>> >>> If you have a repo somewhere that has all these combined, I can just >>> work on the final layer to enable the device tree plumbing on my >>> board. I just need the imx8mm.dtsi changes for this, DSIM, and the >>> LCDIF so I can finish the task. >> >> On that note, I also have a i.MX8M Nano board which is similar to my >> 8MM. If I understood the 8MM clock block driver better, I hope to >> adapt your changes for the Nano too. Once the GPCv2 driver is >> accepted, I was also going to look at updating it to support the Nano >> as well which also has the same DSIM and LCDIF as the 8MM as well and >> a better GPU than the Mini but lacking the VPU. > > I don't have a branch, but I sent you the collected patches off-list. > I would also be interested in the patch collection for BLK_CTL, DSIM, etc. Marek, would you mind sending me those, too? Adam, did you already set up a branch and do some tests with the full stack? Thanks, Frieder ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL 2020-11-30 11:47 ` Frieder Schrempf @ 2020-11-30 15:43 ` Adam Ford 2020-12-10 15:14 ` Frieder Schrempf 0 siblings, 1 reply; 17+ messages in thread From: Adam Ford @ 2020-11-30 15:43 UTC (permalink / raw) To: Frieder Schrempf Cc: Marek Vasut, Dong Aisheng, devicetree, Abel Vesa, Fabio Estevam, Guido Günther, Rob Herring, NXP Linux Team, Shawn Guo, arm-soc, Lucas Stach On Mon, Nov 30, 2020 at 5:47 AM Frieder Schrempf <frieder.schrempf@kontron.de> wrote: > > Hi, > > On 07.10.20 22:50, Marek Vasut wrote: > > On 10/7/20 10:17 PM, Adam Ford wrote: > >> On Wed, Oct 7, 2020 at 3:08 PM Adam Ford <aford173@gmail.com> wrote: > >>> > >>> On Wed, Oct 7, 2020 at 3:03 PM Marek Vasut <marex@denx.de> wrote: > >>>> > >>>> On 10/7/20 9:52 PM, Adam Ford wrote: > >>>>> On Sun, Oct 4, 2020 at 12:53 AM Marek Vasut <marex@denx.de> wrote: > >>>>>> > >>>>>> Add the i.MX8MM BLK_CTL compatible string to the list. > >>>> [...] > >>>>>> --- > >>>>>> Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml | 1 + > >>>>>> 1 file changed, 1 insertion(+) > >>>>>> > >>>>> > >>>>> Is there a DTSI change part of this patch? I was going to try to test > >>>>> it, but I am not seeing a change to the imx8mm.dtsi, and I am not > >>>>> sure where to put the node. > >>>> > >>>> There are in fact quite a few other pieces you need to have in place, > >>>> this patchset in itself is not particularly useful, it is just infra for > >>>> the LCDIF and MIPI DSIM block control. You might want to wait until they > >>>> all land in next and test that result. > >>> > >>> I have several patches in place, the GPCv2, this block driver, > >>> enabling GPU DT node, I'm also working on the DSIM patch you posted. > >>> I was hoping to test them all together and reply to the various > >>> threads with tested-by. I also want to get my device tree stuff ready > >>> on the beacon boards so when everything lands, I can post DTS updates > >>> to enable the LCDIF, DSI, and the HDMI bridge. > >>> > >>> If you have a repo somewhere that has all these combined, I can just > >>> work on the final layer to enable the device tree plumbing on my > >>> board. I just need the imx8mm.dtsi changes for this, DSIM, and the > >>> LCDIF so I can finish the task. > >> > >> On that note, I also have a i.MX8M Nano board which is similar to my > >> 8MM. If I understood the 8MM clock block driver better, I hope to > >> adapt your changes for the Nano too. Once the GPCv2 driver is > >> accepted, I was also going to look at updating it to support the Nano > >> as well which also has the same DSIM and LCDIF as the 8MM as well and > >> a better GPU than the Mini but lacking the VPU. > > > > I don't have a branch, but I sent you the collected patches off-list. > > > > I would also be interested in the patch collection for BLK_CTL, DSIM, > etc. Marek, would you mind sending me those, too? > > Adam, did you already set up a branch and do some tests with the full stack? Frieder, I have been monitoring some of the activity on the BLK_CTL. It seems like there is some disagreement on how to connect the power domain controller with the BLK_CTL. Someone reported that it causes a hang on the 8MP, so until that gets resolved, I doubt we'll be able to use the display system. Some of the DSIM changes happening are being pushed back for further changes, so it seems like having the full integration might be a while. adam > > Thanks, > Frieder ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL 2020-11-30 15:43 ` Adam Ford @ 2020-12-10 15:14 ` Frieder Schrempf 2020-12-16 21:24 ` Tim Harvey 0 siblings, 1 reply; 17+ messages in thread From: Frieder Schrempf @ 2020-12-10 15:14 UTC (permalink / raw) To: Adam Ford, Marek Vasut Cc: Dong Aisheng, devicetree, Abel Vesa, Fabio Estevam, Guido Günther, Rob Herring, NXP Linux Team, Shawn Guo, arm-soc, Lucas Stach Hi, On 30.11.20 16:43, Adam Ford wrote: > On Mon, Nov 30, 2020 at 5:47 AM Frieder Schrempf > <frieder.schrempf@kontron.de> wrote: >> >> Hi, >> >> On 07.10.20 22:50, Marek Vasut wrote: >>> On 10/7/20 10:17 PM, Adam Ford wrote: >>>> On Wed, Oct 7, 2020 at 3:08 PM Adam Ford <aford173@gmail.com> wrote: >>>>> >>>>> On Wed, Oct 7, 2020 at 3:03 PM Marek Vasut <marex@denx.de> wrote: >>>>>> >>>>>> On 10/7/20 9:52 PM, Adam Ford wrote: >>>>>>> On Sun, Oct 4, 2020 at 12:53 AM Marek Vasut <marex@denx.de> wrote: >>>>>>>> >>>>>>>> Add the i.MX8MM BLK_CTL compatible string to the list. >>>>>> [...] >>>>>>>> --- >>>>>>>> Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml | 1 + >>>>>>>> 1 file changed, 1 insertion(+) >>>>>>>> >>>>>>> >>>>>>> Is there a DTSI change part of this patch? I was going to try to test >>>>>>> it, but I am not seeing a change to the imx8mm.dtsi, and I am not >>>>>>> sure where to put the node. >>>>>> >>>>>> There are in fact quite a few other pieces you need to have in place, >>>>>> this patchset in itself is not particularly useful, it is just infra for >>>>>> the LCDIF and MIPI DSIM block control. You might want to wait until they >>>>>> all land in next and test that result. >>>>> >>>>> I have several patches in place, the GPCv2, this block driver, >>>>> enabling GPU DT node, I'm also working on the DSIM patch you posted. >>>>> I was hoping to test them all together and reply to the various >>>>> threads with tested-by. I also want to get my device tree stuff ready >>>>> on the beacon boards so when everything lands, I can post DTS updates >>>>> to enable the LCDIF, DSI, and the HDMI bridge. >>>>> >>>>> If you have a repo somewhere that has all these combined, I can just >>>>> work on the final layer to enable the device tree plumbing on my >>>>> board. I just need the imx8mm.dtsi changes for this, DSIM, and the >>>>> LCDIF so I can finish the task. >>>> >>>> On that note, I also have a i.MX8M Nano board which is similar to my >>>> 8MM. If I understood the 8MM clock block driver better, I hope to >>>> adapt your changes for the Nano too. Once the GPCv2 driver is >>>> accepted, I was also going to look at updating it to support the Nano >>>> as well which also has the same DSIM and LCDIF as the 8MM as well and >>>> a better GPU than the Mini but lacking the VPU. >>> >>> I don't have a branch, but I sent you the collected patches off-list. >>> >> >> I would also be interested in the patch collection for BLK_CTL, DSIM, >> etc. Marek, would you mind sending me those, too? >> >> Adam, did you already set up a branch and do some tests with the full stack? > > Frieder, > > I have been monitoring some of the activity on the BLK_CTL. It seems > like there is some disagreement on how to connect the power domain > controller with the BLK_CTL. Someone reported that it causes a hang > on the 8MP, so until that gets resolved, I doubt we'll be able to use > the display system. Some of the DSIM changes happening are being > pushed back for further changes, so it seems like having the full > integration might be a while. I have pulled all the latest patches, including Marek's off-list patches together in one branch based on v5.10-rc7 [1] if anyone is interested. I added some fixes on top, that I needed to get my display behind another Toshiba DSI-DPI bridge working. Those are probably not upstreamable at all and need further investigation. I'm hoping to reply to the individual threads for more feedback. I see that there are some blocking issues, but we hopefully get them resolved somehow. Thanks Frieder [1] https://github.com/fschrempf/linux/commits/v5.10-mx8mm-graphics ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL 2020-12-10 15:14 ` Frieder Schrempf @ 2020-12-16 21:24 ` Tim Harvey 2020-12-22 9:07 ` Frieder Schrempf 0 siblings, 1 reply; 17+ messages in thread From: Tim Harvey @ 2020-12-16 21:24 UTC (permalink / raw) To: Frieder Schrempf Cc: Adam Ford, Marek Vasut, Dong Aisheng, devicetree, Abel Vesa, Shawn Guo, Guido Günther, Rob Herring, NXP Linux Team, Fabio Estevam, arm-soc, Lucas Stach, Michael Tretter 'On Thu, Dec 10, 2020 at 7:15 AM Frieder Schrempf <frieder.schrempf@kontron.de> wrote: > > Hi, > > On 30.11.20 16:43, Adam Ford wrote: > > On Mon, Nov 30, 2020 at 5:47 AM Frieder Schrempf > > <frieder.schrempf@kontron.de> wrote: > >> > >> Hi, > >> > >> On 07.10.20 22:50, Marek Vasut wrote: > >>> On 10/7/20 10:17 PM, Adam Ford wrote: > >>>> On Wed, Oct 7, 2020 at 3:08 PM Adam Ford <aford173@gmail.com> wrote: > >>>>> > >>>>> On Wed, Oct 7, 2020 at 3:03 PM Marek Vasut <marex@denx.de> wrote: > >>>>>> > >>>>>> On 10/7/20 9:52 PM, Adam Ford wrote: > >>>>>>> On Sun, Oct 4, 2020 at 12:53 AM Marek Vasut <marex@denx.de> wrote: > >>>>>>>> > >>>>>>>> Add the i.MX8MM BLK_CTL compatible string to the list. > >>>>>> [...] > >>>>>>>> --- > >>>>>>>> Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml | 1 + > >>>>>>>> 1 file changed, 1 insertion(+) > >>>>>>>> > >>>>>>> > >>>>>>> Is there a DTSI change part of this patch? I was going to try to test > >>>>>>> it, but I am not seeing a change to the imx8mm.dtsi, and I am not > >>>>>>> sure where to put the node. > >>>>>> > >>>>>> There are in fact quite a few other pieces you need to have in place, > >>>>>> this patchset in itself is not particularly useful, it is just infra for > >>>>>> the LCDIF and MIPI DSIM block control. You might want to wait until they > >>>>>> all land in next and test that result. > >>>>> > >>>>> I have several patches in place, the GPCv2, this block driver, > >>>>> enabling GPU DT node, I'm also working on the DSIM patch you posted. > >>>>> I was hoping to test them all together and reply to the various > >>>>> threads with tested-by. I also want to get my device tree stuff ready > >>>>> on the beacon boards so when everything lands, I can post DTS updates > >>>>> to enable the LCDIF, DSI, and the HDMI bridge. > >>>>> > >>>>> If you have a repo somewhere that has all these combined, I can just > >>>>> work on the final layer to enable the device tree plumbing on my > >>>>> board. I just need the imx8mm.dtsi changes for this, DSIM, and the > >>>>> LCDIF so I can finish the task. > >>>> > >>>> On that note, I also have a i.MX8M Nano board which is similar to my > >>>> 8MM. If I understood the 8MM clock block driver better, I hope to > >>>> adapt your changes for the Nano too. Once the GPCv2 driver is > >>>> accepted, I was also going to look at updating it to support the Nano > >>>> as well which also has the same DSIM and LCDIF as the 8MM as well and > >>>> a better GPU than the Mini but lacking the VPU. > >>> > >>> I don't have a branch, but I sent you the collected patches off-list. > >>> > >> > >> I would also be interested in the patch collection for BLK_CTL, DSIM, > >> etc. Marek, would you mind sending me those, too? > >> > >> Adam, did you already set up a branch and do some tests with the full stack? > > > > Frieder, > > > > I have been monitoring some of the activity on the BLK_CTL. It seems > > like there is some disagreement on how to connect the power domain > > controller with the BLK_CTL. Someone reported that it causes a hang > > on the 8MP, so until that gets resolved, I doubt we'll be able to use > > the display system. Some of the DSIM changes happening are being > > pushed back for further changes, so it seems like having the full > > integration might be a while. > > I have pulled all the latest patches, including Marek's off-list patches > together in one branch based on v5.10-rc7 [1] if anyone is interested. > > I added some fixes on top, that I needed to get my display behind > another Toshiba DSI-DPI bridge working. Those are probably not > upstreamable at all and need further investigation. > > I'm hoping to reply to the individual threads for more feedback. I see > that there are some blocking issues, but we hopefully get them resolved > somehow. > > Thanks > Frieder > > [1] https://github.com/fschrempf/linux/commits/v5.10-mx8mm-graphics > Frieder, Thanks for sharing your repo as it's getting hard to track these patchsets (gpc/blk-ctl/power-domain/exynos/dsim). I'm also working on display support for IMX8MM and in my case I'm trying to connect to a RaspberryPi 7in display which I see Marek has been doing some work on to split out drivers/gpu/drm/panel/panel-raspberrypi-touchscreen.c to separate bridge, regulator/backlight, and simple-panel driver. Marek, Thanks for your recent work on splitting out the rpi display driver so that it can be bound via device-tree. I have found that I need to move the tc358762_init to enable vs pre-enable when using it with the in-progress samsung-dsim driver else the driver fails writes due to not being enabled yet: diff --git a/drivers/gpu/drm/bridge/tc358762.c b/drivers/gpu/drm/bridge/tc358762.c index 1bfdfc6..0d88e61 100644 --- a/drivers/gpu/drm/bridge/tc358762.c +++ b/drivers/gpu/drm/bridge/tc358762.c @@ -153,11 +153,17 @@ static void tc358762_pre_enable(struct drm_bridge *bridge) if (ret < 0) dev_err(ctx->dev, "error enabling regulators (%d)\n", ret); + ctx->pre_enabled = true; +} + +static void tc358762_enable(struct drm_bridge *bridge) +{ + struct tc358762 *ctx = bridge_to_tc358762(bridge); + int ret; + ret = tc358762_init(ctx); if (ret < 0) dev_err(ctx->dev, "error initializing bridge (%d)\n", ret); - - ctx->pre_enabled = true; } static int tc358762_attach(struct drm_bridge *bridge, @@ -172,6 +178,7 @@ static int tc358762_attach(struct drm_bridge *bridge, static const struct drm_bridge_funcs tc358762_bridge_funcs = { .post_disable = tc358762_post_disable, .pre_enable = tc358762_pre_enable, + .enable = tc358762_enable, .attach = tc358762_attach, }; Frieder, I did find that your "drm/exynos: Fix PLL PMS offset for P value bitfield" patch breaks the samsung_dsim_host_transfer for me with the tc358762 bridge in the rpi panel. If I have that patch I get a timeout on the transfer with some added debugging: [ 4.386387] tc358762_write 0x0210=0x00000003 0 [ 4.387031] samsung_dsim_host_transfer ret: 0 [ 4.387038] tc358762_write 0x0164=0x00000005 0 [ 4.387375] samsung_dsim_host_transfer ret: 0 [ 4.387379] tc358762_write 0x0168=0x00000005 0 [ 4.387409] samsung_dsim_host_transfer ret: 0 [ 4.387413] tc358762_write 0x0144=0x00000000 0 [ 4.387741] samsung_dsim_host_transfer ret: 0 [ 4.387745] tc358762_write 0x0148=0x00000000 0 [ 4.387773] samsung_dsim_host_transfer ret: 0 [ 4.387777] tc358762_write 0x0114=0x00000003 0 [ 4.387804] samsung_dsim_host_transfer ret: 0 [ 4.387808] tc358762_write 0x0450=0x00000000 0 [ 4.387834] samsung_dsim_host_transfer ret: 0 [ 4.387838] tc358762_write 0x0420=0x00100150 0 [ 4.388168] samsung_dsim_host_transfer ret: 0 [ 4.388172] tc358762_write 0x0464=0x0000040f 0 [ 4.388200] samsung_dsim_host_transfer ret: 0 [ 4.493346] tc358762_write 0x0104=0x00000001 0 [ 5.509341] imx-dsim-dsi 32e10000.mipi_dsi: xfer timed out: 29 06 00 00 04 01 01 00 00 00 [ 5.509345] samsung_dsim_host_transfer ret: -110 [ 5.509348] tc358762_write mipi_dsi_generic_write failed err=-110 [ 5.509352] tc358762_write 0x0204=0x00000001 -110 [ 5.617336] tc358762_init failed err=-110 [ 5.617344] tc358762 32e10000.mipi_dsi.0: error initializing bridge (-110) Here is your patch which causes this issue for me: diff --git a/drivers/gpu/drm/bridge/samsung-dsim.c b/drivers/gpu/drm/bridge/samsung-dsim.c index cb1ec3c..fc7c1d0 100644 --- a/drivers/gpu/drm/bridge/samsung-dsim.c +++ b/drivers/gpu/drm/bridge/samsung-dsim.c @@ -174,7 +174,7 @@ /* DSIM_PLLCTRL */ #define DSIM_FREQ_BAND(x) ((x) << 24) #define DSIM_PLL_EN (1 << 23) -#define DSIM_PLL_P(x) ((x) << 13) +#define DSIM_PLL_P(x) ((x) << 14) #define DSIM_PLL_M(x) ((x) << 4) #define DSIM_PLL_S(x) ((x) << 1) I'm not very knowledgeable about MIPI DSI and find it strange that several writes in tc35872_init succeed until the failing writes: tc358762_write(ctx, PPI_STARTPPI, PPI_START_FUNCTION); tc358762_write(ctx, DSI_STARTDSI, DSI_RX_START); For what its worth I've backported Marek's rpi backlight/reglator and simple-pannel driver to the NXP imx_5.4.47_2.2.0 kernel and do not see any MIPI DSI write failure there, although I have the same behavior of the display not showing anything. Marek, are you using the rpi panel with IMX8MM? While I now have the drivers probing without error and have a functional backlight, regulator I see nothing on the display. here is my dt fragment for my IMX8MM: / { panel { compatible = "powertip,ph800480t013-idf02"; power-supply = <&attiny>; backlight = <&attiny>; port { panel_in: endpointpanelin { remote-endpoint = <&bridge_out>; }; }; }; }; &i2c3 { edt-ft5x06@38 { compatible = "edt,edt-ft5x06"; reg = <0x38>; pinctrl-0 = <&pinctrl_touchscreen>; interrupt-parent = <&gpio1>; interrupts = <7 IRQ_TYPE_EDGE_FALLING>; vcc-supply = <&attiny>; invert; screen-x = <800>; screen-y = <480>; }; attiny: regulator@45 { compatible = "raspberrypi,7inch-touchscreen-panel-regulator"; reg = <0x45>; }; }; &mipi_dsi { #address-cells = <1>; #size-cells = <0>; status = "okay"; bridge@0 { compatible = "toshiba,tc358762"; reg = <0>; vddc-supply = <&attiny>; #address-cells = <1>; #size-cells = <0>; status = "okay"; ports { #address-cells = <1>; #size-cells = <0>; port@0 { reg = <0>; bridge_in: endpoint { remote-endpoint = <&dsi_out>; }; }; port@1 { reg = <1>; bridge_out: endpoint { remote-endpoint = <&panel_in>; }; }; }; }; ports { port@1 { reg = <1>; dsi_out: endpoint { remote-endpoint = <&bridge_in>; }; }; }; }; And after booting I have: /sys/class/backlight/7inch-touchscreen-panel-bl/ - backlight controller /sys/class/regulator/regulator.9/ - tc358762-power /sys/class/input/input1/ /dev/input/event1 - generic ft5x06 (79) /sys/class/drm/card0 /sys/class/drm/card0-DPI-1 /sys/class/graphics/fb0 - mxsfb_drv.c I'm using 'video=DPI-1:800x480@65M' in my kernel cmdline although I don't think thats needed for fb display. # fbset -i mode "800x480" geometry 800 480 800 480 32 timings 0 0 0 0 0 0 0 accel true rgba 8/16,8/8,8/0,0/0 endmode Frame buffer device information: Name : mxsfb-drmdrmfb Address : 0 Size : 1536000 Type : PACKED PIXELS Visual : TRUECOLOR XPanStep : 1 YPanStep : 1 YWrapStep : 0 LineLength : 3200 Accelerator : No # gst-launch-1.0 videotestsrc ! fbdevsink # just shows a blank (but backlit) display Any idea what could be going wrong here? Also Marek do you know why the edt-ft5x06 driver never seems to assert it's interrupt? I haven't gotten that working even though I've wired the INT pin from the display to a DIO on the IMX8MM. Best Regards, Tim ^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL 2020-12-16 21:24 ` Tim Harvey @ 2020-12-22 9:07 ` Frieder Schrempf 0 siblings, 0 replies; 17+ messages in thread From: Frieder Schrempf @ 2020-12-22 9:07 UTC (permalink / raw) To: Tim Harvey Cc: Adam Ford, Marek Vasut, Dong Aisheng, devicetree, Abel Vesa, Shawn Guo, Guido Günther, Rob Herring, NXP Linux Team, Fabio Estevam, arm-soc, Lucas Stach, Michael Tretter Hi Tim, On 16.12.20 22:24, Tim Harvey wrote: > 'On Thu, Dec 10, 2020 at 7:15 AM Frieder Schrempf > <frieder.schrempf@kontron.de> wrote: >> >> Hi, >> >> On 30.11.20 16:43, Adam Ford wrote: >>> On Mon, Nov 30, 2020 at 5:47 AM Frieder Schrempf >>> <frieder.schrempf@kontron.de> wrote: >>>> >>>> Hi, >>>> >>>> On 07.10.20 22:50, Marek Vasut wrote: >>>>> On 10/7/20 10:17 PM, Adam Ford wrote: >>>>>> On Wed, Oct 7, 2020 at 3:08 PM Adam Ford <aford173@gmail.com> wrote: >>>>>>> >>>>>>> On Wed, Oct 7, 2020 at 3:03 PM Marek Vasut <marex@denx.de> wrote: >>>>>>>> >>>>>>>> On 10/7/20 9:52 PM, Adam Ford wrote: >>>>>>>>> On Sun, Oct 4, 2020 at 12:53 AM Marek Vasut <marex@denx.de> wrote: >>>>>>>>>> >>>>>>>>>> Add the i.MX8MM BLK_CTL compatible string to the list. >>>>>>>> [...] >>>>>>>>>> --- >>>>>>>>>> Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml | 1 + >>>>>>>>>> 1 file changed, 1 insertion(+) >>>>>>>>>> >>>>>>>>> >>>>>>>>> Is there a DTSI change part of this patch? I was going to try to test >>>>>>>>> it, but I am not seeing a change to the imx8mm.dtsi, and I am not >>>>>>>>> sure where to put the node. >>>>>>>> >>>>>>>> There are in fact quite a few other pieces you need to have in place, >>>>>>>> this patchset in itself is not particularly useful, it is just infra for >>>>>>>> the LCDIF and MIPI DSIM block control. You might want to wait until they >>>>>>>> all land in next and test that result. >>>>>>> >>>>>>> I have several patches in place, the GPCv2, this block driver, >>>>>>> enabling GPU DT node, I'm also working on the DSIM patch you posted. >>>>>>> I was hoping to test them all together and reply to the various >>>>>>> threads with tested-by. I also want to get my device tree stuff ready >>>>>>> on the beacon boards so when everything lands, I can post DTS updates >>>>>>> to enable the LCDIF, DSI, and the HDMI bridge. >>>>>>> >>>>>>> If you have a repo somewhere that has all these combined, I can just >>>>>>> work on the final layer to enable the device tree plumbing on my >>>>>>> board. I just need the imx8mm.dtsi changes for this, DSIM, and the >>>>>>> LCDIF so I can finish the task. >>>>>> >>>>>> On that note, I also have a i.MX8M Nano board which is similar to my >>>>>> 8MM. If I understood the 8MM clock block driver better, I hope to >>>>>> adapt your changes for the Nano too. Once the GPCv2 driver is >>>>>> accepted, I was also going to look at updating it to support the Nano >>>>>> as well which also has the same DSIM and LCDIF as the 8MM as well and >>>>>> a better GPU than the Mini but lacking the VPU. >>>>> >>>>> I don't have a branch, but I sent you the collected patches off-list. >>>>> >>>> >>>> I would also be interested in the patch collection for BLK_CTL, DSIM, >>>> etc. Marek, would you mind sending me those, too? >>>> >>>> Adam, did you already set up a branch and do some tests with the full stack? >>> >>> Frieder, >>> >>> I have been monitoring some of the activity on the BLK_CTL. It seems >>> like there is some disagreement on how to connect the power domain >>> controller with the BLK_CTL. Someone reported that it causes a hang >>> on the 8MP, so until that gets resolved, I doubt we'll be able to use >>> the display system. Some of the DSIM changes happening are being >>> pushed back for further changes, so it seems like having the full >>> integration might be a while. >> >> I have pulled all the latest patches, including Marek's off-list patches >> together in one branch based on v5.10-rc7 [1] if anyone is interested. >> >> I added some fixes on top, that I needed to get my display behind >> another Toshiba DSI-DPI bridge working. Those are probably not >> upstreamable at all and need further investigation. >> >> I'm hoping to reply to the individual threads for more feedback. I see >> that there are some blocking issues, but we hopefully get them resolved >> somehow. >> >> Thanks >> Frieder >> >> [1] https://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ffschrempf%2Flinux%2Fcommits%2Fv5.10-mx8mm-graphics&data=04%7C01%7Cfrieder.schrempf%40kontron.de%7C4b06e39a1030405300be08d8a20913f6%7C8c9d3c973fd941c8a2b1646f3942daf1%7C0%7C0%7C637437507175831939%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=n1Y4HwamMfPukFuVEurgt7KOM9SMLkercFhgFMuE6ro%3D&reserved=0 >> > > Frieder, > > Thanks for sharing your repo as it's getting hard to track these > patchsets (gpc/blk-ctl/power-domain/exynos/dsim). I'm also working on > display support for IMX8MM and in my case I'm trying to connect to a > RaspberryPi 7in display which I see Marek has been doing some work on > to split out drivers/gpu/drm/panel/panel-raspberrypi-touchscreen.c to > separate bridge, regulator/backlight, and simple-panel driver. thanks for the feedback and good to know of other people caring about upstream support. > > Marek, > > Thanks for your recent work on splitting out the rpi display driver so > that it can be bound via device-tree. I have found that I need to move > the tc358762_init to enable vs pre-enable when using it with the > in-progress samsung-dsim driver else the driver fails writes due to > not being enabled yet: > diff --git a/drivers/gpu/drm/bridge/tc358762.c > b/drivers/gpu/drm/bridge/tc358762.c > index 1bfdfc6..0d88e61 100644 > --- a/drivers/gpu/drm/bridge/tc358762.c > +++ b/drivers/gpu/drm/bridge/tc358762.c > @@ -153,11 +153,17 @@ static void tc358762_pre_enable(struct drm_bridge *bridge) > if (ret < 0) > dev_err(ctx->dev, "error enabling regulators (%d)\n", ret); > > + ctx->pre_enabled = true; > +} > + > +static void tc358762_enable(struct drm_bridge *bridge) > +{ > + struct tc358762 *ctx = bridge_to_tc358762(bridge); > + int ret; > + > ret = tc358762_init(ctx); > if (ret < 0) > dev_err(ctx->dev, "error initializing bridge (%d)\n", ret); > - > - ctx->pre_enabled = true; > } > > static int tc358762_attach(struct drm_bridge *bridge, > @@ -172,6 +178,7 @@ static int tc358762_attach(struct drm_bridge *bridge, > static const struct drm_bridge_funcs tc358762_bridge_funcs = { > .post_disable = tc358762_post_disable, > .pre_enable = tc358762_pre_enable, > + .enable = tc358762_enable, > .attach = tc358762_attach, > }; > > Frieder, I did find that your "drm/exynos: Fix PLL PMS offset for P > value bitfield" patch breaks the samsung_dsim_host_transfer for me > with the tc358762 bridge in the rpi panel. If I have that patch I get > a timeout on the transfer with some added debugging: > [ 4.386387] tc358762_write 0x0210=0x00000003 0 > [ 4.387031] samsung_dsim_host_transfer ret: 0 > [ 4.387038] tc358762_write 0x0164=0x00000005 0 > [ 4.387375] samsung_dsim_host_transfer ret: 0 > [ 4.387379] tc358762_write 0x0168=0x00000005 0 > [ 4.387409] samsung_dsim_host_transfer ret: 0 > [ 4.387413] tc358762_write 0x0144=0x00000000 0 > [ 4.387741] samsung_dsim_host_transfer ret: 0 > [ 4.387745] tc358762_write 0x0148=0x00000000 0 > [ 4.387773] samsung_dsim_host_transfer ret: 0 > [ 4.387777] tc358762_write 0x0114=0x00000003 0 > [ 4.387804] samsung_dsim_host_transfer ret: 0 > [ 4.387808] tc358762_write 0x0450=0x00000000 0 > [ 4.387834] samsung_dsim_host_transfer ret: 0 > [ 4.387838] tc358762_write 0x0420=0x00100150 0 > [ 4.388168] samsung_dsim_host_transfer ret: 0 > [ 4.388172] tc358762_write 0x0464=0x0000040f 0 > [ 4.388200] samsung_dsim_host_transfer ret: 0 > [ 4.493346] tc358762_write 0x0104=0x00000001 0 > [ 5.509341] imx-dsim-dsi 32e10000.mipi_dsi: xfer timed out: 29 06 > 00 00 04 01 01 00 00 00 > [ 5.509345] samsung_dsim_host_transfer ret: -110 > [ 5.509348] tc358762_write mipi_dsi_generic_write failed err=-110 > [ 5.509352] tc358762_write 0x0204=0x00000001 -110 > [ 5.617336] tc358762_init failed err=-110 > [ 5.617344] tc358762 32e10000.mipi_dsi.0: error initializing bridge (-110) > > Here is your patch which causes this issue for me: > diff --git a/drivers/gpu/drm/bridge/samsung-dsim.c > b/drivers/gpu/drm/bridge/samsung-dsim.c > index cb1ec3c..fc7c1d0 100644 > --- a/drivers/gpu/drm/bridge/samsung-dsim.c > +++ b/drivers/gpu/drm/bridge/samsung-dsim.c > @@ -174,7 +174,7 @@ > /* DSIM_PLLCTRL */ > #define DSIM_FREQ_BAND(x) ((x) << 24) > #define DSIM_PLL_EN (1 << 23) > -#define DSIM_PLL_P(x) ((x) << 13) > +#define DSIM_PLL_P(x) ((x) << 14) > #define DSIM_PLL_M(x) ((x) << 4) > #define DSIM_PLL_S(x) ((x) << 1) As I already mentioned in the commit message of this change, I have no idea how the "correct" fix should look like or if there even is anything to fix here at all. It's just what I needed to get my setup working and I found it really odd that the NXP vendor implementation differs from the upstream Exynos driver in this place. I have some other hardware setups with different bridges (LVDS/HDMI) behind the DSI and if I find some time, I will try them and see if they behave differently. Unfortunately I don't have any hardware to connect the RPi display to the i.MX8MM to test your setup. Best regards Frieder > > I'm not very knowledgeable about MIPI DSI and find it strange that > several writes in tc35872_init succeed until the failing writes: > tc358762_write(ctx, PPI_STARTPPI, PPI_START_FUNCTION); > tc358762_write(ctx, DSI_STARTDSI, DSI_RX_START); > > For what its worth I've backported Marek's rpi backlight/reglator and > simple-pannel driver to the NXP imx_5.4.47_2.2.0 kernel and do not see > any MIPI DSI write failure there, although I have the same behavior of > the display not showing anything. > > Marek, are you using the rpi panel with IMX8MM? While I now have the > drivers probing without error and have a functional backlight, > regulator I see nothing on the display. ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL 2020-10-03 22:45 [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL Marek Vasut ` (3 preceding siblings ...) 2020-10-07 19:52 ` Adam Ford @ 2021-02-04 12:46 ` Adam Ford 4 siblings, 0 replies; 17+ messages in thread From: Adam Ford @ 2021-02-04 12:46 UTC (permalink / raw) To: Marek Vasut Cc: arm-soc, Dong Aisheng, Abel Vesa, devicetree, Shawn Guo, Guido Günther, Rob Herring, NXP Linux Team, Fabio Estevam, Lucas Stach On Sun, Oct 4, 2020 at 12:53 AM Marek Vasut <marex@denx.de> wrote: > > Add the i.MX8MM BLK_CTL compatible string to the list. It seems that NXP has updated the TRM to Rev 3 for the i.MX8M. For what it's worth, section 13.2 calls this DISPLAY_BLK_CTRL They better document the missing registers. adam > > Signed-off-by: Marek Vasut <marex@denx.de> > Cc: Abel Vesa <abel.vesa@nxp.com> > Cc: Dong Aisheng <aisheng.dong@nxp.com> > Cc: Fabio Estevam <festevam@gmail.com> > Cc: Guido Günther <agx@sigxcpu.org> > Cc: Lucas Stach <l.stach@pengutronix.de> > Cc: Rob Herring <robh+dt@kernel.org> > Cc: Shawn Guo <shawnguo@kernel.org> > Cc: NXP Linux Team <linux-imx@nxp.com> > Cc: devicetree@vger.kernel.org > --- > Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml b/Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml > index 5e9eb402b9b6..346429f49093 100644 > --- a/Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml > +++ b/Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml > @@ -20,6 +20,7 @@ properties: > compatible: > items: > - enum: > + - fsl,imx8mm-dispmix-blk-ctl > - fsl,imx8mp-audio-blk-ctl > - fsl,imx8mp-hdmi-blk-ctl > - fsl,imx8mp-media-blk-ctl > -- > 2.28.0 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2021-02-04 12:47 UTC | newest] Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-10-03 22:45 [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL Marek Vasut 2020-10-03 22:45 ` [PATCH 2/5] dt-bindings: clock: imx8mm: Add media blk_ctl clock IDs Marek Vasut 2020-10-06 21:12 ` Rob Herring 2020-10-03 22:45 ` [PATCH 3/5] dt-bindings: reset: imx8mm: Add media blk_ctl reset IDs Marek Vasut 2020-10-06 21:12 ` Rob Herring 2020-10-06 21:12 ` [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL Rob Herring 2020-10-07 19:52 ` Adam Ford 2020-10-07 20:01 ` Marek Vasut 2020-10-07 20:08 ` Adam Ford 2020-10-07 20:17 ` Adam Ford 2020-10-07 20:50 ` Marek Vasut 2020-11-30 11:47 ` Frieder Schrempf 2020-11-30 15:43 ` Adam Ford 2020-12-10 15:14 ` Frieder Schrempf 2020-12-16 21:24 ` Tim Harvey 2020-12-22 9:07 ` Frieder Schrempf 2021-02-04 12:46 ` Adam Ford
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).