From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH 3/3] dt-bindings: reset: oxnas: Update for OX820 Date: Mon, 19 Sep 2016 16:18:50 -0500 Message-ID: <20160919211850.GA11839@rob-hp-laptop> References: <20160909131955.27334-1-narmstrong@baylibre.com> <20160909131955.27334-4-narmstrong@baylibre.com> <1473428153.3023.34.camel@pengutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1473428153.3023.34.camel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Philipp Zabel Cc: Neil Armstrong , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org On Fri, Sep 09, 2016 at 03:35:53PM +0200, Philipp Zabel wrote: > Hi Neil, > > Am Freitag, den 09.09.2016, 15:19 +0200 schrieb Neil Armstrong: > > Add new compatible string for reset and sys-ctrl for the Oxford > > Semiconductor OX820 Support. > > Drop the OX810SE indices since they moved in a dedicated include file. > > Not sure if the OX820 reset controller should be considered "compatible" > to OX810SE. The register layout and usage is the same, but the reset > indices are different. Does this warrant a new compatible string, or > should this be > compatible = "oxsemi,ox820-reset", "oxsemi,ox810se-reset"? Correct. Different connections are fine since the connections are described in the DT. However you could have some connections not described, so a new compatible may still be needed for that purpose (in addition to other reasons you do a new compatible for new SoCs). Rob -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html