From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755427AbbIAJMt (ORCPT ); Tue, 1 Sep 2015 05:12:49 -0400 Received: from mail-wi0-f178.google.com ([209.85.212.178]:36058 "EHLO mail-wi0-f178.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755217AbbIAJMr (ORCPT ); Tue, 1 Sep 2015 05:12:47 -0400 Date: Tue, 1 Sep 2015 10:12:43 +0100 From: Lee Jones To: Peter Griffin Cc: Nathan Lynch , ohad@wizery.com, devicetree@vger.kernel.org, kernel@stlinux.com, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [STLinux Kernel] [PATCH v2 3/4] remoteproc: Supply controller driver for ST's Remote Processors Message-ID: <20150901091243.GP4796@x1> References: <1440757911-9120-1-git-send-email-lee.jones@linaro.org> <1440757911-9120-4-git-send-email-lee.jones@linaro.org> <55E08B34.9080102@mentor.com> <20150901075538.GJ4796@x1> <20150901081700.GA30542@griffinp-ThinkPad-X1-Carbon-2nd> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20150901081700.GA30542@griffinp-ThinkPad-X1-Carbon-2nd> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 01 Sep 2015, Peter Griffin wrote: > Hi, > > On Tue, 01 Sep 2015, Lee Jones wrote: > > > On Fri, 28 Aug 2015, Nathan Lynch wrote: > > > > > On 08/28/2015 05:31 AM, Lee Jones wrote: > > > > diff --git a/drivers/remoteproc/Kconfig b/drivers/remoteproc/Kconfig > > > > index 28c711f..72e97d7 100644 > > > > --- a/drivers/remoteproc/Kconfig > > > > +++ b/drivers/remoteproc/Kconfig > > > > @@ -77,4 +77,13 @@ config DA8XX_REMOTEPROC > > > > It's safe to say n here if you're not interested in multimedia > > > > offloading. > > > > > > > > +config ST_REMOTEPROC > > > > + tristate "ST remoteproc support" > > > > + depends on ARCH_STI > > > > + select REMOTEPROC > > > > + help > > > > + Say y here to support ST's adjunct processors via the remote > > > > + processor framework. > > > > + This can be either built-in or a loadable module. > > > > + > > > > > > The code uses reset_control_* APIs, so this should depend on > > > RESET_CONTROLLER, no? > > > > There's no need to explicitly depend on RESET_CONTROLLER. > > > > With !RESET_CONTROLLER the user is WARN()ed about using the reset_* > > API. > > ARCH_STI selects RESET_CONTROLLER, so it will always be enabled. That too. Thanks for pointing that out. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog