On Tue, Apr 09, 2019 at 01:45:58PM -0500, [email protected] wrote:
> From: Richard Gong <[email protected]>
> 
> Add a device tree binding for the Intel Stratix10 remote system
> update (RSU) driver
> 
> Signed-off-by: Richard Gong <[email protected]>
> Reviewed-by: Alan Tull <[email protected]>
> ---
>  .../bindings/firmware/intel,stratix10-rsu.txt      | 31 
> ++++++++++++++++++++++
>  1 file changed, 31 insertions(+)
>  create mode 100644 
> Documentation/devicetree/bindings/firmware/intel,stratix10-rsu.txt
> 
> diff --git 
> a/Documentation/devicetree/bindings/firmware/intel,stratix10-rsu.txt 
> b/Documentation/devicetree/bindings/firmware/intel,stratix10-rsu.txt
> new file mode 100644
> index 0000000..b6250eb
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/firmware/intel,stratix10-rsu.txt
> @@ -0,0 +1,31 @@
> +Intel Remote System Update Driver for Stratix10 SoC
> +============================================
> +The Intel Remote System Update (RSU) driver exposes interfaces
> +accessed through the Intel Service Layer to user space via sysfs
> +interface. The RSU interfaces report and control some of the optional
> +RSU features on Intel Stratix 10 SoC.
> +
> +The RSU feature provides a way for customers to update the boot
> +configuration of a Intel Stratix 10 SoC device with significantly reduced
> +risk of corrupting the bitstream storage and bricking the system.
> +
> +Required properties:
> +-------------------
> +The rsu node has the following mandatory properties, must be located under
> +the firmware/svc node.
> +
> +- compatible: "intel,stratix10-rsu"
> +
> +Example:
> +-------
> +     firmware {
> +             svc {
> +                     compatible = "intel,stratix10-svc";
> +                     method = "smc";
> +                     memory-region = <&service_reserved>;
> +
> +                     rsu: rsu {
> +                             compatible = "intel,stratix10-rsu";

There's no DT resources, so why does this need to be in DT. Just make 
the driver for the parent instantiate the driver for this.

Rob

Reply via email to