From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Rutland Subject: Re: [PATCH v3 4/4] documentation: Add secure monitor binding documentation Date: Mon, 23 May 2016 18:11:59 +0100 Message-ID: <20160523171159.GH4976@leverpostej> References: <1464021024-29380-1-git-send-email-carlo@caione.org> <1464021024-29380-5-git-send-email-carlo@caione.org> <20160523163815.GE4976@leverpostej> <20160523165931.GA31299@mephisto> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20160523165931.GA31299@mephisto> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Carlo Caione Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, arnd-r2nGTMty4D4@public.gmane.org, khilman-rdvid1DuHRBWk0Htik3J/w@public.gmane.org, jens.wiklander-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, matthias.bgg-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, linux-amlogic-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-6IF/jdPJHihWk0Htik3J/w@public.gmane.org, afaerber-l3A5Bk7waGM@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: devicetree@vger.kernel.org On Mon, May 23, 2016 at 06:59:31PM +0200, Carlo Caione wrote: > On 23/05/16 17:38, Mark Rutland wrote: > > > +Required properties for the secure monitor node: > > > +- compatible: Should be "amlogic,meson-sm" > > > +- amlogic,sm-cmd-input-base: SMC32 function identifier to read the physical > > > + address of the input buffer > > > +- amlogic,sm-cmd-output-base: SMC32 function identifier to read the physical > > > + address of the output buffer > > > > Do the IDs for these actually differ per board? > > I expect these to differ per SoC (GXBB in this case), not per board. The > driver is generic enough to be (hopefully) used for several SoCs just > changing the related header file that defines the SCM commands. > > > Are some functions simply not implemented on some boards? > > I don't think this is possible. Given that, I think it may be better to just have a "amlogic,meson-gxbb-sm" compatible string, and derive the set of functions and associated IDs from that. That is, unless you know that future revisions have functions with the exact same semantics but differing IDs. Regardless, it would make sense to have a GXBB-specific compatible string prepended to the list. That way in the future we can handle anything specific to the GXBB variant of the secure monitor if necessary. Thanks, Mark. -- 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