* [PATCH] media: schedule removal for legacy staging drivers
@ 2019-06-25 11:48 ` Mauro Carvalho Chehab
0 siblings, 0 replies; 5+ messages in thread
From: Mauro Carvalho Chehab @ 2019-06-25 11:48 UTC (permalink / raw
To: Linux Media Mailing List
Cc: Mauro Carvalho Chehab, Mauro Carvalho Chehab, Greg Kroah-Hartman,
Laurent Pinchart, Sakari Ailus, devel
Keeping legacy problematic code forever is not a good idea.
So, let's schedule a date for those legacy stuff to rest in piece.
If someone wants to steps up and take them from the staging ostracism
and do give them a rejuvenation shower in order to address the
isues pointed on their TODO lists, be our guest!
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
---
drivers/staging/media/bcm2048/TODO | 6 ++++++
drivers/staging/media/davinci_vpfe/TODO | 7 +++++++
drivers/staging/media/omap4iss/TODO | 7 +++++++
drivers/staging/media/soc_camera/TODO | 9 +++++++++
4 files changed, 29 insertions(+)
diff --git a/drivers/staging/media/bcm2048/TODO b/drivers/staging/media/bcm2048/TODO
index 6bee2a2dad68..478984672c9b 100644
--- a/drivers/staging/media/bcm2048/TODO
+++ b/drivers/staging/media/bcm2048/TODO
@@ -1,3 +1,9 @@
+NOTE:
+ The bcm2048 driver fixes on this TODO were not addressed yet.
+ It has been a long time since the last related change.
+ Unless someone steps up addressing those, this driver is
+ scheduled to be removed for Kernel 5.4.
+
TODO:
From the initial code review:
diff --git a/drivers/staging/media/davinci_vpfe/TODO b/drivers/staging/media/davinci_vpfe/TODO
index cc8bd9306f2a..9d4577a911c9 100644
--- a/drivers/staging/media/davinci_vpfe/TODO
+++ b/drivers/staging/media/davinci_vpfe/TODO
@@ -1,3 +1,10 @@
+NOTE:
+ The davinci_vpfe driver fixes on this TODO were not addressed yet.
+ It has been a long time since the last related change.
+ Unless someone steps up addressing those, this driver is
+ scheduled to be removed for Kernel 5.4.
+
+
TODO (general):
==================================
diff --git a/drivers/staging/media/omap4iss/TODO b/drivers/staging/media/omap4iss/TODO
index 4d220ef82653..fb90be3c1f32 100644
--- a/drivers/staging/media/omap4iss/TODO
+++ b/drivers/staging/media/omap4iss/TODO
@@ -1,3 +1,10 @@
+NOTE:
+ The omap4iss driver fixes on this TODO were not addressed yet.
+ It has been a long time since the last related change.
+ Unless someone steps up addressing those, this driver is
+ scheduled to be removed for Kernel 5.4.
+
+
* Fix FIFO/buffer overflows and underflows
* Replace dummy resizer code with a real implementation
* Fix checkpatch errors and warnings
diff --git a/drivers/staging/media/soc_camera/TODO b/drivers/staging/media/soc_camera/TODO
index 932af6443b67..677dcdc1de61 100644
--- a/drivers/staging/media/soc_camera/TODO
+++ b/drivers/staging/media/soc_camera/TODO
@@ -1,3 +1,12 @@
+NOTE:
+ The old drivers that depends on SoC camera framework require
+ conversion. We're not accepting any patches that are doing just
+ checkpatch or style fixes before such conversion.
+
+ If nobody steps up addressing those, this driver is scheduled to be
+ removed for Kernel 5.5.
+
+
The SoC camera framework is obsolete and scheduled for removal in the near
future. Developers are encouraged to convert the drivers to use the
regular V4L2 API if these drivers are still needed (and if someone has the
--
2.21.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH] media: schedule removal for legacy staging drivers
@ 2019-06-25 11:48 ` Mauro Carvalho Chehab
0 siblings, 0 replies; 5+ messages in thread
From: Mauro Carvalho Chehab @ 2019-06-25 11:48 UTC (permalink / raw
To: Linux Media Mailing List
Cc: devel, Greg Kroah-Hartman, Mauro Carvalho Chehab,
Laurent Pinchart, Sakari Ailus, Mauro Carvalho Chehab
Keeping legacy problematic code forever is not a good idea.
So, let's schedule a date for those legacy stuff to rest in piece.
If someone wants to steps up and take them from the staging ostracism
and do give them a rejuvenation shower in order to address the
isues pointed on their TODO lists, be our guest!
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
---
drivers/staging/media/bcm2048/TODO | 6 ++++++
drivers/staging/media/davinci_vpfe/TODO | 7 +++++++
drivers/staging/media/omap4iss/TODO | 7 +++++++
drivers/staging/media/soc_camera/TODO | 9 +++++++++
4 files changed, 29 insertions(+)
diff --git a/drivers/staging/media/bcm2048/TODO b/drivers/staging/media/bcm2048/TODO
index 6bee2a2dad68..478984672c9b 100644
--- a/drivers/staging/media/bcm2048/TODO
+++ b/drivers/staging/media/bcm2048/TODO
@@ -1,3 +1,9 @@
+NOTE:
+ The bcm2048 driver fixes on this TODO were not addressed yet.
+ It has been a long time since the last related change.
+ Unless someone steps up addressing those, this driver is
+ scheduled to be removed for Kernel 5.4.
+
TODO:
From the initial code review:
diff --git a/drivers/staging/media/davinci_vpfe/TODO b/drivers/staging/media/davinci_vpfe/TODO
index cc8bd9306f2a..9d4577a911c9 100644
--- a/drivers/staging/media/davinci_vpfe/TODO
+++ b/drivers/staging/media/davinci_vpfe/TODO
@@ -1,3 +1,10 @@
+NOTE:
+ The davinci_vpfe driver fixes on this TODO were not addressed yet.
+ It has been a long time since the last related change.
+ Unless someone steps up addressing those, this driver is
+ scheduled to be removed for Kernel 5.4.
+
+
TODO (general):
==================================
diff --git a/drivers/staging/media/omap4iss/TODO b/drivers/staging/media/omap4iss/TODO
index 4d220ef82653..fb90be3c1f32 100644
--- a/drivers/staging/media/omap4iss/TODO
+++ b/drivers/staging/media/omap4iss/TODO
@@ -1,3 +1,10 @@
+NOTE:
+ The omap4iss driver fixes on this TODO were not addressed yet.
+ It has been a long time since the last related change.
+ Unless someone steps up addressing those, this driver is
+ scheduled to be removed for Kernel 5.4.
+
+
* Fix FIFO/buffer overflows and underflows
* Replace dummy resizer code with a real implementation
* Fix checkpatch errors and warnings
diff --git a/drivers/staging/media/soc_camera/TODO b/drivers/staging/media/soc_camera/TODO
index 932af6443b67..677dcdc1de61 100644
--- a/drivers/staging/media/soc_camera/TODO
+++ b/drivers/staging/media/soc_camera/TODO
@@ -1,3 +1,12 @@
+NOTE:
+ The old drivers that depends on SoC camera framework require
+ conversion. We're not accepting any patches that are doing just
+ checkpatch or style fixes before such conversion.
+
+ If nobody steps up addressing those, this driver is scheduled to be
+ removed for Kernel 5.5.
+
+
The SoC camera framework is obsolete and scheduled for removal in the near
future. Developers are encouraged to convert the drivers to use the
regular V4L2 API if these drivers are still needed (and if someone has the
--
2.21.0
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [Media summit topic] Criteria to drop drivers from staging - Was: Re: [PATCH] media: schedule removal for legacy staging drivers
2019-06-25 11:48 ` Mauro Carvalho Chehab
(?)
@ 2019-06-25 11:59 ` Mauro Carvalho Chehab
-1 siblings, 0 replies; 5+ messages in thread
From: Mauro Carvalho Chehab @ 2019-06-25 11:59 UTC (permalink / raw
To: Linux Media Mailing List
Cc: Mauro Carvalho Chehab, Greg Kroah-Hartman, Laurent Pinchart,
Sakari Ailus
For the next Media Summit, I'd like to discuss some criteria in order to avoid
drivers to stay at staging from longer than needed.
We have drivers at staging that aren't touched since 2014 (except for trivial
and cleanup changes or due to changes at kABI). It sounds really doubtful that
such drivers will get any care anytime soon.
The way I see from some developers is that they rush to get stuff under
staging, but, once the code is there, they relax and pretend that their
drivers are already at the Linux Kernel and are first class citizens.
I suspect that part of such behavior is due to some sort of pressure from
the vendors that are sponsoring their work, but, once things enter at
staging, they stop their sponsorship to fix the problems there.
Staging is not supposed to work like that: once the driver gets merged there,
developers and their sponsors should actually be committing to do whatever
required to get the driver done to be part of the Linux Kernel's full body,
and not to stay forever on a sort of Kernel's refugee camp.
If, for whatever reason this doesn't happen, we should be dropping those
drivers after a while.
So, I'd like to discuss a set of criteria we'll be adopting in order to
decide when it is time to send unloved staging code to /dev/null.
Regards,
Mauro
Em Tue, 25 Jun 2019 08:48:26 -0300
Mauro Carvalho Chehab <mchehab+samsung@kernel.org> escreveu:
> Keeping legacy problematic code forever is not a good idea.
>
> So, let's schedule a date for those legacy stuff to rest in piece.
>
> If someone wants to steps up and take them from the staging ostracism
> and do give them a rejuvenation shower in order to address the
> isues pointed on their TODO lists, be our guest!
>
> Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
> ---
> drivers/staging/media/bcm2048/TODO | 6 ++++++
> drivers/staging/media/davinci_vpfe/TODO | 7 +++++++
> drivers/staging/media/omap4iss/TODO | 7 +++++++
> drivers/staging/media/soc_camera/TODO | 9 +++++++++
> 4 files changed, 29 insertions(+)
>
> diff --git a/drivers/staging/media/bcm2048/TODO b/drivers/staging/media/bcm2048/TODO
> index 6bee2a2dad68..478984672c9b 100644
> --- a/drivers/staging/media/bcm2048/TODO
> +++ b/drivers/staging/media/bcm2048/TODO
> @@ -1,3 +1,9 @@
> +NOTE:
> + The bcm2048 driver fixes on this TODO were not addressed yet.
> + It has been a long time since the last related change.
> + Unless someone steps up addressing those, this driver is
> + scheduled to be removed for Kernel 5.4.
> +
> TODO:
>
> From the initial code review:
> diff --git a/drivers/staging/media/davinci_vpfe/TODO b/drivers/staging/media/davinci_vpfe/TODO
> index cc8bd9306f2a..9d4577a911c9 100644
> --- a/drivers/staging/media/davinci_vpfe/TODO
> +++ b/drivers/staging/media/davinci_vpfe/TODO
> @@ -1,3 +1,10 @@
> +NOTE:
> + The davinci_vpfe driver fixes on this TODO were not addressed yet.
> + It has been a long time since the last related change.
> + Unless someone steps up addressing those, this driver is
> + scheduled to be removed for Kernel 5.4.
> +
> +
> TODO (general):
> ==================================
>
> diff --git a/drivers/staging/media/omap4iss/TODO b/drivers/staging/media/omap4iss/TODO
> index 4d220ef82653..fb90be3c1f32 100644
> --- a/drivers/staging/media/omap4iss/TODO
> +++ b/drivers/staging/media/omap4iss/TODO
> @@ -1,3 +1,10 @@
> +NOTE:
> + The omap4iss driver fixes on this TODO were not addressed yet.
> + It has been a long time since the last related change.
> + Unless someone steps up addressing those, this driver is
> + scheduled to be removed for Kernel 5.4.
> +
> +
> * Fix FIFO/buffer overflows and underflows
> * Replace dummy resizer code with a real implementation
> * Fix checkpatch errors and warnings
> diff --git a/drivers/staging/media/soc_camera/TODO b/drivers/staging/media/soc_camera/TODO
> index 932af6443b67..677dcdc1de61 100644
> --- a/drivers/staging/media/soc_camera/TODO
> +++ b/drivers/staging/media/soc_camera/TODO
> @@ -1,3 +1,12 @@
> +NOTE:
> + The old drivers that depends on SoC camera framework require
> + conversion. We're not accepting any patches that are doing just
> + checkpatch or style fixes before such conversion.
> +
> + If nobody steps up addressing those, this driver is scheduled to be
> + removed for Kernel 5.5.
> +
> +
> The SoC camera framework is obsolete and scheduled for removal in the near
> future. Developers are encouraged to convert the drivers to use the
> regular V4L2 API if these drivers are still needed (and if someone has the
Thanks,
Mauro
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] media: schedule removal for legacy staging drivers
2019-06-25 11:48 ` Mauro Carvalho Chehab
@ 2019-06-25 12:29 ` Sakari Ailus
-1 siblings, 0 replies; 5+ messages in thread
From: Sakari Ailus @ 2019-06-25 12:29 UTC (permalink / raw
To: Mauro Carvalho Chehab
Cc: Linux Media Mailing List, Mauro Carvalho Chehab,
Greg Kroah-Hartman, Laurent Pinchart, devel
On Tue, Jun 25, 2019 at 08:48:26AM -0300, Mauro Carvalho Chehab wrote:
> Keeping legacy problematic code forever is not a good idea.
>
> So, let's schedule a date for those legacy stuff to rest in piece.
>
> If someone wants to steps up and take them from the staging ostracism
> and do give them a rejuvenation shower in order to address the
> isues pointed on their TODO lists, be our guest!
>
> Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
--
Sakari Ailus
sakari.ailus@linux.intel.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] media: schedule removal for legacy staging drivers
@ 2019-06-25 12:29 ` Sakari Ailus
0 siblings, 0 replies; 5+ messages in thread
From: Sakari Ailus @ 2019-06-25 12:29 UTC (permalink / raw
To: Mauro Carvalho Chehab
Cc: devel, Greg Kroah-Hartman, Mauro Carvalho Chehab,
Laurent Pinchart, Linux Media Mailing List
On Tue, Jun 25, 2019 at 08:48:26AM -0300, Mauro Carvalho Chehab wrote:
> Keeping legacy problematic code forever is not a good idea.
>
> So, let's schedule a date for those legacy stuff to rest in piece.
>
> If someone wants to steps up and take them from the staging ostracism
> and do give them a rejuvenation shower in order to address the
> isues pointed on their TODO lists, be our guest!
>
> Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
--
Sakari Ailus
sakari.ailus@linux.intel.com
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-06-25 12:29 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-06-25 11:48 [PATCH] media: schedule removal for legacy staging drivers Mauro Carvalho Chehab
2019-06-25 11:48 ` Mauro Carvalho Chehab
2019-06-25 11:59 ` [Media summit topic] Criteria to drop drivers from staging - Was: " Mauro Carvalho Chehab
2019-06-25 12:29 ` Sakari Ailus
2019-06-25 12:29 ` Sakari Ailus
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.