All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] soc_camera/omap1: move to staging in preparation for removal
@ 2016-02-23 12:13 Hans Verkuil
  2016-02-23 14:55 ` Guennadi Liakhovetski
  0 siblings, 1 reply; 7+ messages in thread
From: Hans Verkuil @ 2016-02-23 12:13 UTC (permalink / raw)
  To: linux-media, Guennadi Liakhovetski, Sakari Ailus

This driver is deprecated: it needs to be converted to vb2 and
it should become a stand-alone driver instead of using the
soc-camera framework.

Unless someone is willing to take this on (unlikely with such
ancient hardware) it is going to be removed from the kernel
soon.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
---
 drivers/media/platform/soc_camera/Kconfig                   | 10 ----------
 drivers/media/platform/soc_camera/Makefile                  |  1 -
 drivers/staging/media/Kconfig                               |  2 ++
 drivers/staging/media/Makefile                              |  1 +
 drivers/staging/media/omap1/Kconfig                         | 13 +++++++++++++
 drivers/staging/media/omap1/Makefile                        |  3 +++
 .../soc_camera => staging/media/omap1}/omap1_camera.c       |  0
 7 files changed, 19 insertions(+), 11 deletions(-)
 create mode 100644 drivers/staging/media/omap1/Kconfig
 create mode 100644 drivers/staging/media/omap1/Makefile
 rename drivers/{media/platform/soc_camera => staging/media/omap1}/omap1_camera.c (100%)

diff --git a/drivers/media/platform/soc_camera/Kconfig b/drivers/media/platform/soc_camera/Kconfig
index f2776cd..954dd36 100644
--- a/drivers/media/platform/soc_camera/Kconfig
+++ b/drivers/media/platform/soc_camera/Kconfig
@@ -60,16 +60,6 @@ config VIDEO_SH_MOBILE_CEU
 	---help---
 	  This is a v4l2 driver for the SuperH Mobile CEU Interface

-config VIDEO_OMAP1
-	tristate "OMAP1 Camera Interface driver"
-	depends on VIDEO_DEV && SOC_CAMERA
-	depends on ARCH_OMAP1
-	depends on HAS_DMA
-	select VIDEOBUF_DMA_CONTIG
-	select VIDEOBUF_DMA_SG
-	---help---
-	  This is a v4l2 driver for the TI OMAP1 camera interface
-
 config VIDEO_MX2
 	tristate "i.MX27 Camera Sensor Interface driver"
 	depends on VIDEO_DEV && SOC_CAMERA
diff --git a/drivers/media/platform/soc_camera/Makefile b/drivers/media/platform/soc_camera/Makefile
index 2826382..bdd7fc9 100644
--- a/drivers/media/platform/soc_camera/Makefile
+++ b/drivers/media/platform/soc_camera/Makefile
@@ -9,7 +9,6 @@ obj-$(CONFIG_SOC_CAMERA_PLATFORM)	+= soc_camera_platform.o
 obj-$(CONFIG_VIDEO_ATMEL_ISI)		+= atmel-isi.o
 obj-$(CONFIG_VIDEO_MX2)			+= mx2_camera.o
 obj-$(CONFIG_VIDEO_MX3)			+= mx3_camera.o
-obj-$(CONFIG_VIDEO_OMAP1)		+= omap1_camera.o
 obj-$(CONFIG_VIDEO_PXA27x)		+= pxa_camera.o
 obj-$(CONFIG_VIDEO_SH_MOBILE_CEU)	+= sh_mobile_ceu_camera.o
 obj-$(CONFIG_VIDEO_SH_MOBILE_CSI2)	+= sh_mobile_csi2.o
diff --git a/drivers/staging/media/Kconfig b/drivers/staging/media/Kconfig
index d48a5c2..382d868 100644
--- a/drivers/staging/media/Kconfig
+++ b/drivers/staging/media/Kconfig
@@ -29,6 +29,8 @@ source "drivers/staging/media/mn88472/Kconfig"

 source "drivers/staging/media/mn88473/Kconfig"

+source "drivers/staging/media/omap1/Kconfig"
+
 source "drivers/staging/media/omap4iss/Kconfig"

 source "drivers/staging/media/timb/Kconfig"
diff --git a/drivers/staging/media/Makefile b/drivers/staging/media/Makefile
index fb94f04..89d038c 100644
--- a/drivers/staging/media/Makefile
+++ b/drivers/staging/media/Makefile
@@ -2,6 +2,7 @@ obj-$(CONFIG_I2C_BCM2048)	+= bcm2048/
 obj-$(CONFIG_DVB_CXD2099)	+= cxd2099/
 obj-$(CONFIG_LIRC_STAGING)	+= lirc/
 obj-$(CONFIG_VIDEO_DM365_VPFE)	+= davinci_vpfe/
+obj-$(CONFIG_VIDEO_OMAP1)	+= omap1/
 obj-$(CONFIG_VIDEO_OMAP4)	+= omap4iss/
 obj-$(CONFIG_DVB_MN88472)       += mn88472/
 obj-$(CONFIG_DVB_MN88473)       += mn88473/
diff --git a/drivers/staging/media/omap1/Kconfig b/drivers/staging/media/omap1/Kconfig
new file mode 100644
index 0000000..6cfab3a
--- /dev/null
+++ b/drivers/staging/media/omap1/Kconfig
@@ -0,0 +1,13 @@
+config VIDEO_OMAP1
+	tristate "OMAP1 Camera Interface driver"
+	depends on VIDEO_DEV && SOC_CAMERA
+	depends on ARCH_OMAP1
+	depends on HAS_DMA
+	select VIDEOBUF_DMA_CONTIG
+	select VIDEOBUF_DMA_SG
+	---help---
+	  This is a v4l2 driver for the TI OMAP1 camera interface
+
+	  This driver is deprecated and will be removed soon unless someone
+	  will start the work to convert this driver to the vb2 framework
+	  and remove the soc-camera dependency.
diff --git a/drivers/staging/media/omap1/Makefile b/drivers/staging/media/omap1/Makefile
new file mode 100644
index 0000000..2885622
--- /dev/null
+++ b/drivers/staging/media/omap1/Makefile
@@ -0,0 +1,3 @@
+# Makefile for OMAP1 driver
+
+obj-$(CONFIG_VIDEO_OMAP1) += omap1_camera.o
diff --git a/drivers/media/platform/soc_camera/omap1_camera.c b/drivers/staging/media/omap1/omap1_camera.c
similarity index 100%
rename from drivers/media/platform/soc_camera/omap1_camera.c
rename to drivers/staging/media/omap1/omap1_camera.c
-- 
2.7.0


^ permalink raw reply related	[flat|nested] 7+ messages in thread
* Re: [PATCH] soc_camera/omap1: move to staging in preparation for removal
  2016-02-23 12:13 [PATCH] soc_camera/omap1: move to staging in preparation for removal Hans Verkuil
@ 2016-02-23 14:55 ` Guennadi Liakhovetski
  2016-02-23 14:59   ` Hans Verkuil
  0 siblings, 1 reply; 7+ messages in thread
From: Guennadi Liakhovetski @ 2016-02-23 14:55 UTC (permalink / raw)
  To: Hans Verkuil; +Cc: linux-media, Sakari Ailus

Hi Hans,

On Tue, 23 Feb 2016, Hans Verkuil wrote:

> This driver is deprecated: it needs to be converted to vb2 and
> it should become a stand-alone driver instead of using the
> soc-camera framework.
> 
> Unless someone is willing to take this on (unlikely with such
> ancient hardware) it is going to be removed from the kernel
> soon.
> 
> Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

I guess I won't be pulling this through my tree, right?

Acked-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>

Thanks
Guennadi

> ---
>  drivers/media/platform/soc_camera/Kconfig                   | 10 ----------
>  drivers/media/platform/soc_camera/Makefile                  |  1 -
>  drivers/staging/media/Kconfig                               |  2 ++
>  drivers/staging/media/Makefile                              |  1 +
>  drivers/staging/media/omap1/Kconfig                         | 13 +++++++++++++
>  drivers/staging/media/omap1/Makefile                        |  3 +++
>  .../soc_camera => staging/media/omap1}/omap1_camera.c       |  0
>  7 files changed, 19 insertions(+), 11 deletions(-)
>  create mode 100644 drivers/staging/media/omap1/Kconfig
>  create mode 100644 drivers/staging/media/omap1/Makefile
>  rename drivers/{media/platform/soc_camera => staging/media/omap1}/omap1_camera.c (100%)
> 
> diff --git a/drivers/media/platform/soc_camera/Kconfig b/drivers/media/platform/soc_camera/Kconfig
> index f2776cd..954dd36 100644
> --- a/drivers/media/platform/soc_camera/Kconfig
> +++ b/drivers/media/platform/soc_camera/Kconfig
> @@ -60,16 +60,6 @@ config VIDEO_SH_MOBILE_CEU
>  	---help---
>  	  This is a v4l2 driver for the SuperH Mobile CEU Interface
> 
> -config VIDEO_OMAP1
> -	tristate "OMAP1 Camera Interface driver"
> -	depends on VIDEO_DEV && SOC_CAMERA
> -	depends on ARCH_OMAP1
> -	depends on HAS_DMA
> -	select VIDEOBUF_DMA_CONTIG
> -	select VIDEOBUF_DMA_SG
> -	---help---
> -	  This is a v4l2 driver for the TI OMAP1 camera interface
> -
>  config VIDEO_MX2
>  	tristate "i.MX27 Camera Sensor Interface driver"
>  	depends on VIDEO_DEV && SOC_CAMERA
> diff --git a/drivers/media/platform/soc_camera/Makefile b/drivers/media/platform/soc_camera/Makefile
> index 2826382..bdd7fc9 100644
> --- a/drivers/media/platform/soc_camera/Makefile
> +++ b/drivers/media/platform/soc_camera/Makefile
> @@ -9,7 +9,6 @@ obj-$(CONFIG_SOC_CAMERA_PLATFORM)	+= soc_camera_platform.o
>  obj-$(CONFIG_VIDEO_ATMEL_ISI)		+= atmel-isi.o
>  obj-$(CONFIG_VIDEO_MX2)			+= mx2_camera.o
>  obj-$(CONFIG_VIDEO_MX3)			+= mx3_camera.o
> -obj-$(CONFIG_VIDEO_OMAP1)		+= omap1_camera.o
>  obj-$(CONFIG_VIDEO_PXA27x)		+= pxa_camera.o
>  obj-$(CONFIG_VIDEO_SH_MOBILE_CEU)	+= sh_mobile_ceu_camera.o
>  obj-$(CONFIG_VIDEO_SH_MOBILE_CSI2)	+= sh_mobile_csi2.o
> diff --git a/drivers/staging/media/Kconfig b/drivers/staging/media/Kconfig
> index d48a5c2..382d868 100644
> --- a/drivers/staging/media/Kconfig
> +++ b/drivers/staging/media/Kconfig
> @@ -29,6 +29,8 @@ source "drivers/staging/media/mn88472/Kconfig"
> 
>  source "drivers/staging/media/mn88473/Kconfig"
> 
> +source "drivers/staging/media/omap1/Kconfig"
> +
>  source "drivers/staging/media/omap4iss/Kconfig"
> 
>  source "drivers/staging/media/timb/Kconfig"
> diff --git a/drivers/staging/media/Makefile b/drivers/staging/media/Makefile
> index fb94f04..89d038c 100644
> --- a/drivers/staging/media/Makefile
> +++ b/drivers/staging/media/Makefile
> @@ -2,6 +2,7 @@ obj-$(CONFIG_I2C_BCM2048)	+= bcm2048/
>  obj-$(CONFIG_DVB_CXD2099)	+= cxd2099/
>  obj-$(CONFIG_LIRC_STAGING)	+= lirc/
>  obj-$(CONFIG_VIDEO_DM365_VPFE)	+= davinci_vpfe/
> +obj-$(CONFIG_VIDEO_OMAP1)	+= omap1/
>  obj-$(CONFIG_VIDEO_OMAP4)	+= omap4iss/
>  obj-$(CONFIG_DVB_MN88472)       += mn88472/
>  obj-$(CONFIG_DVB_MN88473)       += mn88473/
> diff --git a/drivers/staging/media/omap1/Kconfig b/drivers/staging/media/omap1/Kconfig
> new file mode 100644
> index 0000000..6cfab3a
> --- /dev/null
> +++ b/drivers/staging/media/omap1/Kconfig
> @@ -0,0 +1,13 @@
> +config VIDEO_OMAP1
> +	tristate "OMAP1 Camera Interface driver"
> +	depends on VIDEO_DEV && SOC_CAMERA
> +	depends on ARCH_OMAP1
> +	depends on HAS_DMA
> +	select VIDEOBUF_DMA_CONTIG
> +	select VIDEOBUF_DMA_SG
> +	---help---
> +	  This is a v4l2 driver for the TI OMAP1 camera interface
> +
> +	  This driver is deprecated and will be removed soon unless someone
> +	  will start the work to convert this driver to the vb2 framework
> +	  and remove the soc-camera dependency.
> diff --git a/drivers/staging/media/omap1/Makefile b/drivers/staging/media/omap1/Makefile
> new file mode 100644
> index 0000000..2885622
> --- /dev/null
> +++ b/drivers/staging/media/omap1/Makefile
> @@ -0,0 +1,3 @@
> +# Makefile for OMAP1 driver
> +
> +obj-$(CONFIG_VIDEO_OMAP1) += omap1_camera.o
> diff --git a/drivers/media/platform/soc_camera/omap1_camera.c b/drivers/staging/media/omap1/omap1_camera.c
> similarity index 100%
> rename from drivers/media/platform/soc_camera/omap1_camera.c
> rename to drivers/staging/media/omap1/omap1_camera.c
> -- 
> 2.7.0
> 

^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH] soc_camera/omap1: move to staging in preparation for removal
  2016-02-23 14:55 ` Guennadi Liakhovetski
