From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     =?iso-8859-1?Q?Cl=E9ment_L=E9ger?= <clement.leger@bootlin.com>
CC:     "David S. Miller" <davem@davemloft.net>,
        Jakub Kicinski <kuba@kernel.org>,
        Rob Herring <robh+dt@kernel.org>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        "UNGLinuxDriver@microchip.com" <UNGLinuxDriver@microchip.com>,
        Andrew Lunn <andrew@lunn.ch>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
        Denis Kirjanov <dkirjanov@suse.de>,
        Julian Wiedmann <jwi@linux.ibm.com>
Subject: Re: [PATCH net-next v3 1/4] dt-bindings: net: mscc,vsc7514-switch:
 convert txt bindings to yaml
Thread-Topic: [PATCH net-next v3 1/4] dt-bindings: net: mscc,vsc7514-switch:
 convert txt bindings to yaml
Thread-Index: AQHX4uryalRRqJ6wdEegqX4WYakZdqwWFj8A
Date:   Fri, 26 Nov 2021 17:50:05 +0000
Message-ID: <20211126175004.6dsumhtepek4m36h@skbuf>
References: <20211126172739.329098-1-clement.leger@bootlin.com>
 <20211126172739.329098-2-clement.leger@bootlin.com>
In-Reply-To: <20211126172739.329098-2-clement.leger@bootlin.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?4pASCljO6IpTcYdTqKscd34kybAsEEAVTe2Cn2oOEKeIdo5uAXS3fLRZdA?=
 =?iso-8859-1?Q?Wczumv/vZWqG91mNJfGPdbiAV1TvSqfB6N3qflDOgRYUQ29yDV1I91Xz81?=
 =?iso-8859-1?Q?AEWFHPvgIAVP3M62GvSITd5+2TixNwcBD9iGRGRRiPUz3B51RUvXNz5wh4?=
 =?iso-8859-1?Q?RrRkd6rgeeYqX2qTa8TdAGBligIp9Kz6ukeIPOSOJmwDv2u4AC8bVvTcOa?=
 =?iso-8859-1?Q?BVeSJWNVnhmDN3/T5FleBw82lz0BQKK2DXzNgPnjYYDyfdMBCgYs0NG+LU?=
 =?iso-8859-1?Q?TpjL0v8cdMb21ukIqJ3HE2NXjxPNupEOkMhbz1CG9U98N3jrRE1jw6WFq0?=
 =?iso-8859-1?Q?2mQeIfabOKhP/sFYA7/FnFVZDiICqncNjuyESxmR7RcBCWAGhBDXwqwKyq?=
 =?iso-8859-1?Q?HRWFaO83wOcpVbpwBGrCe6b+inzH+xYI6ujtI7hrudNr+mcgJDCjZBqqBN?=
 =?iso-8859-1?Q?Uo41rhrRX5JZvPRsSlfJ+udms8knf5sjZk2bjtFGHVeTXWDfiN6bgAouAP?=
 =?iso-8859-1?Q?mLQaPMUzYScL7scmB3Du/PmETkveiyy+QU2nMxzrOyMx2WzB9FlfatrdBn?=
 =?iso-8859-1?Q?Wwf+C3DWC4IJH/12Q1fzPAU429wHxu0RddJV+8y+NcJzbVNBrWI7mvBRGb?=
 =?iso-8859-1?Q?SO0Ji9H9yuxQ87KfyY0ivblN5YIByWZCW0/lcdUBM2SsjS0+ydy2iwGrwf?=
 =?iso-8859-1?Q?nLAV0LARVm4DaynUtOe6M/S3zdO5P/FYD31E7LoCZiXOuNo4EWebDXWfzj?=
 =?iso-8859-1?Q?DRJvV5V4IykY7PaNAICsfbwXU0LjJYvXQ3mBB+5uSUmDlY0Kh1JyKDlR4J?=
 =?iso-8859-1?Q?y5LQ9mARMwKuqmCDTMmKsJe2wolNwl/7QSNIky0J0yXIUq65pqPC18oxU0?=
 =?iso-8859-1?Q?xB5n8gHFU9nXOx+N8075yIICC4e0OohSdwgRQe5fBK5KweiJN5qRPmZl7T?=
 =?iso-8859-1?Q?Ixhko1kh35DDD+TFbM+B0esATCuLhhVDre4T0hV7lcpnKsvm357sDnmvdF?=
 =?iso-8859-1?Q?Nuzb5W96WjueU4FnQnnBWy7MQQSWlyTqBzLbE40O4gIS7CqKRMuRZSvaa2?=
 =?iso-8859-1?Q?W1IzLQkVVHfDDvmDM9xKzN3NfryDYNILJhBQM1vD6UKtOUT+7xLmvZyuZB?=
 =?iso-8859-1?Q?i8Exs6IYxRHVHcrJvpOJcWtW2dJHACHiBS436usQr7di6p23D1hNzNIxZ3?=
 =?iso-8859-1?Q?oWG85UdcSdLYC07mk92Qu7ta9fQoczhHTiV4D/7V9uqm036Huw8Hu2XLih?=
 =?iso-8859-1?Q?1wljmB/va5v6fH8u/fTcOVGLJkOGjulYr9/hayX4n5MdztSPCztEgzybqk?=
 =?iso-8859-1?Q?XfCbMwbPh4oX45apevBw81lx2VVPu3j9beK51nKlhgCPN1kyanab43S8ci?=
 =?iso-8859-1?Q?AM/AoX6HE4ik5tFcndKOAdn2WzqhHE2zZhgyeaZDxVibipLeYtSt5/Mtk9?=
 =?iso-8859-1?Q?0TuvlJwjI8HDBwOOSFInDFKX9Ci/J3mWPBYwGzMtA68Fq2V8TaBRzLIFRU?=
 =?iso-8859-1?Q?/hCoxc2yv/q1bLfJ7UIbUsFnzNwP/M/M9FKDEaV8uOSBujBhAaxB5TrT+z?=
 =?iso-8859-1?Q?4/DykcwgMBWlYYBGfWVcrGlf5LCGtX0solofljE6nc527DypmMsSVKUDlV?=
 =?iso-8859-1?Q?Psvwut/gdCrUwwQI2mW7HbAwUYNKXwE5ugaxt3LvkkniyfNO05LjXjuPqc?=
 =?iso-8859-1?Q?N3bMAmB+z4gFTNncUnE=3D?=
