Hi Javier,
On Wed, Feb 9, 2022 at 10:12 AM Javier Martinez Canillas
<javierm@xxxxxxxxxx> wrote:
> The ssd130x driver only provides the core support for these devices but it
> does not have any bus transport logic. Add a driver to interface over SPI.
>
> Signed-off-by: Javier Martinez Canillas <javierm@xxxxxxxxxx>
Thanks for your patch!
> --- /dev/null
> +++ b/drivers/gpu/drm/solomon/ssd130x-spi.c
> +static const struct of_device_id ssd130x_of_match[] = {
> + {
> + .compatible = "solomon,ssd1305fb-spi",
This needs an update to the DT bindings.
Hence this may be a good time to deprecate the existing
"solomon,ssd130*fb-i2c" compatible values, and switch to
"solomon,ssd130*fb" instead, for both I2C and SPI.
Of course the I2C subdriver still has to bind against the old values,
too, for backwards compatibility.
> + .data = (void *)&ssd130x_ssd1305_deviceinfo,
The casts are not needed.
> + },
> + {
> + .compatible = "solomon,ssd1306fb-spi",
> + .data = (void *)&ssd130x_ssd1306_deviceinfo,
> + },
> + {
> + .compatible = "solomon,ssd1307fb-spi",
> + .data = (void *)&ssd130x_ssd1307_deviceinfo,
> + },
> + {
> + .compatible = "solomon,ssd1309fb-spi",
> + .data = (void *)&ssd130x_ssd1309_deviceinfo,
> + },
> + { /* sentinel */ },
> +};
> +MODULE_DEVICE_TABLE(of, ssd130x_of_match);
> +
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
