[PATCH 3/4] mtd: spi-nor: micron-st: add mt35xu01gbba support
Tudor Ambarus
tudor.ambarus at linaro.org
Tue Nov 11 00:23:40 PST 2025
did you use html format when replying? Use plaintext, please.
On 11/11/25 9:54 AM, Bough Chen wrote:
>>> + .size = SZ_128M,
> I did consider to remove the size and other flags since SFDP can handle that, but if remove the .size here.
> Micron spi-nor do not have a chance to execute
>
> params->set_octal_dtr = micron_st_nor_set_octal_dtr;
>
> because in micron-st, params->set_octal_dtr is defined in flash_info->fixups-> default_init.
> And flash_info->fixups-> default_init can only be called in spi_nor_init_params_deprecated-> spi_nor_manufacturer_init_params
> If no .size define in flash_info, spi_nor_needs_sfdp() will return true, no chance to call spi_nor_init_params_deprecated.
>
> If need to remove the .size here, seems to move the params->set_octal_dtr to spi_nor_manufacturer-> fixups-> late_init or flash_info->fixups->late_init.
>
> I notice macronix already did like that, I will add this change in the next version.
Yes, move that to late_int(). We'd like to remove default_init()
if possible.
Cheers,
ta
More information about the linux-mtd
mailing list