Re: [PATCH v1 1/1] drm/i915: Include only needed headers in ascii85.h

From: Andy Shevchenko
Date: Wed Apr 07 2021 - 09:41:43 EST


On Wed, Apr 07, 2021 at 04:40:10PM +0300, Andy Shevchenko wrote:
> On Wed, Apr 07, 2021 at 03:46:37PM +0300, Jani Nikula wrote:
> > On Wed, 07 Apr 2021, Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote:
> > > The ascii85.h is user of exactly two headers, i.e. math.h and types.h.
> > > There is no need to carry on entire kernel.h.
> > >
> > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> >
> > That's hardly drm/i915 specific!
>
> It was originated from i915!
>
> > Reviewed-by: Jani Nikula <jani.nikula@xxxxxxxxx>
>
> Thanks!
>
> > But who's going to pick it up and, hopefully, change the subject prefix
> > to ascii85 or something?
>
> It's drm specific. Should I leave drm and resend? Or what is the general rule
> for drm misc changes?

Since the topic comes up about drm misc, can somebody approve push access from
me to that?

https://gitlab.freedesktop.org/freedesktop/freedesktop/-/issues/311

--
With Best Regards,
Andy Shevchenko