@ 2016-02-23 14:59   ` Hans Verkuil
  2016-02-23 15:05     ` Guennadi Liakhovetski
  0 siblings, 1 reply; 7+ messages in thread
From: Hans Verkuil @ 2016-02-23 14:59 UTC (permalink / raw)
  To: Guennadi Liakhovetski; +Cc: linux-media, Sakari Ailus

On 02/23/16 15:55, Guennadi Liakhovetski wrote:
> Hi Hans,
> 
> On Tue, 23 Feb 2016, Hans Verkuil wrote:
> 
>> This driver is deprecated: it needs to be converted to vb2 and
>> it should become a stand-alone driver instead of using the
>> soc-camera framework.
>>
>> Unless someone is willing to take this on (unlikely with such
>> ancient hardware) it is going to be removed from the kernel
>> soon.
>>
>> Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
> 
> I guess I won't be pulling this through my tree, right?

Right.

> 
> Acked-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>

Thanks!

This one was easy.

For the sh renesas drivers I will take one final look during the weekend.
See what works and what doesn't. I actually have hardware for it, after
all.

Regards,

	Hans

> 
> Thanks
> Guennadi
> 
>> ---
>>  drivers/media/platform/soc_camera/Kconfig                   | 10 ----------
>>  drivers/media/platform/soc_camera/Makefile                  |  1 -
>>  drivers/staging/media/Kconfig                               |  2 ++
>>  drivers/staging/media/Makefile                              |  1 +
>>  drivers/staging/media/omap1/Kconfig                         | 13 +++++++++++++
>>  drivers/staging/media/omap1/Makefile                        |  3 +++
>>  .../soc_camera => staging/media/omap1}/omap1_camera.c       |  0
>>  7 files changed, 19 insertions(+), 11 deletions(-)
>>  create mode 100644 drivers/staging/media/omap1/Kconfig
>>  create mode 100644 drivers/staging/media/omap1/Makefile
>>  rename drivers/{media/platform/soc_camera => staging/media/omap1}/omap1_camera.c (100%)
>>
>> diff --git a/drivers/media/platform/soc_camera/Kconfig b/drivers/media/platform/soc_camera/Kconfig
>> index f2776cd..954dd36 100644
>> --- a/drivers/media/platform/soc_camera/Kconfig
>> +++ b/drivers/media/platform/soc_camera/Kconfig
>> @@ -60,16 +60,6 @@ config VIDEO_SH_MOBILE_CEU
>>  	---help---
>>  	  This is a v4l2 driver for the SuperH Mobile CEU Interface
>>
>> -config VIDEO_OMAP1
>> -	tristate "OMAP1 Camera Interface driver"
>> -	depends on VIDEO_DEV && SOC_CAMERA
>> -	depends on ARCH_OMAP1
>> -	depends on HAS_DMA
>> -	select VIDEOBUF_DMA_CONTIG
>> -	select VIDEOBUF_DMA_SG
>> -	---help---
>> -	  This is a v4l2 driver for the TI OMAP1 camera interface
>> -
>>  config VIDEO_MX2
>>  	tristate "i.MX27 Camera Sensor Interface driver"
>>  	depends on VIDEO_DEV && SOC_CAMERA
>> diff --git a/drivers/media/platform/soc_camera/Makefile b/drivers/media/platform/soc_camera/Makefile
>> index 2826382..bdd7fc9 100644
>> --- a/drivers/media/platform/soc_camera/Makefile
>> +++ b/drivers/media/platform/soc_camera/Makefile
>> @@ -9,7 +9,6 @@ obj-$(CONFIG_SOC_CAMERA_PLATFORM)	+= soc_camera_platform.o
>>  obj-$(CONFIG_VIDEO_ATMEL_ISI)		+= atmel-isi.o
>>  obj-$(CONFIG_VIDEO_MX2)			+= mx2_camera.o
>>  obj-$(CONFIG_VIDEO_MX3)			+= mx3_camera.o
>> -obj-$(CONFIG_VIDEO_OMAP1)		+= omap1_camera.o
>>  obj-$(CONFIG_VIDEO_PXA27x)		+= pxa_camera.o
>>  obj-$(CONFIG_VIDEO_SH_MOBILE_CEU)	+= sh_mobile_ceu_camera.o
>>  obj-$(CONFIG_VIDEO_SH_MOBILE_CSI2)	+= sh_mobile_csi2.o
>> diff --git a/drivers/staging/media/Kconfig b/drivers/staging/media/Kconfig
>> index d48a5c2..382d868 100644
>> --- a/drivers/staging/media/Kconfig
>> +++ b/drivers/staging/media/Kconfig
>> @@ -29,6 +29,8 @@ source "drivers/staging/media/mn88472/Kconfig"
>>
>>  source "drivers/staging/media/mn88473/Kconfig"
>>
>> +source "drivers/staging/media/omap1/Kconfig"
>> +
>>  source "drivers/staging/media/omap4iss/Kconfig"
>>
>>  source "drivers/staging/media/timb/Kconfig"
>> diff --git a/drivers/staging/media/Makefile b/drivers/staging/media/Makefile
>> index fb94f04..89d038c 100644
>> --- a/drivers/staging/media/Makefile
>> +++ b/drivers/staging/media/Makefile
>> @@ -2,6 +2,7 @@ obj-$(CONFIG_I2C_BCM2048)	+= bcm2048/
>>  obj-$(CONFIG_DVB_CXD2099)	+= cxd2099/
>>  obj-$(CONFIG_LIRC_STAGING)	+= lirc/
>>  obj-$(CONFIG_VIDEO_DM365_VPFE)	+= davinci_vpfe/
>> +obj-$(CONFIG_VIDEO_OMAP1)	+= omap1/
>>  obj-$(CONFIG_VIDEO_OMAP4)	+= omap4iss/
>>  obj-$(CONFIG_DVB_MN88472)       += mn88472/
>>  obj-$(CONFIG_DVB_MN88473)       += mn88473/
>> diff --git a/drivers/staging/media/omap1/Kconfig b/drivers/staging/media/omap1/Kconfig
>> new file mode 100644
>> index 0000000..6cfab3a
>> --- /dev/null
>> +++ b/drivers/staging/media/omap1/Kconfig
>> @@ -0,0 +1,13 @@
>> +config VIDEO_OMAP1
>> +	tristate "OMAP1 Camera Interface driver"
>> +	depends on VIDEO_DEV && SOC_CAMERA
>> +	depends on ARCH_OMAP1
>> +	depends on HAS_DMA
>> +	select VIDEOBUF_DMA_CONTIG
>> +	select VIDEOBUF_DMA_SG
>> +	---help---
>> +	  This is a v4l2 driver for the TI OMAP1 camera interface
>> +
>> +	  This driver is deprecated and will be removed soon unless someone
>> +	  will start the work to convert this driver to the vb2 framework
>> +	  and remove the soc-camera dependency.
>> diff --git a/drivers/staging/media/omap1/Makefile b/drivers/staging/media/omap1/Makefile
>> new file mode 100644
>> index 0000000..2885622
>> --- /dev/null
>> +++ b/drivers/staging/media/omap1/Makefile
>> @@ -0,0 +1,3 @@
>> +# Makefile for OMAP1 driver
>> +
>> +obj-$(CONFIG_VIDEO_OMAP1) += omap1_camera.o
>> diff --git a/drivers/media/platform/soc_camera/omap1_camera.c b/drivers/staging/media/omap1/omap1_camera.c
>> similarity index 100%
>> rename from drivers/media/platform/soc_camera/omap1_camera.c
>> rename to drivers/staging/media/omap1/omap1_camera.c
>> -- 
>> 2.7.0
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH] soc_camera/omap1: move to staging in preparation for removal
  2016-02-23 14:59   ` Hans Verkuil
@ 2016-02-23 15:05     ` Guennadi Liakhovetski
  2016-02-23 15:56       ` Hans Verkuil
  0 siblings, 1 reply; 7+ messages in thread
From: Guennadi Liakhovetski @ 2016-02-23 15:05 UTC (permalink / raw)
  To: Hans Verkuil; +Cc: linux-media, Sakari Ailus

On Tue, 23 Feb 2016, Hans Verkuil wrote:

> On 02/23/16 15:55, Guennadi Liakhovetski wrote:
> > Hi Hans,
> > 
> > On Tue, 23 Feb 2016, Hans Verkuil wrote:
> > 
> >> This driver is deprecated: it needs to be converted to vb2 and
> >> it should become a stand-alone driver instead of using the
> >> soc-camera framework.
> >>
> >> Unless someone is willing to take this on (unlikely with such
> >> ancient hardware) it is going to be removed from the kernel
> >> soon.
> >>
> >> Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
> > 
> > I guess I won't be pulling this through my tree, right?
> 
> Right.
> 
> > 
> > Acked-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
> 
> Thanks!
> 
> This one was easy.

mx2, mx3 should be easy too?

Guennadi

> For the sh renesas drivers I will take one final look during the weekend.
> See what works and what doesn't. I actually have hardware for it, after
> all.
> 
> Regards,
> 
> 	Hans
> 
> > 
> > Thanks
> > Guennadi
> > 
> >> ---
> >>  drivers/media/platform/soc_camera/Kconfig                   | 10 ----------
> >>  drivers/media/platform/soc_camera/Makefile                  |  1 -
> >>  drivers/staging/media/Kconfig                               |  2 ++
> >>  drivers/staging/media/Makefile                              |  1 +
> >>  drivers/staging/media/omap1/Kconfig                         | 13 +++++++++++++
> >>  drivers/staging/media/omap1/Makefile                        |  3 +++
> >>  .../soc_camera => staging/media/omap1}/omap1_camera.c       |  0
> >>  7 files changed, 19 insertions(+), 11 deletions(-)
> >>  create mode 100644 drivers/staging/media/omap1/Kconfig
> >>  create mode 100644 drivers/staging/media/omap1/Makefile
> >>  rename drivers/{media/platform/soc_camera => staging/media/omap1}/omap1_camera.c (100%)
> >>
> >> diff --git a/drivers/media/platform/soc_camera/Kconfig b/drivers/media/platform/soc_camera/Kconfig
> >> index f2776cd..954dd36 100644
> >> --- a/drivers/media/platform/soc_camera/Kconfig
> >> +++ b/drivers/media/platform/soc_camera/Kconfig
> >> @@ -60,16 +60,6 @@ config VIDEO_SH_MOBILE_CEU
> >>  	---help---
> >>  	  This is a v4l2 driver for the SuperH Mobile CEU Interface
> >>
> >> -config VIDEO_OMAP1
> >> -	tristate "OMAP1 Camera Interface driver"
> >> -	depends on VIDEO_DEV && SOC_CAMERA
> >> -	depends on ARCH_OMAP1
> >> -	depends on HAS_DMA
> >> -	select VIDEOBUF_DMA_CONTIG
> >> -	select VIDEOBUF_DMA_SG
> >> -	---help---
> >> -	  This is a v4l2 driver for the TI OMAP1 camera interface
> >> -
> >>  config VIDEO_MX2
> >>  	tristate "i.MX27 Camera Sensor Interface driver"
> >>  	depends on VIDEO_DEV && SOC_CAMERA
> >> diff --git a/drivers/media/platform/soc_camera/Makefile b/drivers/media/platform/soc_camera/Makefile
> >> index 2826382..bdd7fc9 100644
> >> --- a/drivers/media/platform/soc_camera/Makefile
> >> +++ b/drivers/media/platform/soc_camera/Makefile
> >> @@ -9,7 +9,6 @@ obj-$(CONFIG_SOC_CAMERA_PLATFORM)	+= soc_camera_platform.o
> >>  obj-$(CONFIG_VIDEO_ATMEL_ISI)		+= atmel-isi.o
> >>  obj-$(CONFIG_VIDEO_MX2)			+= mx2_camera.o
> >>  obj-$(CONFIG_VIDEO_MX3)			+= mx3_camera.o
> >> -obj-$(CONFIG_VIDEO_OMAP1)		+= omap1_camera.o
> >>  obj-$(CONFIG_VIDEO_PXA27x)		+= pxa_camera.o
> >>  obj-$(CONFIG_VIDEO_SH_MOBILE_CEU)	+= sh_mobile_ceu_camera.o
> >>  obj-$(CONFIG_VIDEO_SH_MOBILE_CSI2)	+= sh_mobile_csi2.o
> >> diff --git a/drivers/staging/media/Kconfig b/drivers/staging/media/Kconfig
> >> index d48a5c2..382d868 100644
> >> --- a/drivers/staging/media/Kconfig
> >> +++ b/drivers/staging/media/Kconfig
> >> @@ -29,6 +29,8 @@ source "drivers/staging/media/mn88472/Kconfig"
> >>
> >>  source "drivers/staging/media/mn88473/Kconfig"
> >>
> >> +source "drivers/staging/media/omap1/Kconfig"
> >> +
> >>  source "drivers/staging/media/omap4iss/Kconfig"
> >>
> >>  source "drivers/staging/media/timb/Kconfig"
> >> diff --git a/drivers/staging/media/Makefile b/drivers/staging/media/Makefile
> >> index fb94f04..89d038c 100644
> >> --- a/drivers/staging/media/Makefile
> >> +++ b/drivers/staging/media/Makefile
> >> @@ -2,6 +2,7 @@ obj-$(CONFIG_I2C_BCM2048)	+= bcm2048/
> >>  obj-$(CONFIG_DVB_CXD2099)	+= cxd2099/
> >>  obj-$(CONFIG_LIRC_STAGING)	+= lirc/
> >>  obj-$(CONFIG_VIDEO_DM365_VPFE)	+= davinci_vpfe/
> >> +obj-$(CONFIG_VIDEO_OMAP1)	+= omap1/
> >>  obj-$(CONFIG_VIDEO_OMAP4)	+= omap4iss/
> >>  obj-$(CONFIG_DVB_MN88472)       += mn88472/
> >>  obj-$(CONFIG_DVB_MN88473)       += mn88473/
> >> diff --git a/drivers/staging/media/omap1/Kconfig b/drivers/staging/media/omap1/Kconfig
> >> new file mode 100644
> >> index 0000000..6cfab3a
> >> --- /dev/null
> >> +++ b/drivers/staging/media/omap1/Kconfig
> >> @@ -0,0 +1,13 @@
> >> +config VIDEO_OMAP1
> >> +	tristate "OMAP1 Camera Interface driver"
> >> +	depends on VIDEO_DEV && SOC_CAMERA
> >> +	depends on ARCH_OMAP1
> >> +	depends on HAS_DMA
> >> +	select VIDEOBUF_DMA_CONTIG
> >> +	select VIDEOBUF_DMA_SG
> >> +	---help---
> >> +	  This is a v4l2 driver for the TI OMAP1 camera interface
> >> +
> >> +	  This driver is deprecated and will be removed soon unless someone
> >> +	  will start the work to convert this driver to the vb2 framework
> >> +	  and remove the soc-camera dependency.
> >> diff --git a/drivers/staging/media/omap1/Makefile b/drivers/staging/media/omap1/Makefile
> >> new file mode 100644
> >> index 0000000..2885622
> >> --- /dev/null
> >> +++ b/drivers/staging/media/omap1/Makefile
> >> @@ -0,0 +1,3 @@
> >> +# Makefile for OMAP1 driver
> >> +
> >> +obj-$(CONFIG_VIDEO_OMAP1) += omap1_camera.o
> >> diff --git a/drivers/media/platform/soc_camera/omap1_camera.c b/drivers/staging/media/omap1/omap1_camera.c
> >> similarity index 100%
> >> rename from drivers/media/platform/soc_camera/omap1_camera.c
> >> rename to drivers/staging/media/omap1/omap1_camera.c
> >> -- 
> >> 2.7.0
> >>
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-media" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > 
> 

^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH] soc_camera/omap1: move to staging in preparation for removal
  2016-02-23 15:05     ` Guennadi Liakhovetski
@ 2016-02-23 15:56       ` Hans Verkuil
  2016-02-23 16:03         ` Guennadi Liakhovetski
  0 siblings, 1 reply; 7+ messages in thread
From: Hans Verkuil @ 2016-02-23 15:56 UTC (permalink / raw)
  To: Guennadi Liakhovetski; +Cc: linux-media, Sakari Ailus

On 02/23/2016 04:05 PM, Guennadi Liakhovetski wrote:
> On Tue, 23 Feb 2016, Hans Verkuil wrote:
> 
>> On 02/23/16 15:55, Guennadi Liakhovetski wrote:
>>> Hi Hans,
>>>
>>> On Tue, 23 Feb 2016, Hans Verkuil wrote:
>>>
>>>> This driver is deprecated: it needs to be converted to vb2 and
>>>> it should become a stand-alone driver instead of using the
>>>> soc-camera framework.
>>>>
>>>> Unless someone is willing to take this on (unlikely with such
>>>> ancient hardware) it is going to be removed from the kernel
>>>> soon.
>>>>
>>>> Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
>>>
>>> I guess I won't be pulling this through my tree, right?
>>
>> Right.
>>
>>>
>>> Acked-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
>>
>> Thanks!
>>
>> This one was easy.
> 
> mx2, mx3 should be easy too?

