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

2021-04-07 Thread Andy Shevchenko
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 > > wrote: > > > The ascii85.h is user of exactly two headers, i.e. math.h and types.h. > > > There is no need to carry on

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

2021-04-07 Thread Andy Shevchenko
On Wed, Apr 07, 2021 at 03:46:37PM +0300, Jani Nikula wrote: > On Wed, 07 Apr 2021, Andy Shevchenko > 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 > > That's hardly

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

2021-04-07 Thread Jani Nikula
On Wed, 07 Apr 2021, Andy Shevchenko 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 That's hardly drm/i915 specific! Reviewed-by: Jani Nikula But who's going to pick it up and,

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

2021-04-07 Thread Andy Shevchenko
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 --- include/linux/ascii85.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/include/linux/ascii85.h b/include/linux/ascii85.h