drm/ili9486: Resolve clash in spi_device_id names
authorDave Stevenson <dave.stevenson@raspberrypi.com>
Fri, 1 Sep 2023 11:17:38 +0000 (12:17 +0100)
committerDom Cobley <popcornmix@gmail.com>
Mon, 19 Feb 2024 11:33:38 +0000 (11:33 +0000)
For "Really Good Reasons" [1] the SPI core requires a match
between compatible device strings and the name in spi_device_id.

The ili9486 driver uses compatible strings "waveshare,rpi-lcd-35"
and "ozzmaker,piscreen", but "rpi-lcd-35" and "piscreen" are missing,
so add them.

Compatible string "ilitek,ili9486" is already used by
staging/fbtft/fb_ili9486, therefore leaving it present in ili9486 as an
spi_device_id causes the incorrect module to be loaded, therefore remove
this id.

[1] https://elixir.bootlin.com/linux/latest/source/drivers/spi/spi.c#L487

Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.com>
drivers/gpu/drm/tiny/ili9486.c

index 938bceed599985b84a0fd5d8f5c67b5e3fe08a51..a6bc9df206a86b151efe5097687d7d9d029ea97a 100644 (file)
@@ -188,7 +188,6 @@ static const struct of_device_id ili9486_of_match[] = {
 MODULE_DEVICE_TABLE(of, ili9486_of_match);
 
 static const struct spi_device_id ili9486_id[] = {
-       { "ili9486", 0 },
        { "rpi-lcd-35", 0 },
        { "piscreen", 0 },
        { }