[PATCH v2 04/22] drivers: mtd: spi-nand: add missed add missed MODULE_DEVICE_TABLE()

Frieder Schrempf frieder.schrempf at kontron.de
Tue Aug 5 08:36:48 CEST 2025


Am 05.08.25 um 00:11 schrieb Mikhail Kshevetskiy:
> Signed-off-by: Mikhail Kshevetskiy <mikhail.kshevetskiy at iopsys.eu>

There is some duplication in the subject line of this patch. Also it
should be "missing" not "missed".

The prefix in the subject line for this patch and all the other spinand
patches in this series should not contain "drivers:". And also "spinand"
is more common than "spi-nand", I guess. So the prefix should probably
be "mtd: spinand:".

For simple changes like this that are directly picked from Linux, can
you please try to pick the commits from the Linux tree as-is, meaning we
can preserve the authorship and you can add your SoB-tag for doing the port?

> ---
>  drivers/mtd/nand/spi/core.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/mtd/nand/spi/core.c b/drivers/mtd/nand/spi/core.c
> index d8abd03e919..34e5ac33c63 100644
> --- a/drivers/mtd/nand/spi/core.c
> +++ b/drivers/mtd/nand/spi/core.c
> @@ -1285,12 +1285,14 @@ static const struct spi_device_id spinand_ids[] = {
>  	{ .name = "spi-nand" },
>  	{ /* sentinel */ },
>  };
> +MODULE_DEVICE_TABLE(spi, spinand_ids);
>  
>  #ifdef CONFIG_OF
>  static const struct of_device_id spinand_of_ids[] = {
>  	{ .compatible = "spi-nand" },
>  	{ /* sentinel */ },
>  };
> +MODULE_DEVICE_TABLE(of, spinand_of_ids);
>  #endif
>  
>  static struct spi_mem_driver spinand_drv = {



More information about the U-Boot mailing list