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=-8.3 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_1 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 25724C43141 for ; Thu, 28 Nov 2019 11:55:51 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id E73082176D for ; Thu, 28 Nov 2019 11:55:50 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=ti.com header.i=@ti.com header.b="DTxMyJ8I" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726730AbfK1Lzu (ORCPT ); Thu, 28 Nov 2019 06:55:50 -0500 Received: from fllv0015.ext.ti.com ([198.47.19.141]:59724 "EHLO fllv0015.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726191AbfK1Lzt (ORCPT ); Thu, 28 Nov 2019 06:55:49 -0500 Received: from fllv0035.itg.ti.com ([10.64.41.0]) by fllv0015.ext.ti.com (8.15.2/8.15.2) with ESMTP id xASBtdj3129928; Thu, 28 Nov 2019 05:55:39 -0600 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1574942139; bh=3J+AVC3Pcb6Yw4883ZHaDaBKebxD+28yuArSAn5pnYs=; h=Subject:To:CC:References:From:Date:In-Reply-To; b=DTxMyJ8IM+hI9lzEyMDfVjAXcgvEWo4yuQAAxh045FStAR9dwHNIf2D9heiAneEsk KgNtb4oyXiqgYGAQQPahAyykbgZg1HRGSeS6M8Uh8MaXjetcrggR90rpEgG2MpZUiO JM824LXxaS86t9zrzbLw+g5U1f7WF/YT6E+iMjxE= Received: from DLEE109.ent.ti.com (dlee109.ent.ti.com [157.170.170.41]) by fllv0035.itg.ti.com (8.15.2/8.15.2) with ESMTP id xASBtc3d083362; Thu, 28 Nov 2019 05:55:39 -0600 Received: from DLEE115.ent.ti.com (157.170.170.26) by DLEE109.ent.ti.com (157.170.170.41) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1847.3; Thu, 28 Nov 2019 05:55:38 -0600 Received: from fllv0039.itg.ti.com (10.64.41.19) by DLEE115.ent.ti.com (157.170.170.26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1847.3 via Frontend Transport; Thu, 28 Nov 2019 05:55:38 -0600 Received: from [10.250.100.73] (ileax41-snat.itg.ti.com [10.172.224.153]) by fllv0039.itg.ti.com (8.15.2/8.15.2) with ESMTP id xASBtYaM109133; Thu, 28 Nov 2019 05:55:35 -0600 Subject: Re: [PATCH v7 net-next 06/13] dt-bindings: net: ti: add new cpsw switch driver bindings To: Rob Herring , Srinivas Kandagatla , Bartosz Golaszewski CC: netdev , Ilias Apalodimas , Andrew Lunn , "David S . Miller" , Ivan Khoronzhuk , Jiri Pirko , Florian Fainelli , Sekhar Nori , "linux-kernel@vger.kernel.org" , linux-omap , Murali Karicheri , References: <20191119221925.28426-1-grygorii.strashko@ti.com> <20191119221925.28426-7-grygorii.strashko@ti.com> From: Grygorii Strashko Message-ID: <9dc06cb7-c875-6fc1-e755-3832e9f39a52@ti.com> Date: Thu, 28 Nov 2019 13:55:32 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8"; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Rob, On 21/11/2019 21:24, Rob Herring wrote: > On Tue, Nov 19, 2019 at 4:19 PM Grygorii Strashko > wrote: >> >> Add bindings for the new TI CPSW switch driver. Comparing to the legacy >> bindings (net/cpsw.txt): >> - ports definition follows DSA bindings (net/dsa/dsa.txt) and ports can be >> marked as "disabled" if not physically wired. >> - all deprecated properties dropped; >> - all legacy propertiies dropped which represent constant HW cpapbilities >> (cpdma_channels, ale_entries, bd_ram_size, mac_control, slaves, >> active_slave) >> - TI CPTS DT properties are reused as is, but grouped in "cpts" sub-node >> - TI Davinci MDIO DT bindings are reused as is, because Davinci MDIO is >> reused. >> >> Signed-off-by: Grygorii Strashko >> --- >> .../bindings/net/ti,cpsw-switch.yaml | 240 ++++++++++++++++++ >> 1 file changed, 240 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml > > I see David has applied this already, but it still has numerous > problems. Please send a follow-up. > >> >> diff --git a/Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml b/Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml >> new file mode 100644 >> index 000000000000..81ae8cafabc1 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml >> @@ -0,0 +1,240 @@ >> +# SPDX-License-Identifier: GPL-2.0 > > For new bindings, please dual license: > > (GPL-2.0-only OR BSD-2-Clause) > >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/net/ti,cpsw-switch.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: TI SoC Ethernet Switch Controller (CPSW) Device Tree Bindings >> + >> +maintainers: >> + - Grygorii Strashko >> + - Sekhar Nori >> + >> +description: >> + The 3-port switch gigabit ethernet subsystem provides ethernet packet >> + communication and can be configured as an ethernet switch. It provides the >> + gigabit media independent interface (GMII),reduced gigabit media >> + independent interface (RGMII), reduced media independent interface (RMII), >> + the management data input output (MDIO) for physical layer device (PHY) >> + management. >> + >> +properties: >> + compatible: >> + oneOf: >> + - const: ti,cpsw-switch >> + - items: >> + - const: ti,am335x-cpsw-switch >> + - const: ti,cpsw-switch >> + - items: >> + - const: ti,am4372-cpsw-switch >> + - const: ti,cpsw-switch >> + - items: >> + - const: ti,dra7-cpsw-switch >> + - const: ti,cpsw-switch >> + >> + reg: >> + maxItems: 1 >> + description: >> + The physical base address and size of full the CPSW module IO range >> + >> + ranges: true >> + >> + clocks: >> + maxItems: 1 >> + description: CPSW functional clock >> + >> + clock-names: >> + maxItems: 1 >> + items: >> + - const: fck >> + >> + interrupts: >> + items: >> + - description: RX_THRESH interrupt >> + - description: RX interrupt >> + - description: TX interrupt >> + - description: MISC interrupt >> + >> + interrupt-names: >> + items: >> + - const: "rx_thresh" >> + - const: "rx" >> + - const: "tx" >> + - const: "misc" >> + >> + pinctrl-names: true >> + >> + syscon: >> + $ref: /schemas/types.yaml#definitions/phandle >> + description: >> + Phandle to the system control device node which provides access to >> + efuse IO range with MAC addresses > > Can't you use nvmem binding for this? > I've sent patch to fix other comments except this one. About nvmem,I've been thinking about it for a long time already, but in our case MAC address is encoded in eFuse register in a different way for different SoCs. So even if I'll try to use nvmem and define some MAC cell: efuse: efuse { compatible = "..."; #address-cells = <1>; #size-cells = <1>; eth_mac: eth_mac@34 { reg = <0x34 0x10>; }; }; portX { ... nvmem-cells = <ð_mac>; nvmem-cell-names = "mac-address"; }; the of_get_mac_address() will finally call nvmem->reg_read(priv, offset, val, bytes); and at this point nvmem driver will have no knowledge about the type of the cell (MAC address), so no decoding can not be done and returned mac will be incorrect. Not sure how to proceed here. One of the ways is to pass cell info in struct nvmem_device .reg_read()/.reg_write() callbacks, cell name could be use to perform some actions. Another thing which need to be considered is - MAC can be assigned per port, so dev->of_node != port_of_node (and of_get_mac_addr_nvmem() will fail). -- Best regards, grygorii