On Wed, Dec 07, 2011 at 12:57:44PM +0100, Lothar Waßmann wrote: > Hi, > > Shawn Guo writes: > > I'm fine with this cosmetic patch with the comments below addressed. > > > > On Tue, Dec 06, 2011 at 02:41:27PM +0100, Lothar Waßmann wrote: > > > - Remove bogus parens around numerical arguments in #define's > > > - When defining bit masks, use the #def for the shift count > > > instead of repeating the numbers > > > > This change brings a few 'line over 80 characters' checkpatch warnings. > > > will fix. > > > > - Remove bogus whitespace in indentation > > > - Remove 'break' at end of switch statement > > > > > > Signed-off-by: Lothar Waßmann <LW@xxxxxxxxxxxxxxxxxxx> > > > --- > > > drivers/mmc/host/mxs-mmc.c | 57 +++++++++++++++++++++---------------------- > > > 1 files changed, 28 insertions(+), 29 deletions(-) > > > > > > diff --git a/drivers/mmc/host/mxs-mmc.c b/drivers/mmc/host/mxs-mmc.c > > [...] > > > @@ -277,7 +277,7 @@ static irqreturn_t mxs_mmc_irq_handler(int irq, void *dev_id) > > > > > > stat = readl(host->base + HW_SSP_CTRL1); > > > writel(stat & MXS_MMC_IRQ_BITS, > > > - host->base + HW_SSP_CTRL1 + MXS_CLR_ADDR); > > > > This is an intended indentation with 7 spaces. > > > According to Documentation/CodingStyle spaces are never used for > indentation (except in documentation, Kconfig and comments). > Such nice indentation is all over this driver and even kernel tree. You may want to leave it as it is or fix them all over the driver. -- Regards, Shawn -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
