Re: [PATCH v3 3/3] MAINTAINERS: add it66121 HDMI bridge driver entry

From: Robert Foss <robert.foss@linaro.org>
To: Neil Armstrong <narmstrong@baylibre.com>
Cc: Andrzej Hajda <a.hajda@samsung.com>,
	Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
	paul@crapouillou.net, Jonas Karlman <jonas@kwiboo.se>,
	Jernej Skrabec <jernej.skrabec@siol.net>,
	dri-devel <dri-devel@lists.freedesktop.org>,
	linux-kernel <linux-kernel@vger.kernel.org>,
	Phong LE <ple@baylibre.com>
Subject: Re: [PATCH v3 3/3] MAINTAINERS: add it66121 HDMI bridge driver entry
Date: Tue, 13 Apr 2021 19:53:52 +0200	[thread overview]
Message-ID: <CAG3jFysHBhj-mj_WAmHNaTDdmzkF2UwTmWc8Xnzb1zdcFFYN7A@mail.gmail.com> (raw)
In-Reply-To: <bb9196e3-e2d0-e44b-0cf1-cb469ea2edd7@baylibre.com>

On Mon, 12 Apr 2021 at 17:47, Neil Armstrong <narmstrong@baylibre.com> wrote:
>
> On 12/04/2021 17:46, Neil Armstrong wrote:
> > From: Phong LE <ple@baylibre.com>
> >
> > Add Neil Armstrong and myself as maintainers
> >
> > Signed-off-by: Phong LE <ple@baylibre.com>
>
> Obviously:
> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
>
> > ---
> >  MAINTAINERS | 8 ++++++++
> >  1 file changed, 8 insertions(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 9ae8444c96b4..ff6450c83049 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -9515,6 +9515,14 @@ Q:     http://patchwork.linuxtv.org/project/linux-media/list/
> >  T:   git git://linuxtv.org/anttip/media_tree.git
> >  F:   drivers/media/tuners/it913x*
> >
> > +ITE IT66121 HDMI BRIDGE DRIVER
> > +M:   Phong LE <ple@baylibre.com>
> > +M:   Neil Armstrong <narmstrong@baylibre.com>
> > +S:   Maintained
> > +F:   drivers/gpu/drm/bridge/ite-it66121.c
> > +T:   git git://anongit.freedesktop.org/drm/drm-misc

I'm getting a checkpatch --strict warning here. The two above lines
should switch places.

WARNING: Misordered MAINTAINERS entry - list 'T:' before 'F:'
#27: FILE: MAINTAINERS:9537:
+F:    drivers/gpu/drm/bridge/ite-it66121.c
+T:    git git://anongit.freedesktop.org/drm/drm-misc

With this warning silenced, feel free to add my r-b.

Reviewed-by: Robert Foss <robert.foss@linaro.org>

> > +F:   Documentation/devicetree/bindings/display/bridge/ite,it66121.yaml
> > +
> >  IVTV VIDEO4LINUX DRIVER
> >  M:   Andy Walls <awalls@md.metrocast.net>
> >  L:   linux-media@vger.kernel.org
> >
>