Re: [PATCH v1 33/33] drm/hisilicon: drop use of drmP.h

From: Sam Ravnborg <sam@ravnborg.org>
To: Emil Velikov <emil.l.velikov@gmail.com>
Cc: "Gerd Hoffmann" <kraxel@redhat.com>,
	"Hans de Goede" <hdegoede@redhat.com>,
	"Jani Nikula" <jani.nikula@intel.com>,
	"Chen Feng" <puck.chen@hisilicon.com>,
	"John Garry" <john.garry@huawei.com>,
	"ML dri-devel" <dri-devel@lists.freedesktop.org>,
	"Junwei Zhang" <Jerry.Zhang@amd.com>,
	"Xinliang Liu" <z.liuxinliang@hisilicon.com>,
	"David Airlie" <airlied@linux.ie>,
	"Xinwei Kong" <kong.kongxinwei@hisilicon.com>,
	"Laurent Pinchart" <laurent.pinchart@ideasonboard.com>,
	"Thomas Zimmermann" <tzimmermann@suse.de>,
	"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
	"Rongrong Zou" <zourongrong@gmail.com>,
	"Alex Deucher" <alexander.deucher@amd.com>,
	"Thomas Gleixner" <tglx@linutronix.de>,
	"Christian König" <christian.koenig@amd.com>,
	"Allison Randal" <allison@lohutok.net>,
	"Souptick Joarder" <jrdr.linux@gmail.com>
Subject: Re: [PATCH v1 33/33] drm/hisilicon: drop use of drmP.h
Date: Mon, 15 Jul 2019 18:35:21 +0200	[thread overview]
Message-ID: <20190715163521.GM27038@ravnborg.org> (raw)
In-Reply-To: <20190715103451.GA20839@arch-x1c3>

On Mon, Jul 15, 2019 at 11:34:51AM +0100, Emil Velikov wrote:
> On 2019/07/15, Sam Ravnborg wrote:
> > Hi Emil.
> > 
> > > > ---
> > > > The list of cc: was too large to add all recipients to the cover letter.
> > > > Please find cover letter here:
> > > > https://lists.freedesktop.org/archives/dri-devel/2019-June/thread.html
> > > > Search for "drm: drop use of drmp.h in drm-misc"
> > > >
> > > Speaking of long CC list, most patches are ok yet this has gone a bit crazy.
> > > How did you manage to pull such a long list? The get_maintainer.pl
> > > script shows a total of 17 for all of hibmc and kirin.
> > 
> > Sometimes scripts/get_maintainer.pl outputs a lot of people.
> > Sometimes only a few.
> > I have not looked into details why, and for this patch series
> > I used "dim add-missing-cc" to most patches.
> > 
> This patch seems like a good candidate to fixup dim. Can we volunteer
> you for the task ;-)
Naah, it is somewhere in get_maintainers.pl and this is too much
perl for my liking.

> > > Either way, since you've built-tested these (and conflicts are a
> > > matter of #include) for the lot:
> > > Acked-by: Emil Velikov <emil.velikov@collabora.com>
> > Just to be sure. Was this an Ack for the full series or only this patch?
> > I started processing the patches, and the ones where I have got an
> > explicit ack or r-b is queued already (without your ack).
> > 
> For the lot -> For the series.
Thanks for the feedback.

For now I have added your ack on all, and pushed the ones where I had
feedback from others too.

Will resend the rest of the patches, because my git send-email somehow
lost several recipients in first try.

	Sam

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel