From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ying.Liu@freescale.com (Liu Ying) Date: Wed, 28 May 2014 18:12:45 +0800 Subject: [PATCH] pwm: i.MX: Cleanup indentation for register definitions In-Reply-To: <20140528115636.4ad791aa@ipc1.ka-ro> References: <1401185980-29536-1-git-send-email-Ying.Liu@freescale.com> <20140528115636.4ad791aa@ipc1.ka-ro> Message-ID: <5385B69D.50508@freescale.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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 >> Cc: Sascha Hauer >> Cc: Shawn Guo >> Cc: Lothar Wa?mann >> Cc: linux-pwm at vger.kernel.org >> Cc: linux-arm-kernel at lists.infradead.org >> Signed-off-by: Liu Ying >> --- >> 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