Content-Type: text/plain; charset="iso-8859-1"
Content-ID: <D513B487C0692D44AE17B90F048E454A@eurprd04.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 05b540cc-1d24-46cc-6d42-08d9b1052de6
X-MS-Exchange-CrossTenant-originalarrivaltime: 26 Nov 2021 17:50:05.0905
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: bvq3unbDeQE3Nh3KIgXb6utLsPcEOKNRiptuzXLDLe8VOZd7JDf2dqSHWcqJEA+TJgjg6NzR7gMiOGVmZZEGTw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0402MB2864
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Fri, Nov 26, 2021 at 06:27:36PM +0100, Cl=E9ment L=E9ger wrote:
> +  ethernet-ports:
> +    type: object
> +
> +    properties:
> +      '#address-cells':
> +        const: 1
> +      '#size-cells':
> +        const: 0
> +
> +    additionalProperties: false
> +
> +    patternProperties:
> +      "^port@[0-9a-f]+$":
> +        type: object
> +        description: Ethernet ports handled by the switch
> +
> +        $ref: ethernet-controller.yaml#
> +
> +        unevaluatedProperties: false
> +
> +        properties:
> +          reg:
> +            description: Switch port number
> +
> +          phy-handle: true
> +
> +          phy-mode: true
> +
> +          fixed-link: true
> +
> +          mac-address: true
> +
> +        required:
> +          - reg
> +
> +        oneOf:
> +          - required:
> +              - phy-handle
> +              - phy-mode
> +          - required:
> +              - fixed-link

Are you practically saying that a phy-mode would not be required with
fixed-link? Because it still is...

> +
> +required:
> +  - compatible
> +  - reg
> +  - reg-names
> +  - interrupts
> +  - interrupt-names
> +  - ethernet-ports=