[PATCH 02/10] mtd: add a helper to check the SLC/MLC nand chip

Huang Shijie b32955 at freescale.com
Mon Aug 12 22:35:30 EDT 2013


于 2013年08月13日 08:52, Brian Norris 写道:
>> -	if (denali->nand.cellinfo&  NAND_CI_CELLTYPE_MSK&&
>> >  +	if (!nand_is_slc(&denali->nand)&&
> According to my recommendations in patch 1, this would only need to be:
>
>    if (denali->nand.bits_per_cell>  1&&  ...)
>
> IMO, that is plenty readable, then we don't need the helper.
>

The drivers may also check the SLC/MLC.

IMO, using a helper is more readable :)

Huang Shijie





More information about the linux-mtd mailing list