mbox series

[0/7] fbdev: Clean up include dependencies in header

Message ID 20240212101712.23675-1-tzimmermann@suse.de (mailing list archive)
Headers show
Series fbdev: Clean up include dependencies in header | expand

Message

Thomas Zimmermann Feb. 12, 2024, 10:13 a.m. UTC 
Remove unnecessary dependencies in the include statements of the
header file <linux/fb.h>. Several files throughout the kernel include
the fbdev header, so reducing dependencies positively affects other
subsystems as well. Also fix up nouveau, which needs backlight.h in
one of its source files.

Thomas Zimmermann (7):
  drm/nouveau: Include <linux/backlight.h>
  fbdev: Do not include <linux/backlight.h> in header
  fbdev: Do not include <linux/fs.h> in header
  fbdev: Do not include <linux/notifier.h> in header
  fbdev: Do not include <linux/slab.h> in header
  fbdev: Clean up forward declarations in header file
  fbdev: Clean up include statements in header file

 drivers/gpu/drm/nouveau/dispnv50/disp.c |  1 +
 include/linux/fb.h                      | 24 +++++++++++++-----------
 2 files changed, 14 insertions(+), 11 deletions(-)

Comments

Jani Nikula Feb. 12, 2024, 10:24 a.m. UTC | #1 
On Mon, 12 Feb 2024, Thomas Zimmermann <tzimmermann@suse.de> wrote:
> Remove unnecessary dependencies in the include statements of the
> header file <linux/fb.h>. Several files throughout the kernel include
> the fbdev header, so reducing dependencies positively affects other
> subsystems as well. Also fix up nouveau, which needs backlight.h in
> one of its source files.

On the series,

Reviewed-by: Jani Nikula <jani.nikula@intel.com>

> Thomas Zimmermann (7):
>   drm/nouveau: Include <linux/backlight.h>
>   fbdev: Do not include <linux/backlight.h> in header
>   fbdev: Do not include <linux/fs.h> in header
>   fbdev: Do not include <linux/notifier.h> in header
>   fbdev: Do not include <linux/slab.h> in header
>   fbdev: Clean up forward declarations in header file
>   fbdev: Clean up include statements in header file
>
>  drivers/gpu/drm/nouveau/dispnv50/disp.c |  1 +
>  include/linux/fb.h                      | 24 +++++++++++++-----------
>  2 files changed, 14 insertions(+), 11 deletions(-)