Sure. Are you certain these can be deprecated? I haven't heard
anything regarding the status of these driver. You probably know that
best.

If you give me the go ahead, then I'll prepare patches for these to
move them to staging.

Regards,

	Hans

> 
> Guennadi
> 
>> For the sh renesas drivers I will take one final look during the weekend.
>> See what works and what doesn't. I actually have hardware for it, after
>> all.
>>
>> Regards,
>>
>> 	Hans
>>
>>>
>>> Thanks
>>> Guennadi
>>>
>>>> ---
>>>>  drivers/media/platform/soc_camera/Kconfig                   | 10 ----------
>>>>  drivers/media/platform/soc_camera/Makefile                  |  1 -
>>>>  drivers/staging/media/Kconfig                               |  2 ++
>>>>  drivers/staging/media/Makefile                              |  1 +
>>>>  drivers/staging/media/omap1/Kconfig                         | 13 +++++++++++++
>>>>  drivers/staging/media/omap1/Makefile                        |  3 +++
>>>>  .../soc_camera => staging/media/omap1}/omap1_camera.c       |  0
>>>>  7 files changed, 19 insertions(+), 11 deletions(-)
>>>>  create mode 100644 drivers/staging/media/omap1/Kconfig
>>>>  create mode 100644 drivers/staging/media/omap1/Makefile
>>>>  rename drivers/{media/platform/soc_camera => staging/media/omap1}/omap1_camera.c (100%)
>>>>
>>>> diff --git a/drivers/media/platform/soc_camera/Kconfig b/drivers/media/platform/soc_camera/Kconfig
>>>> index f2776cd..954dd36 100644
>>>> --- a/drivers/media/platform/soc_camera/Kconfig
>>>> +++ b/drivers/media/platform/soc_camera/Kconfig
>>>> @@ -60,16 +60,6 @@ config VIDEO_SH_MOBILE_CEU
>>>>  	---help---
>>>>  	  This is a v4l2 driver for the SuperH Mobile CEU Interface
>>>>
>>>> -config VIDEO_OMAP1
>>>> -	tristate "OMAP1 Camera Interface driver"
>>>> -	depends on VIDEO_DEV && SOC_CAMERA
>>>> -	depends on ARCH_OMAP1
>>>> -	depends on HAS_DMA
>>>> -	select VIDEOBUF_DMA_CONTIG
>>>> -	select VIDEOBUF_DMA_SG
>>>> -	---help---
>>>> -	  This is a v4l2 driver for the TI OMAP1 camera interface
>>>> -
>>>>  config VIDEO_MX2
>>>>  	tristate "i.MX27 Camera Sensor Interface driver"
>>>>  	depends on VIDEO_DEV && SOC_CAMERA
>>>> diff --git a/drivers/media/platform/soc_camera/Makefile b/drivers/media/platform/soc_camera/Makefile
>>>> index 2826382..bdd7fc9 100644
>>>> --- a/drivers/media/platform/soc_camera/Makefile
>>>> +++ b/drivers/media/platform/soc_camera/Makefile
>>>> @@ -9,7 +9,6 @@ obj-$(CONFIG_SOC_CAMERA_PLATFORM)	+= soc_camera_platform.o
>>>>  obj-$(CONFIG_VIDEO_ATMEL_ISI)		+= atmel-isi.o
>>>>  obj-$(CONFIG_VIDEO_MX2)			+= mx2_camera.o
>>>>  obj-$(CONFIG_VIDEO_MX3)			+= mx3_camera.o
>>>> -obj-$(CONFIG_VIDEO_OMAP1)		+= omap1_camera.o
>>>>  obj-$(CONFIG_VIDEO_PXA27x)		+= pxa_camera.o
>>>>  obj-$(CONFIG_VIDEO_SH_MOBILE_CEU)	+= sh_mobile_ceu_camera.o
>>>>  obj-$(CONFIG_VIDEO_SH_MOBILE_CSI2)	+= sh_mobile_csi2.o
>>>> diff --git a/drivers/staging/media/Kconfig b/drivers/staging/media/Kconfig
>>>> index d48a5c2..382d868 100644
>>>> --- a/drivers/staging/media/Kconfig
>>>> +++ b/drivers/staging/media/Kconfig
>>>> @@ -29,6 +29,8 @@ source "drivers/staging/media/mn88472/Kconfig"
>>>>
>>>>  source "drivers/staging/media/mn88473/Kconfig"
>>>>
>>>> +source "drivers/staging/media/omap1/Kconfig"
>>>> +
>>>>  source "drivers/staging/media/omap4iss/Kconfig"
>>>>
>>>>  source "drivers/staging/media/timb/Kconfig"
>>>> diff --git a/drivers/staging/media/Makefile b/drivers/staging/media/Makefile
>>>> index fb94f04..89d038c 100644
>>>> --- a/drivers/staging/media/Makefile
>>>> +++ b/drivers/staging/media/Makefile
>>>> @@ -2,6 +2,7 @@ obj-$(CONFIG_I2C_BCM2048)	+= bcm2048/
>>>>  obj-$(CONFIG_DVB_CXD2099)	+= cxd2099/
>>>>  obj-$(CONFIG_LIRC_STAGING)	+= lirc/
>>>>  obj-$(CONFIG_VIDEO_DM365_VPFE)	+= davinci_vpfe/
>>>> +obj-$(CONFIG_VIDEO_OMAP1)	+= omap1/
>>>>  obj-$(CONFIG_VIDEO_OMAP4)	+= omap4iss/
>>>>  obj-$(CONFIG_DVB_MN88472)       += mn88472/
>>>>  obj-$(CONFIG_DVB_MN88473)       += mn88473/
>>>> diff --git a/drivers/staging/media/omap1/Kconfig b/drivers/staging/media/omap1/Kconfig
>>>> new file mode 100644
>>>> index 0000000..6cfab3a
>>>> --- /dev/null
>>>> +++ b/drivers/staging/media/omap1/Kconfig
>>>> @@ -0,0 +1,13 @@
>>>> +config VIDEO_OMAP1
>>>> +	tristate "OMAP1 Camera Interface driver"
>>>> +	depends on VIDEO_DEV && SOC_CAMERA
>>>> +	depends on ARCH_OMAP1
>>>> +	depends on HAS_DMA
>>>> +	select VIDEOBUF_DMA_CONTIG
>>>> +	select VIDEOBUF_DMA_SG
>>>> +	---help---
>>>> +	  This is a v4l2 driver for the TI OMAP1 camera interface
>>>> +
>>>> +	  This driver is deprecated and will be removed soon unless someone
>>>> +	  will start the work to convert this driver to the vb2 framework
>>>> +	  and remove the soc-camera dependency.
>>>> diff --git a/drivers/staging/media/omap1/Makefile b/drivers/staging/media/omap1/Makefile
>>>> new file mode 100644
>>>> index 0000000..2885622
>>>> --- /dev/null
>>>> +++ b/drivers/staging/media/omap1/Makefile
>>>> @@ -0,0 +1,3 @@
>>>> +# Makefile for OMAP1 driver
>>>> +
>>>> +obj-$(CONFIG_VIDEO_OMAP1) += omap1_camera.o
>>>> diff --git a/drivers/media/platform/soc_camera/omap1_camera.c b/drivers/staging/media/omap1/omap1_camera.c
>>>> similarity index 100%
>>>> rename from drivers/media/platform/soc_camera/omap1_camera.c
>>>> rename to drivers/staging/media/omap1/omap1_camera.c
>>>> -- 
>>>> 2.7.0
>>>>
>>> --
>>> To unsubscribe from this list: send the line "unsubscribe linux-media" in
>>> the body of a message to majordomo@vger.kernel.org
>>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>>
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 


^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH] soc_camera/omap1: move to staging in preparation for removal
  2016-02-23 15:56       ` Hans Verkuil
@ 2016-02-23 16:03         ` Guennadi Liakhovetski
  2016-02-23 16:09           ` Hans Verkuil
  0 siblings, 1 reply; 7+ messages in thread
From: Guennadi Liakhovetski @ 2016-02-23 16:03 UTC (permalink / raw)
  To: Hans Verkuil; +Cc: linux-media, Sakari Ailus

On Tue, 23 Feb 2016, Hans Verkuil wrote:

> On 02/23/2016 04:05 PM, Guennadi Liakhovetski wrote:
> > On Tue, 23 Feb 2016, Hans Verkuil wrote:
> > 
> >> On 02/23/16 15:55, Guennadi Liakhovetski wrote:
> >>> Hi Hans,
> >>>
> >>> On Tue, 23 Feb 2016, Hans Verkuil wrote:
> >>>
> >>>> This driver is deprecated: it needs to be converted to vb2 and
> >>>> it should become a stand-alone driver instead of using the
> >>>> soc-camera framework.
> >>>>
> >>>> Unless someone is willing to take this on (unlikely with such
> >>>> ancient hardware) it is going to be removed from the kernel
> >>>> soon.
> >>>>
> >>>> Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
> >>>
> >>> I guess I won't be pulling this through my tree, right?
> >>
> >> Right.
> >>
> >>>
> >>> Acked-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
> >>
> >> Thanks!
> >>
> >> This one was easy.
> > 
> > mx2, mx3 should be easy too?
> 
> Sure. Are you certain these can be deprecated? I haven't heard
> anything regarding the status of these driver. You probably know that
> best.
> 
> If you give me the go ahead, then I'll prepare patches for these to
> move them to staging.

I asked pengutronix guys, they don't care about them any more. We had 
someone from freescale on CC in the original discussion, they didn't 
report back either, so... Up to you really, I'd put the probability of 
someone coming forward with a strong enough case to keep them at 4% :-) Is 
it good enough for you to prepare that patch?

Thanks
Guennadi

> Regards,
> 
> 	Hans
> 
> > 
> > Guennadi
> > 
> >> For the sh renesas drivers I will take one final look during the weekend.
> >> See what works and what doesn't. I actually have hardware for it, after
> >> all.
> >>
> >> Regards,
> >>
> >> 	Hans
> >>
> >>>
> >>> Thanks
> >>> Guennadi
> >>>
> >>>> ---
> >>>>  drivers/media/platform/soc_camera/Kconfig                   | 10 ----------
> >>>>  drivers/media/platform/soc_camera/Makefile                  |  1 -
> >>>>  drivers/staging/media/Kconfig                               |  2 ++
> >>>>  drivers/staging/media/Makefile                              |  1 +
> >>>>  drivers/staging/media/omap1/Kconfig                         | 13 +++++++++++++
> >>>>  drivers/staging/media/omap1/Makefile                        |  3 +++
> >>>>  .../soc_camera => staging/media/omap1}/omap1_camera.c       |  0
> >>>>  7 files changed, 19 insertions(+), 11 deletions(-)
> >>>>  create mode 100644 drivers/staging/media/omap1/Kconfig
> >>>>  create mode 100644 drivers/staging/media/omap1/Makefile
> >>>>  rename drivers/{media/platform/soc_camera => staging/media/omap1}/omap1_camera.c (100%)
> >>>>
> >>>> diff --git a/drivers/media/platform/soc_camera/Kconfig b/drivers/media/platform/soc_camera/Kconfig
> >>>> index f2776cd..954dd36 100644
> >>>> --- a/drivers/media/platform/soc_camera/Kconfig
> >>>> +++ b/drivers/media/platform/soc_camera/Kconfig
> >>>> @@ -60,16 +60,6 @@ config VIDEO_SH_MOBILE_CEU
> >>>>  	---help---
> >>>>  	  This is a v4l2 driver for the SuperH Mobile CEU Interface
> >>>>
> >>>> -config VIDEO_OMAP1
> >>>> -	tristate "OMAP1 Camera Interface driver"
> >>>> -	depends on VIDEO_DEV && SOC_CAMERA
> >>>> -	depends on ARCH_OMAP1
> >>>> -	depends on HAS_DMA
> >>>> -	select VIDEOBUF_DMA_CONTIG
> >>>> -	select VIDEOBUF_DMA_SG
> >>>> -	---help---
> >>>> -	  This is a v4l2 driver for the TI OMAP1 camera interface
> >>>> -
> >>>>  config VIDEO_MX2
> >>>>  	tristate "i.MX27 Camera Sensor Interface driver"
> >>>>  	depends on VIDEO_DEV && SOC_CAMERA
> >>>> diff --git a/drivers/media/platform/soc_camera/Makefile b/drivers/media/platform/soc_camera/Makefile
> >>>> index 2826382..bdd7fc9 100644
> >>>> --- a/drivers/media/platform/soc_camera/Makefile
> >>>> +++ b/drivers/media/platform/soc_camera/Makefile
> >>>> @@ -9,7 +9,6 @@ obj-$(CONFIG_SOC_CAMERA_PLATFORM)	+= soc_camera_platform.o
> >>>>  obj-$(CONFIG_VIDEO_ATMEL_ISI)		+= atmel-isi.o
> >>>>  obj-$(CONFIG_VIDEO_MX2)			+= mx2_camera.o
> >>>>  obj-$(CONFIG_VIDEO_MX3)			+= mx3_camera.o
> >>>> -obj-$(CONFIG_VIDEO_OMAP1)		+= omap1_camera.o
> >>>>  obj-$(CONFIG_VIDEO_PXA27x)		+= pxa_camera.o
> >>>>  obj-$(CONFIG_VIDEO_SH_MOBILE_CEU)	+= sh_mobile_ceu_camera.o
> >>>>  obj-$(CONFIG_VIDEO_SH_MOBILE_CSI2)	+= sh_mobile_csi2.o
> >>>> diff --git a/drivers/staging/media/Kconfig b/drivers/staging/media/Kconfig
> >>>> index d48a5c2..382d868 100644
> >>>> --- a/drivers/staging/media/Kconfig
> >>>> +++ b/drivers/staging/media/Kconfig
> >>>> @@ -29,6 +29,8 @@ source "drivers/staging/media/mn88472/Kconfig"
> >>>>
> >>>>  source "drivers/staging/media/mn88473/Kconfig"
> >>>>
> >>>> +source "drivers/staging/media/omap1/Kconfig"
> >>>> +
> >>>>  source "drivers/staging/media/omap4iss/Kconfig"
> >>>>
> >>>>  source "drivers/staging/media/timb/Kconfig"
> >>>> diff --git a/drivers/staging/media/Makefile b/drivers/staging/media/Makefile
> >>>> index fb94f04..89d038c 100644
> >>>> --- a/drivers/staging/media/Makefile
> >>>> +++ b/drivers/staging/media/Makefile
> >>>> @@ -2,6 +2,7 @@ obj-$(CONFIG_I2C_BCM2048)	+= bcm2048/
> >>>>  obj-$(CONFIG_DVB_CXD2099)	+= cxd2099/
> >>>>  obj-$(CONFIG_LIRC_STAGING)	+= lirc/
> >>>>  obj-$(CONFIG_VIDEO_DM365_VPFE)	+= davinci_vpfe/
> >>>> +obj-$(CONFIG_VIDEO_OMAP1)	+= omap1/
> >>>>  obj-$(CONFIG_VIDEO_OMAP4)	+= omap4iss/
> >>>>  obj-$(CONFIG_DVB_MN88472)       += mn88472/
> >>>>  obj-$(CONFIG_DVB_MN88473)       += mn88473/
> >>>> diff --git a/drivers/staging/media/omap1/Kconfig b/drivers/staging/media/omap1/Kconfig
> >>>> new file mode 100644
> >>>> index 0000000..6cfab3a
> >>>> --- /dev/null
> >>>> +++ b/drivers/staging/media/omap1/Kconfig
> >>>> @@ -0,0 +1,13 @@
> >>>> +config VIDEO_OMAP1
> >>>> +	tristate "OMAP1 Camera Interface driver"
> >>>> +	depends on VIDEO_DEV && SOC_CAMERA
> >>>> +	depends on ARCH_OMAP1
> >>>> +	depends on HAS_DMA
> >>>> +	select VIDEOBUF_DMA_CONTIG
> >>>> +	select VIDEOBUF_DMA_SG
> >>>> +	---help---
> >>>> +	  This is a v4l2 driver for the TI OMAP1 camera interface
> >>>> +
> >>>> +	  This driver is deprecated and will be removed soon unless someone
> >>>> +	  will start the work to convert this driver to the vb2 framework
> >>>> +	  and remove the soc-camera dependency.
> >>>> diff --git a/drivers/staging/media/omap1/Makefile b/drivers/staging/media/omap1/Makefile
> >>>> new file mode 100644
> >>>> index 0000000..2885622
> >>>> --- /dev/null
> >>>> +++ b/drivers/staging/media/omap1/Makefile
> >>>> @@ -0,0 +1,3 @@
> >>>> +# Makefile for OMAP1 driver
> >>>> +
> >>>> +obj-$(CONFIG_VIDEO_OMAP1) += omap1_camera.o
> >>>> diff --git a/drivers/media/platform/soc_camera/omap1_camera.c b/drivers/staging/media/omap1/omap1_camera.c
> >>>> similarity index 100%
> >>>> rename from drivers/media/platform/soc_camera/omap1_camera.c
> >>>> rename to drivers/staging/media/omap1/omap1_camera.c
> >>>> -- 
> >>>> 2.7.0
> >>>>
> >>> --
> >>> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> >>> the body of a message to majordomo@vger.kernel.org
> >>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> >>>
> >>
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-media" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > 
> 

^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH] soc_camera/omap1: move to staging in preparation for removal
  2016-02-23 16:03         ` Guennadi Liakhovetski
