drivers/mtd/nand/nand_mrvl_nfc.c

Robert Jarzmik robert.jarzmik at free.fr
Wed May 20 11:37:02 PDT 2015


Sascha Hauer <s.hauer at pengutronix.de> writes:

> Hi Andrei,
>
> +Cc Robert Jarzmik <robert.jarzmik at free.fr>
>
> On Tue, May 19, 2015 at 12:23:22PM -0700, Andrei Yakimov wrote:
>> Please,
>>  change 
>>   case NAND_CMD_PARAM:
>>         host->buf_count = 256;
>> To:
>>   host->buf_count = 768;
>> 
>> If first copy of parameter page corrupted you will not able
>> to use ONFI flash.
>
> Thanks for noting. However, like many other opensource projects we
> communicate changes and change requests with formalized patches. Please
> generate one using git format-patch or similar and send it to the list.
> I'll happily make that change then.

Same from me, I'll happily review a patch.
May I add also that host->ndcb3 and host->data_size deserve the same change
within the NAND_CMD_PARAM case statement.

Cheers.

-- 
Robert



More information about the barebox mailing list