On 05/28/2014 05:56 PM, Lothar Waßmann wrote: > Hi, > > Liu Ying wrote: >> This patch contains no logic change to cleanup indentation >> for register definitions only. >> >> Cc: Thierry Reding <[email protected]> >> Cc: Sascha Hauer <[email protected]> >> Cc: Shawn Guo <[email protected]> >> Cc: Lothar Waßmann <[email protected]> >> Cc: [email protected] >> Cc: [email protected] >> Signed-off-by: Liu Ying <[email protected]> >> --- >> drivers/pwm/pwm-imx.c | 26 +++++++++++++------------- >> 1 file changed, 13 insertions(+), 13 deletions(-) >> >> diff --git a/drivers/pwm/pwm-imx.c b/drivers/pwm/pwm-imx.c >> index cc47733..0784554 100644 >> --- a/drivers/pwm/pwm-imx.c >> +++ b/drivers/pwm/pwm-imx.c >> @@ -21,24 +21,24 @@ >> >> /* i.MX1 and i.MX21 share the same PWM function block: */ >> >> -#define MX1_PWMC 0x00 /* PWM Control Register */ >> -#define MX1_PWMS 0x04 /* PWM Sample Register */ >> -#define MX1_PWMP 0x08 /* PWM Period Register */ >> +#define MX1_PWMC 0x00 /* PWM Control Register */ >> +#define MX1_PWMS 0x04 /* PWM Sample Register */ >> +#define MX1_PWMP 0x08 /* PWM Period Register */ >> >> -#define MX1_PWMC_EN (1 << 4) >> +#define MX1_PWMC_EN (1 << 4) >> >> /* i.MX27, i.MX31, i.MX35 share the same PWM function block: */ >> >> -#define MX3_PWMCR 0x00 /* PWM Control Register */ >> -#define MX3_PWMSAR 0x0C /* PWM Sample Register */ >> -#define MX3_PWMPR 0x10 /* PWM Period Register */ >> -#define MX3_PWMCR_PRESCALER(x) (((x - 1) & 0xFFF) << 4) >> -#define MX3_PWMCR_DOZEEN (1 << 24) >> -#define MX3_PWMCR_WAITEN (1 << 23) >> +#define MX3_PWMCR 0x00 /* PWM Control Register */ >> +#define MX3_PWMSAR 0x0C /* PWM Sample Register */ >> +#define MX3_PWMPR 0x10 /* PWM Period Register */ >> +#define MX3_PWMCR_PRESCALER(x) (((x - 1) & 0xFFF) << 4) > ^ > You could also add the missing () around the macro argument when you > are changing this line anyway. >
I may generate a separate patch to do that. Thanks. > > Lothar Waßmann > -- Liu Ying -- To unsubscribe from this list: send the line "unsubscribe linux-pwm" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
