From mboxrd@z Thu Jan 1 00:00:00 1970 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=-2.2 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_MUTT autolearn=unavailable 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 2CC3FC46477 for ; Wed, 19 Jun 2019 14:03:35 +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 06518208CA for ; Wed, 19 Jun 2019 14:03:34 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="PfaVHgIF"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=lunn.ch header.i=@lunn.ch header.b="5MzChjyZ" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 06518208CA Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=lunn.ch Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-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.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=i4jjjfmeHGiXGbFoP25uUABGd7C1/aWk75LJGQwAt+g=; b=PfaVHgIFEOecKX SIeXpMJTIt5dMvt+W8TtLLr55OJRQXxCi/iKMT9g2vzqJzObxtP0TPWj2W5pPlb78SCrRcAGu1Pqf aR729eo2zQ3DTGmyUHKnUPWRosGr/r2pxvpKEAh1RsT3v9irwxnjlhiDpe12rguT2uHPYppULrDYn Jr3fvMwqXZvF11TyVBsfXwjHAFqKbMaLm5yialqHm+A3TE3xOywirdV61O5chRGbMnm2Wp9L1LaEm wxQOVJg4OhpNxdVIGoSoU+1RwMGFddpr0QXf4mp0CIGET1hgRbMrwVzzmaBG+GYEuyI94ULwJmdUn aQ7zsSIjw9PI3KeP2iZA==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92 #3 (Red Hat Linux)) id 1hdbBW-0003x2-Fk; Wed, 19 Jun 2019 14:03:34 +0000 Received: from vps0.lunn.ch ([185.16.172.187]) by bombadil.infradead.org with esmtps (Exim 4.92 #3 (Red Hat Linux)) id 1hdbBT-0003vo-MZ for linux-arm-kernel@lists.infradead.org; Wed, 19 Jun 2019 14:03:33 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lunn.ch; s=20171124; h=In-Reply-To:Content-Type:MIME-Version:References:Message-ID: Subject:Cc:To:From:Date:Sender:Reply-To:Content-Transfer-Encoding:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=g/6LlihmMyA9edalIxizKqgeDYFzECLoXeUK/dZh79M=; b=5MzChjyZGap1Y6rGKxn+CiNR/P 34z6le3HpVkCKRfX2P6R5Fuv+EcVLn/CkrSkFlu70ZkNck4VhFnPTz3pRFR3oA8GM/mAjItrM4Kav z/YpNbdI/CmKi6uaqf/30+8Rto7w75+NYufMlHfTW6R2Z1sZUtWqMHmTK7YuHZc/OBMM=; Received: from andrew by vps0.lunn.ch with local (Exim 4.89) (envelope-from ) id 1hdbBD-0001kz-0q; Wed, 19 Jun 2019 16:03:15 +0200 Date: Wed, 19 Jun 2019 16:03:14 +0200 From: Andrew Lunn To: Maxime Ripard Subject: Re: [PATCH v3 01/16] dt-bindings: net: Add YAML schemas for the generic Ethernet options Message-ID: <20190619140314.GC18352@lunn.ch> References: <27aeb33cf5b896900d5d11bd6957eda268014f0c.1560937626.git-series.maxime.ripard@bootlin.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <27aeb33cf5b896900d5d11bd6957eda268014f0c.1560937626.git-series.maxime.ripard@bootlin.com> User-Agent: Mutt/1.5.23 (2014-03-12) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190619_070331_891656_852213A2 X-CRM114-Status: GOOD ( 14.44 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Mark Rutland , devicetree@vger.kernel.org, Florian Fainelli , Alexandre Torgue , Rob Herring , Antoine =?iso-8859-1?Q?T=E9nart?= , netdev@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, Chen-Yu Tsai , Rob Herring , Maxime Coquelin , Maxime Chevallier , Frank Rowand , "David S . Miller" , linux-arm-kernel@lists.infradead.org, Heiner Kallweit Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Maxime > + phy-connection-type: > + description: > + Operation mode of the PHY interface > + enum: > + # There is not a standard bus between the MAC and the PHY, > + # something proprietary is being used to embed the PHY in the > + # MAC. ... > + > + phy-mode: > + $ref: "#/properties/phy-connection-type" > + deprecated: true I don't think phy-mode is actually deprecated. ethernet.txt actually says: "This is now a de-facto standard property;" and no mentions that is should not be used. Looking at actual device trees, phy-mode is by far more popular than phy-connection-type. fwnode_get_phy_mode() first looks for phy-mode and only falls back to phy-connection-type if it is not present. The same is true for of_get_phy_mode(). > + fixed-link: > + allOf: > + - if: > + type: array > + then: > + minItems: 1 > + maxItems: 1 > + items: > + items: > + - minimum: 0 > + maximum: 31 > + description: > + Emulated PHY ID, choose any but unique to the all > + specified fixed-links > + > + - enum: [0, 1] > + description: > + Duplex configuration. 0 for half duplex or 1 for > + full duplex > + > + - enum: [10, 100, 1000] > + description: > + Link speed in Mbits/sec. > + > + - enum: [0, 1] > + description: > + Pause configuration. 0 for no pause, 1 for pause > + > + - enum: [0, 1] > + description: > + Asymmetric pause configuration. 0 for no asymmetric > + pause, 1 for asymmetric pause > + This array of 5 values format should be marked as deprecated. > + > + - if: > + type: object > + then: > + properties: > + speed: > + allOf: > + - $ref: /schemas/types.yaml#definitions/uint32 > + - enum: [10, 100, 1000] This recently changed, depending on context. If PHYLINK is being used, any speed is allowed. If phylib is used, then only these speeds are allowed. And we are starting to see some speeds other than listed here. Andrew _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel