Message ID | 1411424546-12718-1-git-send-email-srinivas.kandagatla@linaro.org (mailing list archive) |
---|---|
State | Accepted, archived |
Headers |
Received: from mail.tu-berlin.de ([130.149.7.33]) by www.linuxtv.org with esmtp (Exim 4.72) (envelope-from <linux-media-owner@vger.kernel.org>) id 1XWC0R-0003mH-UA; Tue, 23 Sep 2014 00:22:51 +0200 X-tubIT-Incoming-IP: 209.132.180.67 Received: from vger.kernel.org ([209.132.180.67]) by mail.tu-berlin.de (exim-4.72/mailfrontend-8) with esmtp id 1XWC0P-0004kB-mI; Tue, 23 Sep 2014 00:22:51 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755103AbaIVWWd (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Mon, 22 Sep 2014 18:22:33 -0400 Received: from mail-we0-f176.google.com ([74.125.82.176]:45290 "EHLO mail-we0-f176.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755080AbaIVWWc (ORCPT <rfc822;linux-media@vger.kernel.org>); Mon, 22 Sep 2014 18:22:32 -0400 Received: by mail-we0-f176.google.com with SMTP id w61so2810381wes.35 for <linux-media@vger.kernel.org>; Mon, 22 Sep 2014 15:22:30 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=qKfuuyBGwV3N9zE6rKzB+/U36euMn0mE2xPWL/GcJDs=; b=Xby8KzZKxSl7XbZSRvuKTt3PykNetvPHeKA8cj7mM07iLw9QiTKPp6pfBF1SBD6FBG qLVWkv7URRjvEPM8UZfFCmnp3pPdKOZdcLGans3aDooZW6ZoxYCcyMkbeOkCi/KO8Lwf BWISQJedM3nUgP1Z92lmnFcZpxJZFC5XrSd1xaGPay/7TAJ3M7I/wQ1+6FELPffpyyce 2x0NYB3usydZdMr6MM2FiQTzlkqpeibE2TUB4FXzGGaYaivRHejlm/PioYksneiSR4So 8ocGFV4ZeVxvR9MtLPVjI5xl92lxsdr5GY/bTjNhe94EoJHuTwgABZT4KK+uHkvQYSMU 0yaw== X-Gm-Message-State: ALoCoQkSVYK9X1PBhkDQmVJoMqNnIa7ez4hHU5BgA8OATjK07o3dtn1crGwOPi+RvX8Dpscntx9l X-Received: by 10.194.205.196 with SMTP id li4mr22952675wjc.46.1411424550672; Mon, 22 Sep 2014 15:22:30 -0700 (PDT) Received: from srini-ThinkPad-X1-Carbon-2nd.dlink.com (host-78-149-10-230.as13285.net. [78.149.10.230]) by mx.google.com with ESMTPSA id eq9sm117612wic.20.2014.09.22.15.22.29 for <multiple recipients> (version=TLSv1.2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Mon, 22 Sep 2014 15:22:29 -0700 (PDT) From: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> To: Mauro Carvalho Chehab <m.chehab@samsung.com> Cc: Maxime Coquelin <maxime.coquelin@st.com>, Patrice Chotard <patrice.chotard@st.com>, linux-arm-kernel@lists.infradead.org, kernel@stlinux.com, linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, Srinivas Kandagatla <srinivas.kandagatla@linaro.org> Subject: [PATCH 1/3] media: st-rc: move to using reset_control_get_optional Date: Mon, 22 Sep 2014 23:22:26 +0100 Message-Id: <1411424546-12718-1-git-send-email-srinivas.kandagatla@linaro.org> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1411424501-12673-1-git-send-email-srinivas.kandagatla@linaro.org> References: <1411424501-12673-1-git-send-email-srinivas.kandagatla@linaro.org> Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-PMX-Version: 6.0.0.2142326, Antispam-Engine: 2.7.2.2107409, Antispam-Data: 2014.9.22.221520 X-PMX-Spam: Gauge=IIIIIIII, Probability=8%, Report=' MULTIPLE_RCPTS 0.1, HTML_00_01 0.05, HTML_00_10 0.05, BODYTEXTP_SIZE_3000_LESS 0, BODY_SIZE_1300_1399 0, BODY_SIZE_2000_LESS 0, BODY_SIZE_5000_LESS 0, BODY_SIZE_7000_LESS 0, URI_ENDS_IN_HTML 0, __ANY_URI 0, __CP_URI_IN_BODY 0, __FRAUD_BODY_WEBMAIL 0, __FRAUD_WEBMAIL 0, __HAS_FROM 0, __HAS_MSGID 0, __HAS_X_MAILER 0, __HAS_X_MAILING_LIST 0, __IN_REP_TO 0, __MIME_TEXT_ONLY 0, __MULTIPLE_RCPTS_CC_X2 0, __SANE_MSGID 0, __TO_MALFORMED_2 0, __URI_NO_WWW 0, __URI_NS , __YOUTUBE_RCVD 0' |
Commit Message
Srinivas Kandagatla
Sept. 22, 2014, 10:22 p.m. UTC
This patch fixes a compilation error while building with the
random kernel configuration.
drivers/media/rc/st_rc.c: In function 'st_rc_probe':
drivers/media/rc/st_rc.c:281:2: error: implicit declaration of
function 'reset_control_get' [-Werror=implicit-function-declaration]
rc_dev->rstc = reset_control_get(dev, NULL);
drivers/media/rc/st_rc.c:281:15: warning: assignment makes pointer
from integer without a cast [enabled by default]
rc_dev->rstc = reset_control_get(dev, NULL);
Reported-by: Jim Davis <jim.epost@gmail.com>
Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
---
drivers/media/rc/st_rc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
Hi Srini, On Mon, 22 Sep 2014, Srinivas Kandagatla wrote: > This patch fixes a compilation error while building with the > random kernel configuration. > > drivers/media/rc/st_rc.c: In function 'st_rc_probe': > drivers/media/rc/st_rc.c:281:2: error: implicit declaration of > function 'reset_control_get' [-Werror=implicit-function-declaration] > rc_dev->rstc = reset_control_get(dev, NULL); > > drivers/media/rc/st_rc.c:281:15: warning: assignment makes pointer > from integer without a cast [enabled by default] > rc_dev->rstc = reset_control_get(dev, NULL); Is managing the reset line actually optional though? I can't test atm as I don't have access to my board, but quite often if the IP's aren't taken out of reset reads / writes to the perhpiheral will hang the SoC. If managing the reset line isn't optional then I think the correct fix is to add depends on RESET_CONTROLLER in the kconfig. This will then do the right thing for randconfig builds as well. regards, Peter. -- 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
Hi Pete, On 23/09/14 19:02, Peter Griffin wrote: > Hi Srini, > > On Mon, 22 Sep 2014, Srinivas Kandagatla wrote: > >> This patch fixes a compilation error while building with the >> random kernel configuration. >> >> drivers/media/rc/st_rc.c: In function 'st_rc_probe': >> drivers/media/rc/st_rc.c:281:2: error: implicit declaration of >> function 'reset_control_get' [-Werror=implicit-function-declaration] >> rc_dev->rstc = reset_control_get(dev, NULL); >> >> drivers/media/rc/st_rc.c:281:15: warning: assignment makes pointer >> from integer without a cast [enabled by default] >> rc_dev->rstc = reset_control_get(dev, NULL); > > Is managing the reset line actually optional though? I can't test atm as I don't have > access to my board, but quite often if the IP's aren't taken out of reset reads / writes > to the perhpiheral will hang the SoC. > Yes and No. AFAIK reset line is optional on SOCs like 7108, 7141. I think having the driver function without reset might is a value add in case we plan to reuse the mainline driver for these SOCs. On latest ARM SOCs with SBC the IRB IP is moved to SBC and held in reset. Am not sure, if the reset line is optional in next generation SOCs? > If managing the reset line isn't optional then I think the correct fix is to add > depends on RESET_CONTROLLER in the kconfig. I agree. This would make the COMPILE_TEST less useful though. thanks, srini > > This will then do the right thing for randconfig builds as well. > > regards, > > Peter. > -- 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
Hi Srini, Peter, On 09/24/2014 10:12 AM, Srinivas Kandagatla wrote: > Hi Pete, > > On 23/09/14 19:02, Peter Griffin wrote: >> Hi Srini, >> >> On Mon, 22 Sep 2014, Srinivas Kandagatla wrote: >> >>> This patch fixes a compilation error while building with the >>> random kernel configuration. >>> >>> drivers/media/rc/st_rc.c: In function 'st_rc_probe': >>> drivers/media/rc/st_rc.c:281:2: error: implicit declaration of >>> function 'reset_control_get' [-Werror=implicit-function-declaration] >>> rc_dev->rstc = reset_control_get(dev, NULL); >>> >>> drivers/media/rc/st_rc.c:281:15: warning: assignment makes pointer >>> from integer without a cast [enabled by default] >>> rc_dev->rstc = reset_control_get(dev, NULL); >> >> Is managing the reset line actually optional though? I can't test atm >> as I don't have >> access to my board, but quite often if the IP's aren't taken out of >> reset reads / writes >> to the perhpiheral will hang the SoC. >> > Yes and No. > AFAIK reset line is optional on SOCs like 7108, 7141. > I think having the driver function without reset might is a value add in > case we plan to reuse the mainline driver for these SOCs. > > On latest ARM SOCs with SBC the IRB IP is moved to SBC and held in reset. > Am not sure, if the reset line is optional in next generation SOCs? I don't know for next SoCs, but I think it makes sense to make it optional. Regards, Maxime > >> If managing the reset line isn't optional then I think the correct fix >> is to add >> depends on RESET_CONTROLLER in the kconfig. > I agree. > This would make the COMPILE_TEST less useful though. > > > thanks, > srini >> >> This will then do the right thing for randconfig builds as well. >> >> regards, >> >> Peter. >> > > _______________________________________________ > Kernel mailing list > Kernel@stlinux.com > http://www.stlinux.com/mailman/listinfo/kernel -- 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
Hi Srini, Sorry seems I never replied to this. > >>drivers/media/rc/st_rc.c:281:15: warning: assignment makes pointer > >>from integer without a cast [enabled by default] > >> rc_dev->rstc = reset_control_get(dev, NULL); > > > >Is managing the reset line actually optional though? I can't test atm as I don't have > >access to my board, but quite often if the IP's aren't taken out of reset reads / writes > >to the perhpiheral will hang the SoC. > > > Yes and No. > AFAIK reset line is optional on SOCs like 7108, 7141. > I think having the driver function without reset might is a value > add in case we plan to reuse the mainline driver for these SOCs. Yes that is a good point, for the series: - Acked-by: Peter Griffin <peter.griffin@linaro.org> regards, Peter. -- 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
diff --git a/drivers/media/rc/st_rc.c b/drivers/media/rc/st_rc.c index 5c15135..e0f1312 100644 --- a/drivers/media/rc/st_rc.c +++ b/drivers/media/rc/st_rc.c @@ -278,7 +278,7 @@ static int st_rc_probe(struct platform_device *pdev) rc_dev->rx_base = rc_dev->base; - rc_dev->rstc = reset_control_get(dev, NULL); + rc_dev->rstc = reset_control_get_optional(dev, NULL); if (IS_ERR(rc_dev->rstc)) rc_dev->rstc = NULL;