From patchwork Wed Jul 21 10:12:19 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Joakim Zhang X-Patchwork-Id: 12390549 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-18.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,MSGID_FROM_MTA_HEADER,SPF_HELO_NONE, SPF_PASS,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 19483C12002 for ; Wed, 21 Jul 2021 10:14:12 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id DA74A6121F for ; Wed, 21 Jul 2021 10:14:11 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org DA74A6121F Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=nxp.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-Id:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=ZwXJoGq5yI/i99JKE5ep9uhUeMOnXr61axPorkKvyMQ=; b=SoGI6/pulfQNKJ ODdWqrLIohT2SaosIPw2GoqxBk1VlV1ugghRDOmwDCJ4CUuwogh+NGlU3e8c/GY6A9EXlCZOLba17 OdyrESUVBL2eKW/OQk0nD1f7Y2noH3ht+fSSdvPce4P0joEsBeBBe6KjxzSgEpWPaZ9F55pMoZ2aM NUWSKbjTN3gFTZBep9YFDFHO1Ia08ohjPl3w1CRIoPmW/MCWLYvBkLabX7rP6HYZv01DzQFR96Yf1 f4oKUS3QY/ZmJZRAgJ/7tWWusJ7rbSdLGQtmvtX8kahaZ+YZea/6wczQV6csoe0yoRLfvHeMdclPr /GJ58MrVxSGoASPeXfDw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1m69DE-00FByJ-Js; Wed, 21 Jul 2021 10:12:24 +0000 Received: from mail-eopbgr60057.outbound.protection.outlook.com ([40.107.6.57] helo=EUR04-DB3-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1m69Cv-00FBsy-0u for linux-arm-kernel@lists.infradead.org; Wed, 21 Jul 2021 10:12:06 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=EW96/CIiSPC+DUVbvh6QnlTTnev48vgQZA1BNdGEv0JluNd7jfyhgo4EaXC9XkSFMEML4GQ4N7U9HTc20pF9MdTTPuDl6qbHOGr4Ta0739/fNbF0gqxBfbv2iByy0P41DoDmok0mZYFcLfwMYmImL4k+os2s9PW28j56djUDvlnwO4GsxxPGK9awPnb/vwQCkXyqMCfwiKlj4ZQ15BHV2JJFzcYcTaASM2f4gGUOVHor5DopZrO8lUxHeXzDa6hVxHzgc376srNCXxtRvsXF5dKmNkkFskE0RWcpIkVcSAOc4P3/VCVZFj/5jLjjbu1D4A/mYcMQkWcEh0b705ultw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=+eUnslCw/qHp+T0wyI5XAUkzUatwF5S7M7N5Ysi0R4Q=; b=dmgV9lUfMn+OoGTMZkqVrDNrjulDertBDZt8GWpiMT3lnqvTCSyKkgR9mIUHpV0582NZwCBIS7zCMG35XzpzDy0rxdpsz1tKvBe5upYU8eC5fouDV/nBGnQuS/hKgjrpolFLuIsP/9VQYc0dYCLrTCvy3g8HwzADtkfU6chcvFdgHvcAw4yLnsml/srrdElFsjYO2K6BxY4O1i9cGGObIl2Fnq2Fh//9+99eY6zXjAos+1StSKEJ5r+PgAI1YX1PX3rjTezuxq4PH20yjRMoaOFgSrcdJGOemEbBK0r05btc9iW1y1RDop77ZCEdjAyQnOpmPZx3hFmBazWjSjoGJg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=+eUnslCw/qHp+T0wyI5XAUkzUatwF5S7M7N5Ysi0R4Q=; b=XraNbnIwLzOXbt74jXm4sAJHzlvilc9ihirEEQBNXV8aQbiI2FSjbO40yajFZEfjsBRSUY+p0pTmhMoYeeBQFoy7JdlARkHRdHsQxq+62k0biUTVm9xiyx0mcNWlp23gLgsfEvBDo5XojB9V7Kp2d+EQtU8wY/e8bM6+JzUQ7X4= Authentication-Results: davemloft.net; dkim=none (message not signed) header.d=none;davemloft.net; dmarc=none action=none header.from=nxp.com; Received: from DB8PR04MB6795.eurprd04.prod.outlook.com (2603:10a6:10:fa::15) by DBAPR04MB7480.eurprd04.prod.outlook.com (2603:10a6:10:1a3::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4331.23; Wed, 21 Jul 2021 10:12:02 +0000 Received: from DB8PR04MB6795.eurprd04.prod.outlook.com ([fe80::9c70:fd2f:f676:4802]) by DB8PR04MB6795.eurprd04.prod.outlook.com ([fe80::9c70:fd2f:f676:4802%9]) with mapi id 15.20.4331.034; Wed, 21 Jul 2021 10:12:01 +0000 From: Joakim Zhang To: davem@davemloft.net, kuba@kernel.org, robh+dt@kernel.org, shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com Cc: linux-imx@nxp.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: [PATCH net-next 1/2] dt-bindings: net: fsl, fec: improve the binding a bit Date: Wed, 21 Jul 2021 18:12:19 +0800 Message-Id: <20210721101220.22781-2-qiangqing.zhang@nxp.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20210721101220.22781-1-qiangqing.zhang@nxp.com> References: <20210721101220.22781-1-qiangqing.zhang@nxp.com> X-ClientProxiedBy: SG2PR04CA0195.apcprd04.prod.outlook.com (2603:1096:4:14::33) To DB8PR04MB6795.eurprd04.prod.outlook.com (2603:10a6:10:fa::15) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from localhost.localdomain (119.31.174.71) by SG2PR04CA0195.apcprd04.prod.outlook.com (2603:1096:4:14::33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4331.21 via Frontend Transport; Wed, 21 Jul 2021 10:11:58 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 856bddb7-abe2-4b80-a7ba-08d94c2ffb89 X-MS-TrafficTypeDiagnostic: DBAPR04MB7480: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:6790; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: Xc7rkrWK6dL3b+E05GqcLFumZ/UFd5M81CxQKO9WO8eBEHaVhXOGdPmTgJC5HFz3F7G9jkF+Oy+0rFRrL/ulU/Mv0UG7limRWRmujSq0IowJ/4UtgtzeJS1JPGNYyWLjtupAevL3AgGhVC3BegDQ4a0qOvZhfgfygl/J3ZGhVbWh/uKTEY6DbG8O9pRCa6AtyL6dg5PcGA2aMLEFe+cLMRQ/344GGLfICn29G/F4EKAVPgjK/oFJcZTKod5aiLg0tOe5R+wsZABqQWPR3X3xdkQQpY4NfELIcc5Xu1rEg4wpBvWpGaE9cfJOI2mAa3mvhqnfiCCAHggaq/DwA4pFpWChtO9QHrEZZ4b5DzrVqa31KYF7cFpQP3enuQA0ylhydZo1f9eDFYWvFWSL92gjeq/tRTfDstTT8mAy5szYbAoZ/h9QilZiiB5+3q0MWBdQmNKJV1fOeNFEA+bTgITEcsYCQYuOl0Ipoz2p3Jvjg7TWF5idsxM79CWoCumgwAWO/C2SuGop8Dn1GVTTHEtlVeqkA8kZXEVbnuwCpNV7ZKpeB/Tza4MN+u2b4MTZ9oEBnWpkE2KSbRmgZjz8B6E9NzVUyVkWnImywzwMGacOHN14xnn4AtJTE4qsXd7ESevim+1tBaqYpOKZf9xSAqc30hGV8zn8tiNdup5N+DVZKQzTbyZHzFeNuyIS5cd/YI+r43l+ioEHiNFBGRGlOpWS8g== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DB8PR04MB6795.eurprd04.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(366004)(346002)(376002)(136003)(396003)(39860400002)(1076003)(4326008)(186003)(66476007)(66946007)(5660300002)(6486002)(8676002)(478600001)(7416002)(2906002)(26005)(52116002)(83380400001)(6506007)(86362001)(38350700002)(6666004)(8936002)(38100700002)(2616005)(6512007)(36756003)(956004)(316002)(66556008); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: Lyu8FaDcNqBW0XBSRrUSKmHpQxFBLaXgpd1G0eYYbJ+DZVto3C0W4VFjU/AupGwmxCB5PMEREMRlsoIj/qcr6YEsLLCFrWxrgIQ1d0XPO7sZSkQirarVIuB1r7RuvjY1bRw2F1RiXk5JdQXJay+qNwoNrLri/fsKo4JBthlknQybkUf92Ps4YE5w3B39UonveK65QEqKpeNRkF5xtVnX+zxJkPJwGOllbX977Eg8uNLYQd0DTYVdwlvg8QF618GsrFa4KZdMG2irnzWYIwXj0CPxPlzCy/Jm9KRMQ5on5svtDN0w7s8mfezI8UTv9vTYyMvuFYQB+KlQXZgVYoxP9dykntqc4IeqcBKlx/1jHpZlBE3ax3dSyTlFmpKM495tlDrIWasOz+TO+oCdDpvpFutPLQ7Gd3aqQlZaKOgCXSQZPL8hkPv/FAsKQNi3rriyNhYSEHPVmmTzDVWSMuiVW2+pGiRxWL0j+MA40ZJMKyTQOuv/4I/8f4y5D/YFZV8OtcMuCk162c8vmmyRpBbX1yiEx734+wUm+9GccELhFerShILL5iBZ6kouTPUeoHt0Viusef2WNzyYxfYPfWSUNEcPjieSlUGDOm8nv1gpZj68yM4jf2vQ1dlv/lJlaoQEv6F7NN2i6N/vfTSNuvflrHfxrwCvpNhh0WpEk5XZIcMeavbWnlxcmugetT2K6RIWsvUhag2upNCPvw8xa/X2DzXpRIMbOXsNqfmVy0iDFw+UInOEmxgj/f5NohuMtM/wM/tszCry+W2X8ycVkScAoh3lEDQuwHc3m6VoWn38xUh396g3k3Qda0fCW/snXOJgd+E53Lb/t+N9sz5BzN144nKlMJgmvdKckagfK3BJ7sF9Y2R1Iix72oTESylRlONK5C70lJVMONB5Fp+6e8zWUE5WV0xus4A8SViSfxnhzcu5cmpXUZaQTwjHd8DY0XmCAazt9ZHYSMRdKeDSQ/bhFOfq3lzfc3qK7dtJCzA0GCTVluy9n1SMt7yG+LpQ2nNgHk6oyRdmilJNRw68woDgqdDvKcHsg3USPjxi52IgCqhjrv4NqkgLzAR5Z6yOU0HBEbQb1BLXZj0I0g6k6XDWS3a7dvnvhskY8yUhXm/RcJ5o0jqMpKDk5c+U2ZfjqoZGRmSxATxskSqoW5lXqEEBFgcKJNVyybES0q6abG91qFDianYNH25Kxa/gxeGeDPNXJtqURZuBVDwWBd5pYfXRZgmUm+J78/mh5lfwqjZOGJjCOB/8oOTta/clw5CdlOpB/KOQ2Ct118QCb3Axzx3wt+lUbB4N7NlFT78gvMY0ecO+RdGyt3LQVKWolL8Tnxaz X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 856bddb7-abe2-4b80-a7ba-08d94c2ffb89 X-MS-Exchange-CrossTenant-AuthSource: DB8PR04MB6795.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Jul 2021 10:12:01.8535 (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: 25hhv9O+KA8IKZLqPmT6XM+p9WbFBMQQWool3ZmJXOECrbtj7YhepYDMX8JeVN9iW2CzSDaP1bSgeT8tgQEovA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBAPR04MB7480 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210721_031205_103520_289A594E X-CRM114-Status: GOOD ( 14.28 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org This patch improves the yaml a bit according to Rob Herring comments: 1) normalize interrupt-names property, there is no reason to support random order. 2) validate each string in clock-names property. 3) add constraints for fsl,num-tx-queues/fsl,num-rx-queues property. 4) change additionalProperties to false in order to do strict checking. Signed-off-by: Joakim Zhang --- .../devicetree/bindings/net/fsl,fec.yaml | 34 +++++++++++-------- 1 file changed, 19 insertions(+), 15 deletions(-) diff --git a/Documentation/devicetree/bindings/net/fsl,fec.yaml b/Documentation/devicetree/bindings/net/fsl,fec.yaml index 0f8ca4e574c6..dbcbec95fc9e 100644 --- a/Documentation/devicetree/bindings/net/fsl,fec.yaml +++ b/Documentation/devicetree/bindings/net/fsl,fec.yaml @@ -49,19 +49,21 @@ properties: maxItems: 4 interrupt-names: - description: - Names of the interrupts listed in interrupts property in the same order. - The defaults if not specified are - __Number of interrupts__ __Default__ - 1 "int0" - 2 "int0", "pps" - 3 "int0", "int1", "int2" - 4 "int0", "int1", "int2", "pps" - The order may be changed as long as they correspond to the interrupts - property. Currently, only i.mx7 uses "int1" and "int2". They correspond to - tx/rx queues 1 and 2. "int0" will be used for queue 0 and ENET_MII interrupts. - For imx6sx, "int0" handles all 3 queues and ENET_MII. "pps" is for the pulse - per second interrupt associated with 1588 precision time protocol(PTP). + oneOf: + - items: + - const: int0 + - items: + - const: int0 + - const: pps + - items: + - const: int0 + - const: int1 + - const: int2 + - items: + - const: int0 + - const: int1 + - const: int2 + - const: pps clocks: minItems: 2 @@ -80,7 +82,7 @@ properties: clock-names: minItems: 2 maxItems: 5 - contains: + items: enum: - ipg - ahb @@ -107,12 +109,14 @@ properties: description: The property is valid for enet-avb IP, which supports hw multi queues. Should specify the tx queue number, otherwise set tx queue number to 1. + enum: [1, 2, 3] fsl,num-rx-queues: $ref: /schemas/types.yaml#/definitions/uint32 description: The property is valid for enet-avb IP, which supports hw multi queues. Should specify the rx queue number, otherwise set rx queue number to 1. + enum: [1, 2, 3] fsl,magic-packet: $ref: /schemas/types.yaml#/definitions/flag @@ -179,7 +183,7 @@ required: # least undocumented properties. However, PHY may have a deprecated option to # place PHY OF properties in the MAC node, such as Micrel PHY, and we can find # these boards which is based on i.MX6QDL. -additionalProperties: true +additionalProperties: false examples: - |