[PATCH 09/45] drivers: media: remove duplicate includes
These duplicate includes have been found with scripts/checkincludes.pl but they have been removed manually to avoid removing false positives. Signed-off-by: Pravin Shedge --- drivers/media/platform/pxa_camera.c | 1 - drivers/media/platform/ti-vpe/cal.c | 3 --- drivers/media/v4l2-core/v4l2-mc.c | 2 -- 3 files changed, 6 deletions(-) diff --git a/drivers/media/platform/pxa_camera.c b/drivers/media/platform/pxa_camera.c index 9d3f0cb..2a9f02e 100644 --- a/drivers/media/platform/pxa_camera.c +++ b/drivers/media/platform/pxa_camera.c @@ -32,7 +32,6 @@ #include #include #include -#include #include #include diff --git a/drivers/media/platform/ti-vpe/cal.c b/drivers/media/platform/ti-vpe/cal.c index 8b586c8..719ed1d 100644 --- a/drivers/media/platform/ti-vpe/cal.c +++ b/drivers/media/platform/ti-vpe/cal.c @@ -28,10 +28,7 @@ #include #include #include -#include #include -#include -#include #include #include #include "cal_regs.h" diff --git a/drivers/media/v4l2-core/v4l2-mc.c b/drivers/media/v4l2-core/v4l2-mc.c index 303980b..1d550af 100644 --- a/drivers/media/v4l2-core/v4l2-mc.c +++ b/drivers/media/v4l2-core/v4l2-mc.c @@ -25,8 +25,6 @@ #include #include #include -#include -#include #include int v4l2_mc_create_media_graph(struct media_device *mdev) -- 2.7.4
Re: [PATCH 09/45] drivers: media: remove duplicate includes
Hi Pravin, On Wed, Dec 06, 2017 at 10:22:02PM +0530, Pravin Shedge wrote: > These duplicate includes have been found with scripts/checkincludes.pl but > they have been removed manually to avoid removing false positives. > > Signed-off-by: Pravin Shedge While at it, how about ordering the headers alphabetically as well? Having such a large number of headers there unordered may well be the reason why they're included more than once... -- Sakari Ailus e-mail: sakari.ai...@iki.fi
Re: [PATCH 09/45] drivers: media: remove duplicate includes
On Thu, Dec 7, 2017 at 7:05 PM, Sakari Ailus wrote: > Hi Pravin, > > On Wed, Dec 06, 2017 at 10:22:02PM +0530, Pravin Shedge wrote: >> These duplicate includes have been found with scripts/checkincludes.pl but >> they have been removed manually to avoid removing false positives. >> >> Signed-off-by: Pravin Shedge > > While at it, how about ordering the headers alphabetically as well? Having > such a large number of headers there unordered may well be the reason why > they're included more than once... > > -- > Sakari Ailus > e-mail: sakari.ai...@iki.fi Hi Sakari, Sorry for the late reply. Ordering the header files alphabetically helps to avoid problems such as inclusion of duplicate header files. My personal preference is to go from local to global, each subsection in alphabetical order. Ideally, all header files should be self-contained, and inclusion order should not matter. Simple reordering the headers should not break build. Reordering header files aways helpful for big projects like Linux-Kernel. But this requires changes tree wide and modifies lots of files. Such change requires huge audience to be participated in discussion & take a final call. With this patch I just handled inclusion of header file multiple times to avoid code duplication after preprocessing. Thanks & Regards, PraviN
Re: [PATCH 09/45] drivers: media: remove duplicate includes
On Sun, Dec 10, 2017 at 11:03:29PM +0530, Pravin Shedge wrote: > On Thu, Dec 7, 2017 at 7:05 PM, Sakari Ailus wrote: > > Hi Pravin, > > > > On Wed, Dec 06, 2017 at 10:22:02PM +0530, Pravin Shedge wrote: > >> These duplicate includes have been found with scripts/checkincludes.pl but > >> they have been removed manually to avoid removing false positives. > >> > >> Signed-off-by: Pravin Shedge > > > > While at it, how about ordering the headers alphabetically as well? Having > > such a large number of headers there unordered may well be the reason why > > they're included more than once... > > > > -- > > Sakari Ailus > > e-mail: sakari.ai...@iki.fi > > > Hi Sakari, > > Sorry for the late reply. > > Ordering the header files alphabetically helps to avoid problems such > as inclusion of duplicate header files. > My personal preference is to go from local to global, each subsection > in alphabetical order. > Ideally, all header files should be self-contained, and inclusion > order should not matter. > Simple reordering the headers should not break build. > > Reordering header files aways helpful for big projects like Linux-Kernel. > But this requires changes tree wide and modifies lots of files. > Such change requires huge audience to be participated in discussion & > take a final call. Hmm. I'm not quite sure what do you mean. You're already changing the three files, there's no need to arrange others at the same time. > > With this patch I just handled inclusion of header file multiple times > to avoid code duplication after preprocessing. > > Thanks & Regards, >PraviN -- Sakari Ailus e-mail: sakari.ai...@iki.fi