Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Subject: Re: [PATCH net-next v3 03/47] dt-bindings: net: Convert FMan MAC
 bindings to yaml
To:     Rob Herring <robh@kernel.org>
Cc:     devicetree@vger.kernel.org, Rob Herring <robh+dt@kernel.org>,
        Russell King <linux@armlinux.org.uk>, netdev@vger.kernel.org,
        Paolo Abeni <pabeni@redhat.com>,
        Madalin Bucur <madalin.bucur@nxp.com>,
        Eric Dumazet <edumazet@google.com>,
        "David S . Miller" <davem@davemloft.net>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Jakub Kicinski <kuba@kernel.org>,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org
References: <20220715215954.1449214-1-sean.anderson@seco.com>
 <20220715215954.1449214-4-sean.anderson@seco.com>
 <1657926388.246596.1631478.nullmailer@robh.at.kernel.org>
From:   Sean Anderson <sean.anderson@seco.com>
Message-ID: <1b694d96-5ea8-e4eb-65bf-0ad7ffefa8eb@seco.com>
Date:   Sat, 16 Jul 2022 18:47:08 -0400
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.12.0
In-Reply-To: <1657926388.246596.1631478.nullmailer@robh.at.kernel.org>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?cDVwRTJacWJEVC9WeXRLWHEvSktkZnQ2VUZmTzgrM2MybEZvWXVicEVYTmpx?=
 =?utf-8?B?dlkwb2ZCN3B0dWRIcU1KUzUyTGMvTFBrOU1idjRObndRRHZNWmNCZlFuOFAx?=
 =?utf-8?B?SW91SEZGM1RXNXBlT2lCbHlHOGFyeU9ZVWZYWEtiaUMyemdWSXFHc3VpUFVC?=
 =?utf-8?B?OHIyMHNIVE9JTkVnVk9MbVlsRlBRWXY0Zjd0ZkxCc2R3SjM1Qko3S05mZ0l0?=
 =?utf-8?B?dUdZdDhJYVhrODNxNUFnUFhGc01Kb0toWG10LzM5SDhVUWs0bVZ2WGtuZGox?=
 =?utf-8?B?RmdMbHFqc2pqRWdBYWZ2bjBCaUtQbGNqK2ZXZXhwWnRkZE13UUFzcmRkT2lj?=
 =?utf-8?B?N1RGMU9VMW9KNzYzdGVBRkhwOWRmSnBDcXhvRE1JUlhmSUdIclZKUjU1VUJ2?=
 =?utf-8?B?bHV1b1Q0clRyTXRrT1VRZlNPRW11MWVUcG9PQmdDNmRRNVZBNTk2MmhvSmhB?=
 =?utf-8?B?aVU2UWtabkZKTEErTHUxOFpCZkQ3dnp1OU1RSmlSZWZkb0hOT25ENGlwWDNC?=
 =?utf-8?B?TW1Cd3ZLUmR0NUhtSThzeENDRlpJdlRsaHhpNjEvZFlVNTBoTmtHNWVJRmhh?=
 =?utf-8?B?S0hvZXhpYTVjQXpYaVVMS0ZMQ2VLdS9GK3JScXFMMWM3UXBWRlFIcGNmMmVo?=
 =?utf-8?B?WmlNcjUzL0NPb0xKMjcwZHBiUytnSGE2QjVra01GNlNuZldWcGFyNFRQSjZw?=
 =?utf-8?B?TGR6WVpVb2p4b0orQ0MyYzF6UzMzVitoaU40UjZ0L1BmckVQRUNUVlk5YUgz?=
 =?utf-8?B?MTc3U1hXbWUwYXRGVFBFeEtCdGtyVWVEcWw4d3p1TXYwWkMxVk1PS25MV001?=
 =?utf-8?B?SmpDelBmSHJ1cUQ1T25ZU2VnRFZoSnlmNWZGYlNSNUovdjRSZERJM2tFYXhT?=
 =?utf-8?B?VW1oK0FRdVB6elh5UG5KMHd5Sk9nQ2kzUkh2TmZJTnFmWm1jR2JqMTVKSVo4?=
 =?utf-8?B?aXNzR3VtVTMvZmZ3ZzByZTRMK2JwRlY5cGRzcEYwTm9zNUpkeVlWU0t5SHEx?=
 =?utf-8?B?WUVzOEdkSUJDRjRUK3RCNUZMS1JmTDVERUcrcmFJRG5EbWptZ0QyVUdDZEJr?=
 =?utf-8?B?UWw5U2JFOXhUUHNGYXQ0V0hpaU5idHpnNnUxUDA3MHhxR25wajJTVDE1ZlJm?=
 =?utf-8?B?aHFQOThZYzBiQlJnek95T2dsYXYxRERQdUlVMUVyWk1wSFUrbFMzRHkrWjcy?=
 =?utf-8?B?dUxuNkRya2R1SVZZUC9QUW03Q1hPLzlPZE1VZTdTaUxtdUcwaEYraFZ4TUVv?=
 =?utf-8?B?dUVhUlJiV09GVGdHaU1SQWhMbVIyK2hOTnpsZ3hweklIanh6NjhiQ3hXMnJo?=
 =?utf-8?B?YkxvY1FMSUUzS3ZkUm9meHZIbE1Bc3V6bzdkdzhPbC9xV2cxeFd5em52aUhL?=
 =?utf-8?B?NWs4cVpsUkZzY3l1NmZaYW9tUmJ6Tlp6WVBhdUd5TitTNG90VVZ6eFRrQ21M?=
 =?utf-8?B?Q3ZCZG9ESXgyWTF3aDFhcHVpc2JKdVZvWXFQVkVSZXNiQWZDbnR0RDZudko4?=
 =?utf-8?B?MDVDdGdoS0ROZDRxaXorY2RRTWE5NU5NWjVmdFhkZk9FVzVSRjlKMUp2QXZK?=
 =?utf-8?B?OHZKVUFhd0czbWQ2Z3hsdzR0cE5vTVg3Wi9xd2pTWTB3ODdTVjZnVFJaeFIx?=
 =?utf-8?B?SkdWU0gvQ3cyVU1ObVFIL3BIM1hOSEYyN3JCb0dsU0NTR1hab1RPQkl0dkVQ?=
 =?utf-8?B?ZDB1Q0JzSXR5bFlTRWpJbERKSnEvTUw1cllVY2NxV2JVZzNBM203UVlURTNC?=
 =?utf-8?B?L2FGLzNhYU0vMFhwMWlJcFJVb0docWczU1duOS8wUXoxYTlDQ0VhTTliV3ZJ?=
 =?utf-8?B?N2l1bXZOcS9QK05ibFJDc0paMnQwaDl6cXk2ZmNMamdZU2xNZEFHVkVzRFVJ?=
 =?utf-8?B?UXRYcHZ6UzZaRThrV2MzVkEyYi9QbGFudXZzZ1R2WmxHS1N3RFhaT1BGZU9q?=
 =?utf-8?B?S1A0N09ZVjh3bTZ5dXROMVo3TWRiSzNMTE1rSURqcnlxYUZUTFlFRC9CU1JU?=
 =?utf-8?B?TGp2Sk1semQxUDRNM3JlNzF6RUdOZFlrMnhSN3dBaTB6TjBRWWw1WWlJclp5?=
 =?utf-8?B?VWhTaXdMTWVkZktVTUVCVTljaWpkbnFlMmRHcVdNT1BOcVlIVWhiWmhORFN5?=
 =?utf-8?Q?eoemRXGElqxMCiCZIbFP9Ibny?=
