From: Hans Verkuil <hverkuil@xs4all.nl>
To: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Cc: linux-media <linux-media@vger.kernel.org>,
Sakari Ailus <sakari.ailus@iki.fi>
Subject: Re: [PATCH] soc_camera/omap1: move to staging in preparation for removal
Date: Tue, 23 Feb 2016 16:56:21 +0100 [thread overview]
Message-ID: <56CC8125.60205@xs4all.nl> (raw)
In-Reply-To: <Pine.LNX.4.64.1602231604460.17650@axis700.grange>
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
>
next prev parent reply other threads:[~2016-02-23 15:56 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
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 [this message]
2016-02-23 16:03 ` Guennadi Liakhovetski
2016-02-23 16:09 ` Hans Verkuil
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=56CC8125.60205@xs4all.nl \
--to=hverkuil@xs4all.nl \
--cc=g.liakhovetski@gmx.de \
--cc=linux-media@vger.kernel.org \
--cc=sakari.ailus@iki.fi \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox