Re: [PATCH v2 2/2] dt-bindings: arm: List more IXP4xx devices

[Conor Dooley <conor@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 2159 bytes --]

On Mon, Sep 25, 2023 at 11:03:39AM +0200, Linus Walleij wrote:
> The ixp4xx bindings are lacking some of the devices we have
> out there in the wild, so let's add them.
> 
> The "linksys,wrv54g", "gemtek,gtwx5715", "intel,ixp42x"
> triplet is a pretty special case where the Gemtek device is
> licensed as OEM product by Linksys (now Belkin) hence the
> special entry for this one.
> 
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
>  .../devicetree/bindings/arm/intel-ixp4xx.yaml          | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/arm/intel-ixp4xx.yaml b/Documentation/devicetree/bindings/arm/intel-ixp4xx.yaml
> index 553dcbc70e35..1fecd4692a62 100644
> --- a/Documentation/devicetree/bindings/arm/intel-ixp4xx.yaml
> +++ b/Documentation/devicetree/bindings/arm/intel-ixp4xx.yaml
> @@ -16,12 +16,30 @@ properties:
>      oneOf:
>        - items:
>            - enum:
> +              - adieng,coyote
> +              - arcom,vulcan
> +              - dlink,dsm-g600-a
> +              - freecom,fsg-3
> +              - gateway,7001
> +              - gateworks,gw2348
> +              - goramo,multilink-router
> +              - intel,ixdp425
> +              - intel,ixdpg425
> +              - iom,nas-100d
>                - linksys,nslu2
> +              - netgear,wg302v1
> +              - netgear,wg302v2
> +              - usr,8200
>                - welltech,epbx100
>            - const: intel,ixp42x
> +      - items:
> +          - const: linksys,wrv54g
> +          - const: gemtek,gtwx5715
> +          - const: intel,ixp42x

One question, not so much about this particular case, but is there no
"standalone" version of the gtwx5715 sold by gemtek? Asking as I don't
see it in the enum above. The description sounds like it is both a
product in its own right & sold rebadged.

Thanks,
Conor.

>        - items:
>            - enum:
>                - gateworks,gw2358
> +              - intel,kixrp435
>            - const: intel,ixp43x
>  
>  additionalProperties: true
> 
> -- 
> 2.41.0
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]