* [PATCH] gpu:drm: Remove duplicate headers @ 2019-03-27 21:11 jagdsh.linux 2019-03-28 6:41 ` Mukesh Ojha 2019-03-28 6:51 ` Laurent Pinchart 0 siblings, 2 replies; 5+ messages in thread From: jagdsh.linux @ 2019-03-27 21:11 UTC (permalink / raw) To: a.hajda, Laurent.pinchart, airlied, daniel, jani.nikula, joonas.lahtinen, rodrigo.vivi Cc: dri-devel, linux-kernel, intel-gfx, Jagadeesh Pagadala From: Jagadeesh Pagadala <jagdsh.linux@gmail.com> Remove duplicate headers which are included twice. Signed-off-by: Jagadeesh Pagadala <jagdsh.linux@gmail.com> --- drivers/gpu/drm/bridge/panel.c | 1 - drivers/gpu/drm/i915/intel_display.c | 7 ------- 2 files changed, 8 deletions(-) diff --git a/drivers/gpu/drm/bridge/panel.c b/drivers/gpu/drm/bridge/panel.c index 38eeaf8..eb9567d 100644 --- a/drivers/gpu/drm/bridge/panel.c +++ b/drivers/gpu/drm/bridge/panel.c @@ -15,7 +15,6 @@ #include <drm/drm_encoder.h> #include <drm/drm_modeset_helper_vtables.h> #include <drm/drm_probe_helper.h> -#include <drm/drm_panel.h> struct panel_bridge { struct drm_bridge bridge; diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index ccb6163..1166342 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c @@ -51,14 +51,7 @@ #include "intel_dsi.h" #include "intel_frontbuffer.h" -#include "intel_drv.h" -#include "intel_dsi.h" -#include "intel_frontbuffer.h" - -#include "i915_drv.h" -#include "i915_gem_clflush.h" #include "i915_reset.h" -#include "i915_trace.h" /* Primary plane formats for gen <= 3 */ static const u32 i8xx_primary_formats[] = { -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] gpu:drm: Remove duplicate headers 2019-03-27 21:11 [PATCH] gpu:drm: Remove duplicate headers jagdsh.linux @ 2019-03-28 6:41 ` Mukesh Ojha 2019-03-28 6:51 ` Laurent Pinchart 1 sibling, 0 replies; 5+ messages in thread From: Mukesh Ojha @ 2019-03-28 6:41 UTC (permalink / raw) To: jagdsh.linux, a.hajda, Laurent.pinchart, airlied, daniel, jani.nikula, joonas.lahtinen, rodrigo.vivi Cc: dri-devel, linux-kernel, intel-gfx On 3/28/2019 2:41 AM, jagdsh.linux@gmail.com wrote: > From: Jagadeesh Pagadala <jagdsh.linux@gmail.com> > > Remove duplicate headers which are included twice. > > Signed-off-by: Jagadeesh Pagadala <jagdsh.linux@gmail.com> Reviewed-by: Mukesh Ojha <mojha@codeaurora.org> -Mukesh > --- > drivers/gpu/drm/bridge/panel.c | 1 - > drivers/gpu/drm/i915/intel_display.c | 7 ------- > 2 files changed, 8 deletions(-) > > diff --git a/drivers/gpu/drm/bridge/panel.c b/drivers/gpu/drm/bridge/panel.c > index 38eeaf8..eb9567d 100644 > --- a/drivers/gpu/drm/bridge/panel.c > +++ b/drivers/gpu/drm/bridge/panel.c > @@ -15,7 +15,6 @@ > #include <drm/drm_encoder.h> > #include <drm/drm_modeset_helper_vtables.h> > #include <drm/drm_probe_helper.h> > -#include <drm/drm_panel.h> > > struct panel_bridge { > struct drm_bridge bridge; > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c > index ccb6163..1166342 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -51,14 +51,7 @@ > #include "intel_dsi.h" > #include "intel_frontbuffer.h" > > -#include "intel_drv.h" > -#include "intel_dsi.h" > -#include "intel_frontbuffer.h" > - > -#include "i915_drv.h" > -#include "i915_gem_clflush.h" > #include "i915_reset.h" > -#include "i915_trace.h" > > /* Primary plane formats for gen <= 3 */ > static const u32 i8xx_primary_formats[] = { ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] gpu:drm: Remove duplicate headers 2019-03-27 21:11 [PATCH] gpu:drm: Remove duplicate headers jagdsh.linux 2019-03-28 6:41 ` Mukesh Ojha @ 2019-03-28 6:51 ` Laurent Pinchart [not found] ` <20190328160206.GA94653@CentOS76.localdomain> 1 sibling, 1 reply; 5+ messages in thread From: Laurent Pinchart @ 2019-03-28 6:51 UTC (permalink / raw) To: jagdsh.linux Cc: a.hajda, airlied, daniel, jani.nikula, joonas.lahtinen, rodrigo.vivi, dri-devel, linux-kernel, intel-gfx Hi Jagadeesh, Thank you for the patch. On Thu, Mar 28, 2019 at 02:41:56AM +0530, jagdsh.linux@gmail.com wrote: > From: Jagadeesh Pagadala <jagdsh.linux@gmail.com> > > Remove duplicate headers which are included twice. Could you, while at it, also sort the existing #include statements alphabetically ? This should help avoiding similar issues in the future. > Signed-off-by: Jagadeesh Pagadala <jagdsh.linux@gmail.com> > --- > drivers/gpu/drm/bridge/panel.c | 1 - > drivers/gpu/drm/i915/intel_display.c | 7 ------- > 2 files changed, 8 deletions(-) > > diff --git a/drivers/gpu/drm/bridge/panel.c b/drivers/gpu/drm/bridge/panel.c > index 38eeaf8..eb9567d 100644 > --- a/drivers/gpu/drm/bridge/panel.c > +++ b/drivers/gpu/drm/bridge/panel.c > @@ -15,7 +15,6 @@ > #include <drm/drm_encoder.h> > #include <drm/drm_modeset_helper_vtables.h> > #include <drm/drm_probe_helper.h> > -#include <drm/drm_panel.h> > > struct panel_bridge { > struct drm_bridge bridge; > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c > index ccb6163..1166342 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -51,14 +51,7 @@ > #include "intel_dsi.h" > #include "intel_frontbuffer.h" > > -#include "intel_drv.h" > -#include "intel_dsi.h" > -#include "intel_frontbuffer.h" > - > -#include "i915_drv.h" > -#include "i915_gem_clflush.h" > #include "i915_reset.h" > -#include "i915_trace.h" > > /* Primary plane formats for gen <= 3 */ > static const u32 i8xx_primary_formats[] = { -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 5+ messages in thread
[parent not found: <20190328160206.GA94653@CentOS76.localdomain>]
[parent not found: <20190328141210.GA4813@pendragon.ideasonboard.com>]
* Re: [PATCH v2] gpu:drm: Remove duplicate headers [not found] ` <20190328141210.GA4813@pendragon.ideasonboard.com> @ 2019-03-28 19:02 ` Jagadeesh Pagadala 2019-03-29 6:32 ` Mukesh Ojha 0 siblings, 1 reply; 5+ messages in thread From: Jagadeesh Pagadala @ 2019-03-28 19:02 UTC (permalink / raw) To: Laurent Pinchart, a.hajda, airlied, daniel, jani.nikula, joonas.lahtinen, rodrigo.vivi Cc: dri-devel, linux-kernel, intel-gfx [-- Attachment #1: Type: text/plain, Size: 3952 bytes --] On Thu, Mar 28, 2019 at 04:12:10PM +0200, Laurent Pinchart wrote: > Hi Jagadeesh, > > On Thu, Mar 28, 2019 at 09:32:06PM +0530, Jagadeesh Pagadala wrote: > > On Thu, Mar 28, 2019 at 08:51:24AM +0200, Laurent Pinchart wrote: > > > On Thu, Mar 28, 2019 at 02:41:56AM +0530, jagdsh.linux@gmail.com wrote: > > > > From: Jagadeesh Pagadala <jagdsh.linux@gmail.com> > > > > > > > > Remove duplicate headers which are included twice. > > > > > > Could you, while at it, also sort the existing #include statements > > > alphabetically ? This should help avoiding similar issues in the future. > > > > > > > Signed-off-by: Jagadeesh Pagadala <jagdsh.linux@gmail.com> > > > > --- > > > > drivers/gpu/drm/bridge/panel.c | 1 - > > > > drivers/gpu/drm/i915/intel_display.c | 7 ------- > > > > 2 files changed, 8 deletions(-) > > > > > > > > diff --git a/drivers/gpu/drm/bridge/panel.c b/drivers/gpu/drm/bridge/panel.c > > > > index 38eeaf8..eb9567d 100644 > > > > --- a/drivers/gpu/drm/bridge/panel.c > > > > +++ b/drivers/gpu/drm/bridge/panel.c > > > > @@ -15,7 +15,6 @@ > > > > #include <drm/drm_encoder.h> > > > > #include <drm/drm_modeset_helper_vtables.h> > > > > #include <drm/drm_probe_helper.h> > > > > -#include <drm/drm_panel.h> > > > > > > > > struct panel_bridge { > > > > struct drm_bridge bridge; > > > > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c > > > > index ccb6163..1166342 100644 > > > > --- a/drivers/gpu/drm/i915/intel_display.c > > > > +++ b/drivers/gpu/drm/i915/intel_display.c > > > > @@ -51,14 +51,7 @@ > > > > #include "intel_dsi.h" > > > > #include "intel_frontbuffer.h" > > > > > > > > -#include "intel_drv.h" > > > > -#include "intel_dsi.h" > > > > -#include "intel_frontbuffer.h" > > > > - > > > > -#include "i915_drv.h" > > > > -#include "i915_gem_clflush.h" > > > > #include "i915_reset.h" > > > > -#include "i915_trace.h" > > > > > > > > /* Primary plane formats for gen <= 3 */ > > > > static const u32 i8xx_primary_formats[] = { > > > From c9a68b204fe4bb013c2b8481ca8239c957dd69cc Mon Sep 17 00:00:00 2001 > > From: Jagadeesh Pagadala <jagdsh.linux@gmail.com> > > Date: Thu, 28 Mar 2019 21:20:12 +0530 > > Subject: [PATCH] gpu:drm: sort the existing #includes alphabetically > > > > Hi Laurent, > > > > Hopefully this helps. > > It does, thanks. If you squash it with your original patch and send a > v2, you can add my > > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > > Signed-off-by: Jagadeesh Pagadala <jagdsh.linux@gmail.com> > > --- > > drivers/gpu/drm/bridge/panel.c | 2 +- > > drivers/gpu/drm/i915/intel_display.c | 4 ++-- > > 2 files changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/drivers/gpu/drm/bridge/panel.c b/drivers/gpu/drm/bridge/panel.c > > index eb9567d..000ba7c 100644 > > --- a/drivers/gpu/drm/bridge/panel.c > > +++ b/drivers/gpu/drm/bridge/panel.c > > @@ -9,11 +9,11 @@ > > */ > > > > #include <drm/drmP.h> > > -#include <drm/drm_panel.h> > > #include <drm/drm_atomic_helper.h> > > #include <drm/drm_connector.h> > > #include <drm/drm_encoder.h> > > #include <drm/drm_modeset_helper_vtables.h> > > +#include <drm/drm_panel.h> > > #include <drm/drm_probe_helper.h> > > > > struct panel_bridge { > > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c > > index 1166342..7956e89 100644 > > --- a/drivers/gpu/drm/i915/intel_display.c > > +++ b/drivers/gpu/drm/i915/intel_display.c > > @@ -46,13 +46,13 @@ > > > > #include "i915_drv.h" > > #include "i915_gem_clflush.h" > > +#include "i915_reset.h" > > #include "i915_trace.h" > > + > > #include "intel_drv.h" > > #include "intel_dsi.h" > > #include "intel_frontbuffer.h" > > > > -#include "i915_reset.h" > > - > > /* Primary plane formats for gen <= 3 */ > > static const u32 i8xx_primary_formats[] = { > > DRM_FORMAT_C8, > > -- > Regards, > > Laurent Pinchart [-- Attachment #2: 0001-gpu-drm-Remove-duplicate-headers.patch --] [-- Type: text/plain, Size: 1864 bytes --] From 25776c74d289f8c2e6c89d8838f3110981ce2515 Mon Sep 17 00:00:00 2001 From: Jagadeesh Pagadala <jagdsh.linux@gmail.com> Date: Fri, 29 Mar 2019 00:04:29 +0530 Subject: [PATCH v2] gpu:drm: Remove duplicate headers 1. Remove duplicate headers which are included twice. 2. Sort the existing #includes alphabetically Signed-off-by: Jagadeesh Pagadala <jagdsh.linux@gmail.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> --- drivers/gpu/drm/bridge/panel.c | 3 +-- drivers/gpu/drm/i915/intel_display.c | 9 +-------- 2 files changed, 2 insertions(+), 10 deletions(-) diff --git a/drivers/gpu/drm/bridge/panel.c b/drivers/gpu/drm/bridge/panel.c index 38eeaf8..000ba7c 100644 --- a/drivers/gpu/drm/bridge/panel.c +++ b/drivers/gpu/drm/bridge/panel.c @@ -9,13 +9,12 @@ */ #include <drm/drmP.h> -#include <drm/drm_panel.h> #include <drm/drm_atomic_helper.h> #include <drm/drm_connector.h> #include <drm/drm_encoder.h> #include <drm/drm_modeset_helper_vtables.h> -#include <drm/drm_probe_helper.h> #include <drm/drm_panel.h> +#include <drm/drm_probe_helper.h> struct panel_bridge { struct drm_bridge bridge; diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index ccb6163..7956e89 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c @@ -46,20 +46,13 @@ #include "i915_drv.h" #include "i915_gem_clflush.h" +#include "i915_reset.h" #include "i915_trace.h" -#include "intel_drv.h" -#include "intel_dsi.h" -#include "intel_frontbuffer.h" #include "intel_drv.h" #include "intel_dsi.h" #include "intel_frontbuffer.h" -#include "i915_drv.h" -#include "i915_gem_clflush.h" -#include "i915_reset.h" -#include "i915_trace.h" - /* Primary plane formats for gen <= 3 */ static const u32 i8xx_primary_formats[] = { DRM_FORMAT_C8, -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v2] gpu:drm: Remove duplicate headers 2019-03-28 19:02 ` [PATCH v2] " Jagadeesh Pagadala @ 2019-03-29 6:32 ` Mukesh Ojha 0 siblings, 0 replies; 5+ messages in thread From: Mukesh Ojha @ 2019-03-29 6:32 UTC (permalink / raw) To: Jagadeesh Pagadala, Laurent Pinchart, a.hajda, airlied, daniel, jani.nikula, joonas.lahtinen, rodrigo.vivi Cc: dri-devel, linux-kernel, intel-gfx On 3/29/2019 12:32 AM, Jagadeesh Pagadala wrote: > On Thu, Mar 28, 2019 at 04:12:10PM +0200, Laurent Pinchart wrote: >> Hi Jagadeesh, >> >> On Thu, Mar 28, 2019 at 09:32:06PM +0530, Jagadeesh Pagadala wrote: >>> On Thu, Mar 28, 2019 at 08:51:24AM +0200, Laurent Pinchart wrote: >>>> On Thu, Mar 28, 2019 at 02:41:56AM +0530, jagdsh.linux@gmail.com wrote: >>>>> From: Jagadeesh Pagadala <jagdsh.linux@gmail.com> >>>>> >>>>> Remove duplicate headers which are included twice. >>>> Could you, while at it, also sort the existing #include statements >>>> alphabetically ? This should help avoiding similar issues in the future. >>>> >>>>> Signed-off-by: Jagadeesh Pagadala <jagdsh.linux@gmail.com> Reviewed-by: Mukesh Ojha <mojha@codeaurora.org> -Mukesh >>>>> --- >>>>> drivers/gpu/drm/bridge/panel.c | 1 - >>>>> drivers/gpu/drm/i915/intel_display.c | 7 ------- >>>>> 2 files changed, 8 deletions(-) >>>>> >>>>> diff --git a/drivers/gpu/drm/bridge/panel.c b/drivers/gpu/drm/bridge/panel.c >>>>> index 38eeaf8..eb9567d 100644 >>>>> --- a/drivers/gpu/drm/bridge/panel.c >>>>> +++ b/drivers/gpu/drm/bridge/panel.c >>>>> @@ -15,7 +15,6 @@ >>>>> #include <drm/drm_encoder.h> >>>>> #include <drm/drm_modeset_helper_vtables.h> >>>>> #include <drm/drm_probe_helper.h> >>>>> -#include <drm/drm_panel.h> >>>>> >>>>> struct panel_bridge { >>>>> struct drm_bridge bridge; >>>>> diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c >>>>> index ccb6163..1166342 100644 >>>>> --- a/drivers/gpu/drm/i915/intel_display.c >>>>> +++ b/drivers/gpu/drm/i915/intel_display.c >>>>> @@ -51,14 +51,7 @@ >>>>> #include "intel_dsi.h" >>>>> #include "intel_frontbuffer.h" >>>>> >>>>> -#include "intel_drv.h" >>>>> -#include "intel_dsi.h" >>>>> -#include "intel_frontbuffer.h" >>>>> - >>>>> -#include "i915_drv.h" >>>>> -#include "i915_gem_clflush.h" >>>>> #include "i915_reset.h" >>>>> -#include "i915_trace.h" >>>>> >>>>> /* Primary plane formats for gen <= 3 */ >>>>> static const u32 i8xx_primary_formats[] = { >>> From c9a68b204fe4bb013c2b8481ca8239c957dd69cc Mon Sep 17 00:00:00 2001 >>> From: Jagadeesh Pagadala <jagdsh.linux@gmail.com> >>> Date: Thu, 28 Mar 2019 21:20:12 +0530 >>> Subject: [PATCH] gpu:drm: sort the existing #includes alphabetically >>> >>> Hi Laurent, >>> >>> Hopefully this helps. >> It does, thanks. If you squash it with your original patch and send a >> v2, you can add my >> >> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> >> >>> Signed-off-by: Jagadeesh Pagadala <jagdsh.linux@gmail.com> >>> --- >>> drivers/gpu/drm/bridge/panel.c | 2 +- >>> drivers/gpu/drm/i915/intel_display.c | 4 ++-- >>> 2 files changed, 3 insertions(+), 3 deletions(-) >>> >>> diff --git a/drivers/gpu/drm/bridge/panel.c b/drivers/gpu/drm/bridge/panel.c >>> index eb9567d..000ba7c 100644 >>> --- a/drivers/gpu/drm/bridge/panel.c >>> +++ b/drivers/gpu/drm/bridge/panel.c >>> @@ -9,11 +9,11 @@ >>> */ >>> >>> #include <drm/drmP.h> >>> -#include <drm/drm_panel.h> >>> #include <drm/drm_atomic_helper.h> >>> #include <drm/drm_connector.h> >>> #include <drm/drm_encoder.h> >>> #include <drm/drm_modeset_helper_vtables.h> >>> +#include <drm/drm_panel.h> >>> #include <drm/drm_probe_helper.h> >>> >>> struct panel_bridge { >>> diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c >>> index 1166342..7956e89 100644 >>> --- a/drivers/gpu/drm/i915/intel_display.c >>> +++ b/drivers/gpu/drm/i915/intel_display.c >>> @@ -46,13 +46,13 @@ >>> >>> #include "i915_drv.h" >>> #include "i915_gem_clflush.h" >>> +#include "i915_reset.h" >>> #include "i915_trace.h" >>> + >>> #include "intel_drv.h" >>> #include "intel_dsi.h" >>> #include "intel_frontbuffer.h" >>> >>> -#include "i915_reset.h" >>> - >>> /* Primary plane formats for gen <= 3 */ >>> static const u32 i8xx_primary_formats[] = { >>> DRM_FORMAT_C8, >> -- >> Regards, >> >> Laurent Pinchart ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-03-29 6:32 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-03-27 21:11 [PATCH] gpu:drm: Remove duplicate headers jagdsh.linux
2019-03-28 6:41 ` Mukesh Ojha
2019-03-28 6:51 ` Laurent Pinchart
[not found] ` <20190328160206.GA94653@CentOS76.localdomain>
[not found] ` <20190328141210.GA4813@pendragon.ideasonboard.com>
2019-03-28 19:02 ` [PATCH v2] " Jagadeesh Pagadala
2019-03-29 6:32 ` Mukesh Ojha
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox