From mboxrd@z Thu Jan 1 00:00:00 1970 From: Neil Armstrong Subject: Re: [PATCH 7/9] dt-bindings: phy: meson-g12a-usb3-pcie-phy: convert to yaml Date: Fri, 9 Aug 2019 09:00:01 +0200 Message-ID: <2e6ade0e-175c-e77c-e767-28f4bbfbe575@baylibre.com> References: <20190808085139.21438-1-narmstrong@baylibre.com> <20190808085139.21438-8-narmstrong@baylibre.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Martin Blumenstingl Cc: devicetree@vger.kernel.org, Rob Herring , kishon@ti.com, robh+dt@kernel.org, linux-amlogic@lists.infradead.org, linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org Hi, On 08/08/2019 21:50, Martin Blumenstingl wrote: > On Thu, Aug 8, 2019 at 10:54 AM Neil Armstrong wrote: >> >> Now that we have the DT validation in place, let's convert the device tree >> bindings for the Amlogic G12A USB3 + PCIE Combo PHY over to a YAML schemas. >> >> While the original phy bindings specifies phy-supply as required, >> the examples and implementations makes it optional, thus phy-supply >> is not present in the properties and required lists. > nit-pick: the original bindings didn't mention the phy-supply property at all > I'm not sure if you have to re-send it, maybe this can be fixed up > while applying? It didn't, but the phy.txt made it mandatory, but was optional in all examples and implementation, thus rob asked me to precise it in the commit log. Neil > >> Reviewed-by: Rob Herring >> Signed-off-by: Neil Armstrong > Reviewed-by: Martin Blumenstingl >