Re: [PATCH 3/3] MAINTAINERS: Add entry for Sitronix ST7920 driver

[Maxime Ripard <mripard@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1241 bytes --]

On Wed, Aug 20, 2025 at 03:10:16PM +0200, Krzysztof Kozlowski wrote:
> On 20/08/2025 14:23, Iker Pedrosa wrote:
> >>>
> >>> +DRM DRIVER FOR SITRONIX ST7920 LCD DISPLAYS
> >>> +M:   Iker Pedrosa <ikerpedrosam@gmail.com>
> >>> +S:   Maintained
> >>> +T:   git https://gitlab.freedesktop.org/drm/misc/kernel.git
> >>
> >>
> >> Drop, unless you have commit rights there. Parent entry already covers
> >> this, doesn't it?
> >>
> > 
> > I don't have them, but I'm working with Javier and I think he does have
> > permissions. Let me ask him when he gets back.
> 
> Javier is not mentioned here. You are adding redundant and useless
> information. T: is for subsystem maintainers, not for individual drivers.

Kinda. I mean, you're absolutely right for pretty that it's implicit in
most places in the kernel.

However, it's not here. The drm-misc tree is meant to collect the
patches for all those small drivers, and we don't have a folder to put
these drivers under.

It was pretty confusing to differentiate a driver maintained through its
own tree, and one maintained through drm-misc, so at least explicitly
having the git tree set to drm-misc is how we show that's where the
patches are going to land.

Maxime

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 273 bytes --]