lkml.org 
[lkml]   [2023]   [Oct]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH 1/8] drm/solomon: Rename ssd130x driver to ssd13xx
Hi,

On Mon, Oct 09, 2023 at 08:34:15PM +0200, Javier Martinez Canillas wrote:
> The driver only supports the SSD130x family of Solomon OLED controllers
> now, but will be extended to also support the SSD132x (4-bit grayscale)
> and SSD133x (16-bit RGB) controller families. Rename driver to ssd13xx.
>
> Signed-off-by: Javier Martinez Canillas <javierm@redhat.com>

I'm not sure it's worth it. I understand what you want to achieve, but
this will create conflicts, affect the configuration of everyone
enabling that driver, etc.

And we have plenty of drivers that don't match all the devices they
support anyway.

Plus ....

> @@ -11,10 +11,10 @@
> #include <linux/i2c.h>
> #include <linux/module.h>
>
> -#include "ssd130x.h"
> +#include "ssd13xx.h"
>
> -#define DRIVER_NAME "ssd130x-i2c"
> -#define DRIVER_DESC "DRM driver for Solomon SSD130x OLED displays (I2C)"
> +#define DRIVER_NAME "ssd13xx-i2c"
> +#define DRIVER_DESC "DRM driver for Solomon SSD13xx OLED displays (I2C)"
>
> static const struct regmap_config ssd130x_i2c_regmap_config = {
> .reg_bits = 8,

... We now end up with a lot of inconsistencies where some things are
called ssd130x and others ssd13xx.

Maxime
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2023-10-10 10:09    [W:0.175 / U:0.720 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site