* Re: [PATCH] clk: imx8qxp: Use devm_platform_ioremap_resource() [not found] ` <20190606165546.6675520693@mail.kernel.org> @ 2019-06-06 17:15 ` Leonard Crestez 2019-06-06 17:22 ` Stephen Boyd 2019-06-06 17:57 ` Fabio Estevam 0 siblings, 2 replies; 4+ messages in thread From: Leonard Crestez @ 2019-06-06 17:15 UTC (permalink / raw) To: Stephen Boyd, Fabio Estevam Cc: shawnguo@kernel.org, dl-linux-imx, linux-clk@vger.kernel.org, Anson Huang, Aisheng Dong, kernel@pengutronix.de, devicetree@vger.kernel.org On 06.06.2019 19:55, Stephen Boyd wrote: > Quoting Fabio Estevam (2019-06-06 09:44:43) >> Use devm_platform_ioremap_resource() to simplify the code a bit. >> >> Signed-off-by: Fabio Estevam <festevam@gmail.com> >> --- > > Reviewed-by: Stephen Boyd <sboyd@kernel.org> An extremely similar patch was already submitted and then reverted because it breaks boot: https://patchwork.kernel.org/patch/10908807/ I tested and this new patch also breaks boot. The current imx8 lpcg driver maps entire subsystems at once and if devm_platform_ioremap_resource is used then devices inside the subsystem will fail to probe, including lpuart! The hardware on imx8qxp and related parts (imx8qm) has multiple separate LPCG blocks interspersed between devices. Some refactoring patches were posted by Aisheng to split LPCG into multiple blocks but apparently got stuck in review: https://patchwork.kernel.org/cover/10924029/ https://patchwork.kernel.org/cover/10824443/ https://patchwork.kernel.org/cover/10824537/ There were some disagreements regarding DT bindings for split imx8 clocks, what would it take to move those patches forward? -- Regards, Leonard ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] clk: imx8qxp: Use devm_platform_ioremap_resource() 2019-06-06 17:15 ` [PATCH] clk: imx8qxp: Use devm_platform_ioremap_resource() Leonard Crestez @ 2019-06-06 17:22 ` Stephen Boyd 2019-06-06 17:55 ` Leonard Crestez 2019-06-06 17:57 ` Fabio Estevam 1 sibling, 1 reply; 4+ messages in thread From: Stephen Boyd @ 2019-06-06 17:22 UTC (permalink / raw) To: Fabio Estevam, Leonard Crestez Cc: shawnguo@kernel.org, dl-linux-imx, linux-clk@vger.kernel.org, Anson Huang, Aisheng Dong, kernel@pengutronix.de, devicetree@vger.kernel.org Quoting Leonard Crestez (2019-06-06 10:15:32) > On 06.06.2019 19:55, Stephen Boyd wrote: > > Quoting Fabio Estevam (2019-06-06 09:44:43) > >> Use devm_platform_ioremap_resource() to simplify the code a bit. > >> > >> Signed-off-by: Fabio Estevam <festevam@gmail.com> > >> --- > > > > Reviewed-by: Stephen Boyd <sboyd@kernel.org> > > An extremely similar patch was already submitted and then reverted > because it breaks boot: > > https://patchwork.kernel.org/patch/10908807/ > > I tested and this new patch also breaks boot. > > The current imx8 lpcg driver maps entire subsystems at once and if > devm_platform_ioremap_resource is used then devices inside the subsystem > will fail to probe, including lpuart! > > The hardware on imx8qxp and related parts (imx8qm) has multiple separate > LPCG blocks interspersed between devices. Some refactoring patches were > posted by Aisheng to split LPCG into multiple blocks but apparently got > stuck in review: > > https://patchwork.kernel.org/cover/10924029/ > https://patchwork.kernel.org/cover/10824443/ > https://patchwork.kernel.org/cover/10824537/ > > There were some disagreements regarding DT bindings for split imx8 > clocks, what would it take to move those patches forward? > Don't know. I'll have to read those patches on the list and reply there. ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] clk: imx8qxp: Use devm_platform_ioremap_resource() 2019-06-06 17:22 ` Stephen Boyd @ 2019-06-06 17:55 ` Leonard Crestez 0 siblings, 0 replies; 4+ messages in thread From: Leonard Crestez @ 2019-06-06 17:55 UTC (permalink / raw) To: Stephen Boyd Cc: Fabio Estevam, shawnguo@kernel.org, dl-linux-imx, linux-clk@vger.kernel.org, Anson Huang, Aisheng Dong, kernel@pengutronix.de, devicetree@vger.kernel.org On 06.06.2019 20:22, Stephen Boyd wrote: > Quoting Leonard Crestez (2019-06-06 10:15:32) >> On 06.06.2019 19:55, Stephen Boyd wrote: >>> Quoting Fabio Estevam (2019-06-06 09:44:43) >>>> Use devm_platform_ioremap_resource() to simplify the code a bit. >>>> >>>> Signed-off-by: Fabio Estevam <festevam@gmail.com> >>>> --- >>> >>> Reviewed-by: Stephen Boyd <sboyd@kernel.org> >> >> An extremely similar patch was already submitted and then reverted >> because it breaks boot: >> >> The current imx8 lpcg driver maps entire subsystems at once and if >> devm_platform_ioremap_resource is used then devices inside the subsystem >> will fail to probe, including lpuart! >> >> The hardware on imx8qxp and related parts (imx8qm) has multiple separate >> LPCG blocks interspersed between devices. Some refactoring patches were >> posted by Aisheng to split LPCG into multiple blocks but apparently got >> stuck in review: >> >> There were some disagreements regarding DT bindings for split imx8 >> clocks, what would it take to move those patches forward? > > Don't know. I'll have to read those patches on the list and reply there. Some specific advice on how to refactor and split imx8qxp clk in a way that is acceptable to upstream would be very helpful to us. -- Regards, Leonard ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] clk: imx8qxp: Use devm_platform_ioremap_resource() 2019-06-06 17:15 ` [PATCH] clk: imx8qxp: Use devm_platform_ioremap_resource() Leonard Crestez 2019-06-06 17:22 ` Stephen Boyd @ 2019-06-06 17:57 ` Fabio Estevam 1 sibling, 0 replies; 4+ messages in thread From: Fabio Estevam @ 2019-06-06 17:57 UTC (permalink / raw) To: Leonard Crestez Cc: Stephen Boyd, shawnguo@kernel.org, dl-linux-imx, linux-clk@vger.kernel.org, Anson Huang, Aisheng Dong, kernel@pengutronix.de, devicetree@vger.kernel.org On Thu, Jun 6, 2019 at 2:15 PM Leonard Crestez <leonard.crestez@nxp.com> wrote: > An extremely similar patch was already submitted and then reverted > because it breaks boot: > > https://patchwork.kernel.org/patch/10908807/ > > I tested and this new patch also breaks boot. Ops, sorry about that! ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-06-06 17:57 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20190606164443.6991-1-festevam@gmail.com>
[not found] ` <20190606165546.6675520693@mail.kernel.org>
2019-06-06 17:15 ` [PATCH] clk: imx8qxp: Use devm_platform_ioremap_resource() Leonard Crestez
2019-06-06 17:22 ` Stephen Boyd
2019-06-06 17:55 ` Leonard Crestez
2019-06-06 17:57 ` Fabio Estevam
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox