From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753870AbcFPKqz (ORCPT ); Thu, 16 Jun 2016 06:46:55 -0400 Received: from metis.ext.4.pengutronix.de ([92.198.50.35]:49161 "EHLO metis.ext.4.pengutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751136AbcFPKqy (ORCPT ); Thu, 16 Jun 2016 06:46:54 -0400 Message-ID: <1466074010.3343.15.camel@pengutronix.de> Subject: Re: [PATCH v2 1/3] reset: hisilicon: Add media reset controller binding From: Philipp Zabel To: Xinliang Liu Cc: linux-kernel@vger.kernel.org, robh+dt@kernel.org, xuwei5@hisilicon.com, puck.chen@hisilicon.com, saberlily.xia@hisilicon.com, kong.kongxinwei@hisilicon.com, guodong.xu@linaro.org Date: Thu, 16 Jun 2016 12:46:50 +0200 In-Reply-To: <20160616103000.239669-2-xinliang.liu@linaro.org> References: <20160616103000.239669-1-xinliang.liu@linaro.org> <20160616103000.239669-2-xinliang.liu@linaro.org> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.12.9-1+b1 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit X-SA-Exim-Connect-IP: 2001:67c:670:100:96de:80ff:fec2:9969 X-SA-Exim-Mail-From: p.zabel@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-kernel@vger.kernel.org Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Am Donnerstag, den 16.06.2016, 18:29 +0800 schrieb Xinliang Liu: > Add compatible for media reset controller. > > Signed-off-by: Xinliang Liu > --- > Documentation/devicetree/bindings/reset/hisilicon,hi6220-reset.txt | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/reset/hisilicon,hi6220-reset.txt b/Documentation/devicetree/bindings/reset/hisilicon,hi6220-reset.txt > index e0b185a944ba..c25da39df707 100644 > --- a/Documentation/devicetree/bindings/reset/hisilicon,hi6220-reset.txt > +++ b/Documentation/devicetree/bindings/reset/hisilicon,hi6220-reset.txt > @@ -8,7 +8,9 @@ The reset controller registers are part of the system-ctl block on > hi6220 SoC. > > Required properties: > -- compatible: may be "hisilicon,hi6220-sysctrl" > +- compatible: should be one of the following: > + - "hisilicon,hi6220-sysctrl", "syscon" : For peripheral reset controller. > + - "hisilicon,hi6220-mediactrl", "syscon" : For media reset controller. This does more than the commit message says. Why do you add syscon here? Do the sysctrl and mediactrl register ranges have bits other than reset controls that will be used by other drivers via the syscon mechanism? regards Philipp