RE: [PATCH 1/2] clk: imx6ul: add GPIO clock gates

2018-06-02 Thread Anson Huang
ngicam.com; > mich...@amarulasolutions.com; mturque...@baylibre.com; > robh...@kernel.org; shawn...@kernel.org > Cc: linux-...@vger.kernel.org; dl-linux-imx ; > devicet...@vger.kernel.org; linux-kernel@vger.kernel.org; > linux-arm-ker...@lists.infradead.org > Subject: Re: [PATCH 1/

RE: [PATCH 1/2] clk: imx6ul: add GPIO clock gates

2018-06-02 Thread Anson Huang
ngicam.com; > mich...@amarulasolutions.com; mturque...@baylibre.com; > robh...@kernel.org; shawn...@kernel.org > Cc: linux-...@vger.kernel.org; dl-linux-imx ; > devicet...@vger.kernel.org; linux-kernel@vger.kernel.org; > linux-arm-ker...@lists.infradead.org > Subject: Re: [PATCH 1/

Re: [PATCH 1/2] clk: imx6ul: add GPIO clock gates

2018-06-02 Thread Michael Nazzareno Trimarchi
Hi Fabio On Sat, Jun 2, 2018 at 4:07 PM, Fabio Estevam wrote: > Hi Michael, > > On Sat, Jun 2, 2018 at 11:04 AM, Michael Nazzareno Trimarchi > wrote: > >> ull is a preatty new platform so one board was listed. Are you sure >> that we need? > > There are several imx6ul based dts in mainline and

Re: [PATCH 1/2] clk: imx6ul: add GPIO clock gates

2018-06-02 Thread Michael Nazzareno Trimarchi
Hi Fabio On Sat, Jun 2, 2018 at 4:07 PM, Fabio Estevam wrote: > Hi Michael, > > On Sat, Jun 2, 2018 at 11:04 AM, Michael Nazzareno Trimarchi > wrote: > >> ull is a preatty new platform so one board was listed. Are you sure >> that we need? > > There are several imx6ul based dts in mainline and

Re: [PATCH 1/2] clk: imx6ul: add GPIO clock gates

2018-06-02 Thread Fabio Estevam
Hi Michael, On Sat, Jun 2, 2018 at 11:04 AM, Michael Nazzareno Trimarchi wrote: > ull is a preatty new platform so one board was listed. Are you sure > that we need? There are several imx6ul based dts in mainline and it is better if we can avoid dtb breakage when possible. In this case we can

Re: [PATCH 1/2] clk: imx6ul: add GPIO clock gates

2018-06-02 Thread Fabio Estevam
Hi Michael, On Sat, Jun 2, 2018 at 11:04 AM, Michael Nazzareno Trimarchi wrote: > ull is a preatty new platform so one board was listed. Are you sure > that we need? There are several imx6ul based dts in mainline and it is better if we can avoid dtb breakage when possible. In this case we can

Re: [PATCH 1/2] clk: imx6ul: add GPIO clock gates

2018-06-02 Thread Michael Nazzareno Trimarchi
Hi On Sat, Jun 2, 2018 at 3:48 PM, Fabio Estevam wrote: > Hi Stefan, > > On Tue, May 22, 2018 at 9:25 AM, Stefan Wahren wrote: > >>> --- a/include/dt-bindings/clock/imx6ul-clock.h >>> +++ b/include/dt-bindings/clock/imx6ul-clock.h >>> @@ -242,20 +242,25 @@ >>> #define IMX6UL_CLK_CKO2_PODF

Re: [PATCH 1/2] clk: imx6ul: add GPIO clock gates

2018-06-02 Thread Michael Nazzareno Trimarchi
Hi On Sat, Jun 2, 2018 at 3:48 PM, Fabio Estevam wrote: > Hi Stefan, > > On Tue, May 22, 2018 at 9:25 AM, Stefan Wahren wrote: > >>> --- a/include/dt-bindings/clock/imx6ul-clock.h >>> +++ b/include/dt-bindings/clock/imx6ul-clock.h >>> @@ -242,20 +242,25 @@ >>> #define IMX6UL_CLK_CKO2_PODF

Re: [PATCH 1/2] clk: imx6ul: add GPIO clock gates

2018-06-02 Thread Fabio Estevam
Hi Stefan, On Tue, May 22, 2018 at 9:25 AM, Stefan Wahren wrote: >> --- a/include/dt-bindings/clock/imx6ul-clock.h >> +++ b/include/dt-bindings/clock/imx6ul-clock.h >> @@ -242,20 +242,25 @@ >> #define IMX6UL_CLK_CKO2_PODF 229 >> #define IMX6UL_CLK_CKO2 230 >>

Re: [PATCH 1/2] clk: imx6ul: add GPIO clock gates

2018-06-02 Thread Fabio Estevam
Hi Stefan, On Tue, May 22, 2018 at 9:25 AM, Stefan Wahren wrote: >> --- a/include/dt-bindings/clock/imx6ul-clock.h >> +++ b/include/dt-bindings/clock/imx6ul-clock.h >> @@ -242,20 +242,25 @@ >> #define IMX6UL_CLK_CKO2_PODF 229 >> #define IMX6UL_CLK_CKO2 230 >>

Re: [PATCH 1/2] clk: imx6ul: add GPIO clock gates

2018-06-02 Thread Stephen Boyd
Quoting Stefan Wahren (2018-05-22 05:25:35) > > +++ b/include/dt-bindings/clock/imx6ul-clock.h > > @@ -242,20 +242,25 @@ > > #define IMX6UL_CLK_CKO2_PODF 229 > > #define IMX6UL_CLK_CKO2 230 > > #define IMX6UL_CLK_CKO 231 > > +#define

Re: [PATCH 1/2] clk: imx6ul: add GPIO clock gates

2018-06-02 Thread Stephen Boyd
Quoting Stefan Wahren (2018-05-22 05:25:35) > > +++ b/include/dt-bindings/clock/imx6ul-clock.h > > @@ -242,20 +242,25 @@ > > #define IMX6UL_CLK_CKO2_PODF 229 > > #define IMX6UL_CLK_CKO2 230 > > #define IMX6UL_CLK_CKO 231 > > +#define

Re: [PATCH 1/2] clk: imx6ul: add GPIO clock gates

2018-05-22 Thread Stefan Wahren
Hi Anson, > Anson Huang hat am 22. Mai 2018 um 05:25 geschrieben: > > > i.MX6UL has GPIO clock gates in CCM CCGR, add > them into clock tree for clock management. > > Signed-off-by: Anson Huang > --- > drivers/clk/imx/clk-imx6ul.c | 5

Re: [PATCH 1/2] clk: imx6ul: add GPIO clock gates

2018-05-22 Thread Stefan Wahren
Hi Anson, > Anson Huang hat am 22. Mai 2018 um 05:25 geschrieben: > > > i.MX6UL has GPIO clock gates in CCM CCGR, add > them into clock tree for clock management. > > Signed-off-by: Anson Huang > --- > drivers/clk/imx/clk-imx6ul.c | 5 + >

Re: [PATCH 1/2] clk: imx6ul: add GPIO clock gates

2018-05-22 Thread Fabio Estevam
On Tue, May 22, 2018 at 12:25 AM, Anson Huang wrote: > i.MX6UL has GPIO clock gates in CCM CCGR, add > them into clock tree for clock management. > > Signed-off-by: Anson Huang Reviewed-by: Fabio Estevam

Re: [PATCH 1/2] clk: imx6ul: add GPIO clock gates

2018-05-22 Thread Fabio Estevam
On Tue, May 22, 2018 at 12:25 AM, Anson Huang wrote: > i.MX6UL has GPIO clock gates in CCM CCGR, add > them into clock tree for clock management. > > Signed-off-by: Anson Huang Reviewed-by: Fabio Estevam

[PATCH 1/2] clk: imx6ul: add GPIO clock gates

2018-05-21 Thread Anson Huang
i.MX6UL has GPIO clock gates in CCM CCGR, add them into clock tree for clock management. Signed-off-by: Anson Huang --- drivers/clk/imx/clk-imx6ul.c | 5 + include/dt-bindings/clock/imx6ul-clock.h | 31 ++- 2 files changed, 23

[PATCH 1/2] clk: imx6ul: add GPIO clock gates

2018-05-21 Thread Anson Huang
i.MX6UL has GPIO clock gates in CCM CCGR, add them into clock tree for clock management. Signed-off-by: Anson Huang --- drivers/clk/imx/clk-imx6ul.c | 5 + include/dt-bindings/clock/imx6ul-clock.h | 31 ++- 2 files changed, 23 insertions(+), 13