[U-Boot] [PATCH v2 3/5] i.MX6: define struct pwm_regs and PWMCR_* defines

[Heiko Schocher <hs@denx.de>]

Hello Stefano,

Am 15.07.2014 10:58, schrieb Stefano Babic:
> Hi Heiko,
>
> On 15/07/2014 10:52, Heiko Schocher wrote:
>
>>> I see. What do you mind to add a little effort and move the setup of the
>>> PWM from the aristaneos board to a PWM driver ? I see there is not (yet)
>>> such a driver, but why not ?
>>
>> Yes, I can do that ... Hmm, where do you think is a good place for this
>> driver?
>>
>
> IMHO you can create a drivers/pwm directory and add a pwm-imx.c driver.
> This reflects the same we have in kernel.

Yes, that was also my first thought, but we have a API for this in
include/pwm.h
but no drivers/pwm ... so I searched pwm_init and found:

pollux:u-boot hs [20140715] $ grep -lr pwm_init .
./include/.pwm.h.swp
./include/pwm.h
./arch/arm/include/asm/arch-tegra20/pwm.h
./arch/arm/cpu/armv7/s5p-common/pwm.c
./arch/arm/cpu/armv7/s5p-common/timer.c
./arch/arm/cpu/armv7/tegra20/pwm.c
./board/nvidia/common/board.c
pollux:u-boot hs [20140715] $

I think, it would better to add a "arch/arm/cpu/armv7/mx6/pwm.c"

Would this be ok?

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany