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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 27457C46CA1 for ; Mon, 18 Sep 2023 20:28:56 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229830AbjIRU27 (ORCPT ); Mon, 18 Sep 2023 16:28:59 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47628 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229605AbjIRU25 (ORCPT ); Mon, 18 Sep 2023 16:28:57 -0400 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A3BFC8F for ; Mon, 18 Sep 2023 13:28:51 -0700 (PDT) Received: by smtp.kernel.org (Postfix) with ESMTPSA id D942FC433C8; Mon, 18 Sep 2023 20:28:49 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1695068931; bh=pUlBBuz1FK2J0cyfaQ4KvrItSJbWh9d+uorfnqujgLI=; h=Date:Subject:To:References:From:In-Reply-To:From; b=L7UXMY1k5WmTCEz3nJJH5T79ESygRsChx6oSvJiO5+j6bg/BaSUmONosJP7oJsJt8 0d1wogo6meA3o4KKhq4ocspMuc1nK0WHf+g3qyCQbJYXJBr1RuL0b+D8XRrAhdDX+H JNhKCCJIR3ouQCF6YXBoCZZjkpH5xi7NQchu+uQmxf9WnpAQcSjiiv0IyrkkiGUxbO Nn5giMTbLxNvRNb4pWZn7cU+yYj+pto3gAHZRnEtqWsCCcDZb0t3U3gLd0FXCSrR72 4u4z76lOj7aRqVy1V5hEuUOpEDGrB+Ud2JU6H8ZNdoBKknw2rziHJEeoeSv/ddnlAs YTigZMLrMFqhg== Message-ID: <5194bd75-9562-8375-5748-ccce560b67cf@kernel.org> Date: Mon, 18 Sep 2023 22:28:47 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.15.1 Subject: Re: [PATCH 2/4] dt-bindings: net: Add onsemi NCN26010 ethernet controller To: Jay Monkman , devicetree@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Piergiorgio Beruto , Arndt Schuebel References: Content-Language: en-US From: Krzysztof Kozlowski In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 18/09/2023 08:58, Jay Monkman wrote: > Drop stray blank line. > Add devicetree bindings for onsemi's NCN26010 10BASE-T1S > ethernet controller. Please use scripts/get_maintainers.pl to get a list of necessary people and lists to CC (and consider --no-git-fallback argument). It might happen, that command when run on an older kernel, gives you outdated entries. Therefore please be sure you base your patches on recent Linux kernel. > > Signed-off-by: Jay Monkman > --- > .../devicetree/bindings/net/onnn,macphy.yaml | 94 +++++++++++++++++++ > 1 file changed, 94 insertions(+) > create mode 100644 Documentation/devicetree/bindings/net/onnn,macphy.yaml > > diff --git a/Documentation/devicetree/bindings/net/onnn,macphy.yaml b/Documentation/devicetree/bindings/net/onnn,macphy.yaml > new file mode 100644 > index 000000000000..1813da81b95f > --- /dev/null > +++ b/Documentation/devicetree/bindings/net/onnn,macphy.yaml > @@ -0,0 +1,94 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/net/onnn,macphy.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: onsemi 10BASE-T1S Ethernet Controller > + > +maintainers: > + - Jay Monkman > + > +description: | Do not need '|' unless you need to preserve formatting. > + Bindings for onsemi 10BASE-T1S ethernet controller. Drop "bindings for" and instead describe the hardware. > + > + Supported devices: > + ncn26010 Supported by what? By bindings? Or driver? Drop, instead, describe the hardware. > + > +allOf: > + - $ref: ethernet-controller.yaml# > + > +properties: > + compatible: > + const: onnn,ncn26010 > + > + ctrl-protected: > + description: | Do not need '|' unless you need to preserve formatting. Is it a generic property? Does not look like. All non-standard properties need vendor prefix. > + Enables control data read/write protection mode. I don't understand this and it is very close to duplicate the property name. Describe what it does in the hardware, not in the driver, so it will be justified to have it in bindings in the first place. > + type: boolean > + > + poll-interval: Missing units. > + $ref: /schemas/types.yaml#/definitions/uint32 > + description: | > + Defines how often the MACPHY thread runs in milliseconds. thread? So Linux kthread? Nope, not a bindings property. > + If not specified, a default value of 5 is used. > + > + tx-fcs-calc: > + description: | > + Enables driver calculation of the FCS on transmitted frames. > + type: boolean Driver? Not hardware? So drop the property. > + > + rx-fcs-check: > + description: | > + Enables driver checking of the FCS on received frames > + type: boolean Not a bindings property. > + > + interrupts: > + maxItems: 1 > + > + reg: > + maxItems: 1 > + > + mdio: > + $ref: mdio.yaml# > + unevaluatedProperties: false > + > +required: > + - compatible Nothing else is required? > + > +additionalProperties: false > + > +examples: > + - | > + spi { > + #address-cells = <1>; > + #size-cells = <0>; If this is a SPI device, then you miss spi-peripheral-props. > + > + ethernet@0 { > + compatible = "onnn,ncn26010"; > + reg = <0>; /* CE0 */ CE0? Isn't chip select obvious from the reg=0? Best regards, Krzysztof