[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