[U-Boot] [PATCH v2 3/4] mtd: nand: mxs: Add comment for calculating ECC strength
Jörg Krause
joerg.krause at embedded.rocks
Tue Apr 14 08:29:11 CEST 2015
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 at 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
More information about the U-Boot
mailing list