Re: drivers/mtd/nand/nand_mrvl_nfc.c

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Sascha Hauer <s.hauer@xxxxxxxxxxxxxx> writes:

> Hi Andrei,
>
> +Cc Robert Jarzmik <robert.jarzmik@xxxxxxx>
>
> 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

_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox




[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux