From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754841AbbDJUUi (ORCPT ); Fri, 10 Apr 2015 16:20:38 -0400 Received: from mout.kundenserver.de ([212.227.126.131]:57533 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751798AbbDJUUg (ORCPT ); Fri, 10 Apr 2015 16:20:36 -0400 From: Arnd Bergmann To: linux-media@vger.kernel.org, Tony Lindgren Cc: Tero Kristo , Laurent Pinchart , Mauro Carvalho Chehab , devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-omap@vger.kernel.org Subject: [PATCH] [media] omap4iss: avoid broken OMAP4 dependency Date: Fri, 10 Apr 2015 22:20:20 +0200 Message-ID: <3292592.cickJMVhRq@wuerfel> User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; ) MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:j/a0r7dHnZJm/1ScelgVjZkCxkws/t6iVDFv5AsI3K/OWwkZWC3 G8HfCIUeYLVB6cn+asbzLQXLEoe5VuB/CuU/VzRuF8k2u+Xr5OTnSWBHSciov1W0nci7qrU +wLNeX+cywnbWYVunWWPKpacOgIwg5N0N/U947Za4riadKBdZloOCdlAG9imGl+LY2AkoTi /6mQPdfXjx1S/0jFVwOqg== X-UI-Out-Filterresults: notjunk:1; Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The omap4iss driver uses an interface that used to be provided by OMAP4 but has now been removed and replaced with a WARN_ON(1) statement, which likely broke the iss_csiphy code at runtime. It also broke compiling the driver when CONFIG_ARCH_OMAP2PLUS is set, which is implied by OMAP4: drivers/staging/media/omap4iss/iss_csiphy.c: In function 'omap4iss_csiphy_config': drivers/staging/media/omap4iss/iss_csiphy.c:167:2: error: implicit declaration of function 'omap4_ctrl_pad_writel' [-Werror=implicit-function-declaration] omap4_ctrl_pad_writel(cam_rx_ctrl, In turn, this broke ARM allyesconfig builds. Replacing the omap4_ctrl_pad_writel call with WARN_ON(1) won't make the situation any worse than it already is, but fixes the build problem. Signed-off-by: Arnd Bergmann Fixes: efde234674d9 ("ARM: OMAP4+: control: remove support for legacy pad read/write") --- diff --git a/drivers/staging/media/omap4iss/iss_csiphy.c b/drivers/staging/media/omap4iss/iss_csiphy.c index 7c3d55d811ef..24f56ed90ac3 100644 --- a/drivers/staging/media/omap4iss/iss_csiphy.c +++ b/drivers/staging/media/omap4iss/iss_csiphy.c @@ -140,9 +140,7 @@ int omap4iss_csiphy_config(struct iss_device *iss, * - bit [18] : CSIPHY1 CTRLCLK enable * - bit [17:16] : CSIPHY1 config: 00 d-phy, 01/10 ccp2 */ - cam_rx_ctrl = omap4_ctrl_pad_readl( - OMAP4_CTRL_MODULE_PAD_CORE_CONTROL_CAMERA_RX); - + cam_rx_ctrl = WARN_ON(1); if (subdevs->interface == ISS_INTERFACE_CSI2A_PHY1) { cam_rx_ctrl &= ~(OMAP4_CAMERARX_CSI21_LANEENABLE_MASK | @@ -166,8 +164,7 @@ int omap4iss_csiphy_config(struct iss_device *iss, cam_rx_ctrl |= OMAP4_CAMERARX_CSI22_CTRLCLKEN_MASK; } - omap4_ctrl_pad_writel(cam_rx_ctrl, - OMAP4_CTRL_MODULE_PAD_CORE_CONTROL_CAMERA_RX); + WARN_ON(1); /* Reset used lane count */ csi2->phy->used_data_lanes = 0;