On Tuesday, December 05, 2017 03:02:13 PM Bartlomiej Zolnierkiewicz wrote: > On Saturday, November 25, 2017 04:04:07 PM Arvind Yadav wrote: > > Fix these checkpatch.pl error: > > ERROR: space prohibited before open square bracket '['. > > > > ERROR: space prohibited after that '~' (ctx:WxW) > > + mask &= ~ (1 << (6 + ATA_SHIFT_UDMA)); > > > > ERROR: spaces required around that '?' (ctx:VxW) > > + long pout_required = board_idx? PDC_133_MHZ:PDC_100_MHZ; > > > > ERROR: that open brace { should be on the previous line > > + const struct ata_port_info *ppi[] = > > + { &pdc2027x_port_info[board_idx], NULL }; > > > > Signed-off-by: Arvind Yadav <arvind.yadav.cs@xxxxxxxxx> > > Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@xxxxxxxxxxx> PS two small things than still can be fixed: * s/sytle error/style errors/ in the patch summary * s/error/errors/ in the patch description Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html