mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* Re: [PATCH] drm/ili9163: Add "1.8-128160EF" to spi_device_id table
       [not found] ` <20231116185124.13887-1-mweshahy@yahoo.com>
@ 2023-11-17  8:03   ` Javier Martinez Canillas
  0 siblings, 0 replies; only message in thread
From: Javier Martinez Canillas @ 2023-11-17  8:03 UTC (permalink / raw)
  To: Mostafa Weshahy, maarten.lankhorst, mripard, tzimmermann,
	airlied, daniel, noralf, mweshahy, dri-devel, linux-kernel,
	daniel

Mostafa Weshahy <mweshahy@yahoo.com> writes:

Hello Mostafa,

> drm/ili9163: Add "1.8-128160EF" to spi_device_id table
>
> Currently enabling CONFIG_TINYDRM_ILI9163 driver - regardless of the device
> tree - results in the below confusing log line:
> SPI driver ili9163 has no spi_device_id for newhaven,1.8-128160EF
>
> This commit fixes this false alarm by adding "1.8-128160EF" to
> spi_device_id table of ili9163 driver.
>

It's not actually a false alarm, this is needed if you want the driver
module to be automatically loaded. Because the SPI core always reports
a MODALIAS of the form "spi:1.8-128160EF" even if the device has been
registered by OF.

So this is an actual fix since this driver can be built as a module.

> Signed-off-by: Mostafa Weshahy <mweshahy@yahoo.com>
> ---

Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2023-11-17  8:04 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20231116185124.13887-1-mweshahy.ref@yahoo.com>
     [not found] ` <20231116185124.13887-1-mweshahy@yahoo.com>
2023-11-17  8:03   ` [PATCH] drm/ili9163: Add "1.8-128160EF" to spi_device_id table Javier Martinez Canillas

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®