| Message ID | 20190525204319.28977-1-sam@ravnborg.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | drm/gma500: drop use of drmP.h | expand |
On Sat, May 25, 2019 at 10:43 PM Sam Ravnborg <sam@ravnborg.org> wrote: > > Just a quick (final) probe. If there are no further feedback I will > commit this set sunday. > Added intel-gfx@lists.freedesktop.org just to get a bit more coverage. Hi Sam v2 of this series is already applied to drm-misc-next Thanks Patrik > > > The following patchset remove use of the deprecated drmP.h > header file in the gma500 driver. > > As preparation an empty header file is removed and a dependency on > drm_os_linux.h is dropped. > The local header files are made more self-contained to allow > them to be included in alphabetical order in the files where they are used. > > When removing drmP.h the include files are divided up in blocks: > \#include <linux/*> > > \#include <asm/*> > > \#include <drm/*> > > \#include "" > > Within each block the include files are sorted alphabetically > > Build tested with allmodconfig and allyesconfig for x86, arm, alpha and more. > > Patchset made on top of drm-misc-next > > v2: > - Be consitent in dividing includes files into blocks > - Sort all include files, not only the blocks touched > - Made local header files more self-contained > To allow them to be included in alphabetically order > > v3: > - Collect acks from Daniel and Patrik > > Sam > > Sam Ravnborg (5): > drm/gma500: remove empty gma_drm.h header file > drm/gma500: drop drmP.h from header files > drm/gma500: make local header files more self-contained > drm/gma500: drop use of DRM_UDELAY wrapper > drm/gma500: drop drmp.h include from all .c files > > > drivers/gpu/drm/gma500/accel_2d.c | 18 +++++++-------- > drivers/gpu/drm/gma500/blitter.h | 2 ++ > drivers/gpu/drm/gma500/cdv_device.c | 13 ++++++----- > drivers/gpu/drm/gma500/cdv_device.h | 4 ++++ > drivers/gpu/drm/gma500/cdv_intel_crt.c | 8 +++---- > drivers/gpu/drm/gma500/cdv_intel_display.c | 10 ++++---- > drivers/gpu/drm/gma500/cdv_intel_dp.c | 9 ++++---- > drivers/gpu/drm/gma500/cdv_intel_hdmi.c | 9 ++++---- > drivers/gpu/drm/gma500/cdv_intel_lvds.c | 9 ++++---- > drivers/gpu/drm/gma500/framebuffer.c | 24 +++++++++---------- > drivers/gpu/drm/gma500/framebuffer.h | 1 - > drivers/gpu/drm/gma500/gem.c | 5 ++-- > drivers/gpu/drm/gma500/gma_device.c | 1 - > drivers/gpu/drm/gma500/gma_device.h | 1 + > drivers/gpu/drm/gma500/gma_display.c | 12 +++++++--- > drivers/gpu/drm/gma500/gma_display.h | 3 +++ > drivers/gpu/drm/gma500/gtt.c | 5 ++-- > drivers/gpu/drm/gma500/gtt.h | 1 - > drivers/gpu/drm/gma500/intel_bios.c | 6 ++--- > drivers/gpu/drm/gma500/intel_bios.h | 3 +-- > drivers/gpu/drm/gma500/intel_gmbus.c | 11 +++++---- > drivers/gpu/drm/gma500/intel_i2c.c | 4 +++- > drivers/gpu/drm/gma500/mdfld_device.c | 16 +++++++------ > drivers/gpu/drm/gma500/mdfld_dsi_dpi.c | 4 +++- > drivers/gpu/drm/gma500/mdfld_dsi_output.c | 12 ++++++---- > drivers/gpu/drm/gma500/mdfld_dsi_output.h | 8 +++---- > drivers/gpu/drm/gma500/mdfld_dsi_pkg_sender.c | 4 +++- > drivers/gpu/drm/gma500/mdfld_intel_display.c | 11 +++++---- > drivers/gpu/drm/gma500/mdfld_tmd_vid.c | 2 ++ > drivers/gpu/drm/gma500/mid_bios.c | 5 ++-- > drivers/gpu/drm/gma500/mid_bios.h | 1 + > drivers/gpu/drm/gma500/mmu.c | 6 +++-- > drivers/gpu/drm/gma500/oaktrail.h | 2 ++ > drivers/gpu/drm/gma500/oaktrail_crtc.c | 8 ++++--- > drivers/gpu/drm/gma500/oaktrail_device.c | 20 ++++++++-------- > drivers/gpu/drm/gma500/oaktrail_hdmi.c | 8 ++++--- > drivers/gpu/drm/gma500/oaktrail_lvds.c | 6 ++--- > drivers/gpu/drm/gma500/oaktrail_lvds_i2c.c | 11 ++++----- > drivers/gpu/drm/gma500/power.h | 4 +++- > drivers/gpu/drm/gma500/psb_device.c | 12 +++++----- > drivers/gpu/drm/gma500/psb_drv.c | 33 +++++++++++++++++---------- > drivers/gpu/drm/gma500/psb_drv.h | 16 ++++++------- > drivers/gpu/drm/gma500/psb_intel_display.c | 7 +++--- > drivers/gpu/drm/gma500/psb_intel_lvds.c | 5 ++-- > drivers/gpu/drm/gma500/psb_intel_modes.c | 2 +- > drivers/gpu/drm/gma500/psb_intel_sdvo.c | 15 ++++++------ > drivers/gpu/drm/gma500/psb_irq.c | 9 ++++---- > drivers/gpu/drm/gma500/psb_irq.h | 2 +- > drivers/gpu/drm/gma500/psb_lid.c | 6 ++--- > drivers/gpu/drm/gma500/tc35876x-dsi-lvds.c | 13 +++++++---- > include/drm/gma_drm.h | 25 -------------------- > 51 files changed, 233 insertions(+), 199 deletions(-) >
Hi Patrik.
> v2 of this series is already applied to drm-misc-next
Thanks, I had not rebased on top of drm-misc-next - obviously.
Sam
Just a quick (final) probe. If there are no further feedback I will commit this set sunday. Added intel-gfx@lists.freedesktop.org just to get a bit more coverage. The following patchset remove use of the deprecated drmP.h header file in the gma500 driver. As preparation an empty header file is removed and a dependency on drm_os_linux.h is dropped. The local header files are made more self-contained to allow them to be included in alphabetical order in the files where they are used. When removing drmP.h the include files are divided up in blocks: \#include <linux/*> \#include <asm/*> \#include <drm/*> \#include "" Within each block the include files are sorted alphabetically Build tested with allmodconfig and allyesconfig for x86, arm, alpha and more. Patchset made on top of drm-misc-next v2: - Be consitent in dividing includes files into blocks - Sort all include files, not only the blocks touched - Made local header files more self-contained To allow them to be included in alphabetically order v3: - Collect acks from Daniel and Patrik Sam Sam Ravnborg (5): drm/gma500: remove empty gma_drm.h header file drm/gma500: drop drmP.h from header files drm/gma500: make local header files more self-contained drm/gma500: drop use of DRM_UDELAY wrapper drm/gma500: drop drmp.h include from all .c files drivers/gpu/drm/gma500/accel_2d.c | 18 +++++++-------- drivers/gpu/drm/gma500/blitter.h | 2 ++ drivers/gpu/drm/gma500/cdv_device.c | 13 ++++++----- drivers/gpu/drm/gma500/cdv_device.h | 4 ++++ drivers/gpu/drm/gma500/cdv_intel_crt.c | 8 +++---- drivers/gpu/drm/gma500/cdv_intel_display.c | 10 ++++---- drivers/gpu/drm/gma500/cdv_intel_dp.c | 9 ++++---- drivers/gpu/drm/gma500/cdv_intel_hdmi.c | 9 ++++---- drivers/gpu/drm/gma500/cdv_intel_lvds.c | 9 ++++---- drivers/gpu/drm/gma500/framebuffer.c | 24 +++++++++---------- drivers/gpu/drm/gma500/framebuffer.h | 1 - drivers/gpu/drm/gma500/gem.c | 5 ++-- drivers/gpu/drm/gma500/gma_device.c | 1 - drivers/gpu/drm/gma500/gma_device.h | 1 + drivers/gpu/drm/gma500/gma_display.c | 12 +++++++--- drivers/gpu/drm/gma500/gma_display.h | 3 +++ drivers/gpu/drm/gma500/gtt.c | 5 ++-- drivers/gpu/drm/gma500/gtt.h | 1 - drivers/gpu/drm/gma500/intel_bios.c | 6 ++--- drivers/gpu/drm/gma500/intel_bios.h | 3 +-- drivers/gpu/drm/gma500/intel_gmbus.c | 11 +++++---- drivers/gpu/drm/gma500/intel_i2c.c | 4 +++- drivers/gpu/drm/gma500/mdfld_device.c | 16 +++++++------ drivers/gpu/drm/gma500/mdfld_dsi_dpi.c | 4 +++- drivers/gpu/drm/gma500/mdfld_dsi_output.c | 12 ++++++---- drivers/gpu/drm/gma500/mdfld_dsi_output.h | 8 +++---- drivers/gpu/drm/gma500/mdfld_dsi_pkg_sender.c | 4 +++- drivers/gpu/drm/gma500/mdfld_intel_display.c | 11 +++++---- drivers/gpu/drm/gma500/mdfld_tmd_vid.c | 2 ++ drivers/gpu/drm/gma500/mid_bios.c | 5 ++-- drivers/gpu/drm/gma500/mid_bios.h | 1 + drivers/gpu/drm/gma500/mmu.c | 6 +++-- drivers/gpu/drm/gma500/oaktrail.h | 2 ++ drivers/gpu/drm/gma500/oaktrail_crtc.c | 8 ++++--- drivers/gpu/drm/gma500/oaktrail_device.c | 20 ++++++++-------- drivers/gpu/drm/gma500/oaktrail_hdmi.c | 8 ++++--- drivers/gpu/drm/gma500/oaktrail_lvds.c | 6 ++--- drivers/gpu/drm/gma500/oaktrail_lvds_i2c.c | 11 ++++----- drivers/gpu/drm/gma500/power.h | 4 +++- drivers/gpu/drm/gma500/psb_device.c | 12 +++++----- drivers/gpu/drm/gma500/psb_drv.c | 33 +++++++++++++++++---------- drivers/gpu/drm/gma500/psb_drv.h | 16 ++++++------- drivers/gpu/drm/gma500/psb_intel_display.c | 7 +++--- drivers/gpu/drm/gma500/psb_intel_lvds.c | 5 ++-- drivers/gpu/drm/gma500/psb_intel_modes.c | 2 +- drivers/gpu/drm/gma500/psb_intel_sdvo.c | 15 ++++++------ drivers/gpu/drm/gma500/psb_irq.c | 9 ++++---- drivers/gpu/drm/gma500/psb_irq.h | 2 +- drivers/gpu/drm/gma500/psb_lid.c | 6 ++--- drivers/gpu/drm/gma500/tc35876x-dsi-lvds.c | 13 +++++++---- include/drm/gma_drm.h | 25 -------------------- 51 files changed, 233 insertions(+), 199 deletions(-)