@ 2016-02-23 16:09           ` Hans Verkuil
  0 siblings, 0 replies; 7+ messages in thread
From: Hans Verkuil @ 2016-02-23 16:09 UTC (permalink / raw)
  To: Guennadi Liakhovetski; +Cc: linux-media, Sakari Ailus

On 02/23/2016 05:03 PM, Guennadi Liakhovetski wrote:
> On Tue, 23 Feb 2016, Hans Verkuil wrote:
> 
>> On 02/23/2016 04:05 PM, Guennadi Liakhovetski wrote:
>>> On Tue, 23 Feb 2016, Hans Verkuil wrote:
>>>
>>>> On 02/23/16 15:55, Guennadi Liakhovetski wrote:
>>>>> Hi Hans,
>>>>>
>>>>> On Tue, 23 Feb 2016, Hans Verkuil wrote:
>>>>>
>>>>>> This driver is deprecated: it needs to be converted to vb2 and
>>>>>> it should become a stand-alone driver instead of using the
>>>>>> soc-camera framework.
>>>>>>
>>>>>> Unless someone is willing to take this on (unlikely with such
>>>>>> ancient hardware) it is going to be removed from the kernel
>>>>>> soon.
>>>>>>
>>>>>> Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
>>>>>
>>>>> I guess I won't be pulling this through my tree, right?
>>>>
>>>> Right.
>>>>
>>>>>
>>>>> Acked-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
>>>>
>>>> Thanks!
>>>>
>>>> This one was easy.
>>>
>>> mx2, mx3 should be easy too?
>>
>> Sure. Are you certain these can be deprecated? I haven't heard
>> anything regarding the status of these driver. You probably know that
>> best.
>>
>> If you give me the go ahead, then I'll prepare patches for these to
>> move them to staging.
> 
> I asked pengutronix guys, they don't care about them any more. We had 
> someone from freescale on CC in the original discussion, they didn't 
> report back either, so... Up to you really, I'd put the probability of 
> someone coming forward with a strong enough case to keep them at 4% :-) Is 
> it good enough for you to prepare that patch?

Yes, that's good enough. It's not as if we remove it right away, it'll be
in staging for a while.

Thanks for the info,

	Hans


^ permalink raw reply	[flat|nested] 7+ messages in thread
end of thread, other threads:[~2016-02-23 16:09 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-23 12:13 [PATCH] soc_camera/omap1: move to staging in preparation for removal Hans Verkuil
2016-02-23 14:55 ` Guennadi Liakhovetski
2016-02-23 14:59   ` Hans Verkuil
2016-02-23 15:05     ` Guennadi Liakhovetski
2016-02-23 15:56       ` Hans Verkuil
2016-02-23 16:03         ` Guennadi Liakhovetski
2016-02-23 16:09           ` Hans Verkuil

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.