Message ID | 20220316090013.8150-5-tommaso.merciai@amarulasolutions.com |
---|---|
State | Superseded |
Delegated to: | Stefano Babic |
Headers | show |
Series | imx8mm: add pwm-imx backlight support | expand |
Hi On Wed, Mar 16, 2022 at 10:02 AM Tommaso Merciai <tommaso.merciai@amarulasolutions.com> wrote: > > Add function to enable_pwm_clck function into clock_imx8mm.c. This > function first configure, then enable pwm1 clock from clock control > register. The following configuration is used: > > source(0) -> 24 MHz ref clock > div(0) -> no division for this clock > > References: > - iMX8MMRM.pdf p 303 > > Signed-off-by: Tommaso Merciai <tommaso.merciai@amarulasolutions.com> > --- > arch/arm/mach-imx/imx8m/clock_imx8mm.c | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/arch/arm/mach-imx/imx8m/clock_imx8mm.c b/arch/arm/mach-imx/imx8m/clock_imx8mm.c > index 49945faf2c..5f2eddf715 100644 > --- a/arch/arm/mach-imx/imx8m/clock_imx8mm.c > +++ b/arch/arm/mach-imx/imx8m/clock_imx8mm.c > @@ -313,6 +313,17 @@ void enable_usboh3_clk(unsigned int enable) > } > } > > +void enable_pwm_clk(unsigned char enable) > +{ > + if (enable) { > + clock_enable(CCGR_PWM1, false); > + clock_set_target_val(PWM1_CLK_ROOT, CLK_ROOT_ON | CLK_ROOT_SOURCE_SEL(0) |CLK_ROOT_PRE_DIV(CLK_ROOT_PRE_DIV1)); > + clock_enable(CCGR_PWM1, true); > + } else { > + clock_enable(CCGR_PWM1, false); > + } > +} > + Show not be somenthing like enable_pwm_clk(enum pwm_id, bool enable) ? > void init_uart_clk(u32 index) > { > /* > -- > 2.25.1 >
On Wed, Mar 16, 2022 at 10:05:47AM +0100, Michael Nazzareno Trimarchi wrote: > Hi > > On Wed, Mar 16, 2022 at 10:02 AM Tommaso Merciai > <tommaso.merciai@amarulasolutions.com> wrote: > > > > Add function to enable_pwm_clck function into clock_imx8mm.c. This > > function first configure, then enable pwm1 clock from clock control > > register. The following configuration is used: > > > > source(0) -> 24 MHz ref clock > > div(0) -> no division for this clock > > > > References: > > - iMX8MMRM.pdf p 303 > > > > Signed-off-by: Tommaso Merciai <tommaso.merciai@amarulasolutions.com> > > --- > > arch/arm/mach-imx/imx8m/clock_imx8mm.c | 11 +++++++++++ > > 1 file changed, 11 insertions(+) > > > > diff --git a/arch/arm/mach-imx/imx8m/clock_imx8mm.c b/arch/arm/mach-imx/imx8m/clock_imx8mm.c > > index 49945faf2c..5f2eddf715 100644 > > --- a/arch/arm/mach-imx/imx8m/clock_imx8mm.c > > +++ b/arch/arm/mach-imx/imx8m/clock_imx8mm.c > > @@ -313,6 +313,17 @@ void enable_usboh3_clk(unsigned int enable) > > } > > } > > > > +void enable_pwm_clk(unsigned char enable) > > +{ > > + if (enable) { > > + clock_enable(CCGR_PWM1, false); > > + clock_set_target_val(PWM1_CLK_ROOT, CLK_ROOT_ON | CLK_ROOT_SOURCE_SEL(0) |CLK_ROOT_PRE_DIV(CLK_ROOT_PRE_DIV1)); > > + clock_enable(CCGR_PWM1, true); > > + } else { > > + clock_enable(CCGR_PWM1, false); > > + } > > +} > > + > > Show not be somenthing like > enable_pwm_clk(enum pwm_id, bool enable) > > ? Hi, Like init_clk_usdhc(u32 index) function in clock_imx8mm.c? Ack, I'll send v2. Thanks, Tommaso > > > void init_uart_clk(u32 index) > > { > > /* > > -- > > 2.25.1 > > > > > -- > Michael Nazzareno Trimarchi > Co-Founder & Chief Executive Officer > M. +39 347 913 2170 > michael@amarulasolutions.com > __________________________________ > > Amarula Solutions BV > Joop Geesinkweg 125, 1114 AB, Amsterdam, NL > T. +31 (0)85 111 9172 > info@amarulasolutions.com > www.amarulasolutions.com
diff --git a/arch/arm/mach-imx/imx8m/clock_imx8mm.c b/arch/arm/mach-imx/imx8m/clock_imx8mm.c index 49945faf2c..5f2eddf715 100644 --- a/arch/arm/mach-imx/imx8m/clock_imx8mm.c +++ b/arch/arm/mach-imx/imx8m/clock_imx8mm.c @@ -313,6 +313,17 @@ void enable_usboh3_clk(unsigned int enable) } } +void enable_pwm_clk(unsigned char enable) +{ + if (enable) { + clock_enable(CCGR_PWM1, false); + clock_set_target_val(PWM1_CLK_ROOT, CLK_ROOT_ON | CLK_ROOT_SOURCE_SEL(0) |CLK_ROOT_PRE_DIV(CLK_ROOT_PRE_DIV1)); + clock_enable(CCGR_PWM1, true); + } else { + clock_enable(CCGR_PWM1, false); + } +} + void init_uart_clk(u32 index) { /*
Add function to enable_pwm_clck function into clock_imx8mm.c. This function first configure, then enable pwm1 clock from clock control register. The following configuration is used: source(0) -> 24 MHz ref clock div(0) -> no division for this clock References: - iMX8MMRM.pdf p 303 Signed-off-by: Tommaso Merciai <tommaso.merciai@amarulasolutions.com> --- arch/arm/mach-imx/imx8m/clock_imx8mm.c | 11 +++++++++++ 1 file changed, 11 insertions(+)