[PATCH] media: mx1_camera: Remove driver
Hans Verkuil
hverkuil at xs4all.nl
Mon May 12 09:39:57 PDT 2014
On 05/12/2014 05:46 PM, Alexander Shiyan wrote:
> Mon, 12 May 2014 22:25:34 +0800 от Shawn Guo <shawn.guo at freescale.com>:
>> On Mon, May 12, 2014 at 06:18:00PM +0400, Alexander Shiyan wrote:
>>> Mon, 12 May 2014 22:09:34 +0800 от Shawn Guo <shawn.guo at freescale.com>:
>>>> On Sun, May 11, 2014 at 10:09:11AM +0400, Alexander Shiyan wrote:
>>>>> That driver hasn't been really maintained for a long time. It doesn't
>>>>> compile in any way, it includes non-existent headers, has no users,
>>>>> and marked as "broken" more than year. Due to these factors, mx1_camera
>>>>> is now removed from the tree.
>>>>>
>>>>> Signed-off-by: Alexander Shiyan <shc_work at mail.ru>
>>>>> ---
>>>>> arch/arm/mach-imx/Makefile | 3 -
>>>>> arch/arm/mach-imx/devices/Kconfig | 3 -
>>>>> arch/arm/mach-imx/devices/Makefile | 1 -
>>>>> arch/arm/mach-imx/devices/devices-common.h | 10 -
>>>>> arch/arm/mach-imx/devices/platform-mx1-camera.c | 42 --
>>>>> arch/arm/mach-imx/mx1-camera-fiq-ksym.c | 18 -
>>>>> arch/arm/mach-imx/mx1-camera-fiq.S | 35 -
>>>>> drivers/media/platform/soc_camera/Kconfig | 13 -
>>>>> drivers/media/platform/soc_camera/Makefile | 1 -
>>>>> drivers/media/platform/soc_camera/mx1_camera.c | 866 ------------------------
>>>>> include/linux/platform_data/camera-mx1.h | 35 -
>>>>> 11 files changed, 1027 deletions(-)
>>>>> delete mode 100644 arch/arm/mach-imx/devices/platform-mx1-camera.c
>>>>> delete mode 100644 arch/arm/mach-imx/mx1-camera-fiq-ksym.c
>>>>> delete mode 100644 arch/arm/mach-imx/mx1-camera-fiq.S
>>>>> delete mode 100644 drivers/media/platform/soc_camera/mx1_camera.c
>>>>> delete mode 100644 include/linux/platform_data/camera-mx1.h
>>>>
>>>> Can this patch be split into arch and driver part? Recently, arm-soc
>>>> folks do not want to have arch changes go via driver tree, unless that's
>>>> absolutely necessary.
>>>
>>> Can this patch be applied through arm-soc (imx) tree if it will be approved
>>> by the linux-media maintainers?
>>
>> Yes, it can, if linux-media maintainers want. But I still prefer to two
>> patches, since I do not see any thing requiring it be one. Doing that
>> will ensure we do not run into any merge conflicts.
>
> ARM part and linux-media part are interconnected by using single header
> <linux/platform_data/camera-mx1.h>. So removing a driver in a separated
> patch will touch ARM part in any case.
But the linux-media driver depends on BROKEN, so that isn't build in any
case, right?
Regards,
Hans
More information about the linux-arm-kernel
mailing list