Re: [PATCH 16/25] mtd: spi-nor: winbond: Prepare introduction of W25QxxPW-Q/N parts

[Miquel Raynal <miquel.raynal@bootlin.com>]

>> +static bool winbond_pw_with_ecc_match(const struct spi_nor *nor)
>> +{
>> +	const struct spi_nor_id *id = nor->info->id;
>> +
>> +	if (!winbond_pw_match(nor))
>> +		return false;
>> +
>> +	/* W25Q33PW chips (id[2] == 0x16) do not have built-in ECC support */
>
> The commit message mentions ECC is available for >= 32Mbit. Doesn't
> align with this comment. Also about the 8MBit and 16Mbit ones?

You're right, there are 8 and 16Mb chips which I haven't in hands and
therefore forgot about. The return line below is correct.

> "W25QxxPW chips smaller than 32MBit doesn't provide built-in ECC
> support"?

32Mb chips do not have ECC capability either, so maybe:

/* W25QxxPW densities <= 32 Mbit (id[2] <= 0x16) do not have built-in ECC support */

> If you like I could rewrite it as I'll apply it. The rest of this
> series looks good.

Yes, thank you!

>
>> +	return id->len == 3 && id->bytes[2] >= 0x17;
>
> Then this makes more sense.
>
> -michael

Thanks,
Miquèl