Jagan,
thanks for your review!
On Thu, Jul 14, 2016 at 12:01 AM, Jagan Teki <jagannadh.t...@gmail.com> wrote:
> On 14 July 2016 at 00:00, Moritz Fischer <moritz.fisc...@ettus.com> wrote:
>> Signed-off-by: Moritz Fischer <moritz.fisc...@ettus.com>
>> ---
>> drivers/mtd/spi/sf_params.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/mtd/spi/sf_params.c b/drivers/mtd/spi/sf_params.c
>> index 4f37e33..8ae4eea 100644
>> --- a/drivers/mtd/spi/sf_params.c
>> +++ b/drivers/mtd/spi/sf_params.c
>> @@ -83,6 +83,7 @@ const struct spi_flash_params spi_flash_params_table[] = {
>> {"M25P64", 0x202017, 0x0, 64 * 1024, 128, RD_NORM,
>> 0},
>> {"M25P128", 0x202018, 0x0, 256 * 1024, 64, RD_NORM,
>> 0},
>> {"M25PX64", 0x207117, 0x0, 64 * 1024, 128, RD_NORM,
>> SECT_4K},
>> + {"N25Q016A", 0x20bb15, 0x1000, 64 * 1024, 32, RD_NORM,
>> SECT_4K},
>
> Why would we need this ext_id (0x1000) here? I understand this is true
> number for this part but it shouldn't require for probe or require
> somewhere else?
You're right. I just sent v3.
Cheers,
Moritz
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
