Re: [PATCH 4/4] drm: ssd130x: Add support for SINO WEALTH SH1106
From: Chen-Yu Tsai
Date: Mon Apr 04 2022 - 17:38:33 EST
On Fri, Apr 1, 2022 at 6:10 PM Javier Martinez Canillas
> On Wed, Mar 30, 2022 at 9:09 PM Chen-Yu Tsai <wens@xxxxxxxxxx> wrote:
> > From: Chen-Yu Tsai <wens@xxxxxxxx>
> > The SINO WEALTH SH1106 is an OLED display driver that is somewhat
> > compatible with the SSD1306. It supports a slightly wider display,
> > at 132 instead of 128 pixels. The basic commands are the same, but
> > the SH1106 doesn't support the horizontal or vertical address modes.
> > Add support for this display driver. The default values for some of
> > the hardware settings are taken from the datasheet.
> > Signed-off-by: Chen-Yu Tsai <wens@xxxxxxxx>
> > ---
> > drivers/gpu/drm/solomon/ssd130x-i2c.c | 11 +++++++++++
> Thanks a lot for this patch. It's very nice to see that another
> variant of the OLED controller is being supported!
> I wonder if we should also list SH1106 in the
> drivers/gpu/drm/solomon/Kconfig file so people can find it ?
I can add it to the help text if that helps?
Recently someone mentioned that users are more likely to find drivers
via compatible strings though. And I believe there's also a tool in-tree
that finds all drivers given a device tree.
> ah, one comment I forgot in 3/4 but that also applies to this patch, I
> believe the convention in DRM is for the subject line to be
> "drm/ssd130x:" instead of "drm: ssd130x:"
> Reviewed-by: Javier Martinez Canillas <javierm@xxxxxxxxxx>