X-OriginatorOrg: seco.com
X-MS-Exchange-CrossTenant-Network-Message-Id: de676978-7ce9-49fa-1be0-08da677d1ffc
X-MS-Exchange-CrossTenant-AuthSource: DB7PR03MB4972.eurprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Jul 2022 22:47:13.2785
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: bebe97c3-6438-442e-ade3-ff17aa50e733
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: I1piClGBMC6OiUleGA2Pl0gM5jMoo7VbusLHyejz8VzZ69WV+m/YiKC4sQUnzJFVs3Kr7ixdkIkW/71XvOxMCA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM6PR03MB5541
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 7/15/22 7:06 PM, Rob Herring wrote:
> On Fri, 15 Jul 2022 17:59:10 -0400, Sean Anderson wrote:
>> This converts the MAC portion of the FMan MAC bindings to yaml.
>>
>> Signed-off-by: Sean Anderson <sean.anderson@seco.com>
>> Reviewed-by: Rob Herring <robh@kernel.org>
>> ---
>>
>> Changes in v3:
>> - Incorporate some minor changes into the first FMan binding commit
>>
>> Changes in v2:
>> - New
>>
>>   .../bindings/net/fsl,fman-dtsec.yaml          | 145 ++++++++++++++++++
>>   .../devicetree/bindings/net/fsl-fman.txt      | 128 +---------------
>>   2 files changed, 146 insertions(+), 127 deletions(-)
>>   create mode 100644 Documentation/devicetree/bindings/net/fsl,fman-dtsec.yaml
>>
> 
> My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
> on your patch (DT_CHECKER_FLAGS is new in v5.13):
> 
> yamllint warnings/errors:
> 
> dtschema/dtc warnings/errors:
> /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/net/fsl,fman-dtsec.example.dtb: ethernet@e8000: 'phy-connection-type', 'phy-handle' do not match any of the regexes: 'pinctrl-[0-9]+'
> 	From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/net/fsl,fman-dtsec.yaml
> 
> doc reference errors (make refcheckdocs):

What's the correct way to do this? I have '$ref: ethernet-controller.yaml#'
under allOf, but it doesn't seem to apply. IIRC this doesn't occur for actual dts files.

--Sean