* [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs @ 2021-11-01 22:28 ` Tommaso Merciai 0 siblings, 0 replies; 20+ messages in thread From: Tommaso Merciai @ 2021-11-01 22:28 UTC (permalink / raw) Cc: tomm.merciai, Rob Herring, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Joakim Zhang, Lucas Stach, Peng Fan, Krzysztof Kozlowski, Alice Guo, Adam Ford, devicetree, linux-arm-kernel, linux-kernel Add system controller node for registers of module Display Block Control (DISPLAY_BLK_CTRL, base address: 0x32e28000). The DISPLAY_BLK_CTRL module contains general purpose registers (GPRs), which control varied features of the associated peripherals. Reference: IMX8MMRM Rev. 3, 11/2020, p 3897 --- arch/arm64/boot/dts/freescale/imx8mm.dtsi | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi index 2f632e8ca388..3e496b457e1a 100644 --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi @@ -961,6 +961,11 @@ aips4: bus@32c00000 { #size-cells = <1>; ranges = <0x32c00000 0x32c00000 0x400000>; + dispmix_gpr: display-gpr@32e28000 { + compatible = "fsl, imx8mm-iomuxc-gpr", "syscon"; + reg = <0x32e28000 0x100>; + }; + usbotg1: usb@32e40000 { compatible = "fsl,imx8mm-usb", "fsl,imx7d-usb"; reg = <0x32e40000 0x200>; -- 2.25.1 ^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs @ 2021-11-01 22:28 ` Tommaso Merciai 0 siblings, 0 replies; 20+ messages in thread From: Tommaso Merciai @ 2021-11-01 22:28 UTC (permalink / raw) Cc: tomm.merciai, Rob Herring, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Joakim Zhang, Lucas Stach, Peng Fan, Krzysztof Kozlowski, Alice Guo, Adam Ford, devicetree, linux-arm-kernel, linux-kernel Add system controller node for registers of module Display Block Control (DISPLAY_BLK_CTRL, base address: 0x32e28000). The DISPLAY_BLK_CTRL module contains general purpose registers (GPRs), which control varied features of the associated peripherals. Reference: IMX8MMRM Rev. 3, 11/2020, p 3897 --- arch/arm64/boot/dts/freescale/imx8mm.dtsi | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi index 2f632e8ca388..3e496b457e1a 100644 --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi @@ -961,6 +961,11 @@ aips4: bus@32c00000 { #size-cells = <1>; ranges = <0x32c00000 0x32c00000 0x400000>; + dispmix_gpr: display-gpr@32e28000 { + compatible = "fsl, imx8mm-iomuxc-gpr", "syscon"; + reg = <0x32e28000 0x100>; + }; + usbotg1: usb@32e40000 { compatible = "fsl,imx8mm-usb", "fsl,imx7d-usb"; reg = <0x32e40000 0x200>; -- 2.25.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 20+ messages in thread
* Re: [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs 2021-11-01 22:28 ` Tommaso Merciai @ 2021-11-01 22:35 ` Ahmad Fatoum -1 siblings, 0 replies; 20+ messages in thread From: Ahmad Fatoum @ 2021-11-01 22:35 UTC (permalink / raw) To: Tommaso Merciai Cc: linux-arm-kernel, devicetree, Peng Fan, Alice Guo, Shawn Guo, Sascha Hauer, Joakim Zhang, Krzysztof Kozlowski, linux-kernel, Rob Herring, NXP Linux Team, Pengutronix Kernel Team, Fabio Estevam, Adam Ford, Lucas Stach Hello Tommaso, On 01.11.21 23:28, Tommaso Merciai wrote: > Add system controller node for registers of module Display Block Control > (DISPLAY_BLK_CTRL, base address: 0x32e28000). > The DISPLAY_BLK_CTRL module contains general purpose registers (GPRs), > which control varied features of the associated peripherals. > Reference: IMX8MMRM Rev. 3, 11/2020, p 3897 > --- > arch/arm64/boot/dts/freescale/imx8mm.dtsi | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > index 2f632e8ca388..3e496b457e1a 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > @@ -961,6 +961,11 @@ aips4: bus@32c00000 { > #size-cells = <1>; > ranges = <0x32c00000 0x32c00000 0x400000>; > > + dispmix_gpr: display-gpr@32e28000 { > + compatible = "fsl, imx8mm-iomuxc-gpr", "syscon"; Please read vendor patches before submitting them. The space is out-of-place in the compatible and the compatible is wrong: This doesn't look like a i.MX8MM pin controller. Cheers, Ahmad > + reg = <0x32e28000 0x100>; > + }; > + > usbotg1: usb@32e40000 { > compatible = "fsl,imx8mm-usb", "fsl,imx7d-usb"; > reg = <0x32e40000 0x200>; > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs @ 2021-11-01 22:35 ` Ahmad Fatoum 0 siblings, 0 replies; 20+ messages in thread From: Ahmad Fatoum @ 2021-11-01 22:35 UTC (permalink / raw) To: Tommaso Merciai Cc: linux-arm-kernel, devicetree, Peng Fan, Alice Guo, Shawn Guo, Sascha Hauer, Joakim Zhang, Krzysztof Kozlowski, linux-kernel, Rob Herring, NXP Linux Team, Pengutronix Kernel Team, Fabio Estevam, Adam Ford, Lucas Stach Hello Tommaso, On 01.11.21 23:28, Tommaso Merciai wrote: > Add system controller node for registers of module Display Block Control > (DISPLAY_BLK_CTRL, base address: 0x32e28000). > The DISPLAY_BLK_CTRL module contains general purpose registers (GPRs), > which control varied features of the associated peripherals. > Reference: IMX8MMRM Rev. 3, 11/2020, p 3897 > --- > arch/arm64/boot/dts/freescale/imx8mm.dtsi | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > index 2f632e8ca388..3e496b457e1a 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > @@ -961,6 +961,11 @@ aips4: bus@32c00000 { > #size-cells = <1>; > ranges = <0x32c00000 0x32c00000 0x400000>; > > + dispmix_gpr: display-gpr@32e28000 { > + compatible = "fsl, imx8mm-iomuxc-gpr", "syscon"; Please read vendor patches before submitting them. The space is out-of-place in the compatible and the compatible is wrong: This doesn't look like a i.MX8MM pin controller. Cheers, Ahmad > + reg = <0x32e28000 0x100>; > + }; > + > usbotg1: usb@32e40000 { > compatible = "fsl,imx8mm-usb", "fsl,imx7d-usb"; > reg = <0x32e40000 0x200>; > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ 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] 20+ messages in thread
* Re: [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs 2021-11-01 22:35 ` Ahmad Fatoum @ 2021-11-01 22:58 ` Tommaso Merciai -1 siblings, 0 replies; 20+ messages in thread From: Tommaso Merciai @ 2021-11-01 22:58 UTC (permalink / raw) To: Ahmad Fatoum Cc: linux-arm-kernel, devicetree, Peng Fan, Alice Guo, Shawn Guo, Sascha Hauer, Joakim Zhang, Krzysztof Kozlowski, linux-kernel, Rob Herring, NXP Linux Team, Pengutronix Kernel Team, Fabio Estevam, Adam Ford, Lucas Stach On Mon, Nov 01, 2021 at 11:35:49PM +0100, Ahmad Fatoum wrote: > Hello Tommaso, > > On 01.11.21 23:28, Tommaso Merciai wrote: > > Add system controller node for registers of module Display Block Control > > (DISPLAY_BLK_CTRL, base address: 0x32e28000). > > The DISPLAY_BLK_CTRL module contains general purpose registers (GPRs), > > which control varied features of the associated peripherals. > > Reference: IMX8MMRM Rev. 3, 11/2020, p 3897 > > --- > > arch/arm64/boot/dts/freescale/imx8mm.dtsi | 5 +++++ > > 1 file changed, 5 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > index 2f632e8ca388..3e496b457e1a 100644 > > --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > @@ -961,6 +961,11 @@ aips4: bus@32c00000 { > > #size-cells = <1>; > > ranges = <0x32c00000 0x32c00000 0x400000>; > > > > + dispmix_gpr: display-gpr@32e28000 { > > + compatible = "fsl, imx8mm-iomuxc-gpr", "syscon"; > > Please read vendor patches before submitting them. The space > is out-of-place in the compatible and the compatible is wrong: > This doesn't look like a i.MX8MM pin controller. > > Cheers, > Ahmad Hi Ahmad, Thanks for your review. Do you think this is correct? compatible = "fsl,imx8mm-dispmix-gpr", "syscon"; Let me know. Thanks, Tommaso > > > + reg = <0x32e28000 0x100>; > > + }; > > + > > usbotg1: usb@32e40000 { > > compatible = "fsl,imx8mm-usb", "fsl,imx7d-usb"; > > reg = <0x32e40000 0x200>; > > > > > -- > Pengutronix e.K. | | > Steuerwalder Str. 21 | http://www.pengutronix.de/ | > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs @ 2021-11-01 22:58 ` Tommaso Merciai 0 siblings, 0 replies; 20+ messages in thread From: Tommaso Merciai @ 2021-11-01 22:58 UTC (permalink / raw) To: Ahmad Fatoum Cc: linux-arm-kernel, devicetree, Peng Fan, Alice Guo, Shawn Guo, Sascha Hauer, Joakim Zhang, Krzysztof Kozlowski, linux-kernel, Rob Herring, NXP Linux Team, Pengutronix Kernel Team, Fabio Estevam, Adam Ford, Lucas Stach On Mon, Nov 01, 2021 at 11:35:49PM +0100, Ahmad Fatoum wrote: > Hello Tommaso, > > On 01.11.21 23:28, Tommaso Merciai wrote: > > Add system controller node for registers of module Display Block Control > > (DISPLAY_BLK_CTRL, base address: 0x32e28000). > > The DISPLAY_BLK_CTRL module contains general purpose registers (GPRs), > > which control varied features of the associated peripherals. > > Reference: IMX8MMRM Rev. 3, 11/2020, p 3897 > > --- > > arch/arm64/boot/dts/freescale/imx8mm.dtsi | 5 +++++ > > 1 file changed, 5 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > index 2f632e8ca388..3e496b457e1a 100644 > > --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > @@ -961,6 +961,11 @@ aips4: bus@32c00000 { > > #size-cells = <1>; > > ranges = <0x32c00000 0x32c00000 0x400000>; > > > > + dispmix_gpr: display-gpr@32e28000 { > > + compatible = "fsl, imx8mm-iomuxc-gpr", "syscon"; > > Please read vendor patches before submitting them. The space > is out-of-place in the compatible and the compatible is wrong: > This doesn't look like a i.MX8MM pin controller. > > Cheers, > Ahmad Hi Ahmad, Thanks for your review. Do you think this is correct? compatible = "fsl,imx8mm-dispmix-gpr", "syscon"; Let me know. Thanks, Tommaso > > > + reg = <0x32e28000 0x100>; > > + }; > > + > > usbotg1: usb@32e40000 { > > compatible = "fsl,imx8mm-usb", "fsl,imx7d-usb"; > > reg = <0x32e40000 0x200>; > > > > > -- > Pengutronix e.K. | | > Steuerwalder Str. 21 | http://www.pengutronix.de/ | > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ 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] 20+ messages in thread
* Re: [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs 2021-11-01 22:58 ` Tommaso Merciai @ 2021-11-02 4:22 ` Adam Ford -1 siblings, 0 replies; 20+ messages in thread From: Adam Ford @ 2021-11-02 4:22 UTC (permalink / raw) To: Tommaso Merciai Cc: Ahmad Fatoum, arm-soc, devicetree, Peng Fan, Alice Guo, Shawn Guo, Sascha Hauer, Joakim Zhang, Krzysztof Kozlowski, Linux Kernel Mailing List, Rob Herring, NXP Linux Team, Pengutronix Kernel Team, Fabio Estevam, Lucas Stach On Mon, Nov 1, 2021 at 5:58 PM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > On Mon, Nov 01, 2021 at 11:35:49PM +0100, Ahmad Fatoum wrote: > > Hello Tommaso, > > > > On 01.11.21 23:28, Tommaso Merciai wrote: > > > Add system controller node for registers of module Display Block Control > > > (DISPLAY_BLK_CTRL, base address: 0x32e28000). > > > The DISPLAY_BLK_CTRL module contains general purpose registers (GPRs), > > > which control varied features of the associated peripherals. > > > Reference: IMX8MMRM Rev. 3, 11/2020, p 3897 > > > --- > > > arch/arm64/boot/dts/freescale/imx8mm.dtsi | 5 +++++ > > > 1 file changed, 5 insertions(+) > > > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > index 2f632e8ca388..3e496b457e1a 100644 > > > --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > @@ -961,6 +961,11 @@ aips4: bus@32c00000 { > > > #size-cells = <1>; > > > ranges = <0x32c00000 0x32c00000 0x400000>; > > > > > > + dispmix_gpr: display-gpr@32e28000 { > > > + compatible = "fsl, imx8mm-iomuxc-gpr", "syscon"; > > > > Please read vendor patches before submitting them. The space > > is out-of-place in the compatible and the compatible is wrong: > > This doesn't look like a i.MX8MM pin controller. > > > > Cheers, > > Ahmad > > Hi Ahmad, > Thanks for your review. Do you think this is correct? > > compatible = "fsl,imx8mm-dispmix-gpr", "syscon"; > > Let me know. There was already a driver created for the blk-ctrl stuff and it has a device tree binding at 32e28000. It's tied into the power-domain system, so if you want to enable the csi, dsi, or lcd, etc. you can just reference the blt-ctrl power domain index, and it enables the device's gpc power domain and takes the corresponding device out of reset. adam > > Thanks, > Tommaso > > > > > > + reg = <0x32e28000 0x100>; > > > + }; > > > + > > > usbotg1: usb@32e40000 { > > > compatible = "fsl,imx8mm-usb", "fsl,imx7d-usb"; > > > reg = <0x32e40000 0x200>; > > > > > > > > > -- > > Pengutronix e.K. | | > > Steuerwalder Str. 21 | http://www.pengutronix.de/ | > > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs @ 2021-11-02 4:22 ` Adam Ford 0 siblings, 0 replies; 20+ messages in thread From: Adam Ford @ 2021-11-02 4:22 UTC (permalink / raw) To: Tommaso Merciai Cc: Ahmad Fatoum, arm-soc, devicetree, Peng Fan, Alice Guo, Shawn Guo, Sascha Hauer, Joakim Zhang, Krzysztof Kozlowski, Linux Kernel Mailing List, Rob Herring, NXP Linux Team, Pengutronix Kernel Team, Fabio Estevam, Lucas Stach On Mon, Nov 1, 2021 at 5:58 PM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > On Mon, Nov 01, 2021 at 11:35:49PM +0100, Ahmad Fatoum wrote: > > Hello Tommaso, > > > > On 01.11.21 23:28, Tommaso Merciai wrote: > > > Add system controller node for registers of module Display Block Control > > > (DISPLAY_BLK_CTRL, base address: 0x32e28000). > > > The DISPLAY_BLK_CTRL module contains general purpose registers (GPRs), > > > which control varied features of the associated peripherals. > > > Reference: IMX8MMRM Rev. 3, 11/2020, p 3897 > > > --- > > > arch/arm64/boot/dts/freescale/imx8mm.dtsi | 5 +++++ > > > 1 file changed, 5 insertions(+) > > > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > index 2f632e8ca388..3e496b457e1a 100644 > > > --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > @@ -961,6 +961,11 @@ aips4: bus@32c00000 { > > > #size-cells = <1>; > > > ranges = <0x32c00000 0x32c00000 0x400000>; > > > > > > + dispmix_gpr: display-gpr@32e28000 { > > > + compatible = "fsl, imx8mm-iomuxc-gpr", "syscon"; > > > > Please read vendor patches before submitting them. The space > > is out-of-place in the compatible and the compatible is wrong: > > This doesn't look like a i.MX8MM pin controller. > > > > Cheers, > > Ahmad > > Hi Ahmad, > Thanks for your review. Do you think this is correct? > > compatible = "fsl,imx8mm-dispmix-gpr", "syscon"; > > Let me know. There was already a driver created for the blk-ctrl stuff and it has a device tree binding at 32e28000. It's tied into the power-domain system, so if you want to enable the csi, dsi, or lcd, etc. you can just reference the blt-ctrl power domain index, and it enables the device's gpc power domain and takes the corresponding device out of reset. adam > > Thanks, > Tommaso > > > > > > + reg = <0x32e28000 0x100>; > > > + }; > > > + > > > usbotg1: usb@32e40000 { > > > compatible = "fsl,imx8mm-usb", "fsl,imx7d-usb"; > > > reg = <0x32e40000 0x200>; > > > > > > > > > -- > > Pengutronix e.K. | | > > Steuerwalder Str. 21 | http://www.pengutronix.de/ | > > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ 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] 20+ messages in thread
* Re: [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs 2021-11-02 4:22 ` Adam Ford @ 2021-11-02 11:59 ` Tommaso Merciai -1 siblings, 0 replies; 20+ messages in thread From: Tommaso Merciai @ 2021-11-02 11:59 UTC (permalink / raw) To: Adam Ford Cc: Ahmad Fatoum, arm-soc, devicetree, Peng Fan, Alice Guo, Shawn Guo, Sascha Hauer, Joakim Zhang, Krzysztof Kozlowski, Linux Kernel Mailing List, Rob Herring, NXP Linux Team, Pengutronix Kernel Team, Fabio Estevam, Lucas Stach On Mon, Nov 01, 2021 at 11:22:21PM -0500, Adam Ford wrote: > On Mon, Nov 1, 2021 at 5:58 PM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > > > On Mon, Nov 01, 2021 at 11:35:49PM +0100, Ahmad Fatoum wrote: > > > Hello Tommaso, > > > > > > On 01.11.21 23:28, Tommaso Merciai wrote: > > > > Add system controller node for registers of module Display Block Control > > > > (DISPLAY_BLK_CTRL, base address: 0x32e28000). > > > > The DISPLAY_BLK_CTRL module contains general purpose registers (GPRs), > > > > which control varied features of the associated peripherals. > > > > Reference: IMX8MMRM Rev. 3, 11/2020, p 3897 > > > > --- > > > > arch/arm64/boot/dts/freescale/imx8mm.dtsi | 5 +++++ > > > > 1 file changed, 5 insertions(+) > > > > > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > index 2f632e8ca388..3e496b457e1a 100644 > > > > --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > @@ -961,6 +961,11 @@ aips4: bus@32c00000 { > > > > #size-cells = <1>; > > > > ranges = <0x32c00000 0x32c00000 0x400000>; > > > > > > > > + dispmix_gpr: display-gpr@32e28000 { > > > > + compatible = "fsl, imx8mm-iomuxc-gpr", "syscon"; > > > > > > Please read vendor patches before submitting them. The space > > > is out-of-place in the compatible and the compatible is wrong: > > > This doesn't look like a i.MX8MM pin controller. > > > > > > Cheers, > > > Ahmad > > > > Hi Ahmad, > > Thanks for your review. Do you think this is correct? > > > > compatible = "fsl,imx8mm-dispmix-gpr", "syscon"; > > > > Let me know. > > There was already a driver created for the blk-ctrl stuff and it has a > device tree binding at 32e28000. It's tied into the power-domain > system, so if you want to enable the csi, dsi, or lcd, etc. you can > just reference the blt-ctrl power domain index, and it enables the > device's gpc power domain and takes the corresponding device out of > reset. Hi Adam, You mean using the gpcv2.c driver? drivers/soc/imx/gpcv2.c With the following node, to put out of reset eLCDIF and mipi_dsi: gpc: gpc@303a0000 { compatible = "fsl,imx8mm-gpc"; reg = <0x303a0000 0x10000>; interrupts = <GIC_SPI 87 IRQ_TYPE_LEVEL_HIGH>; interrupt-parent = <&gic>; interrupt-controller; #interrupt-cells = <3>; pgc { #address-cells = <1>; #size-cells = <0>; pgc_mipi: power-domain@0 { #power-domain-cells = <0>; reg = <IMX8M_POWER_DOMAIN_MIPI>; }; pgc_disp: power-domain@7 { #power-domain-cells = <0>; reg = <IMX8M_POWER_DOMAIN_DISP>; }; }; }; Let me know. Thanks, Tommaso > > adam > > > > Thanks, > > Tommaso > > > > > > > > > + reg = <0x32e28000 0x100>; > > > > + }; > > > > + > > > > usbotg1: usb@32e40000 { > > > > compatible = "fsl,imx8mm-usb", "fsl,imx7d-usb"; > > > > reg = <0x32e40000 0x200>; > > > > > > > > > > > > > -- > > > Pengutronix e.K. | | > > > Steuerwalder Str. 21 | http://www.pengutronix.de/ | > > > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > > > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs @ 2021-11-02 11:59 ` Tommaso Merciai 0 siblings, 0 replies; 20+ messages in thread From: Tommaso Merciai @ 2021-11-02 11:59 UTC (permalink / raw) To: Adam Ford Cc: Ahmad Fatoum, arm-soc, devicetree, Peng Fan, Alice Guo, Shawn Guo, Sascha Hauer, Joakim Zhang, Krzysztof Kozlowski, Linux Kernel Mailing List, Rob Herring, NXP Linux Team, Pengutronix Kernel Team, Fabio Estevam, Lucas Stach On Mon, Nov 01, 2021 at 11:22:21PM -0500, Adam Ford wrote: > On Mon, Nov 1, 2021 at 5:58 PM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > > > On Mon, Nov 01, 2021 at 11:35:49PM +0100, Ahmad Fatoum wrote: > > > Hello Tommaso, > > > > > > On 01.11.21 23:28, Tommaso Merciai wrote: > > > > Add system controller node for registers of module Display Block Control > > > > (DISPLAY_BLK_CTRL, base address: 0x32e28000). > > > > The DISPLAY_BLK_CTRL module contains general purpose registers (GPRs), > > > > which control varied features of the associated peripherals. > > > > Reference: IMX8MMRM Rev. 3, 11/2020, p 3897 > > > > --- > > > > arch/arm64/boot/dts/freescale/imx8mm.dtsi | 5 +++++ > > > > 1 file changed, 5 insertions(+) > > > > > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > index 2f632e8ca388..3e496b457e1a 100644 > > > > --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > @@ -961,6 +961,11 @@ aips4: bus@32c00000 { > > > > #size-cells = <1>; > > > > ranges = <0x32c00000 0x32c00000 0x400000>; > > > > > > > > + dispmix_gpr: display-gpr@32e28000 { > > > > + compatible = "fsl, imx8mm-iomuxc-gpr", "syscon"; > > > > > > Please read vendor patches before submitting them. The space > > > is out-of-place in the compatible and the compatible is wrong: > > > This doesn't look like a i.MX8MM pin controller. > > > > > > Cheers, > > > Ahmad > > > > Hi Ahmad, > > Thanks for your review. Do you think this is correct? > > > > compatible = "fsl,imx8mm-dispmix-gpr", "syscon"; > > > > Let me know. > > There was already a driver created for the blk-ctrl stuff and it has a > device tree binding at 32e28000. It's tied into the power-domain > system, so if you want to enable the csi, dsi, or lcd, etc. you can > just reference the blt-ctrl power domain index, and it enables the > device's gpc power domain and takes the corresponding device out of > reset. Hi Adam, You mean using the gpcv2.c driver? drivers/soc/imx/gpcv2.c With the following node, to put out of reset eLCDIF and mipi_dsi: gpc: gpc@303a0000 { compatible = "fsl,imx8mm-gpc"; reg = <0x303a0000 0x10000>; interrupts = <GIC_SPI 87 IRQ_TYPE_LEVEL_HIGH>; interrupt-parent = <&gic>; interrupt-controller; #interrupt-cells = <3>; pgc { #address-cells = <1>; #size-cells = <0>; pgc_mipi: power-domain@0 { #power-domain-cells = <0>; reg = <IMX8M_POWER_DOMAIN_MIPI>; }; pgc_disp: power-domain@7 { #power-domain-cells = <0>; reg = <IMX8M_POWER_DOMAIN_DISP>; }; }; }; Let me know. Thanks, Tommaso > > adam > > > > Thanks, > > Tommaso > > > > > > > > > + reg = <0x32e28000 0x100>; > > > > + }; > > > > + > > > > usbotg1: usb@32e40000 { > > > > compatible = "fsl,imx8mm-usb", "fsl,imx7d-usb"; > > > > reg = <0x32e40000 0x200>; > > > > > > > > > > > > > -- > > > Pengutronix e.K. | | > > > Steuerwalder Str. 21 | http://www.pengutronix.de/ | > > > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > > > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ 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] 20+ messages in thread
* Re: [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs 2021-11-02 11:59 ` Tommaso Merciai @ 2021-11-02 12:23 ` Adam Ford -1 siblings, 0 replies; 20+ messages in thread From: Adam Ford @ 2021-11-02 12:23 UTC (permalink / raw) To: Tommaso Merciai Cc: Ahmad Fatoum, arm-soc, devicetree, Peng Fan, Alice Guo, Shawn Guo, Sascha Hauer, Joakim Zhang, Krzysztof Kozlowski, Linux Kernel Mailing List, Rob Herring, NXP Linux Team, Pengutronix Kernel Team, Fabio Estevam, Lucas Stach The upcoming 5.16 kernel will have a new blk-ctrl driver which will work in conjunction with the GPC. You can see it in linux-next [1], and I would expect it to be present in 5.16-rc1 once the merge is done. In [1], Look for : disp_blk_ctrl: blk-ctrl@32e28000 { compatible = "fsl,imx8mm-disp-blk-ctrl", "syscon"; It creates a bunch of virtual power domains which are effectively the resets for the VPU, CSI, DSI, and LCDIF [2]. Basically, to pull the respective device out of reset, you'd reference them using power-domains. I have an RFC patch for the CSI located [3] which should bring the GPC power domain up, then take the CSI bridge and MIPI_CSI out of reset using the blk-ctrl. A few of us are still investigating the CSI bridge and mipi_csi drivers to determine what's going wrong, but inside that patch, you'll see that we reference "power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_CSI_BRIDGE>;" and "power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_MIPI_CSI>;" which are part of the new blk-ctrl driver @32e2800. Other peripherals like LCD, DSI, and the VPU's should be able to reference their respective power domains to activate the corresponding resets after enabling the proper GPC power domain. [1] - https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/arch/arm64/boot/dts/freescale/imx8mm.dtsi?h=next-20211102 [2] - https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/include/dt-bindings/power/imx8mm-power.h?h=next-20211102 [3] - https://patchwork.kernel.org/project/linux-arm-kernel/patch/20211023203457.1217821-2-aford173@gmail.com/ On Tue, Nov 2, 2021 at 6:59 AM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > On Mon, Nov 01, 2021 at 11:22:21PM -0500, Adam Ford wrote: > > On Mon, Nov 1, 2021 at 5:58 PM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > > > > > On Mon, Nov 01, 2021 at 11:35:49PM +0100, Ahmad Fatoum wrote: > > > > Hello Tommaso, > > > > > > > > On 01.11.21 23:28, Tommaso Merciai wrote: > > > > > Add system controller node for registers of module Display Block Control > > > > > (DISPLAY_BLK_CTRL, base address: 0x32e28000). > > > > > The DISPLAY_BLK_CTRL module contains general purpose registers (GPRs), > > > > > which control varied features of the associated peripherals. > > > > > Reference: IMX8MMRM Rev. 3, 11/2020, p 3897 > > > > > --- > > > > > arch/arm64/boot/dts/freescale/imx8mm.dtsi | 5 +++++ > > > > > 1 file changed, 5 insertions(+) > > > > > > > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > > index 2f632e8ca388..3e496b457e1a 100644 > > > > > --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > > +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > > @@ -961,6 +961,11 @@ aips4: bus@32c00000 { > > > > > #size-cells = <1>; > > > > > ranges = <0x32c00000 0x32c00000 0x400000>; > > > > > > > > > > + dispmix_gpr: display-gpr@32e28000 { > > > > > + compatible = "fsl, imx8mm-iomuxc-gpr", "syscon"; > > > > > > > > Please read vendor patches before submitting them. The space > > > > is out-of-place in the compatible and the compatible is wrong: > > > > This doesn't look like a i.MX8MM pin controller. > > > > > > > > Cheers, > > > > Ahmad > > > > > > Hi Ahmad, > > > Thanks for your review. Do you think this is correct? > > > > > > compatible = "fsl,imx8mm-dispmix-gpr", "syscon"; > > > > > > Let me know. > > > > There was already a driver created for the blk-ctrl stuff and it has a > > device tree binding at 32e28000. It's tied into the power-domain > > system, so if you want to enable the csi, dsi, or lcd, etc. you can > > just reference the blt-ctrl power domain index, and it enables the > > device's gpc power domain and takes the corresponding device out of > > reset. > > Hi Adam, > You mean using the gpcv2.c driver? > > drivers/soc/imx/gpcv2.c > > With the following node, to put out of reset eLCDIF and mipi_dsi: > > gpc: gpc@303a0000 { > compatible = "fsl,imx8mm-gpc"; > reg = <0x303a0000 0x10000>; > interrupts = <GIC_SPI 87 IRQ_TYPE_LEVEL_HIGH>; > interrupt-parent = <&gic>; > interrupt-controller; > #interrupt-cells = <3>; > > pgc { > > #address-cells = <1>; > #size-cells = <0>; > pgc_mipi: power-domain@0 { > #power-domain-cells = <0>; > reg = <IMX8M_POWER_DOMAIN_MIPI>; > }; > > pgc_disp: power-domain@7 { > #power-domain-cells = <0>; > reg = <IMX8M_POWER_DOMAIN_DISP>; > }; > }; > }; > > Let me know. > > Thanks, > Tommaso > > > > > adam > > > > > > Thanks, > > > Tommaso > > > > > > > > > > > > + reg = <0x32e28000 0x100>; > > > > > + }; > > > > > + > > > > > usbotg1: usb@32e40000 { > > > > > compatible = "fsl,imx8mm-usb", "fsl,imx7d-usb"; > > > > > reg = <0x32e40000 0x200>; > > > > > > > > > > > > > > > > > -- > > > > Pengutronix e.K. | | > > > > Steuerwalder Str. 21 | http://www.pengutronix.de/ | > > > > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > > > > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs @ 2021-11-02 12:23 ` Adam Ford 0 siblings, 0 replies; 20+ messages in thread From: Adam Ford @ 2021-11-02 12:23 UTC (permalink / raw) To: Tommaso Merciai Cc: Ahmad Fatoum, arm-soc, devicetree, Peng Fan, Alice Guo, Shawn Guo, Sascha Hauer, Joakim Zhang, Krzysztof Kozlowski, Linux Kernel Mailing List, Rob Herring, NXP Linux Team, Pengutronix Kernel Team, Fabio Estevam, Lucas Stach The upcoming 5.16 kernel will have a new blk-ctrl driver which will work in conjunction with the GPC. You can see it in linux-next [1], and I would expect it to be present in 5.16-rc1 once the merge is done. In [1], Look for : disp_blk_ctrl: blk-ctrl@32e28000 { compatible = "fsl,imx8mm-disp-blk-ctrl", "syscon"; It creates a bunch of virtual power domains which are effectively the resets for the VPU, CSI, DSI, and LCDIF [2]. Basically, to pull the respective device out of reset, you'd reference them using power-domains. I have an RFC patch for the CSI located [3] which should bring the GPC power domain up, then take the CSI bridge and MIPI_CSI out of reset using the blk-ctrl. A few of us are still investigating the CSI bridge and mipi_csi drivers to determine what's going wrong, but inside that patch, you'll see that we reference "power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_CSI_BRIDGE>;" and "power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_MIPI_CSI>;" which are part of the new blk-ctrl driver @32e2800. Other peripherals like LCD, DSI, and the VPU's should be able to reference their respective power domains to activate the corresponding resets after enabling the proper GPC power domain. [1] - https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/arch/arm64/boot/dts/freescale/imx8mm.dtsi?h=next-20211102 [2] - https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/include/dt-bindings/power/imx8mm-power.h?h=next-20211102 [3] - https://patchwork.kernel.org/project/linux-arm-kernel/patch/20211023203457.1217821-2-aford173@gmail.com/ On Tue, Nov 2, 2021 at 6:59 AM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > On Mon, Nov 01, 2021 at 11:22:21PM -0500, Adam Ford wrote: > > On Mon, Nov 1, 2021 at 5:58 PM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > > > > > On Mon, Nov 01, 2021 at 11:35:49PM +0100, Ahmad Fatoum wrote: > > > > Hello Tommaso, > > > > > > > > On 01.11.21 23:28, Tommaso Merciai wrote: > > > > > Add system controller node for registers of module Display Block Control > > > > > (DISPLAY_BLK_CTRL, base address: 0x32e28000). > > > > > The DISPLAY_BLK_CTRL module contains general purpose registers (GPRs), > > > > > which control varied features of the associated peripherals. > > > > > Reference: IMX8MMRM Rev. 3, 11/2020, p 3897 > > > > > --- > > > > > arch/arm64/boot/dts/freescale/imx8mm.dtsi | 5 +++++ > > > > > 1 file changed, 5 insertions(+) > > > > > > > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > > index 2f632e8ca388..3e496b457e1a 100644 > > > > > --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > > +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > > @@ -961,6 +961,11 @@ aips4: bus@32c00000 { > > > > > #size-cells = <1>; > > > > > ranges = <0x32c00000 0x32c00000 0x400000>; > > > > > > > > > > + dispmix_gpr: display-gpr@32e28000 { > > > > > + compatible = "fsl, imx8mm-iomuxc-gpr", "syscon"; > > > > > > > > Please read vendor patches before submitting them. The space > > > > is out-of-place in the compatible and the compatible is wrong: > > > > This doesn't look like a i.MX8MM pin controller. > > > > > > > > Cheers, > > > > Ahmad > > > > > > Hi Ahmad, > > > Thanks for your review. Do you think this is correct? > > > > > > compatible = "fsl,imx8mm-dispmix-gpr", "syscon"; > > > > > > Let me know. > > > > There was already a driver created for the blk-ctrl stuff and it has a > > device tree binding at 32e28000. It's tied into the power-domain > > system, so if you want to enable the csi, dsi, or lcd, etc. you can > > just reference the blt-ctrl power domain index, and it enables the > > device's gpc power domain and takes the corresponding device out of > > reset. > > Hi Adam, > You mean using the gpcv2.c driver? > > drivers/soc/imx/gpcv2.c > > With the following node, to put out of reset eLCDIF and mipi_dsi: > > gpc: gpc@303a0000 { > compatible = "fsl,imx8mm-gpc"; > reg = <0x303a0000 0x10000>; > interrupts = <GIC_SPI 87 IRQ_TYPE_LEVEL_HIGH>; > interrupt-parent = <&gic>; > interrupt-controller; > #interrupt-cells = <3>; > > pgc { > > #address-cells = <1>; > #size-cells = <0>; > pgc_mipi: power-domain@0 { > #power-domain-cells = <0>; > reg = <IMX8M_POWER_DOMAIN_MIPI>; > }; > > pgc_disp: power-domain@7 { > #power-domain-cells = <0>; > reg = <IMX8M_POWER_DOMAIN_DISP>; > }; > }; > }; > > Let me know. > > Thanks, > Tommaso > > > > > adam > > > > > > Thanks, > > > Tommaso > > > > > > > > > > > > + reg = <0x32e28000 0x100>; > > > > > + }; > > > > > + > > > > > usbotg1: usb@32e40000 { > > > > > compatible = "fsl,imx8mm-usb", "fsl,imx7d-usb"; > > > > > reg = <0x32e40000 0x200>; > > > > > > > > > > > > > > > > > -- > > > > Pengutronix e.K. | | > > > > Steuerwalder Str. 21 | http://www.pengutronix.de/ | > > > > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > > > > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ 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] 20+ messages in thread
* Re: [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs 2021-11-02 12:23 ` Adam Ford @ 2021-11-02 15:47 ` Tommaso Merciai -1 siblings, 0 replies; 20+ messages in thread From: Tommaso Merciai @ 2021-11-02 15:47 UTC (permalink / raw) To: Adam Ford Cc: Ahmad Fatoum, arm-soc, devicetree, Peng Fan, Alice Guo, Shawn Guo, Sascha Hauer, Joakim Zhang, Krzysztof Kozlowski, Linux Kernel Mailing List, Rob Herring, NXP Linux Team, Pengutronix Kernel Team, Fabio Estevam, Lucas Stach On Tue, Nov 02, 2021 at 07:23:06AM -0500, Adam Ford wrote: > The upcoming 5.16 kernel will have a new blk-ctrl driver which will > work in conjunction with the GPC. You can see it in linux-next [1], > and I would expect it to be present in 5.16-rc1 once the merge is > done. > > In [1], Look for : > > disp_blk_ctrl: blk-ctrl@32e28000 { > compatible = "fsl,imx8mm-disp-blk-ctrl", "syscon"; > > It creates a bunch of virtual power domains which are effectively the > resets for the VPU, CSI, DSI, and LCDIF [2]. > > Basically, to pull the respective device out of reset, you'd reference > them using power-domains. I have an RFC patch for the CSI located [3] > which should bring the GPC power domain up, then take the CSI bridge > and MIPI_CSI out of reset using the blk-ctrl. A few of us are still > investigating the CSI bridge and mipi_csi drivers to determine what's > going wrong, but inside that patch, you'll see that we reference > "power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_CSI_BRIDGE>;" and > "power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_MIPI_CSI>;" which > are part of the new blk-ctrl driver @32e2800. Other peripherals like > LCD, DSI, and the VPU's should be able to reference their respective > power domains to activate the corresponding resets after enabling the > proper GPC power domain. Hi Adam, Then is all done right. Using this this new driver/dts node eLCDIF/mipi_dsi module are out of reset. Thanks for the tips. I'm trying to get eLCDIF/mipi_dsi work on mainline. I try to get work - eLCDIF using: mxsfb_drv.c - mipi_dsi using: nwl-dsi.c What do you think about? You think that can be a good way ( taking imx8mq as reference )? Let me know. Thanks, Tommaso > > > [1] - https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/arch/arm64/boot/dts/freescale/imx8mm.dtsi?h=next-20211102 > [2] - https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/include/dt-bindings/power/imx8mm-power.h?h=next-20211102 > [3] - https://patchwork.kernel.org/project/linux-arm-kernel/patch/20211023203457.1217821-2-aford173@gmail.com/ > > On Tue, Nov 2, 2021 at 6:59 AM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > > > On Mon, Nov 01, 2021 at 11:22:21PM -0500, Adam Ford wrote: > > > On Mon, Nov 1, 2021 at 5:58 PM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > > > > > > > On Mon, Nov 01, 2021 at 11:35:49PM +0100, Ahmad Fatoum wrote: > > > > > Hello Tommaso, > > > > > > > > > > On 01.11.21 23:28, Tommaso Merciai wrote: > > > > > > Add system controller node for registers of module Display Block Control > > > > > > (DISPLAY_BLK_CTRL, base address: 0x32e28000). > > > > > > The DISPLAY_BLK_CTRL module contains general purpose registers (GPRs), > > > > > > which control varied features of the associated peripherals. > > > > > > Reference: IMX8MMRM Rev. 3, 11/2020, p 3897 > > > > > > --- > > > > > > arch/arm64/boot/dts/freescale/imx8mm.dtsi | 5 +++++ > > > > > > 1 file changed, 5 insertions(+) > > > > > > > > > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > > > index 2f632e8ca388..3e496b457e1a 100644 > > > > > > --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > > > +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > > > @@ -961,6 +961,11 @@ aips4: bus@32c00000 { > > > > > > #size-cells = <1>; > > > > > > ranges = <0x32c00000 0x32c00000 0x400000>; > > > > > > > > > > > > + dispmix_gpr: display-gpr@32e28000 { > > > > > > + compatible = "fsl, imx8mm-iomuxc-gpr", "syscon"; > > > > > > > > > > Please read vendor patches before submitting them. The space > > > > > is out-of-place in the compatible and the compatible is wrong: > > > > > This doesn't look like a i.MX8MM pin controller. > > > > > > > > > > Cheers, > > > > > Ahmad > > > > > > > > Hi Ahmad, > > > > Thanks for your review. Do you think this is correct? > > > > > > > > compatible = "fsl,imx8mm-dispmix-gpr", "syscon"; > > > > > > > > Let me know. > > > > > > There was already a driver created for the blk-ctrl stuff and it has a > > > device tree binding at 32e28000. It's tied into the power-domain > > > system, so if you want to enable the csi, dsi, or lcd, etc. you can > > > just reference the blt-ctrl power domain index, and it enables the > > > device's gpc power domain and takes the corresponding device out of > > > reset. > > > > Hi Adam, > > You mean using the gpcv2.c driver? > > > > drivers/soc/imx/gpcv2.c > > > > With the following node, to put out of reset eLCDIF and mipi_dsi: > > > > gpc: gpc@303a0000 { > > compatible = "fsl,imx8mm-gpc"; > > reg = <0x303a0000 0x10000>; > > interrupts = <GIC_SPI 87 IRQ_TYPE_LEVEL_HIGH>; > > interrupt-parent = <&gic>; > > interrupt-controller; > > #interrupt-cells = <3>; > > > > pgc { > > > > #address-cells = <1>; > > #size-cells = <0>; > > pgc_mipi: power-domain@0 { > > #power-domain-cells = <0>; > > reg = <IMX8M_POWER_DOMAIN_MIPI>; > > }; > > > > pgc_disp: power-domain@7 { > > #power-domain-cells = <0>; > > reg = <IMX8M_POWER_DOMAIN_DISP>; > > }; > > }; > > }; > > > > Let me know. > > > > Thanks, > > Tommaso > > > > > > > > adam > > > > > > > > Thanks, > > > > Tommaso > > > > > > > > > > > > > > > + reg = <0x32e28000 0x100>; > > > > > > + }; > > > > > > + > > > > > > usbotg1: usb@32e40000 { > > > > > > compatible = "fsl,imx8mm-usb", "fsl,imx7d-usb"; > > > > > > reg = <0x32e40000 0x200>; > > > > > > > > > > > > > > > > > > > > > -- > > > > > Pengutronix e.K. | | > > > > > Steuerwalder Str. 21 | http://www.pengutronix.de/ | > > > > > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > > > > > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs @ 2021-11-02 15:47 ` Tommaso Merciai 0 siblings, 0 replies; 20+ messages in thread From: Tommaso Merciai @ 2021-11-02 15:47 UTC (permalink / raw) To: Adam Ford Cc: Ahmad Fatoum, arm-soc, devicetree, Peng Fan, Alice Guo, Shawn Guo, Sascha Hauer, Joakim Zhang, Krzysztof Kozlowski, Linux Kernel Mailing List, Rob Herring, NXP Linux Team, Pengutronix Kernel Team, Fabio Estevam, Lucas Stach On Tue, Nov 02, 2021 at 07:23:06AM -0500, Adam Ford wrote: > The upcoming 5.16 kernel will have a new blk-ctrl driver which will > work in conjunction with the GPC. You can see it in linux-next [1], > and I would expect it to be present in 5.16-rc1 once the merge is > done. > > In [1], Look for : > > disp_blk_ctrl: blk-ctrl@32e28000 { > compatible = "fsl,imx8mm-disp-blk-ctrl", "syscon"; > > It creates a bunch of virtual power domains which are effectively the > resets for the VPU, CSI, DSI, and LCDIF [2]. > > Basically, to pull the respective device out of reset, you'd reference > them using power-domains. I have an RFC patch for the CSI located [3] > which should bring the GPC power domain up, then take the CSI bridge > and MIPI_CSI out of reset using the blk-ctrl. A few of us are still > investigating the CSI bridge and mipi_csi drivers to determine what's > going wrong, but inside that patch, you'll see that we reference > "power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_CSI_BRIDGE>;" and > "power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_MIPI_CSI>;" which > are part of the new blk-ctrl driver @32e2800. Other peripherals like > LCD, DSI, and the VPU's should be able to reference their respective > power domains to activate the corresponding resets after enabling the > proper GPC power domain. Hi Adam, Then is all done right. Using this this new driver/dts node eLCDIF/mipi_dsi module are out of reset. Thanks for the tips. I'm trying to get eLCDIF/mipi_dsi work on mainline. I try to get work - eLCDIF using: mxsfb_drv.c - mipi_dsi using: nwl-dsi.c What do you think about? You think that can be a good way ( taking imx8mq as reference )? Let me know. Thanks, Tommaso > > > [1] - https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/arch/arm64/boot/dts/freescale/imx8mm.dtsi?h=next-20211102 > [2] - https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/include/dt-bindings/power/imx8mm-power.h?h=next-20211102 > [3] - https://patchwork.kernel.org/project/linux-arm-kernel/patch/20211023203457.1217821-2-aford173@gmail.com/ > > On Tue, Nov 2, 2021 at 6:59 AM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > > > On Mon, Nov 01, 2021 at 11:22:21PM -0500, Adam Ford wrote: > > > On Mon, Nov 1, 2021 at 5:58 PM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > > > > > > > On Mon, Nov 01, 2021 at 11:35:49PM +0100, Ahmad Fatoum wrote: > > > > > Hello Tommaso, > > > > > > > > > > On 01.11.21 23:28, Tommaso Merciai wrote: > > > > > > Add system controller node for registers of module Display Block Control > > > > > > (DISPLAY_BLK_CTRL, base address: 0x32e28000). > > > > > > The DISPLAY_BLK_CTRL module contains general purpose registers (GPRs), > > > > > > which control varied features of the associated peripherals. > > > > > > Reference: IMX8MMRM Rev. 3, 11/2020, p 3897 > > > > > > --- > > > > > > arch/arm64/boot/dts/freescale/imx8mm.dtsi | 5 +++++ > > > > > > 1 file changed, 5 insertions(+) > > > > > > > > > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > > > index 2f632e8ca388..3e496b457e1a 100644 > > > > > > --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > > > +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > > > @@ -961,6 +961,11 @@ aips4: bus@32c00000 { > > > > > > #size-cells = <1>; > > > > > > ranges = <0x32c00000 0x32c00000 0x400000>; > > > > > > > > > > > > + dispmix_gpr: display-gpr@32e28000 { > > > > > > + compatible = "fsl, imx8mm-iomuxc-gpr", "syscon"; > > > > > > > > > > Please read vendor patches before submitting them. The space > > > > > is out-of-place in the compatible and the compatible is wrong: > > > > > This doesn't look like a i.MX8MM pin controller. > > > > > > > > > > Cheers, > > > > > Ahmad > > > > > > > > Hi Ahmad, > > > > Thanks for your review. Do you think this is correct? > > > > > > > > compatible = "fsl,imx8mm-dispmix-gpr", "syscon"; > > > > > > > > Let me know. > > > > > > There was already a driver created for the blk-ctrl stuff and it has a > > > device tree binding at 32e28000. It's tied into the power-domain > > > system, so if you want to enable the csi, dsi, or lcd, etc. you can > > > just reference the blt-ctrl power domain index, and it enables the > > > device's gpc power domain and takes the corresponding device out of > > > reset. > > > > Hi Adam, > > You mean using the gpcv2.c driver? > > > > drivers/soc/imx/gpcv2.c > > > > With the following node, to put out of reset eLCDIF and mipi_dsi: > > > > gpc: gpc@303a0000 { > > compatible = "fsl,imx8mm-gpc"; > > reg = <0x303a0000 0x10000>; > > interrupts = <GIC_SPI 87 IRQ_TYPE_LEVEL_HIGH>; > > interrupt-parent = <&gic>; > > interrupt-controller; > > #interrupt-cells = <3>; > > > > pgc { > > > > #address-cells = <1>; > > #size-cells = <0>; > > pgc_mipi: power-domain@0 { > > #power-domain-cells = <0>; > > reg = <IMX8M_POWER_DOMAIN_MIPI>; > > }; > > > > pgc_disp: power-domain@7 { > > #power-domain-cells = <0>; > > reg = <IMX8M_POWER_DOMAIN_DISP>; > > }; > > }; > > }; > > > > Let me know. > > > > Thanks, > > Tommaso > > > > > > > > adam > > > > > > > > Thanks, > > > > Tommaso > > > > > > > > > > > > > > > + reg = <0x32e28000 0x100>; > > > > > > + }; > > > > > > + > > > > > > usbotg1: usb@32e40000 { > > > > > > compatible = "fsl,imx8mm-usb", "fsl,imx7d-usb"; > > > > > > reg = <0x32e40000 0x200>; > > > > > > > > > > > > > > > > > > > > > -- > > > > > Pengutronix e.K. | | > > > > > Steuerwalder Str. 21 | http://www.pengutronix.de/ | > > > > > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > > > > > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ 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] 20+ messages in thread
* Re: [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs 2021-11-02 15:47 ` Tommaso Merciai @ 2021-11-02 16:07 ` Adam Ford -1 siblings, 0 replies; 20+ messages in thread From: Adam Ford @ 2021-11-02 16:07 UTC (permalink / raw) To: Tommaso Merciai Cc: Ahmad Fatoum, arm-soc, devicetree, Peng Fan, Alice Guo, Shawn Guo, Sascha Hauer, Joakim Zhang, Krzysztof Kozlowski, Linux Kernel Mailing List, Rob Herring, NXP Linux Team, Pengutronix Kernel Team, Fabio Estevam, Lucas Stach On Tue, Nov 2, 2021 at 10:47 AM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > On Tue, Nov 02, 2021 at 07:23:06AM -0500, Adam Ford wrote: > > The upcoming 5.16 kernel will have a new blk-ctrl driver which will > > work in conjunction with the GPC. You can see it in linux-next [1], > > and I would expect it to be present in 5.16-rc1 once the merge is > > done. > > > > In [1], Look for : > > > > disp_blk_ctrl: blk-ctrl@32e28000 { > > compatible = "fsl,imx8mm-disp-blk-ctrl", "syscon"; > > > > It creates a bunch of virtual power domains which are effectively the > > resets for the VPU, CSI, DSI, and LCDIF [2]. > > > > Basically, to pull the respective device out of reset, you'd reference > > them using power-domains. I have an RFC patch for the CSI located [3] > > which should bring the GPC power domain up, then take the CSI bridge > > and MIPI_CSI out of reset using the blk-ctrl. A few of us are still > > investigating the CSI bridge and mipi_csi drivers to determine what's > > going wrong, but inside that patch, you'll see that we reference > > "power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_CSI_BRIDGE>;" and > > "power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_MIPI_CSI>;" which > > are part of the new blk-ctrl driver @32e2800. Other peripherals like > > LCD, DSI, and the VPU's should be able to reference their respective > > power domains to activate the corresponding resets after enabling the > > proper GPC power domain. > > Hi Adam, > Then is all done right. Using this this new driver/dts node eLCDIF/mipi_dsi > module are out of reset. Thanks for the tips. I'm trying to get eLCDIF/mipi_dsi > work on mainline. I try to get work > > - eLCDIF using: mxsfb_drv.c > - mipi_dsi using: nwl-dsi.c > > What do you think about? You think that can be a good way ( taking > imx8mq as reference )? The DSI controller for the 8MM and 8MN is not the same as the DSI controller on the 8MQ, but the LCDIF controller should be compatible. There have been several attempts to support the 8MM DSI, but none of them have been accepted for various reasons. The latest was found here [1], but others [2] and [3] , when used together, do something similar. If memory serves, the main issue has to do with the fact that the DSIM controller in the 8MM and 8MN is also present in one of the Samsung processors, and the goal is to rework those drivers so we'll have one driver that supports both Samsung progressors and NXP instead of having two duplicate drivers doing the same thing. When whatever driver is chosen is ready, it'll be likely that the LCDIF will use power-domains = <&disp_blk_ctrl MX8MM_DISPBLK_PD_LCDIF> and the DSI node will use power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_MIPI_DSI> to pull their respective devices out of reset and enable the gpc. [1] - https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=510489&archive=both&state=* [2] - https://patchwork.kernel.org/project/dri-devel/list/?series=347439&archive=both&state=* [3] - https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=359775&archive=both&state=* > > Let me know. > Thanks, > > Tommaso > > > > > > [1] - https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/arch/arm64/boot/dts/freescale/imx8mm.dtsi?h=next-20211102 > > [2] - https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/include/dt-bindings/power/imx8mm-power.h?h=next-20211102 > > [3] - https://patchwork.kernel.org/project/linux-arm-kernel/patch/20211023203457.1217821-2-aford173@gmail.com/ > > > > On Tue, Nov 2, 2021 at 6:59 AM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > > > > > On Mon, Nov 01, 2021 at 11:22:21PM -0500, Adam Ford wrote: > > > > On Mon, Nov 1, 2021 at 5:58 PM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > > > > > > > > > On Mon, Nov 01, 2021 at 11:35:49PM +0100, Ahmad Fatoum wrote: > > > > > > Hello Tommaso, > > > > > > > > > > > > On 01.11.21 23:28, Tommaso Merciai wrote: > > > > > > > Add system controller node for registers of module Display Block Control > > > > > > > (DISPLAY_BLK_CTRL, base address: 0x32e28000). > > > > > > > The DISPLAY_BLK_CTRL module contains general purpose registers (GPRs), > > > > > > > which control varied features of the associated peripherals. > > > > > > > Reference: IMX8MMRM Rev. 3, 11/2020, p 3897 > > > > > > > --- > > > > > > > arch/arm64/boot/dts/freescale/imx8mm.dtsi | 5 +++++ > > > > > > > 1 file changed, 5 insertions(+) > > > > > > > > > > > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > > > > index 2f632e8ca388..3e496b457e1a 100644 > > > > > > > --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > > > > +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > > > > @@ -961,6 +961,11 @@ aips4: bus@32c00000 { > > > > > > > #size-cells = <1>; > > > > > > > ranges = <0x32c00000 0x32c00000 0x400000>; > > > > > > > > > > > > > > + dispmix_gpr: display-gpr@32e28000 { > > > > > > > + compatible = "fsl, imx8mm-iomuxc-gpr", "syscon"; > > > > > > > > > > > > Please read vendor patches before submitting them. The space > > > > > > is out-of-place in the compatible and the compatible is wrong: > > > > > > This doesn't look like a i.MX8MM pin controller. > > > > > > > > > > > > Cheers, > > > > > > Ahmad > > > > > > > > > > Hi Ahmad, > > > > > Thanks for your review. Do you think this is correct? > > > > > > > > > > compatible = "fsl,imx8mm-dispmix-gpr", "syscon"; > > > > > > > > > > Let me know. > > > > > > > > There was already a driver created for the blk-ctrl stuff and it has a > > > > device tree binding at 32e28000. It's tied into the power-domain > > > > system, so if you want to enable the csi, dsi, or lcd, etc. you can > > > > just reference the blt-ctrl power domain index, and it enables the > > > > device's gpc power domain and takes the corresponding device out of > > > > reset. > > > > > > Hi Adam, > > > You mean using the gpcv2.c driver? > > > > > > drivers/soc/imx/gpcv2.c > > > > > > With the following node, to put out of reset eLCDIF and mipi_dsi: > > > > > > gpc: gpc@303a0000 { > > > compatible = "fsl,imx8mm-gpc"; > > > reg = <0x303a0000 0x10000>; > > > interrupts = <GIC_SPI 87 IRQ_TYPE_LEVEL_HIGH>; > > > interrupt-parent = <&gic>; > > > interrupt-controller; > > > #interrupt-cells = <3>; > > > > > > pgc { > > > > > > #address-cells = <1>; > > > #size-cells = <0>; > > > pgc_mipi: power-domain@0 { > > > #power-domain-cells = <0>; > > > reg = <IMX8M_POWER_DOMAIN_MIPI>; > > > }; > > > > > > pgc_disp: power-domain@7 { > > > #power-domain-cells = <0>; > > > reg = <IMX8M_POWER_DOMAIN_DISP>; > > > }; > > > }; > > > }; > > > > > > Let me know. > > > > > > Thanks, > > > Tommaso > > > > > > > > > > > adam > > > > > > > > > > Thanks, > > > > > Tommaso > > > > > > > > > > > > > > > > > > + reg = <0x32e28000 0x100>; > > > > > > > + }; > > > > > > > + > > > > > > > usbotg1: usb@32e40000 { > > > > > > > compatible = "fsl,imx8mm-usb", "fsl,imx7d-usb"; > > > > > > > reg = <0x32e40000 0x200>; > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > Pengutronix e.K. | | > > > > > > Steuerwalder Str. 21 | http://www.pengutronix.de/ | > > > > > > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > > > > > > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ 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] 20+ messages in thread
* Re: [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs @ 2021-11-02 16:07 ` Adam Ford 0 siblings, 0 replies; 20+ messages in thread From: Adam Ford @ 2021-11-02 16:07 UTC (permalink / raw) To: Tommaso Merciai Cc: Ahmad Fatoum, arm-soc, devicetree, Peng Fan, Alice Guo, Shawn Guo, Sascha Hauer, Joakim Zhang, Krzysztof Kozlowski, Linux Kernel Mailing List, Rob Herring, NXP Linux Team, Pengutronix Kernel Team, Fabio Estevam, Lucas Stach On Tue, Nov 2, 2021 at 10:47 AM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > On Tue, Nov 02, 2021 at 07:23:06AM -0500, Adam Ford wrote: > > The upcoming 5.16 kernel will have a new blk-ctrl driver which will > > work in conjunction with the GPC. You can see it in linux-next [1], > > and I would expect it to be present in 5.16-rc1 once the merge is > > done. > > > > In [1], Look for : > > > > disp_blk_ctrl: blk-ctrl@32e28000 { > > compatible = "fsl,imx8mm-disp-blk-ctrl", "syscon"; > > > > It creates a bunch of virtual power domains which are effectively the > > resets for the VPU, CSI, DSI, and LCDIF [2]. > > > > Basically, to pull the respective device out of reset, you'd reference > > them using power-domains. I have an RFC patch for the CSI located [3] > > which should bring the GPC power domain up, then take the CSI bridge > > and MIPI_CSI out of reset using the blk-ctrl. A few of us are still > > investigating the CSI bridge and mipi_csi drivers to determine what's > > going wrong, but inside that patch, you'll see that we reference > > "power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_CSI_BRIDGE>;" and > > "power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_MIPI_CSI>;" which > > are part of the new blk-ctrl driver @32e2800. Other peripherals like > > LCD, DSI, and the VPU's should be able to reference their respective > > power domains to activate the corresponding resets after enabling the > > proper GPC power domain. > > Hi Adam, > Then is all done right. Using this this new driver/dts node eLCDIF/mipi_dsi > module are out of reset. Thanks for the tips. I'm trying to get eLCDIF/mipi_dsi > work on mainline. I try to get work > > - eLCDIF using: mxsfb_drv.c > - mipi_dsi using: nwl-dsi.c > > What do you think about? You think that can be a good way ( taking > imx8mq as reference )? The DSI controller for the 8MM and 8MN is not the same as the DSI controller on the 8MQ, but the LCDIF controller should be compatible. There have been several attempts to support the 8MM DSI, but none of them have been accepted for various reasons. The latest was found here [1], but others [2] and [3] , when used together, do something similar. If memory serves, the main issue has to do with the fact that the DSIM controller in the 8MM and 8MN is also present in one of the Samsung processors, and the goal is to rework those drivers so we'll have one driver that supports both Samsung progressors and NXP instead of having two duplicate drivers doing the same thing. When whatever driver is chosen is ready, it'll be likely that the LCDIF will use power-domains = <&disp_blk_ctrl MX8MM_DISPBLK_PD_LCDIF> and the DSI node will use power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_MIPI_DSI> to pull their respective devices out of reset and enable the gpc. [1] - https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=510489&archive=both&state=* [2] - https://patchwork.kernel.org/project/dri-devel/list/?series=347439&archive=both&state=* [3] - https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=359775&archive=both&state=* > > Let me know. > Thanks, > > Tommaso > > > > > > [1] - https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/arch/arm64/boot/dts/freescale/imx8mm.dtsi?h=next-20211102 > > [2] - https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/include/dt-bindings/power/imx8mm-power.h?h=next-20211102 > > [3] - https://patchwork.kernel.org/project/linux-arm-kernel/patch/20211023203457.1217821-2-aford173@gmail.com/ > > > > On Tue, Nov 2, 2021 at 6:59 AM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > > > > > On Mon, Nov 01, 2021 at 11:22:21PM -0500, Adam Ford wrote: > > > > On Mon, Nov 1, 2021 at 5:58 PM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > > > > > > > > > On Mon, Nov 01, 2021 at 11:35:49PM +0100, Ahmad Fatoum wrote: > > > > > > Hello Tommaso, > > > > > > > > > > > > On 01.11.21 23:28, Tommaso Merciai wrote: > > > > > > > Add system controller node for registers of module Display Block Control > > > > > > > (DISPLAY_BLK_CTRL, base address: 0x32e28000). > > > > > > > The DISPLAY_BLK_CTRL module contains general purpose registers (GPRs), > > > > > > > which control varied features of the associated peripherals. > > > > > > > Reference: IMX8MMRM Rev. 3, 11/2020, p 3897 > > > > > > > --- > > > > > > > arch/arm64/boot/dts/freescale/imx8mm.dtsi | 5 +++++ > > > > > > > 1 file changed, 5 insertions(+) > > > > > > > > > > > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > > > > index 2f632e8ca388..3e496b457e1a 100644 > > > > > > > --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > > > > +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > > > > > > @@ -961,6 +961,11 @@ aips4: bus@32c00000 { > > > > > > > #size-cells = <1>; > > > > > > > ranges = <0x32c00000 0x32c00000 0x400000>; > > > > > > > > > > > > > > + dispmix_gpr: display-gpr@32e28000 { > > > > > > > + compatible = "fsl, imx8mm-iomuxc-gpr", "syscon"; > > > > > > > > > > > > Please read vendor patches before submitting them. The space > > > > > > is out-of-place in the compatible and the compatible is wrong: > > > > > > This doesn't look like a i.MX8MM pin controller. > > > > > > > > > > > > Cheers, > > > > > > Ahmad > > > > > > > > > > Hi Ahmad, > > > > > Thanks for your review. Do you think this is correct? > > > > > > > > > > compatible = "fsl,imx8mm-dispmix-gpr", "syscon"; > > > > > > > > > > Let me know. > > > > > > > > There was already a driver created for the blk-ctrl stuff and it has a > > > > device tree binding at 32e28000. It's tied into the power-domain > > > > system, so if you want to enable the csi, dsi, or lcd, etc. you can > > > > just reference the blt-ctrl power domain index, and it enables the > > > > device's gpc power domain and takes the corresponding device out of > > > > reset. > > > > > > Hi Adam, > > > You mean using the gpcv2.c driver? > > > > > > drivers/soc/imx/gpcv2.c > > > > > > With the following node, to put out of reset eLCDIF and mipi_dsi: > > > > > > gpc: gpc@303a0000 { > > > compatible = "fsl,imx8mm-gpc"; > > > reg = <0x303a0000 0x10000>; > > > interrupts = <GIC_SPI 87 IRQ_TYPE_LEVEL_HIGH>; > > > interrupt-parent = <&gic>; > > > interrupt-controller; > > > #interrupt-cells = <3>; > > > > > > pgc { > > > > > > #address-cells = <1>; > > > #size-cells = <0>; > > > pgc_mipi: power-domain@0 { > > > #power-domain-cells = <0>; > > > reg = <IMX8M_POWER_DOMAIN_MIPI>; > > > }; > > > > > > pgc_disp: power-domain@7 { > > > #power-domain-cells = <0>; > > > reg = <IMX8M_POWER_DOMAIN_DISP>; > > > }; > > > }; > > > }; > > > > > > Let me know. > > > > > > Thanks, > > > Tommaso > > > > > > > > > > > adam > > > > > > > > > > Thanks, > > > > > Tommaso > > > > > > > > > > > > > > > > > > + reg = <0x32e28000 0x100>; > > > > > > > + }; > > > > > > > + > > > > > > > usbotg1: usb@32e40000 { > > > > > > > compatible = "fsl,imx8mm-usb", "fsl,imx7d-usb"; > > > > > > > reg = <0x32e40000 0x200>; > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > Pengutronix e.K. | | > > > > > > Steuerwalder Str. 21 | http://www.pengutronix.de/ | > > > > > > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > > > > > > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs 2021-11-02 16:07 ` Adam Ford @ 2021-11-02 18:18 ` Tim Harvey -1 siblings, 0 replies; 20+ messages in thread From: Tim Harvey @ 2021-11-02 18:18 UTC (permalink / raw) To: Adam Ford Cc: Tommaso Merciai, Ahmad Fatoum, arm-soc, devicetree, Peng Fan, Alice Guo, Shawn Guo, Sascha Hauer, Joakim Zhang, Krzysztof Kozlowski, Linux Kernel Mailing List, Rob Herring, NXP Linux Team, Pengutronix Kernel Team, Fabio Estevam, Lucas Stach, Schrempf Frieder, Jagan Teki, Michael Tretter On Tue, Nov 2, 2021 at 9:08 AM Adam Ford <aford173@gmail.com> wrote: > > On Tue, Nov 2, 2021 at 10:47 AM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > > > On Tue, Nov 02, 2021 at 07:23:06AM -0500, Adam Ford wrote: > > > The upcoming 5.16 kernel will have a new blk-ctrl driver which will > > > work in conjunction with the GPC. You can see it in linux-next [1], > > > and I would expect it to be present in 5.16-rc1 once the merge is > > > done. > > > > > > In [1], Look for : > > > > > > disp_blk_ctrl: blk-ctrl@32e28000 { > > > compatible = "fsl,imx8mm-disp-blk-ctrl", "syscon"; > > > > > > It creates a bunch of virtual power domains which are effectively the > > > resets for the VPU, CSI, DSI, and LCDIF [2]. > > > > > > Basically, to pull the respective device out of reset, you'd reference > > > them using power-domains. I have an RFC patch for the CSI located [3] > > > which should bring the GPC power domain up, then take the CSI bridge > > > and MIPI_CSI out of reset using the blk-ctrl. A few of us are still > > > investigating the CSI bridge and mipi_csi drivers to determine what's > > > going wrong, but inside that patch, you'll see that we reference > > > "power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_CSI_BRIDGE>;" and > > > "power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_MIPI_CSI>;" which > > > are part of the new blk-ctrl driver @32e2800. Other peripherals like > > > LCD, DSI, and the VPU's should be able to reference their respective > > > power domains to activate the corresponding resets after enabling the > > > proper GPC power domain. > > > > Hi Adam, > > Then is all done right. Using this this new driver/dts node eLCDIF/mipi_dsi > > module are out of reset. Thanks for the tips. I'm trying to get eLCDIF/mipi_dsi > > work on mainline. I try to get work > > > > - eLCDIF using: mxsfb_drv.c > > - mipi_dsi using: nwl-dsi.c > > > > What do you think about? You think that can be a good way ( taking > > imx8mq as reference )? > > The DSI controller for the 8MM and 8MN is not the same as the DSI > controller on the 8MQ, but the LCDIF controller should be compatible. > > There have been several attempts to support the 8MM DSI, but none of > them have been accepted for various reasons. > > The latest was found here [1], but others [2] and [3] , when used > together, do something similar. > > If memory serves, the main issue has to do with the fact that the DSIM > controller in the 8MM and 8MN is also present in one of the Samsung > processors, and the goal is to rework those drivers so we'll have one > driver that supports both Samsung progressors and NXP instead of > having two duplicate drivers doing the same thing. When whatever > driver is chosen is ready, it'll be likely that the LCDIF will use > power-domains = <&disp_blk_ctrl MX8MM_DISPBLK_PD_LCDIF> and the DSI > node will use power-domains = <&disp_blk_ctrl > IMX8MM_DISPBLK_PD_MIPI_DSI> to pull their respective devices out of > reset and enable the gpc. > > > [1] - https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=510489&archive=both&state=* > [2] - https://patchwork.kernel.org/project/dri-devel/list/?series=347439&archive=both&state=* > [3] - https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=359775&archive=both&state=* Adam, Thanks for the good summary... I was just putting this info together as well. I'm also interested to see if anyone has made progress on IMX8MM MIPI DSI display. Now that blk-ctl and most of the dt bindings have been merged for 5.16 I think we are just down to the drm/exynos driver issue. Added Frieder, Jagan, and Michael to the thread. Best Regards, Tim ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs @ 2021-11-02 18:18 ` Tim Harvey 0 siblings, 0 replies; 20+ messages in thread From: Tim Harvey @ 2021-11-02 18:18 UTC (permalink / raw) To: Adam Ford Cc: Tommaso Merciai, Ahmad Fatoum, arm-soc, devicetree, Peng Fan, Alice Guo, Shawn Guo, Sascha Hauer, Joakim Zhang, Krzysztof Kozlowski, Linux Kernel Mailing List, Rob Herring, NXP Linux Team, Pengutronix Kernel Team, Fabio Estevam, Lucas Stach, Schrempf Frieder, Jagan Teki, Michael Tretter On Tue, Nov 2, 2021 at 9:08 AM Adam Ford <aford173@gmail.com> wrote: > > On Tue, Nov 2, 2021 at 10:47 AM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > > > On Tue, Nov 02, 2021 at 07:23:06AM -0500, Adam Ford wrote: > > > The upcoming 5.16 kernel will have a new blk-ctrl driver which will > > > work in conjunction with the GPC. You can see it in linux-next [1], > > > and I would expect it to be present in 5.16-rc1 once the merge is > > > done. > > > > > > In [1], Look for : > > > > > > disp_blk_ctrl: blk-ctrl@32e28000 { > > > compatible = "fsl,imx8mm-disp-blk-ctrl", "syscon"; > > > > > > It creates a bunch of virtual power domains which are effectively the > > > resets for the VPU, CSI, DSI, and LCDIF [2]. > > > > > > Basically, to pull the respective device out of reset, you'd reference > > > them using power-domains. I have an RFC patch for the CSI located [3] > > > which should bring the GPC power domain up, then take the CSI bridge > > > and MIPI_CSI out of reset using the blk-ctrl. A few of us are still > > > investigating the CSI bridge and mipi_csi drivers to determine what's > > > going wrong, but inside that patch, you'll see that we reference > > > "power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_CSI_BRIDGE>;" and > > > "power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_MIPI_CSI>;" which > > > are part of the new blk-ctrl driver @32e2800. Other peripherals like > > > LCD, DSI, and the VPU's should be able to reference their respective > > > power domains to activate the corresponding resets after enabling the > > > proper GPC power domain. > > > > Hi Adam, > > Then is all done right. Using this this new driver/dts node eLCDIF/mipi_dsi > > module are out of reset. Thanks for the tips. I'm trying to get eLCDIF/mipi_dsi > > work on mainline. I try to get work > > > > - eLCDIF using: mxsfb_drv.c > > - mipi_dsi using: nwl-dsi.c > > > > What do you think about? You think that can be a good way ( taking > > imx8mq as reference )? > > The DSI controller for the 8MM and 8MN is not the same as the DSI > controller on the 8MQ, but the LCDIF controller should be compatible. > > There have been several attempts to support the 8MM DSI, but none of > them have been accepted for various reasons. > > The latest was found here [1], but others [2] and [3] , when used > together, do something similar. > > If memory serves, the main issue has to do with the fact that the DSIM > controller in the 8MM and 8MN is also present in one of the Samsung > processors, and the goal is to rework those drivers so we'll have one > driver that supports both Samsung progressors and NXP instead of > having two duplicate drivers doing the same thing. When whatever > driver is chosen is ready, it'll be likely that the LCDIF will use > power-domains = <&disp_blk_ctrl MX8MM_DISPBLK_PD_LCDIF> and the DSI > node will use power-domains = <&disp_blk_ctrl > IMX8MM_DISPBLK_PD_MIPI_DSI> to pull their respective devices out of > reset and enable the gpc. > > > [1] - https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=510489&archive=both&state=* > [2] - https://patchwork.kernel.org/project/dri-devel/list/?series=347439&archive=both&state=* > [3] - https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=359775&archive=both&state=* Adam, Thanks for the good summary... I was just putting this info together as well. I'm also interested to see if anyone has made progress on IMX8MM MIPI DSI display. Now that blk-ctl and most of the dt bindings have been merged for 5.16 I think we are just down to the drm/exynos driver issue. Added Frieder, Jagan, and Michael to the thread. Best Regards, Tim _______________________________________________ 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] 20+ messages in thread
* Re: [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs 2021-11-02 18:18 ` Tim Harvey @ 2021-11-02 22:43 ` Tommaso Merciai -1 siblings, 0 replies; 20+ messages in thread From: Tommaso Merciai @ 2021-11-02 22:43 UTC (permalink / raw) To: Tim Harvey Cc: Adam Ford, Ahmad Fatoum, arm-soc, devicetree, Peng Fan, Alice Guo, Shawn Guo, Sascha Hauer, Joakim Zhang, Krzysztof Kozlowski, Linux Kernel Mailing List, Rob Herring, NXP Linux Team, Pengutronix Kernel Team, Fabio Estevam, Lucas Stach, Schrempf Frieder, Jagan Teki, Michael Tretter On Tue, Nov 02, 2021 at 11:18:53AM -0700, Tim Harvey wrote: > On Tue, Nov 2, 2021 at 9:08 AM Adam Ford <aford173@gmail.com> wrote: > > > > On Tue, Nov 2, 2021 at 10:47 AM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > > > > > On Tue, Nov 02, 2021 at 07:23:06AM -0500, Adam Ford wrote: > > > > The upcoming 5.16 kernel will have a new blk-ctrl driver which will > > > > work in conjunction with the GPC. You can see it in linux-next [1], > > > > and I would expect it to be present in 5.16-rc1 once the merge is > > > > done. > > > > > > > > In [1], Look for : > > > > > > > > disp_blk_ctrl: blk-ctrl@32e28000 { > > > > compatible = "fsl,imx8mm-disp-blk-ctrl", "syscon"; > > > > > > > > It creates a bunch of virtual power domains which are effectively the > > > > resets for the VPU, CSI, DSI, and LCDIF [2]. > > > > > > > > Basically, to pull the respective device out of reset, you'd reference > > > > them using power-domains. I have an RFC patch for the CSI located [3] > > > > which should bring the GPC power domain up, then take the CSI bridge > > > > and MIPI_CSI out of reset using the blk-ctrl. A few of us are still > > > > investigating the CSI bridge and mipi_csi drivers to determine what's > > > > going wrong, but inside that patch, you'll see that we reference > > > > "power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_CSI_BRIDGE>;" and > > > > "power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_MIPI_CSI>;" which > > > > are part of the new blk-ctrl driver @32e2800. Other peripherals like > > > > LCD, DSI, and the VPU's should be able to reference their respective > > > > power domains to activate the corresponding resets after enabling the > > > > proper GPC power domain. > > > > > > Hi Adam, > > > Then is all done right. Using this this new driver/dts node eLCDIF/mipi_dsi > > > module are out of reset. Thanks for the tips. I'm trying to get eLCDIF/mipi_dsi > > > work on mainline. I try to get work > > > > > > - eLCDIF using: mxsfb_drv.c > > > - mipi_dsi using: nwl-dsi.c > > > > > > What do you think about? You think that can be a good way ( taking > > > imx8mq as reference )? > > > > The DSI controller for the 8MM and 8MN is not the same as the DSI > > controller on the 8MQ, but the LCDIF controller should be compatible. > > > > There have been several attempts to support the 8MM DSI, but none of > > them have been accepted for various reasons. > > > > The latest was found here [1], but others [2] and [3] , when used > > together, do something similar. > > > > If memory serves, the main issue has to do with the fact that the DSIM > > controller in the 8MM and 8MN is also present in one of the Samsung > > processors, and the goal is to rework those drivers so we'll have one > > driver that supports both Samsung progressors and NXP instead of > > having two duplicate drivers doing the same thing. When whatever > > driver is chosen is ready, it'll be likely that the LCDIF will use > > power-domains = <&disp_blk_ctrl MX8MM_DISPBLK_PD_LCDIF> and the DSI > > node will use power-domains = <&disp_blk_ctrl > > IMX8MM_DISPBLK_PD_MIPI_DSI> to pull their respective devices out of > > reset and enable the gpc. > > > > > > [1] - https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=510489&archive=both&state=* > > [2] - https://patchwork.kernel.org/project/dri-devel/list/?series=347439&archive=both&state=* > > [3] - https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=359775&archive=both&state=* > > Adam, > > Thanks for the good summary... I was just putting this info together > as well. I'm also interested to see if anyone has made progress on > IMX8MM MIPI DSI display. Now that blk-ctl and most of the dt bindings > have been merged for 5.16 I think we are just down to the drm/exynos > driver issue. > > Added Frieder, Jagan, and Michael to the thread. > > Best Regards, > > Tim Hi Adam, Thanks again for your explanation. Then, now the main goal is refactoring exynos dsim driver as bridge driver in order to support both imx8mm and exynos SOC. I'll investigate on it. Regards, Thanks ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs @ 2021-11-02 22:43 ` Tommaso Merciai 0 siblings, 0 replies; 20+ messages in thread From: Tommaso Merciai @ 2021-11-02 22:43 UTC (permalink / raw) To: Tim Harvey Cc: Adam Ford, Ahmad Fatoum, arm-soc, devicetree, Peng Fan, Alice Guo, Shawn Guo, Sascha Hauer, Joakim Zhang, Krzysztof Kozlowski, Linux Kernel Mailing List, Rob Herring, NXP Linux Team, Pengutronix Kernel Team, Fabio Estevam, Lucas Stach, Schrempf Frieder, Jagan Teki, Michael Tretter On Tue, Nov 02, 2021 at 11:18:53AM -0700, Tim Harvey wrote: > On Tue, Nov 2, 2021 at 9:08 AM Adam Ford <aford173@gmail.com> wrote: > > > > On Tue, Nov 2, 2021 at 10:47 AM Tommaso Merciai <tomm.merciai@gmail.com> wrote: > > > > > > On Tue, Nov 02, 2021 at 07:23:06AM -0500, Adam Ford wrote: > > > > The upcoming 5.16 kernel will have a new blk-ctrl driver which will > > > > work in conjunction with the GPC. You can see it in linux-next [1], > > > > and I would expect it to be present in 5.16-rc1 once the merge is > > > > done. > > > > > > > > In [1], Look for : > > > > > > > > disp_blk_ctrl: blk-ctrl@32e28000 { > > > > compatible = "fsl,imx8mm-disp-blk-ctrl", "syscon"; > > > > > > > > It creates a bunch of virtual power domains which are effectively the > > > > resets for the VPU, CSI, DSI, and LCDIF [2]. > > > > > > > > Basically, to pull the respective device out of reset, you'd reference > > > > them using power-domains. I have an RFC patch for the CSI located [3] > > > > which should bring the GPC power domain up, then take the CSI bridge > > > > and MIPI_CSI out of reset using the blk-ctrl. A few of us are still > > > > investigating the CSI bridge and mipi_csi drivers to determine what's > > > > going wrong, but inside that patch, you'll see that we reference > > > > "power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_CSI_BRIDGE>;" and > > > > "power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_MIPI_CSI>;" which > > > > are part of the new blk-ctrl driver @32e2800. Other peripherals like > > > > LCD, DSI, and the VPU's should be able to reference their respective > > > > power domains to activate the corresponding resets after enabling the > > > > proper GPC power domain. > > > > > > Hi Adam, > > > Then is all done right. Using this this new driver/dts node eLCDIF/mipi_dsi > > > module are out of reset. Thanks for the tips. I'm trying to get eLCDIF/mipi_dsi > > > work on mainline. I try to get work > > > > > > - eLCDIF using: mxsfb_drv.c > > > - mipi_dsi using: nwl-dsi.c > > > > > > What do you think about? You think that can be a good way ( taking > > > imx8mq as reference )? > > > > The DSI controller for the 8MM and 8MN is not the same as the DSI > > controller on the 8MQ, but the LCDIF controller should be compatible. > > > > There have been several attempts to support the 8MM DSI, but none of > > them have been accepted for various reasons. > > > > The latest was found here [1], but others [2] and [3] , when used > > together, do something similar. > > > > If memory serves, the main issue has to do with the fact that the DSIM > > controller in the 8MM and 8MN is also present in one of the Samsung > > processors, and the goal is to rework those drivers so we'll have one > > driver that supports both Samsung progressors and NXP instead of > > having two duplicate drivers doing the same thing. When whatever > > driver is chosen is ready, it'll be likely that the LCDIF will use > > power-domains = <&disp_blk_ctrl MX8MM_DISPBLK_PD_LCDIF> and the DSI > > node will use power-domains = <&disp_blk_ctrl > > IMX8MM_DISPBLK_PD_MIPI_DSI> to pull their respective devices out of > > reset and enable the gpc. > > > > > > [1] - https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=510489&archive=both&state=* > > [2] - https://patchwork.kernel.org/project/dri-devel/list/?series=347439&archive=both&state=* > > [3] - https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=359775&archive=both&state=* > > Adam, > > Thanks for the good summary... I was just putting this info together > as well. I'm also interested to see if anyone has made progress on > IMX8MM MIPI DSI display. Now that blk-ctl and most of the dt bindings > have been merged for 5.16 I think we are just down to the drm/exynos > driver issue. > > Added Frieder, Jagan, and Michael to the thread. > > Best Regards, > > Tim Hi Adam, Thanks again for your explanation. Then, now the main goal is refactoring exynos dsim driver as bridge driver in order to support both imx8mm and exynos SOC. I'll investigate on it. Regards, Thanks _______________________________________________ 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] 20+ messages in thread
end of thread, other threads:[~2021-11-02 22:45 UTC | newest] Thread overview: 20+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-11-01 22:28 [PATCH] arm64: dts: imx8m: add syscon node for display_blk_ctrl module regs Tommaso Merciai 2021-11-01 22:28 ` Tommaso Merciai 2021-11-01 22:35 ` Ahmad Fatoum 2021-11-01 22:35 ` Ahmad Fatoum 2021-11-01 22:58 ` Tommaso Merciai 2021-11-01 22:58 ` Tommaso Merciai 2021-11-02 4:22 ` Adam Ford 2021-11-02 4:22 ` Adam Ford 2021-11-02 11:59 ` Tommaso Merciai 2021-11-02 11:59 ` Tommaso Merciai 2021-11-02 12:23 ` Adam Ford 2021-11-02 12:23 ` Adam Ford 2021-11-02 15:47 ` Tommaso Merciai 2021-11-02 15:47 ` Tommaso Merciai 2021-11-02 16:07 ` Adam Ford 2021-11-02 16:07 ` Adam Ford 2021-11-02 18:18 ` Tim Harvey 2021-11-02 18:18 ` Tim Harvey 2021-11-02 22:43 ` Tommaso Merciai 2021-11-02 22:43 ` Tommaso Merciai
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.