Re: [PATCH v3 1/5] dt-bindings: PCI: toshiba,tc9563: Document embedded GPIO controller

[Krzysztof Kozlowski <krzk@kernel.org>]

On Fri, Sep 04, 2026 at 07:15:18PM +0200, Lorenzo Bianconi wrote:
> The TC9563 PCIe switch embeds a GPIO controller providing 37 GPIO
> lines. The controller is registered as an auxiliary device by the TC9563

"auxiliary device" is Linux term, so not really relevant here.

> power controller and accessed through the same register map.
> 
> Describe the switch node itself as the embedded GPIO controller and
> document the per-port reset-gpios property.

Isn't the switch node already described?

> 
> Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@oss.qualcomm.com>
> ---
>  .../devicetree/bindings/pci/toshiba,tc9563.yaml       | 19 ++++++++++++++++++-
>  1 file changed, 18 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/pci/toshiba,tc9563.yaml b/Documentation/devicetree/bindings/pci/toshiba,tc9563.yaml
> index f084830c6d0c..c4cd162cb084 100644
> --- a/Documentation/devicetree/bindings/pci/toshiba,tc9563.yaml
> +++ b/Documentation/devicetree/bindings/pci/toshiba,tc9563.yaml
> @@ -31,6 +31,11 @@ properties:
>      description:
>        GPIO controlling the RESX# pin.
>  
> +  gpio-controller: true
> +
> +  '#gpio-cells':
> +    const: 2
> +
>    vdd18-supply: true
>  
>    vdd09-supply: true
> @@ -84,6 +89,11 @@ $defs:
>      type: object
>  
>      properties:
> +      reset-gpios:
> +        maxItems: 1
> +        description:
> +          GPIO controlling the reset line connected to the downstream port.

Why do you exactly need this? Isn't this PERST, thus already there?

Best regards,
Krzysztof