On Thu, Oct 02, 2014 at 01:12:35AM +0300, Giedrius Statkevicius wrote: > diff --git a/drivers/staging/rts5208/rtsx_chip.c b/drivers/staging/rts5208/rtsx_chip.c > index 1411471..d604341 100644 > --- a/drivers/staging/rts5208/rtsx_chip.c > +++ b/drivers/staging/rts5208/rtsx_chip.c > @@ -291,7 +291,9 @@ int rtsx_reset_chip(struct rtsx_chip *chip) > if (chip->dynamic_aspm) { > if (CHK_SDIO_EXIST(chip)) { > if (CHECK_PID(chip, 0x5288)) { These two conditions can be combined and the indenting pulled in one tab. if (CHK_SDIO_EXIST(chip) && CHECK_PID(chip, 0x5288)) { } > @@ -988,8 +995,10 @@ void rtsx_polling_func(struct rtsx_chip *chip) > > turn_off_led(chip, LED_GPIO); > > - if (chip->auto_power_down && !chip->card_ready && !chip->sd_io) > - rtsx_force_power_down(chip, SSC_PDCTL | OC_PDCTL); > + if (chip->auto_power_down && !chip->card_ready && > + !chip->sd_io) Line this up like this: if (chip->auto_power_down && !chip->card_ready && !chip->sd_io) [tab][tab][tab][space][space][space][space]!chip->sd_io) > + rtsx_force_power_down(chip, > + SSC_PDCTL | OC_PDCTL); > > } > } regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel