Re: [PATCH v3 1/2] dt-bindings: Update iProc GPIO bindings

[Rob Herring]

On Tue, Jul 12, 2016 at 10:39:17AM -0700, Ray Jui wrote:
> Update the iProc GPIO binding document to add new compatible strings
> "brcm,iproc-gpio-nsp" and "brcm,iproc-gpio-stingray" to support the
> iProc based GPIO controller used in the NSP and Stingray SoCs,
> respectively
> 
> Signed-off-by: Ray Jui <ray.jui@xxxxxxxxxxxx>
> ---
>  .../devicetree/bindings/pinctrl/brcm,iproc-gpio.txt    | 18 ++++++++++++++++--
>  1 file changed, 16 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/pinctrl/brcm,iproc-gpio.txt b/Documentation/devicetree/bindings/pinctrl/brcm,iproc-gpio.txt
> index e427792..7bd1614 100644
> --- a/Documentation/devicetree/bindings/pinctrl/brcm,iproc-gpio.txt
> +++ b/Documentation/devicetree/bindings/pinctrl/brcm,iproc-gpio.txt
> @@ -3,8 +3,22 @@ Broadcom iProc GPIO/PINCONF Controller
>  Required properties:
>  
>  - compatible:
> -    Must be "brcm,cygnus-ccm-gpio", "brcm,cygnus-asiu-gpio",
> -    "brcm,cygnus-crmu-gpio" or "brcm,iproc-gpio"
> +    "brcm,iproc-gpio" for the generic iProc based GPIO controller IP that
> +    supports full-featured pinctrl and GPIO functions used in various iProc
> +    based SoCs
> +
> +    May contain an SoC-specific compatibility string to accommodate any
> +    SoC-specific features
> +
> +    "brcm,cygnus-ccm-gpio", "brcm,cygnus-asiu-gpio", or
> +    "brcm,cygnus-crmu-gpio" for Cygnus SoCs
> +
> +    "brcm,iproc-gpio-nsp" for the iProc NSP SoC that has drive strength support
> +    disabled

The typical ordering is <soc>-<block>, so:

iproc-nsp-gpio

> +
> +    "brcm,iproc-gpio-stingray" for the iProc Stingray SoC that has the general
> +    pinctrl support completely disabled in this IP block. In Stingray, a
> +    different IP block is used to handle pinctrl related functions

iproc-stingray-gpio


>  
>  - reg:
>      Define the base and range of the I/O address space that contains SoC
> -- 
> 2.1.4
>