[U-Boot,v2,3/5] mtd: sf_probe: remove spi-flash compatible

Message ID 20181206095122.7117-4-narmstrong@baylibre.com
State New
Delegated to: Jagannadha Sutradharudu Teki
Headers show
Series
  • spi-flash: switch to "jedec, spi-nor" binding
Related show

Commit Message

Neil Armstrong Dec. 6, 2018, 9:51 a.m.
Now the "spi-flash" compatible has been removed in the DTS files,
remove this compatible from sf_probe to finally use the Linux "jedec,spi-nor"
compatible.

Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
---
 drivers/mtd/spi/sf_probe.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Jagan Teki Dec. 10, 2018, 12:45 p.m. | #1
+ Simon

On Thu, Dec 6, 2018 at 3:22 PM Neil Armstrong <narmstrong@baylibre.com> wrote:
>
> Now the "spi-flash" compatible has been removed in the DTS files,
> remove this compatible from sf_probe to finally use the Linux "jedec,spi-nor"
> compatible.
>
> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
> ---
>  drivers/mtd/spi/sf_probe.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/drivers/mtd/spi/sf_probe.c b/drivers/mtd/spi/sf_probe.c
> index 7a379914d8..aae39dd601 100644
> --- a/drivers/mtd/spi/sf_probe.c
> +++ b/drivers/mtd/spi/sf_probe.c
> @@ -160,7 +160,6 @@ static const struct dm_spi_flash_ops spi_flash_std_ops = {
>  };
>
>  static const struct udevice_id spi_flash_std_ids[] = {
> -       { .compatible = "spi-flash" },
>         { .compatible = "jedec,spi-nor" },

Sandbox is the user for spi-flash binding, but I'm fine with using
know binding from Linux to use sandbox. Simon is that OK?

Patch

diff --git a/drivers/mtd/spi/sf_probe.c b/drivers/mtd/spi/sf_probe.c
index 7a379914d8..aae39dd601 100644
--- a/drivers/mtd/spi/sf_probe.c
+++ b/drivers/mtd/spi/sf_probe.c
@@ -160,7 +160,6 @@  static const struct dm_spi_flash_ops spi_flash_std_ops = {
 };
 
 static const struct udevice_id spi_flash_std_ids[] = {
-	{ .compatible = "spi-flash" },
 	{ .compatible = "jedec,spi-nor" },
 	{ }
 };