Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support
Hi Peng, > > -Original Message- > > From: Lukasz Majewski [mailto:lu...@denx.de] > > Sent: 2019年4月23日 16:46 > > To: Peng Fan > > Cc: Jagan Teki ; Stefano Babic > > ; Fabio Estevam ; Simon Glass > > ; Tom Rini ; Marek Vasut > > ; Neil Armstrong > > ; Philipp Tomsich > > ; Maxime Ripard > > ; Michael Trimarchi > > ; Andre Przywara > > ; U-Boot-Denx ; > > linux-amar...@amarulasolutions.com; dl-uboot-imx > > Subject: Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK > > support > > > > On Tue, 23 Apr 2019 07:47:38 + > > Peng Fan wrote: > > > > > Hi Lukasz, > > > > > > > -Original Message- > > > > From: Lukasz Majewski [mailto:lu...@denx.de] > > > > Sent: 2019年4月20日 6:18 > > > > To: Peng Fan > > > > Cc: Jagan Teki ; Stefano Babic > > > > ; Fabio Estevam ; Simon > > Glass > > > > ; Tom Rini ; Marek Vasut > > > > ; Neil Armstrong > > > > ; Philipp Tomsich > > > > ; Maxime Ripard > > > > ; Michael Trimarchi > > > > ; Andre Przywara > > > > ; U-Boot-Denx ; > > > > linux-amar...@amarulasolutions.com; dl-uboot-imx > > > > > > Subject: Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK > > > > support > > > > > > > > On Fri, 19 Apr 2019 08:52:28 + > > > > Peng Fan wrote: > > > > > > > > > > On Fri, 19 Apr 2019 11:56:25 +0530 Jagan Teki > > > > > > wrote: > > > > > > > > > > > > > On Fri, Apr 5, 2019 at 2:20 AM Lukasz Majewski > > > > > > > wrote: > > > > > > > > > > > > > > > > Hi Jagan, > > > > > > > > > > > > > > > > > On Thu, Apr 4, 2019 at 3:31 PM Lukasz Majewski > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > On Thu, 4 Apr 2019 14:56:36 +0530 Jagan Teki > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > On Thu, Apr 4, 2019 at 2:31 PM Lukasz Majewski > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > On Tue, 2 Apr 2019 16:58:33 +0530 Jagan Teki > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > This is revised version of previous i.MX6 > > > > > > > > > > > > > clock management [1]. > > > > > > > > > > > > > > > > > > > > > > > > > > The main difference between previous version > > > > > > > > > > > > > is > > > > > > > > > > > > > - Group the i.MX6 ccm clocks into gates and > > > > > > > > > > > > > tree instead of handling the clocks in simple > > > > > > > > > > > > > way using case statement. > > > > > > > > > > > > > - use gate clocks for enable/disable > > > > > > > > > > > > > management. > > > > > > > > > > > > > - use tree clocks for get/set rate or parent > > > > > > > > > > > > > traverse management. > > > > > > > > > > > > > - parent clock handling via clock type. > > > > > > > > > > > > > - traverse the parent clock using recursive > > > > > > > > > > > > > functionlaity. > > > > > > > > > > > > > > > > > > > > > > > > > > The main motive behind this tree framework is > > > > > > > > > > > > > to make the clock tree management simple and > > > > > > > > > > > > > useful for U-Boot requirements instead of > > > > > > > > > > > > > garbing Linux clock management code. > > > > > > > > > > > > > > > > > > > > > > > > > > We are trying to manage the Allwinner clocks >
Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support
> -Original Message- > From: Lukasz Majewski [mailto:lu...@denx.de] > Sent: 2019年4月23日 16:46 > To: Peng Fan > Cc: Jagan Teki ; Stefano Babic > ; Fabio Estevam ; Simon Glass > ; Tom Rini ; Marek Vasut > ; Neil Armstrong > ; Philipp Tomsich > ; Maxime Ripard > ; Michael Trimarchi > ; Andre Przywara > ; U-Boot-Denx ; > linux-amar...@amarulasolutions.com; dl-uboot-imx > Subject: Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support > > On Tue, 23 Apr 2019 07:47:38 + > Peng Fan wrote: > > > Hi Lukasz, > > > > > -Original Message- > > > From: Lukasz Majewski [mailto:lu...@denx.de] > > > Sent: 2019年4月20日 6:18 > > > To: Peng Fan > > > Cc: Jagan Teki ; Stefano Babic > > > ; Fabio Estevam ; Simon > Glass > > > ; Tom Rini ; Marek Vasut > > > ; Neil Armstrong > > > ; Philipp Tomsich > > > ; Maxime Ripard > > > ; Michael Trimarchi > > > ; Andre Przywara > > > ; U-Boot-Denx ; > > > linux-amar...@amarulasolutions.com; dl-uboot-imx > > > > Subject: Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK > > > support > > > > > > On Fri, 19 Apr 2019 08:52:28 + > > > Peng Fan wrote: > > > > > > > > On Fri, 19 Apr 2019 11:56:25 +0530 Jagan Teki > > > > > wrote: > > > > > > > > > > > On Fri, Apr 5, 2019 at 2:20 AM Lukasz Majewski > > > > > > wrote: > > > > > > > > > > > > > > Hi Jagan, > > > > > > > > > > > > > > > On Thu, Apr 4, 2019 at 3:31 PM Lukasz Majewski > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > On Thu, 4 Apr 2019 14:56:36 +0530 Jagan Teki > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > On Thu, Apr 4, 2019 at 2:31 PM Lukasz Majewski > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > On Tue, 2 Apr 2019 16:58:33 +0530 Jagan Teki > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > This is revised version of previous i.MX6 clock > > > > > > > > > > > > management [1]. > > > > > > > > > > > > > > > > > > > > > > > > The main difference between previous version is > > > > > > > > > > > > - Group the i.MX6 ccm clocks into gates and tree > > > > > > > > > > > > instead of handling the clocks in simple way using > > > > > > > > > > > > case statement. > > > > > > > > > > > > - use gate clocks for enable/disable management. > > > > > > > > > > > > - use tree clocks for get/set rate or parent > > > > > > > > > > > > traverse management. > > > > > > > > > > > > - parent clock handling via clock type. > > > > > > > > > > > > - traverse the parent clock using recursive > > > > > > > > > > > > functionlaity. > > > > > > > > > > > > > > > > > > > > > > > > The main motive behind this tree framework is to > > > > > > > > > > > > make the clock tree management simple and useful > > > > > > > > > > > > for U-Boot requirements instead of garbing Linux > > > > > > > > > > > > clock management code. > > > > > > > > > > > > > > > > > > > > > > > > We are trying to manage the Allwinner clocks with > > > > > > > > > > > > similar kind, so having this would really help > > > > > > > > > > > > i.MX6 as well. > > > > > > > > > > > > > > > > > > > > > > > > Added simple names for clock macros, but will > > > > > > > > > > > > update it in future version. > > > > > > > > > > > > > > > > > > > > > > > > I have skipped ENET clocks fro
Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support
On Tue, 23 Apr 2019 07:47:38 + Peng Fan wrote: > Hi Lukasz, > > > -Original Message- > > From: Lukasz Majewski [mailto:lu...@denx.de] > > Sent: 2019年4月20日 6:18 > > To: Peng Fan > > Cc: Jagan Teki ; Stefano Babic > > ; Fabio Estevam ; Simon Glass > > ; Tom Rini ; Marek Vasut > > ; Neil Armstrong > > ; Philipp Tomsich > > ; Maxime Ripard > > ; Michael Trimarchi > > ; Andre Przywara > > ; U-Boot-Denx ; > > linux-amar...@amarulasolutions.com; dl-uboot-imx > > Subject: Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK > > support > > > > On Fri, 19 Apr 2019 08:52:28 + > > Peng Fan wrote: > > > > > > On Fri, 19 Apr 2019 11:56:25 +0530 > > > > Jagan Teki wrote: > > > > > > > > > On Fri, Apr 5, 2019 at 2:20 AM Lukasz Majewski > > > > > wrote: > > > > > > > > > > > > Hi Jagan, > > > > > > > > > > > > > On Thu, Apr 4, 2019 at 3:31 PM Lukasz Majewski > > > > > > > wrote: > > > > > > > > > > > > > > > > On Thu, 4 Apr 2019 14:56:36 +0530 Jagan Teki > > > > > > > > wrote: > > > > > > > > > > > > > > > > > On Thu, Apr 4, 2019 at 2:31 PM Lukasz Majewski > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > On Tue, 2 Apr 2019 16:58:33 +0530 Jagan Teki > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > This is revised version of previous i.MX6 clock > > > > > > > > > > > management [1]. > > > > > > > > > > > > > > > > > > > > > > The main difference between previous version is > > > > > > > > > > > - Group the i.MX6 ccm clocks into gates and tree > > > > > > > > > > > instead of handling the clocks in simple way using > > > > > > > > > > > case statement. > > > > > > > > > > > - use gate clocks for enable/disable management. > > > > > > > > > > > - use tree clocks for get/set rate or parent > > > > > > > > > > > traverse management. > > > > > > > > > > > - parent clock handling via clock type. > > > > > > > > > > > - traverse the parent clock using recursive > > > > > > > > > > > functionlaity. > > > > > > > > > > > > > > > > > > > > > > The main motive behind this tree framework is to > > > > > > > > > > > make the clock tree management simple and useful > > > > > > > > > > > for U-Boot requirements instead of garbing Linux > > > > > > > > > > > clock management code. > > > > > > > > > > > > > > > > > > > > > > We are trying to manage the Allwinner clocks with > > > > > > > > > > > similar kind, so having this would really help > > > > > > > > > > > i.MX6 as well. > > > > > > > > > > > > > > > > > > > > > > Added simple names for clock macros, but will > > > > > > > > > > > update it in future version. > > > > > > > > > > > > > > > > > > > > > > I have skipped ENET clocks from previous series, > > > > > > > > > > > will add it in future patches. > > > > > > > > > > > > > > > > > > > > > > Changes for v2: > > > > > > > > > > > - changed framework patches. > > > > > > > > > > > - add support for imx6qdl and imx6ul boards > > > > > > > > > > > - add clock gates, tree. > > > > > > > > > > > > > > > > > > > > > > [1] https://patchwork.ozlabs.org/cover/950964/ > > > > > > > > > > > > > > > > > > > > > > Any inputs? > > > > > > > > > > > > > > > > > >
Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support
Hi Lukasz, > -Original Message- > From: Lukasz Majewski [mailto:lu...@denx.de] > Sent: 2019年4月20日 6:18 > To: Peng Fan > Cc: Jagan Teki ; Stefano Babic > ; Fabio Estevam ; Simon Glass > ; Tom Rini ; Marek Vasut > ; Neil Armstrong > ; Philipp Tomsich > ; Maxime Ripard > ; Michael Trimarchi > ; Andre Przywara > ; U-Boot-Denx ; > linux-amar...@amarulasolutions.com; dl-uboot-imx > Subject: Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support > > On Fri, 19 Apr 2019 08:52:28 + > Peng Fan wrote: > > > > On Fri, 19 Apr 2019 11:56:25 +0530 > > > Jagan Teki wrote: > > > > > > > On Fri, Apr 5, 2019 at 2:20 AM Lukasz Majewski > > > > wrote: > > > > > > > > > > Hi Jagan, > > > > > > > > > > > On Thu, Apr 4, 2019 at 3:31 PM Lukasz Majewski > > > > > > wrote: > > > > > > > > > > > > > > On Thu, 4 Apr 2019 14:56:36 +0530 Jagan Teki > > > > > > > wrote: > > > > > > > > > > > > > > > On Thu, Apr 4, 2019 at 2:31 PM Lukasz Majewski > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > On Tue, 2 Apr 2019 16:58:33 +0530 Jagan Teki > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > This is revised version of previous i.MX6 clock > > > > > > > > > > management [1]. > > > > > > > > > > > > > > > > > > > > The main difference between previous version is > > > > > > > > > > - Group the i.MX6 ccm clocks into gates and tree > > > > > > > > > > instead of handling the clocks in simple way using > > > > > > > > > > case statement. > > > > > > > > > > - use gate clocks for enable/disable management. > > > > > > > > > > - use tree clocks for get/set rate or parent traverse > > > > > > > > > > management. > > > > > > > > > > - parent clock handling via clock type. > > > > > > > > > > - traverse the parent clock using recursive > > > > > > > > > > functionlaity. > > > > > > > > > > > > > > > > > > > > The main motive behind this tree framework is to make > > > > > > > > > > the clock tree management simple and useful for U-Boot > > > > > > > > > > requirements instead of garbing Linux clock management > > > > > > > > > > code. > > > > > > > > > > > > > > > > > > > > We are trying to manage the Allwinner clocks with > > > > > > > > > > similar kind, so having this would really help i.MX6 > > > > > > > > > > as well. > > > > > > > > > > > > > > > > > > > > Added simple names for clock macros, but will update > > > > > > > > > > it in future version. > > > > > > > > > > > > > > > > > > > > I have skipped ENET clocks from previous series, will > > > > > > > > > > add it in future patches. > > > > > > > > > > > > > > > > > > > > Changes for v2: > > > > > > > > > > - changed framework patches. > > > > > > > > > > - add support for imx6qdl and imx6ul boards > > > > > > > > > > - add clock gates, tree. > > > > > > > > > > > > > > > > > > > > [1] https://patchwork.ozlabs.org/cover/950964/ > > > > > > > > > > > > > > > > > > > > Any inputs? > > > > > > > > > > > > > > > > > > Hmm It looks like we are doing some development in > > > > > > > > > parallel. > > > > > > > > > > > > > > > > > > Please look into following commit [1]: > > > > > > > > > https://patchwork.ozlabs.org/patch/1034051/ > > > > > > > > > > > > > > > > > > It ports from Linux 5.0 the CCF framework for i
Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support
On Fri, Apr 19, 2019 at 08:52:28AM +, Peng Fan wrote: > > > On Fri, 19 Apr 2019 11:56:25 +0530 > > Jagan Teki wrote: > > > > > On Fri, Apr 5, 2019 at 2:20 AM Lukasz Majewski wrote: > > > > > > > > Hi Jagan, > > > > > > > > > On Thu, Apr 4, 2019 at 3:31 PM Lukasz Majewski > > > > > wrote: > > > > > > > > > > > > On Thu, 4 Apr 2019 14:56:36 +0530 Jagan Teki > > > > > > wrote: > > > > > > > > > > > > > On Thu, Apr 4, 2019 at 2:31 PM Lukasz Majewski > > > > > > > wrote: > > > > > > > > > > > > > > > > On Tue, 2 Apr 2019 16:58:33 +0530 Jagan Teki > > > > > > > > wrote: > > > > > > > > > > > > > > > > > This is revised version of previous i.MX6 clock management > > > > > > > > > [1]. > > > > > > > > > > > > > > > > > > The main difference between previous version is > > > > > > > > > - Group the i.MX6 ccm clocks into gates and tree instead > > > > > > > > > of handling the clocks in simple way using case statement. > > > > > > > > > - use gate clocks for enable/disable management. > > > > > > > > > - use tree clocks for get/set rate or parent traverse > > > > > > > > > management. > > > > > > > > > - parent clock handling via clock type. > > > > > > > > > - traverse the parent clock using recursive functionlaity. > > > > > > > > > > > > > > > > > > The main motive behind this tree framework is to make the > > > > > > > > > clock tree management simple and useful for U-Boot > > > > > > > > > requirements instead of garbing Linux clock management > > > > > > > > > code. > > > > > > > > > > > > > > > > > > We are trying to manage the Allwinner clocks with similar > > > > > > > > > kind, so having this would really help i.MX6 as well. > > > > > > > > > > > > > > > > > > Added simple names for clock macros, but will update it in > > > > > > > > > future version. > > > > > > > > > > > > > > > > > > I have skipped ENET clocks from previous series, will add > > > > > > > > > it in future patches. > > > > > > > > > > > > > > > > > > Changes for v2: > > > > > > > > > - changed framework patches. > > > > > > > > > - add support for imx6qdl and imx6ul boards > > > > > > > > > - add clock gates, tree. > > > > > > > > > > > > > > > > > > [1] https://patchwork.ozlabs.org/cover/950964/ > > > > > > > > > > > > > > > > > > Any inputs? > > > > > > > > > > > > > > > > Hmm It looks like we are doing some development in > > > > > > > > parallel. > > > > > > > > > > > > > > > > Please look into following commit [1]: > > > > > > > > https://patchwork.ozlabs.org/patch/1034051/ > > > > > > > > > > > > > > > > It ports from Linux 5.0 the CCF framework for iMX6Q, which > > > > > > > > IMHO in the long term is a better approach. > > > > > > > > The code is kept simple and resembles the code from Barebox. > > > > > > > > > > > > > > > > Please correct me if I'm wrong, but the code from your work > > > > > > > > is not modeling muxes, gates and other components from Linux > > > > > > > > CCF. > > > > > > > > > > > > > > The U-Boot implementation of CLK would require as minimal and > > > > > > > simple as possible due to requirement of U-Boot itself. Hope > > > > > > > you agree this point? > > > > > > > > > > > > Now i.MX6 is using clock.c CLK implementation. If we decide to > > > > > > replace it - we shall do it in a way, which would allow us to > > > > > > follow Linux kernel. (the barebox implementation is a stripped > > > > > > CCF from Linux, the same is in patch [1]). > > > > > > > > > > > > > if yes having CCF stack code to handle all clock with > > > > > > > respective separate drivers management is may not require as > > > > > > > of now, IMHO. > > > > > > > > > > > > I do have a gut feeling, that we will end up with the need to > > > > > > have the CCF framework ported anyway. As for example imx7/8 can > > > > > > re-use muxes, gates code. > > > > > > > > > > As per my experience the main the over-ahead to handle clocks in > > > > > U-Boot if we go with separate clock drivers is for Video and > > > > > Ethernet peripherals. these are key IP's which use more clocks > > > > > from U-Boot point-of-view, others can be handle pretty > > > > > straight-forward unless if they don't have too much tree chain. > > > > > > > > > > On this series, the tree management is already supported ENET in > > > > > i.MX6, and Allwinner platforms. > > > > > > > > > > As of now, I'm thinking I can handle reset of the clocks with > > > > > similar way. > > > > > > > > But this code also supports ENET and ESDHCI clocks on i.MX6Q (as > > > > supporting those was the motivator for this work). > > > > > > > > One important thing to be aware of - the problem with SPL's > > > > footprint. The implementation with clock.c is small and simple, but > > > > doesn't scale well. > > > > > > > > > > > > > > > > > > > > > However, those are only my "feelings" after a glimpse look - I > > > > > > will look into your code more thoroughly and provide feedback. > > > > > > > > > > Please have a look, if possible check even the code size by adding > > > > >
Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support
On Fri, 19 Apr 2019 08:52:28 + Peng Fan wrote: > > On Fri, 19 Apr 2019 11:56:25 +0530 > > Jagan Teki wrote: > > > > > On Fri, Apr 5, 2019 at 2:20 AM Lukasz Majewski > > > wrote: > > > > > > > > Hi Jagan, > > > > > > > > > On Thu, Apr 4, 2019 at 3:31 PM Lukasz Majewski > > > > > wrote: > > > > > > > > > > > > On Thu, 4 Apr 2019 14:56:36 +0530 Jagan Teki > > > > > > wrote: > > > > > > > > > > > > > On Thu, Apr 4, 2019 at 2:31 PM Lukasz Majewski > > > > > > > wrote: > > > > > > > > > > > > > > > > On Tue, 2 Apr 2019 16:58:33 +0530 Jagan Teki > > > > > > > > wrote: > > > > > > > > > > > > > > > > > This is revised version of previous i.MX6 clock > > > > > > > > > management [1]. > > > > > > > > > > > > > > > > > > The main difference between previous version is > > > > > > > > > - Group the i.MX6 ccm clocks into gates and tree > > > > > > > > > instead of handling the clocks in simple way using > > > > > > > > > case statement. > > > > > > > > > - use gate clocks for enable/disable management. > > > > > > > > > - use tree clocks for get/set rate or parent traverse > > > > > > > > > management. > > > > > > > > > - parent clock handling via clock type. > > > > > > > > > - traverse the parent clock using recursive > > > > > > > > > functionlaity. > > > > > > > > > > > > > > > > > > The main motive behind this tree framework is to make > > > > > > > > > the clock tree management simple and useful for U-Boot > > > > > > > > > requirements instead of garbing Linux clock management > > > > > > > > > code. > > > > > > > > > > > > > > > > > > We are trying to manage the Allwinner clocks with > > > > > > > > > similar kind, so having this would really help i.MX6 > > > > > > > > > as well. > > > > > > > > > > > > > > > > > > Added simple names for clock macros, but will update > > > > > > > > > it in future version. > > > > > > > > > > > > > > > > > > I have skipped ENET clocks from previous series, will > > > > > > > > > add it in future patches. > > > > > > > > > > > > > > > > > > Changes for v2: > > > > > > > > > - changed framework patches. > > > > > > > > > - add support for imx6qdl and imx6ul boards > > > > > > > > > - add clock gates, tree. > > > > > > > > > > > > > > > > > > [1] https://patchwork.ozlabs.org/cover/950964/ > > > > > > > > > > > > > > > > > > Any inputs? > > > > > > > > > > > > > > > > Hmm It looks like we are doing some development in > > > > > > > > parallel. > > > > > > > > > > > > > > > > Please look into following commit [1]: > > > > > > > > https://patchwork.ozlabs.org/patch/1034051/ > > > > > > > > > > > > > > > > It ports from Linux 5.0 the CCF framework for iMX6Q, > > > > > > > > which IMHO in the long term is a better approach. > > > > > > > > The code is kept simple and resembles the code from > > > > > > > > Barebox. > > > > > > > > > > > > > > > > Please correct me if I'm wrong, but the code from your > > > > > > > > work is not modeling muxes, gates and other components > > > > > > > > from Linux CCF. > > > > > > > > > > > > > > The U-Boot implementation of CLK would require as minimal > > > > > > > and simple as possible due to requirement of U-Boot > > > > > > > itself. Hope you agree this point? > > > > > > > > > > > > Now i.MX6 is using clock.c CLK implementation. If we decide > > > > > > to replace it - we shall do it in a way, which would allow > > > > > > us to follow Linux kernel. (the barebox implementation is a > > > > > > stripped CCF from Linux, the same is in patch [1]). > > > > > > > > > > > > > if yes having CCF stack code to handle all clock with > > > > > > > respective separate drivers management is may not require > > > > > > > as of now, IMHO. > > > > > > > > > > > > I do have a gut feeling, that we will end up with the need > > > > > > to have the CCF framework ported anyway. As for example > > > > > > imx7/8 can re-use muxes, gates code. > > > > > > > > > > As per my experience the main the over-ahead to handle clocks > > > > > in U-Boot if we go with separate clock drivers is for Video > > > > > and Ethernet peripherals. these are key IP's which use more > > > > > clocks from U-Boot point-of-view, others can be handle pretty > > > > > straight-forward unless if they don't have too much tree > > > > > chain. > > > > > > > > > > On this series, the tree management is already supported ENET > > > > > in i.MX6, and Allwinner platforms. > > > > > > > > > > As of now, I'm thinking I can handle reset of the clocks with > > > > > similar way. > > > > > > > > But this code also supports ENET and ESDHCI clocks on i.MX6Q (as > > > > supporting those was the motivator for this work). > > > > > > > > One important thing to be aware of - the problem with SPL's > > > > footprint. The implementation with clock.c is small and simple, > > > > but doesn't scale well. > > > > > > > > > > > > > > > > > > > > > However, those are only my "feelings" after a glimpse look > > > > > > - I will look into your code more thoroughly and
Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support
> On Fri, 19 Apr 2019 11:56:25 +0530 > Jagan Teki wrote: > > > On Fri, Apr 5, 2019 at 2:20 AM Lukasz Majewski wrote: > > > > > > Hi Jagan, > > > > > > > On Thu, Apr 4, 2019 at 3:31 PM Lukasz Majewski > > > > wrote: > > > > > > > > > > On Thu, 4 Apr 2019 14:56:36 +0530 Jagan Teki > > > > > wrote: > > > > > > > > > > > On Thu, Apr 4, 2019 at 2:31 PM Lukasz Majewski > > > > > > wrote: > > > > > > > > > > > > > > On Tue, 2 Apr 2019 16:58:33 +0530 Jagan Teki > > > > > > > wrote: > > > > > > > > > > > > > > > This is revised version of previous i.MX6 clock management > > > > > > > > [1]. > > > > > > > > > > > > > > > > The main difference between previous version is > > > > > > > > - Group the i.MX6 ccm clocks into gates and tree instead > > > > > > > > of handling the clocks in simple way using case statement. > > > > > > > > - use gate clocks for enable/disable management. > > > > > > > > - use tree clocks for get/set rate or parent traverse > > > > > > > > management. > > > > > > > > - parent clock handling via clock type. > > > > > > > > - traverse the parent clock using recursive functionlaity. > > > > > > > > > > > > > > > > The main motive behind this tree framework is to make the > > > > > > > > clock tree management simple and useful for U-Boot > > > > > > > > requirements instead of garbing Linux clock management > > > > > > > > code. > > > > > > > > > > > > > > > > We are trying to manage the Allwinner clocks with similar > > > > > > > > kind, so having this would really help i.MX6 as well. > > > > > > > > > > > > > > > > Added simple names for clock macros, but will update it in > > > > > > > > future version. > > > > > > > > > > > > > > > > I have skipped ENET clocks from previous series, will add > > > > > > > > it in future patches. > > > > > > > > > > > > > > > > Changes for v2: > > > > > > > > - changed framework patches. > > > > > > > > - add support for imx6qdl and imx6ul boards > > > > > > > > - add clock gates, tree. > > > > > > > > > > > > > > > > [1] https://patchwork.ozlabs.org/cover/950964/ > > > > > > > > > > > > > > > > Any inputs? > > > > > > > > > > > > > > Hmm It looks like we are doing some development in > > > > > > > parallel. > > > > > > > > > > > > > > Please look into following commit [1]: > > > > > > > https://patchwork.ozlabs.org/patch/1034051/ > > > > > > > > > > > > > > It ports from Linux 5.0 the CCF framework for iMX6Q, which > > > > > > > IMHO in the long term is a better approach. > > > > > > > The code is kept simple and resembles the code from Barebox. > > > > > > > > > > > > > > Please correct me if I'm wrong, but the code from your work > > > > > > > is not modeling muxes, gates and other components from Linux > > > > > > > CCF. > > > > > > > > > > > > The U-Boot implementation of CLK would require as minimal and > > > > > > simple as possible due to requirement of U-Boot itself. Hope > > > > > > you agree this point? > > > > > > > > > > Now i.MX6 is using clock.c CLK implementation. If we decide to > > > > > replace it - we shall do it in a way, which would allow us to > > > > > follow Linux kernel. (the barebox implementation is a stripped > > > > > CCF from Linux, the same is in patch [1]). > > > > > > > > > > > if yes having CCF stack code to handle all clock with > > > > > > respective separate drivers management is may not require as > > > > > > of now, IMHO. > > > > > > > > > > I do have a gut feeling, that we will end up with the need to > > > > > have the CCF framework ported anyway. As for example imx7/8 can > > > > > re-use muxes, gates code. > > > > > > > > As per my experience the main the over-ahead to handle clocks in > > > > U-Boot if we go with separate clock drivers is for Video and > > > > Ethernet peripherals. these are key IP's which use more clocks > > > > from U-Boot point-of-view, others can be handle pretty > > > > straight-forward unless if they don't have too much tree chain. > > > > > > > > On this series, the tree management is already supported ENET in > > > > i.MX6, and Allwinner platforms. > > > > > > > > As of now, I'm thinking I can handle reset of the clocks with > > > > similar way. > > > > > > But this code also supports ENET and ESDHCI clocks on i.MX6Q (as > > > supporting those was the motivator for this work). > > > > > > One important thing to be aware of - the problem with SPL's > > > footprint. The implementation with clock.c is small and simple, but > > > doesn't scale well. > > > > > > > > > > > > > > > > > However, those are only my "feelings" after a glimpse look - I > > > > > will look into your code more thoroughly and provide feedback. > > > > > > > > Please have a look, if possible check even the code size by adding > > > > USDHC clocks. > > > > > > Yes, code size (especially in SPL) is an _important_ factor here. > > > > > > > > > > > > > > > > > > > > > > > > This series is using recursive calls for handling parenting > > > > > > stuff to handle get or set rates, which is fine for handling > > > > >
Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support
On Fri, 19 Apr 2019 11:56:25 +0530 Jagan Teki wrote: > On Fri, Apr 5, 2019 at 2:20 AM Lukasz Majewski wrote: > > > > Hi Jagan, > > > > > On Thu, Apr 4, 2019 at 3:31 PM Lukasz Majewski > > > wrote: > > > > > > > > On Thu, 4 Apr 2019 14:56:36 +0530 > > > > Jagan Teki wrote: > > > > > > > > > On Thu, Apr 4, 2019 at 2:31 PM Lukasz Majewski > > > > > wrote: > > > > > > > > > > > > On Tue, 2 Apr 2019 16:58:33 +0530 > > > > > > Jagan Teki wrote: > > > > > > > > > > > > > This is revised version of previous i.MX6 clock management > > > > > > > [1]. > > > > > > > > > > > > > > The main difference between previous version is > > > > > > > - Group the i.MX6 ccm clocks into gates and tree instead > > > > > > > of handling the clocks in simple way using case statement. > > > > > > > - use gate clocks for enable/disable management. > > > > > > > - use tree clocks for get/set rate or parent traverse > > > > > > > management. > > > > > > > - parent clock handling via clock type. > > > > > > > - traverse the parent clock using recursive functionlaity. > > > > > > > > > > > > > > The main motive behind this tree framework is to make the > > > > > > > clock tree management simple and useful for U-Boot > > > > > > > requirements instead of garbing Linux clock management > > > > > > > code. > > > > > > > > > > > > > > We are trying to manage the Allwinner clocks with similar > > > > > > > kind, so having this would really help i.MX6 as well. > > > > > > > > > > > > > > Added simple names for clock macros, but will update it in > > > > > > > future version. > > > > > > > > > > > > > > I have skipped ENET clocks from previous series, will add > > > > > > > it in future patches. > > > > > > > > > > > > > > Changes for v2: > > > > > > > - changed framework patches. > > > > > > > - add support for imx6qdl and imx6ul boards > > > > > > > - add clock gates, tree. > > > > > > > > > > > > > > [1] https://patchwork.ozlabs.org/cover/950964/ > > > > > > > > > > > > > > Any inputs? > > > > > > > > > > > > Hmm It looks like we are doing some development in > > > > > > parallel. > > > > > > > > > > > > Please look into following commit [1]: > > > > > > https://patchwork.ozlabs.org/patch/1034051/ > > > > > > > > > > > > It ports from Linux 5.0 the CCF framework for iMX6Q, which > > > > > > IMHO in the long term is a better approach. > > > > > > The code is kept simple and resembles the code from Barebox. > > > > > > > > > > > > Please correct me if I'm wrong, but the code from your work > > > > > > is not modeling muxes, gates and other components from > > > > > > Linux CCF. > > > > > > > > > > The U-Boot implementation of CLK would require as minimal and > > > > > simple as possible due to requirement of U-Boot itself. Hope > > > > > you agree this point? > > > > > > > > Now i.MX6 is using clock.c CLK implementation. If we decide to > > > > replace it - we shall do it in a way, which would allow us to > > > > follow Linux kernel. (the barebox implementation is a stripped > > > > CCF from Linux, the same is in patch [1]). > > > > > > > > > if yes having CCF stack code to handle all clock with > > > > > respective separate drivers management is may not require as > > > > > of now, IMHO. > > > > > > > > I do have a gut feeling, that we will end up with the need to > > > > have the CCF framework ported anyway. As for example imx7/8 can > > > > re-use muxes, gates code. > > > > > > As per my experience the main the over-ahead to handle clocks in > > > U-Boot if we go with separate clock drivers is for Video and > > > Ethernet peripherals. these are key IP's which use more clocks > > > from U-Boot point-of-view, others can be handle pretty > > > straight-forward unless if they don't have too much tree chain. > > > > > > On this series, the tree management is already supported ENET in > > > i.MX6, and Allwinner platforms. > > > > > > As of now, I'm thinking I can handle reset of the clocks with > > > similar way. > > > > But this code also supports ENET and ESDHCI clocks on i.MX6Q (as > > supporting those was the motivator for this work). > > > > One important thing to be aware of - the problem with SPL's > > footprint. The implementation with clock.c is small and simple, but > > doesn't scale well. > > > > > > > > > > > > > However, those are only my "feelings" after a glimpse look - I > > > > will look into your code more thoroughly and provide feedback. > > > > > > Please have a look, if possible check even the code size by adding > > > USDHC clocks. > > > > Yes, code size (especially in SPL) is an _important_ factor here. > > > > > > > > > > > > > > > > > > > This series is using recursive calls for handling parenting > > > > > stuff to handle get or set rates, which is fine for handling > > > > > clock tree management as far as U-Boot point-of-view. We have > > > > > faced similar situation as I explained in commit message > > > > > about Allwinner clocks [2] and we ended up going this way.
Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support
On Fri, Apr 5, 2019 at 2:20 AM Lukasz Majewski wrote: > > Hi Jagan, > > > On Thu, Apr 4, 2019 at 3:31 PM Lukasz Majewski wrote: > > > > > > On Thu, 4 Apr 2019 14:56:36 +0530 > > > Jagan Teki wrote: > > > > > > > On Thu, Apr 4, 2019 at 2:31 PM Lukasz Majewski > > > > wrote: > > > > > > > > > > On Tue, 2 Apr 2019 16:58:33 +0530 > > > > > Jagan Teki wrote: > > > > > > > > > > > This is revised version of previous i.MX6 clock management > > > > > > [1]. > > > > > > > > > > > > The main difference between previous version is > > > > > > - Group the i.MX6 ccm clocks into gates and tree instead of > > > > > > handling the clocks in simple way using case statement. > > > > > > - use gate clocks for enable/disable management. > > > > > > - use tree clocks for get/set rate or parent traverse > > > > > > management. > > > > > > - parent clock handling via clock type. > > > > > > - traverse the parent clock using recursive functionlaity. > > > > > > > > > > > > The main motive behind this tree framework is to make the > > > > > > clock tree management simple and useful for U-Boot > > > > > > requirements instead of garbing Linux clock management code. > > > > > > > > > > > > We are trying to manage the Allwinner clocks with similar > > > > > > kind, so having this would really help i.MX6 as well. > > > > > > > > > > > > Added simple names for clock macros, but will update it in > > > > > > future version. > > > > > > > > > > > > I have skipped ENET clocks from previous series, will add it > > > > > > in future patches. > > > > > > > > > > > > Changes for v2: > > > > > > - changed framework patches. > > > > > > - add support for imx6qdl and imx6ul boards > > > > > > - add clock gates, tree. > > > > > > > > > > > > [1] https://patchwork.ozlabs.org/cover/950964/ > > > > > > > > > > > > Any inputs? > > > > > > > > > > Hmm It looks like we are doing some development in parallel. > > > > > > > > > > Please look into following commit [1]: > > > > > https://patchwork.ozlabs.org/patch/1034051/ > > > > > > > > > > It ports from Linux 5.0 the CCF framework for iMX6Q, which IMHO > > > > > in the long term is a better approach. > > > > > The code is kept simple and resembles the code from Barebox. > > > > > > > > > > Please correct me if I'm wrong, but the code from your work is > > > > > not modeling muxes, gates and other components from Linux CCF. > > > > > > > > The U-Boot implementation of CLK would require as minimal and > > > > simple as possible due to requirement of U-Boot itself. Hope you > > > > agree this point? > > > > > > Now i.MX6 is using clock.c CLK implementation. If we decide to > > > replace it - we shall do it in a way, which would allow us to follow > > > Linux kernel. (the barebox implementation is a stripped CCF from > > > Linux, the same is in patch [1]). > > > > > > > if yes having CCF stack code to handle all clock with > > > > respective separate drivers management is may not require as of > > > > now, IMHO. > > > > > > I do have a gut feeling, that we will end up with the need to have > > > the CCF framework ported anyway. As for example imx7/8 can re-use > > > muxes, gates code. > > > > As per my experience the main the over-ahead to handle clocks in > > U-Boot if we go with separate clock drivers is for Video and Ethernet > > peripherals. these are key IP's which use more clocks from U-Boot > > point-of-view, others can be handle pretty straight-forward unless if > > they don't have too much tree chain. > > > > On this series, the tree management is already supported ENET in > > i.MX6, and Allwinner platforms. > > > > As of now, I'm thinking I can handle reset of the clocks with similar > > way. > > But this code also supports ENET and ESDHCI clocks on i.MX6Q (as > supporting those was the motivator for this work). > > One important thing to be aware of - the problem with SPL's footprint. > The implementation with clock.c is small and simple, but doesn't scale > well. > > > > > > > > > However, those are only my "feelings" after a glimpse look - I will > > > look into your code more thoroughly and provide feedback. > > > > Please have a look, if possible check even the code size by adding > > USDHC clocks. > > Yes, code size (especially in SPL) is an _important_ factor here. > > > > > > > > > > > > > > This series is using recursive calls for handling parenting stuff > > > > to handle get or set rates, which is fine for handling clock tree > > > > management as far as U-Boot point-of-view. We have faced similar > > > > situation as I explained in commit message about Allwinner clocks > > > > [2] and we ended up going this way. > > > > > > I'm not Allwinner expert - but if I may ask - how far away is this > > > implementation from mainline Linux kernel? > > > > > > How difficult is it to port the new code (or update it)? > > > > Allwinner clocks also has similar gates, muxs, and with other platform > > stuff which has too much scope in Linux to use CCM. > > For example the barebox managed
Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support
Hi Jagan, > On Thu, Apr 4, 2019 at 3:31 PM Lukasz Majewski wrote: > > > > On Thu, 4 Apr 2019 14:56:36 +0530 > > Jagan Teki wrote: > > > > > On Thu, Apr 4, 2019 at 2:31 PM Lukasz Majewski > > > wrote: > > > > > > > > On Tue, 2 Apr 2019 16:58:33 +0530 > > > > Jagan Teki wrote: > > > > > > > > > This is revised version of previous i.MX6 clock management > > > > > [1]. > > > > > > > > > > The main difference between previous version is > > > > > - Group the i.MX6 ccm clocks into gates and tree instead of > > > > > handling the clocks in simple way using case statement. > > > > > - use gate clocks for enable/disable management. > > > > > - use tree clocks for get/set rate or parent traverse > > > > > management. > > > > > - parent clock handling via clock type. > > > > > - traverse the parent clock using recursive functionlaity. > > > > > > > > > > The main motive behind this tree framework is to make the > > > > > clock tree management simple and useful for U-Boot > > > > > requirements instead of garbing Linux clock management code. > > > > > > > > > > We are trying to manage the Allwinner clocks with similar > > > > > kind, so having this would really help i.MX6 as well. > > > > > > > > > > Added simple names for clock macros, but will update it in > > > > > future version. > > > > > > > > > > I have skipped ENET clocks from previous series, will add it > > > > > in future patches. > > > > > > > > > > Changes for v2: > > > > > - changed framework patches. > > > > > - add support for imx6qdl and imx6ul boards > > > > > - add clock gates, tree. > > > > > > > > > > [1] https://patchwork.ozlabs.org/cover/950964/ > > > > > > > > > > Any inputs? > > > > > > > > Hmm It looks like we are doing some development in parallel. > > > > > > > > Please look into following commit [1]: > > > > https://patchwork.ozlabs.org/patch/1034051/ > > > > > > > > It ports from Linux 5.0 the CCF framework for iMX6Q, which IMHO > > > > in the long term is a better approach. > > > > The code is kept simple and resembles the code from Barebox. > > > > > > > > Please correct me if I'm wrong, but the code from your work is > > > > not modeling muxes, gates and other components from Linux CCF. > > > > > > The U-Boot implementation of CLK would require as minimal and > > > simple as possible due to requirement of U-Boot itself. Hope you > > > agree this point? > > > > Now i.MX6 is using clock.c CLK implementation. If we decide to > > replace it - we shall do it in a way, which would allow us to follow > > Linux kernel. (the barebox implementation is a stripped CCF from > > Linux, the same is in patch [1]). > > > > > if yes having CCF stack code to handle all clock with > > > respective separate drivers management is may not require as of > > > now, IMHO. > > > > I do have a gut feeling, that we will end up with the need to have > > the CCF framework ported anyway. As for example imx7/8 can re-use > > muxes, gates code. > > As per my experience the main the over-ahead to handle clocks in > U-Boot if we go with separate clock drivers is for Video and Ethernet > peripherals. these are key IP's which use more clocks from U-Boot > point-of-view, others can be handle pretty straight-forward unless if > they don't have too much tree chain. > > On this series, the tree management is already supported ENET in > i.MX6, and Allwinner platforms. > > As of now, I'm thinking I can handle reset of the clocks with similar > way. But this code also supports ENET and ESDHCI clocks on i.MX6Q (as supporting those was the motivator for this work). One important thing to be aware of - the problem with SPL's footprint. The implementation with clock.c is small and simple, but doesn't scale well. > > > > > However, those are only my "feelings" after a glimpse look - I will > > look into your code more thoroughly and provide feedback. > > Please have a look, if possible check even the code size by adding > USDHC clocks. Yes, code size (especially in SPL) is an _important_ factor here. > > > > > > > > > This series is using recursive calls for handling parenting stuff > > > to handle get or set rates, which is fine for handling clock tree > > > management as far as U-Boot point-of-view. We have faced similar > > > situation as I explained in commit message about Allwinner clocks > > > [2] and we ended up going this way. > > > > I'm not Allwinner expert - but if I may ask - how far away is this > > implementation from mainline Linux kernel? > > > > How difficult is it to port the new code (or update it)? > > Allwinner clocks also has similar gates, muxs, and with other platform > stuff which has too much scope in Linux to use CCM. For example the barebox managed to get subset of Linux CCF ported, without loosing the CCF similarity. Important factors/requirements for the i.MX clock code: 1. Easy maintenance in long-term 2. Reusing the code in SPL (with a very important factor of _code_size_). 3. Reuse the code for
Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support
On Thu, Apr 04, 2019 at 09:35:43PM +0530, Jagan Teki wrote: > On Thu, Apr 4, 2019 at 9:26 PM Tom Rini wrote: > > > > On Thu, Apr 04, 2019 at 12:48:58PM -0300, Fabio Estevam wrote: > > > On Thu, Apr 4, 2019 at 7:01 AM Lukasz Majewski wrote: > > > > > > > Fabio, Stefano what do you think? > > > > > > > > As we change the clock.c code, IMHO we shall do the new port properly. > > > > > > I think the CCF solution proposed by Lukasz looks good and it will be > > > easier to maintain and sync with the kernel. > > > > This sounds like an important goal as well, to me. Thanks! > > I don't know why we rely too-much on Linux to import the big stack > code, since the requirement of U-Boot here is to handle the clocks as > minimum(as required) as compared to what OS is looking for. > > Are we looking for handling clock tree management for a whole or > looking as required (or as simple) is the main criteria to think > about. We rely on leveraging Linux when possible for a lot of reasons. First, it's generally going to have to solve most of the same problems we have to solve. Second, it's what most folks are going to be familiar with. So if we can strip down that same framework to work for us, it'll make life easier on everyone involved. -- Tom signature.asc Description: PGP signature ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support
On Thu, Apr 4, 2019 at 3:31 PM Lukasz Majewski wrote: > > On Thu, 4 Apr 2019 14:56:36 +0530 > Jagan Teki wrote: > > > On Thu, Apr 4, 2019 at 2:31 PM Lukasz Majewski wrote: > > > > > > On Tue, 2 Apr 2019 16:58:33 +0530 > > > Jagan Teki wrote: > > > > > > > This is revised version of previous i.MX6 clock management [1]. > > > > > > > > The main difference between previous version is > > > > - Group the i.MX6 ccm clocks into gates and tree instead of > > > > handling the clocks in simple way using case statement. > > > > - use gate clocks for enable/disable management. > > > > - use tree clocks for get/set rate or parent traverse management. > > > > - parent clock handling via clock type. > > > > - traverse the parent clock using recursive functionlaity. > > > > > > > > The main motive behind this tree framework is to make the clock > > > > tree management simple and useful for U-Boot requirements instead > > > > of garbing Linux clock management code. > > > > > > > > We are trying to manage the Allwinner clocks with similar kind, so > > > > having this would really help i.MX6 as well. > > > > > > > > Added simple names for clock macros, but will update it in future > > > > version. > > > > > > > > I have skipped ENET clocks from previous series, will add it in > > > > future patches. > > > > > > > > Changes for v2: > > > > - changed framework patches. > > > > - add support for imx6qdl and imx6ul boards > > > > - add clock gates, tree. > > > > > > > > [1] https://patchwork.ozlabs.org/cover/950964/ > > > > > > > > Any inputs? > > > > > > Hmm It looks like we are doing some development in parallel. > > > > > > Please look into following commit [1]: > > > https://patchwork.ozlabs.org/patch/1034051/ > > > > > > It ports from Linux 5.0 the CCF framework for iMX6Q, which IMHO in > > > the long term is a better approach. > > > The code is kept simple and resembles the code from Barebox. > > > > > > Please correct me if I'm wrong, but the code from your work is not > > > modeling muxes, gates and other components from Linux CCF. > > > > The U-Boot implementation of CLK would require as minimal and simple > > as possible due to requirement of U-Boot itself. Hope you agree this > > point? > > Now i.MX6 is using clock.c CLK implementation. If we decide to > replace it - we shall do it in a way, which would allow us to follow > Linux kernel. (the barebox implementation is a stripped CCF from > Linux, the same is in patch [1]). > > > if yes having CCF stack code to handle all clock with > > respective separate drivers management is may not require as of now, > > IMHO. > > I do have a gut feeling, that we will end up with the need to have the > CCF framework ported anyway. As for example imx7/8 can re-use muxes, > gates code. As per my experience the main the over-ahead to handle clocks in U-Boot if we go with separate clock drivers is for Video and Ethernet peripherals. these are key IP's which use more clocks from U-Boot point-of-view, others can be handle pretty straight-forward unless if they don't have too much tree chain. On this series, the tree management is already supported ENET in i.MX6, and Allwinner platforms. As of now, I'm thinking I can handle reset of the clocks with similar way. > > However, those are only my "feelings" after a glimpse look - I will look > into your code more thoroughly and provide feedback. Please have a look, if possible check even the code size by adding USDHC clocks. > > > > > This series is using recursive calls for handling parenting stuff to > > handle get or set rates, which is fine for handling clock tree > > management as far as U-Boot point-of-view. We have faced similar > > situation as I explained in commit message about Allwinner clocks [2] > > and we ended up going this way. > > I'm not Allwinner expert - but if I may ask - how far away is this > implementation from mainline Linux kernel? > > How difficult is it to port the new code (or update it)? Allwinner clocks also has similar gates, muxs, and with other platform stuff which has too much scope in Linux to use CCM. ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support
On Thu, Apr 4, 2019 at 9:26 PM Tom Rini wrote: > > On Thu, Apr 04, 2019 at 12:48:58PM -0300, Fabio Estevam wrote: > > On Thu, Apr 4, 2019 at 7:01 AM Lukasz Majewski wrote: > > > > > Fabio, Stefano what do you think? > > > > > > As we change the clock.c code, IMHO we shall do the new port properly. > > > > I think the CCF solution proposed by Lukasz looks good and it will be > > easier to maintain and sync with the kernel. > > This sounds like an important goal as well, to me. Thanks! I don't know why we rely too-much on Linux to import the big stack code, since the requirement of U-Boot here is to handle the clocks as minimum(as required) as compared to what OS is looking for. Are we looking for handling clock tree management for a whole or looking as required (or as simple) is the main criteria to think about. ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support
On Thu, Apr 04, 2019 at 12:48:58PM -0300, Fabio Estevam wrote: > On Thu, Apr 4, 2019 at 7:01 AM Lukasz Majewski wrote: > > > Fabio, Stefano what do you think? > > > > As we change the clock.c code, IMHO we shall do the new port properly. > > I think the CCF solution proposed by Lukasz looks good and it will be > easier to maintain and sync with the kernel. This sounds like an important goal as well, to me. Thanks! -- Tom signature.asc Description: PGP signature ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support
On Thu, Apr 4, 2019 at 7:01 AM Lukasz Majewski wrote: > Fabio, Stefano what do you think? > > As we change the clock.c code, IMHO we shall do the new port properly. I think the CCF solution proposed by Lukasz looks good and it will be easier to maintain and sync with the kernel. Thanks ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support
On Thu, 4 Apr 2019 14:56:36 +0530 Jagan Teki wrote: > On Thu, Apr 4, 2019 at 2:31 PM Lukasz Majewski wrote: > > > > On Tue, 2 Apr 2019 16:58:33 +0530 > > Jagan Teki wrote: > > > > > This is revised version of previous i.MX6 clock management [1]. > > > > > > The main difference between previous version is > > > - Group the i.MX6 ccm clocks into gates and tree instead of > > > handling the clocks in simple way using case statement. > > > - use gate clocks for enable/disable management. > > > - use tree clocks for get/set rate or parent traverse management. > > > - parent clock handling via clock type. > > > - traverse the parent clock using recursive functionlaity. > > > > > > The main motive behind this tree framework is to make the clock > > > tree management simple and useful for U-Boot requirements instead > > > of garbing Linux clock management code. > > > > > > We are trying to manage the Allwinner clocks with similar kind, so > > > having this would really help i.MX6 as well. > > > > > > Added simple names for clock macros, but will update it in future > > > version. > > > > > > I have skipped ENET clocks from previous series, will add it in > > > future patches. > > > > > > Changes for v2: > > > - changed framework patches. > > > - add support for imx6qdl and imx6ul boards > > > - add clock gates, tree. > > > > > > [1] https://patchwork.ozlabs.org/cover/950964/ > > > > > > Any inputs? > > > > Hmm It looks like we are doing some development in parallel. > > > > Please look into following commit [1]: > > https://patchwork.ozlabs.org/patch/1034051/ > > > > It ports from Linux 5.0 the CCF framework for iMX6Q, which IMHO in > > the long term is a better approach. > > The code is kept simple and resembles the code from Barebox. > > > > Please correct me if I'm wrong, but the code from your work is not > > modeling muxes, gates and other components from Linux CCF. > > The U-Boot implementation of CLK would require as minimal and simple > as possible due to requirement of U-Boot itself. Hope you agree this > point? Now i.MX6 is using clock.c CLK implementation. If we decide to replace it - we shall do it in a way, which would allow us to follow Linux kernel. (the barebox implementation is a stripped CCF from Linux, the same is in patch [1]). > if yes having CCF stack code to handle all clock with > respective separate drivers management is may not require as of now, > IMHO. I do have a gut feeling, that we will end up with the need to have the CCF framework ported anyway. As for example imx7/8 can re-use muxes, gates code. However, those are only my "feelings" after a glimpse look - I will look into your code more thoroughly and provide feedback. > > This series is using recursive calls for handling parenting stuff to > handle get or set rates, which is fine for handling clock tree > management as far as U-Boot point-of-view. We have faced similar > situation as I explained in commit message about Allwinner clocks [2] > and we ended up going this way. I'm not Allwinner expert - but if I may ask - how far away is this implementation from mainline Linux kernel? How difficult is it to port the new code (or update it)? > > The patches where I get introduced clock tree is based on muxes, gates > which were similar like Linux but I've managed to update according to > U-Boot need. > I have tried enet, enet_ref clocks as well and those are > working out-of-box. > > Feel free to comments, I have no intention to block anything. let's > have a proper discussion. Fabio, Stefano what do you think? As we change the clock.c code, IMHO we shall do the new port properly. > > [2] https://patchwork.ozlabs.org/patch/1019673/ Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lu...@denx.de pgpkIZ6MgUre6.pgp Description: OpenPGP digital signature ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support
On Thu, Apr 4, 2019 at 2:31 PM Lukasz Majewski wrote: > > On Tue, 2 Apr 2019 16:58:33 +0530 > Jagan Teki wrote: > > > This is revised version of previous i.MX6 clock management [1]. > > > > The main difference between previous version is > > - Group the i.MX6 ccm clocks into gates and tree instead of handling > > the clocks in simple way using case statement. > > - use gate clocks for enable/disable management. > > - use tree clocks for get/set rate or parent traverse management. > > - parent clock handling via clock type. > > - traverse the parent clock using recursive functionlaity. > > > > The main motive behind this tree framework is to make the clock tree > > management simple and useful for U-Boot requirements instead of > > garbing Linux clock management code. > > > > We are trying to manage the Allwinner clocks with similar kind, so > > having this would really help i.MX6 as well. > > > > Added simple names for clock macros, but will update it in future > > version. > > > > I have skipped ENET clocks from previous series, will add it in future > > patches. > > > > Changes for v2: > > - changed framework patches. > > - add support for imx6qdl and imx6ul boards > > - add clock gates, tree. > > > > [1] https://patchwork.ozlabs.org/cover/950964/ > > > > Any inputs? > > Hmm It looks like we are doing some development in parallel. > > Please look into following commit [1]: > https://patchwork.ozlabs.org/patch/1034051/ > > It ports from Linux 5.0 the CCF framework for iMX6Q, which IMHO in the > long term is a better approach. > The code is kept simple and resembles the code from Barebox. > > Please correct me if I'm wrong, but the code from your work is not > modeling muxes, gates and other components from Linux CCF. The U-Boot implementation of CLK would require as minimal and simple as possible due to requirement of U-Boot itself. Hope you agree this point? if yes having CCF stack code to handle all clock with respective separate drivers management is may not require as of now, IMHO. This series is using recursive calls for handling parenting stuff to handle get or set rates, which is fine for handling clock tree management as far as U-Boot point-of-view. We have faced similar situation as I explained in commit message about Allwinner clocks [2] and we ended up going this way. The patches where I get introduced clock tree is based on muxes, gates which were similar like Linux but I've managed to update according to U-Boot need. I have tried enet, enet_ref clocks as well and those are working out-of-box. Feel free to comments, I have no intention to block anything. let's have a proper discussion. [2] https://patchwork.ozlabs.org/patch/1019673/ ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support
On Tue, 2 Apr 2019 16:58:33 +0530 Jagan Teki wrote: > This is revised version of previous i.MX6 clock management [1]. > > The main difference between previous version is > - Group the i.MX6 ccm clocks into gates and tree instead of handling > the clocks in simple way using case statement. > - use gate clocks for enable/disable management. > - use tree clocks for get/set rate or parent traverse management. > - parent clock handling via clock type. > - traverse the parent clock using recursive functionlaity. > > The main motive behind this tree framework is to make the clock tree > management simple and useful for U-Boot requirements instead of > garbing Linux clock management code. > > We are trying to manage the Allwinner clocks with similar kind, so > having this would really help i.MX6 as well. > > Added simple names for clock macros, but will update it in future > version. > > I have skipped ENET clocks from previous series, will add it in future > patches. > > Changes for v2: > - changed framework patches. > - add support for imx6qdl and imx6ul boards > - add clock gates, tree. > > [1] https://patchwork.ozlabs.org/cover/950964/ > > Any inputs? Hmm It looks like we are doing some development in parallel. Please look into following commit [1]: https://patchwork.ozlabs.org/patch/1034051/ It ports from Linux 5.0 the CCF framework for iMX6Q, which IMHO in the long term is a better approach. The code is kept simple and resembles the code from Barebox. Please correct me if I'm wrong, but the code from your work is not modeling muxes, gates and other components from Linux CCF. Unfortunately for [1] - I did not have time recently to finish it ... (address Simon's comments about uclass). > Jagan. > > Jagan Teki (10): > clk: imx: Kconfig: Make CONFIG_CLK available for selection > clk: imx: Add i.MX6Q clock driver > clk: imx: Add i.MX6UL clock driver > clk: Add clk_div_mask helper > clk: imx: Add imx6q clock tree support > clk: imx6: Add imx6ul clock tree support > ARM: dts: i.MX6QDL: Add u-boot,dm-spl for clks > ARM: dts: i.MX6UL: Add u-boot,dm-spl for clks > configs: icore_mipi: Enable CLK > ARM: imx6: Enable CLK for Engicam i.MX6UL boards > > arch/arm/dts/imx6qdl-u-boot.dtsi | 4 + > arch/arm/dts/imx6ul-u-boot.dtsi | 4 + > arch/arm/include/asm/arch-mx6/clock.h | 109 > arch/arm/mach-imx/mx6/Kconfig | 2 + > configs/imx6qdl_icore_mipi_defconfig | 2 + > configs/imx8qxp_mek_defconfig | 2 +- > drivers/clk/imx/Kconfig | 29 - > drivers/clk/imx/Makefile | 6 + > drivers/clk/imx/clk-imx6-common.c | 172 > ++ drivers/clk/imx/clk-imx6q.c | > 109 drivers/clk/imx/clk-imx6ul.c | 85 > + include/clk-uclass.h | 2 + > 12 files changed, 523 insertions(+), 3 deletions(-) > create mode 100644 drivers/clk/imx/clk-imx6-common.c > create mode 100644 drivers/clk/imx/clk-imx6q.c > create mode 100644 drivers/clk/imx/clk-imx6ul.c > Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lu...@denx.de pgpxVlphskq6j.pgp Description: OpenPGP digital signature ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] [PATCH v2 00/10] clk: imx: Add i.MX6 CLK support
This is revised version of previous i.MX6 clock management [1]. The main difference between previous version is - Group the i.MX6 ccm clocks into gates and tree instead of handling the clocks in simple way using case statement. - use gate clocks for enable/disable management. - use tree clocks for get/set rate or parent traverse management. - parent clock handling via clock type. - traverse the parent clock using recursive functionlaity. The main motive behind this tree framework is to make the clock tree management simple and useful for U-Boot requirements instead of garbing Linux clock management code. We are trying to manage the Allwinner clocks with similar kind, so having this would really help i.MX6 as well. Added simple names for clock macros, but will update it in future version. I have skipped ENET clocks from previous series, will add it in future patches. Changes for v2: - changed framework patches. - add support for imx6qdl and imx6ul boards - add clock gates, tree. [1] https://patchwork.ozlabs.org/cover/950964/ Any inputs? Jagan. Jagan Teki (10): clk: imx: Kconfig: Make CONFIG_CLK available for selection clk: imx: Add i.MX6Q clock driver clk: imx: Add i.MX6UL clock driver clk: Add clk_div_mask helper clk: imx: Add imx6q clock tree support clk: imx6: Add imx6ul clock tree support ARM: dts: i.MX6QDL: Add u-boot,dm-spl for clks ARM: dts: i.MX6UL: Add u-boot,dm-spl for clks configs: icore_mipi: Enable CLK ARM: imx6: Enable CLK for Engicam i.MX6UL boards arch/arm/dts/imx6qdl-u-boot.dtsi | 4 + arch/arm/dts/imx6ul-u-boot.dtsi | 4 + arch/arm/include/asm/arch-mx6/clock.h | 109 arch/arm/mach-imx/mx6/Kconfig | 2 + configs/imx6qdl_icore_mipi_defconfig | 2 + configs/imx8qxp_mek_defconfig | 2 +- drivers/clk/imx/Kconfig | 29 - drivers/clk/imx/Makefile | 6 + drivers/clk/imx/clk-imx6-common.c | 172 ++ drivers/clk/imx/clk-imx6q.c | 109 drivers/clk/imx/clk-imx6ul.c | 85 + include/clk-uclass.h | 2 + 12 files changed, 523 insertions(+), 3 deletions(-) create mode 100644 drivers/clk/imx/clk-imx6-common.c create mode 100644 drivers/clk/imx/clk-imx6q.c create mode 100644 drivers/clk/imx/clk-imx6ul.c -- 2.18.0.321.gffc6fa0e3 ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
