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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id DE64DC00140 for ; Fri, 5 Aug 2022 14:11:20 +0000 (UTC) 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:MIME-Version:List-Subscribe:List-Help: List-Post:List-Archive:List-Unsubscribe:List-Id:Message-Id:Date:Subject: References:In-Reply-To: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=jJGpz7sUxUhr20nmvgyxEwBI9jhXJ9brPoiJ9BuGy+g=; b=RDDdTBo/UEFrBz +/gZWpzQeovRRVyU9u9LwpSyhUSpfz5nYrEtujmX8bxNI2qa/WT70i1YosrFbTEOLt1f6lAla2HX3 A0yWDvU1sLvUKaD3Oqvb4Qx7v4IuNC2EECUxRb4dXk0I8/JAXM6JyCrGPG+MD8VgHoJJRBAOk+0m3 799aJH3cSrI/KBQ6vYR7m8ljTEghKAOYDoOzCC1ReYHzkB+P2nQYXI6d+WkJR46jzoRG6/n+xXJjt Xoeg32vnr5AdO3pqf1roqHB7U7fkKTnLYwBs4Df1fuWQAEK6+fGXLsQbh0jhHOefR4JUu0lTrk4iU a0LUyK/Ot1r8gWLA7JEg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oJy1b-00Fivo-8L; Fri, 05 Aug 2022 14:10:04 +0000 Received: from mail-il1-f172.google.com ([209.85.166.172]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oJy1S-00Fisd-G3; Fri, 05 Aug 2022 14:09:56 +0000 Received: by mail-il1-f172.google.com with SMTP id t15so1417029ilm.7; Fri, 05 Aug 2022 07:09:53 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=message-id:date:subject:references:in-reply-to:cc:to:from :x-gm-message-state:from:to:cc; bh=4wU29egtLMNG+tGI0yk/oLxzbleDgRoflVboG4WD0GM=; b=h6OhITkZ82ghNuVorDvez820pmCdQvej2j73dXsJS6FtJSz4fbBezgdkrPl7hE2MfH 9ExexkL6o4DohkS82bWvoX/vdJ7OTsxiTgJNt1HoMsz1eMC5uf8QyLnVO1eX3AHd/o/w gnR1Iwhr5qfUNvSJ2q2rltdN6g55RuewHj7p4Wfd+rr9xeJDZ05k0vf7O27ZMl4Jdroi AhKGevAfFde/MdSJEMjPNpl5KFh3W+Lj65mvev+IhbPorc+G3UQrosXy0h/t/6LzgTY7 osH+JLavai8XwbHwKHjtIXOVj53kPqcOYwOXY61R+OXlHxtrjwz+UT3THg++DGXmzzMY VlFg== X-Gm-Message-State: ACgBeo0M79yFNxnQY9Pwo3I2EDQ5l2rbNFlH9d5nEbMMhZXCylFQyMf/ HA4pAg0RX82Fp7mwdm6eDEy4T6qk0A== X-Google-Smtp-Source: AA6agR5/LQJibOTqVIimFgDhK095NQ+fg7nigus3m4Dfff2GNYlsUbKl8lciS3gMG3znsPQfY2wOsg== X-Received: by 2002:a05:6e02:1a87:b0:2de:8da2:9ecf with SMTP id k7-20020a056e021a8700b002de8da29ecfmr3104073ilv.263.1659708593198; Fri, 05 Aug 2022 07:09:53 -0700 (PDT) Received: from robh.at.kernel.org ([64.188.179.248]) by smtp.gmail.com with ESMTPSA id s17-20020a92cc11000000b002ddf58d8828sm1687134ilp.19.2022.08.05.07.09.51 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 05 Aug 2022 07:09:52 -0700 (PDT) Received: (nullmailer pid 1802483 invoked by uid 1000); Fri, 05 Aug 2022 14:09:50 -0000 From: Rob Herring To: Sean Anderson Cc: Kishon Vijay Abraham I , Ioana Ciornei , linux-phy@lists.infradead.org, Camelia Alexandra Groza , Madalin Bucur , Vinod Koul , linuxppc-dev@lists.ozlabs.org, Rob Herring , Stephen Boyd , devicetree@vger.kernel.org, linux-clk@vger.kernel.org, Michael Turquette , linux-arm-kernel@lists.infradead.org, Krzysztof Kozlowski In-Reply-To: <20220804220602.477589-3-sean.anderson@seco.com> References: <20220804220602.477589-1-sean.anderson@seco.com> <20220804220602.477589-3-sean.anderson@seco.com> Subject: Re: [PATCH v4 2/8] dt-bindings: phy: Add Lynx 10G phy binding Date: Fri, 05 Aug 2022 08:09:50 -0600 Message-Id: <1659708590.965354.1802482.nullmailer@robh.at.kernel.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220805_070954_591723_B6907444 X-CRM114-Status: GOOD ( 24.87 ) 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: , MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Thu, 04 Aug 2022 18:05:56 -0400, Sean Anderson wrote: > This adds a binding for the SerDes module found on QorIQ processors. > Each phy is a subnode of the top-level device, possibly supporting > multiple lanes and protocols. This "thick" #phy-cells is used due to > allow for better organization of parameters. Note that the particular > parameters necessary to select a protocol-controller/lane combination > vary across different SoCs, and even within different SerDes on the same > SoC. > > The driver is designed to be able to completely reconfigure lanes at > runtime. Generally, the phy consumer can select the appropriate > protocol using set_mode. > > There are two PLLs, each of which can be used as the master clock for > each lane. Each PLL has its own reference. For the moment they are > required, because it simplifies the driver implementation. Absent > reference clocks can be modeled by a fixed-clock with a rate of 0. > > Signed-off-by: Sean Anderson > --- > > Changes in v4: > - Use subnodes to describe lane configuration, instead of describing > PCCRs. This is the same style used by phy-cadence-sierra et al. > > Changes in v3: > - Manually expand yaml references > - Add mode configuration to device tree > > Changes in v2: > - Rename to fsl,lynx-10g.yaml > - Refer to the device in the documentation, rather than the binding > - Move compatible first > - Document phy cells in the description > - Allow a value of 1 for phy-cells. This allows for compatibility with > the similar (but according to Ioana Ciornei different enough) lynx-28g > binding. > - Remove minItems > - Use list for clock-names > - Fix example binding having too many cells in regs > - Add #clock-cells. This will allow using assigned-clocks* to configure > the PLLs. > - Document the structure of the compatible strings > > .../devicetree/bindings/phy/fsl,lynx-10g.yaml | 236 ++++++++++++++++++ > 1 file changed, 236 insertions(+) > create mode 100644 Documentation/devicetree/bindings/phy/fsl,lynx-10g.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: Error: Documentation/devicetree/bindings/phy/fsl,lynx-10g.example.dts:51.27-28 syntax error FATAL ERROR: Unable to parse input tree make[1]: *** [scripts/Makefile.lib:383: Documentation/devicetree/bindings/phy/fsl,lynx-10g.example.dtb] Error 1 make[1]: *** Waiting for unfinished jobs.... make: *** [Makefile:1404: dt_binding_check] Error 2 doc reference errors (make refcheckdocs): See https://patchwork.ozlabs.org/patch/ This check can fail if there are any dependencies. The base for a patch series is generally the most recent rc1. If you already ran 'make dt_binding_check' and didn't see the above error(s), then make sure 'yamllint' is installed and dt-schema is up to date: pip3 install dtschema --upgrade Please check and re-submit. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel