Hello Heiko, On Di, 2015-04-14 at 08:12 +0200, Heiko Schocher wrote: > Hello Jörg, > > Am 13.04.2015 22:17, schrieb Jörg Krause: > > Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks> > > --- > > drivers/mtd/nand/mxs_nand.c | 7 +++++++ > > 1 file changed, 7 insertions(+) > > nitpick only ...
I'm unsure what this comment means translated to German. Something like small changes only? > > > diff --git a/drivers/mtd/nand/mxs_nand.c > > b/drivers/mtd/nand/mxs_nand.c > > index 912fed8..76e47ab 100644 > > --- a/drivers/mtd/nand/mxs_nand.c > > +++ b/drivers/mtd/nand/mxs_nand.c > > @@ -148,6 +148,13 @@ static inline uint32_t > > mxs_nand_get_ecc_strength(uint32_t page_data_size, > > { > > int ecc_strength; > > > > + /* Determine the ECC layout with the formula: > > wrong comment style ... please fix also in patch 4/4... thanks. Checkpatch did not complain and I did not know there is a coding style for comments. Should it be: /* * Determine the ECC layout... Bye Jörg _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot