Re: [PATCH v2 3/5] drm/solomon: Add ssd130x new compatible strings and deprecate old ones.

2022-04-12 Thread Javier Martinez Canillas
On 4/12/22 09:19, Geert Uytterhoeven wrote:
> Hi Javier,
> 
> On Mon, Apr 11, 2022 at 11:12 PM Javier Martinez Canillas
>  wrote:
>> The current compatible strings for SSD130x I2C controllers contain an "fb"
>> and "-i2c" suffixes. These have been deprecated and more correct ones were
>> added, that don't encode a subsystem or bus used to interface the devices.
>>
>> Signed-off-by: Javier Martinez Canillas 
>> Acked-by: Mark Brown 
>> ---
>>
>> (no changes since v1)
> 
> That's not really true ;-)
>

Ups, I did indeed update the compatible strings to drop the "-i2c" as you
asked. I'll add that in the patch history for v2 when posting v3.
 
> Reviewed-by: Geert Uytterhoeven 
> 

Thanks!

-- 
Best regards,

Javier Martinez Canillas
Linux Engineering
Red Hat



Re: [PATCH v2 3/5] drm/solomon: Add ssd130x new compatible strings and deprecate old ones.

2022-04-12 Thread Geert Uytterhoeven
Hi Javier,

On Mon, Apr 11, 2022 at 11:12 PM Javier Martinez Canillas
 wrote:
> The current compatible strings for SSD130x I2C controllers contain an "fb"
> and "-i2c" suffixes. These have been deprecated and more correct ones were
> added, that don't encode a subsystem or bus used to interface the devices.
>
> Signed-off-by: Javier Martinez Canillas 
> Acked-by: Mark Brown 
> ---
>
> (no changes since v1)

That's not really true ;-)

Reviewed-by: Geert Uytterhoeven 

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

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


[PATCH v2 3/5] drm/solomon: Add ssd130x new compatible strings and deprecate old ones.

2022-04-11 Thread Javier Martinez Canillas
The current compatible strings for SSD130x I2C controllers contain an "fb"
and "-i2c" suffixes. These have been deprecated and more correct ones were
added, that don't encode a subsystem or bus used to interface the devices.

Signed-off-by: Javier Martinez Canillas 
Acked-by: Mark Brown 
---

(no changes since v1)

 drivers/gpu/drm/solomon/ssd130x-i2c.c | 21 +
 1 file changed, 21 insertions(+)

diff --git a/drivers/gpu/drm/solomon/ssd130x-i2c.c 
b/drivers/gpu/drm/solomon/ssd130x-i2c.c
index d099b241dd3f..87abe1fe31fc 100644
--- a/drivers/gpu/drm/solomon/ssd130x-i2c.c
+++ b/drivers/gpu/drm/solomon/ssd130x-i2c.c
@@ -87,6 +87,27 @@ static struct ssd130x_deviceinfo ssd130x_ssd1309_deviceinfo 
= {
 };
 
 static const struct of_device_id ssd130x_of_match[] = {
+   {
+   .compatible = "sinowealth,sh1106",
+   .data = _sh1106_deviceinfo,
+   },
+   {
+   .compatible = "solomon,ssd1305",
+   .data = _ssd1305_deviceinfo,
+   },
+   {
+   .compatible = "solomon,ssd1306",
+   .data = _ssd1306_deviceinfo,
+   },
+   {
+   .compatible = "solomon,ssd1307",
+   .data = _ssd1307_deviceinfo,
+   },
+   {
+   .compatible = "solomon,ssd1309",
+   .data = _ssd1309_deviceinfo,
+   },
+   /* Deprecated but kept for backward compatibility */
{
.compatible = "sinowealth,sh1106-i2c",
.data = _sh1106_deviceinfo,
-- 
2.35.1