From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from lb1-smtp-cloud6.xs4all.net ([194.109.24.24]:35401 "EHLO lb1-smtp-cloud6.xs4all.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757988AbcBXHUm (ORCPT ); Wed, 24 Feb 2016 02:20:42 -0500 Subject: Re: [PATCH] v4l2: remove MIPI CSI-2 driver for SH-Mobile platforms To: Simon Horman , Laurent Pinchart References: <1456279679-11342-1-git-send-email-horms+renesas@verge.net.au> <2212155.BHpL65I02t@avalon> <20160224061721.GK5435@verge.net.au> Cc: Guennadi Liakhovetski , Laurent Pinchart , Magnus Damm , linux-media@vger.kernel.org, linux-renesas-soc@vger.kernel.org From: Hans Verkuil Message-ID: <56CD59C3.2030401@xs4all.nl> Date: Wed, 24 Feb 2016 08:20:35 +0100 MIME-Version: 1.0 In-Reply-To: <20160224061721.GK5435@verge.net.au> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: On 02/24/2016 07:17 AM, Simon Horman wrote: > On Wed, Feb 24, 2016 at 07:59:57AM +0200, Laurent Pinchart wrote: >> Hi Simon, >> >> Thank you for the patch. >> >> On Wednesday 24 February 2016 11:07:59 Simon Horman wrote: >>> This driver does not appear to have ever been used by any SoC's defconfig >>> and does not appear to support DT. In sort it seems unused an unlikely >>> to be used. >>> >>> Signed-off-by: Simon Horman >>> --- >>> drivers/media/platform/soc_camera/Kconfig | 7 - >>> drivers/media/platform/soc_camera/Makefile | 1 - >>> drivers/media/platform/soc_camera/sh_mobile_csi2.c | 400 ------------------ >> >> Shouldn't you also remove include/media/drv-intf/sh_mobile_csi2.h ? You would >> then need to update drivers/media/platform/soc_camera/sh_mobile_ceu.c >> accordingly, or remove it altogether. > > Thanks. > > sh_mobile_ceu appears to be used by several SH boards so I'd rather > not remove it, at least not for this reason. > > So I'd prefer to look into updating sh_mobile_ceu.c and removing > sh_mobile_csi2.h. Last time I checked the ceu driver failed to work (missing clock). I'll test again this weekend with the latest kernel. See what the status is of this driver. I got some patches as well for the ceu, but I'm not sure if I ever got it to fully work. I'll report back after my tests. Regards, Hans > >>> 3 files changed, 408 deletions(-) >>> delete mode 100644 drivers/media/platform/soc_camera/sh_mobile_csi2.c >>> >>> Based on the master branch of media_tree >> >> -- >> Regards, >> >> Laurent Pinchart > -- > 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 >