Re: [PATCH] spi-nor: Remove unused spi-nor.o rule

From: Joel Stanley
Date: Wed Jul 22 2020 - 05:31:58 EST


On Wed, 22 Jul 2020 at 09:00, Joel Stanley <joel@xxxxxxxxx> wrote:
>
> In "mtd: spi-nor: Prepare core / manufacturer code split" the file was
> refactored into seperate files, so the rule is no longer required.

Please ignore this patch. I misunderstood how spi-nor-objs worked, and
didn't properly test the patch.

I am still a bit confused as to how spi-nor.o gets linked in, as
there's no spi-nor.o on my filesystem. Can someone explain how it
works?

> Fixes: a0900d0195d2 ("mtd: spi-nor: Prepare core / manufacturer code split")
> Reported-by: CÃdric Le Goater <clg@xxxxxxxx>
> Signed-off-by: Joel Stanley <joel@xxxxxxxxx>
> ---
> drivers/mtd/spi-nor/Makefile | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/mtd/spi-nor/Makefile b/drivers/mtd/spi-nor/Makefile
> index 653923896205..2ede5ef79b01 100644
> --- a/drivers/mtd/spi-nor/Makefile
> +++ b/drivers/mtd/spi-nor/Makefile
> @@ -17,6 +17,5 @@ spi-nor-objs += sst.o
> spi-nor-objs += winbond.o
> spi-nor-objs += xilinx.o
> spi-nor-objs += xmc.o
> -obj-$(CONFIG_MTD_SPI_NOR) += spi-nor.o
>
> obj-$(CONFIG_MTD_SPI_NOR) += controllers/
> --
> 2.27.0
>