linux-kernel - Re: [PATCH v3 0/5] drm/sitronix/st7571-i2c: Add support for the ST7567 Controller

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <87y0sobftx.fsf@minerva.mail-host-address-is-not-set>
Date: Wed, 16 Jul 2025 12:16:58 +0200
From: Javier Martinez Canillas <javierm@...hat.com>
To: linux-kernel@...r.kernel.org
Cc: ipedrosa@...hat.com, Conor Dooley <conor+dt@...nel.org>, David Airlie
 <airlied@...il.com>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Maarten
 Lankhorst <maarten.lankhorst@...ux.intel.com>, Marcus Folkesson
 <marcus.folkesson@...il.com>, Maxime Ripard <mripard@...nel.org>, Rob
 Herring <robh@...nel.org>, Simona Vetter <simona@...ll.ch>, Thomas
 Zimmermann <tzimmermann@...e.de>, devicetree@...r.kernel.org,
 dri-devel@...ts.freedesktop.org
Subject: Re: [PATCH v3 0/5] drm/sitronix/st7571-i2c: Add support for the
 ST7567 Controller

Javier Martinez Canillas <javierm@...hat.com> writes:

> This patch-series adds support for the Sitronix ST7567 Controller, which is a
> monochrome Dot Matrix LCD Controller that has SPI, I2C and parallel interfaces.
>
> The st7571-i2c driver only has support for I2C so displays using other transport
> interfaces are currently not supported.
>
> The DRM_FORMAT_R1 pixel format and data commands are the same than what is used
> by the ST7571 controller, so only is needed a different callback that implements
> the expected initialization sequence for the ST7567 chip and a different callback
> to parse the sub-set of DT properties needed by the ST7567.
>
> Patches #1 and #2 are some trivial cleanups for the driver.
>
> Patch #3 is a preparatory change that adds the level of indirection for the DT
> parsing logic.
>
> Patch #4 adds a Device Tree binding schema for the ST7567 Controller.
>
> Patch #5 finally extends the st7571-i2c driver to also support the ST7567 device.
>
> Changes in v3:
> - Fix reset typo in commit message (Marcus Folkesson).
> - Explicitly set ST7571_SET_REVERSE(0) instead of relying on defaults.
>
> Changes in v2:
> - Use a different parse DT function (Thomas Zimmermann).
>
> Javier Martinez Canillas (5):
>   drm/sitronix/st7571-i2c: Fix encoder callbacks function names
>   drm/sitronix/st7571-i2c: Log probe deferral cause for GPIO get failure
>   drm/sitronix/st7571-i2c: Add an indirection level to parse DT
>   dt-bindings: display: Add Sitronix ST7567 LCD Controller
>   drm/sitronix/st7571-i2c: Add support for the ST7567 Controller
>

Pushed to drm-misc (drm-misc-next). Thanks!

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat