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 65201C38A2D for ; Tue, 25 Oct 2022 19:00:37 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231751AbiJYTA3 (ORCPT ); Tue, 25 Oct 2022 15:00:29 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41036 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232442AbiJYTAZ (ORCPT ); Tue, 25 Oct 2022 15:00:25 -0400 Received: from mail-qt1-x835.google.com (mail-qt1-x835.google.com [IPv6:2607:f8b0:4864:20::835]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A4CA43471E for ; Tue, 25 Oct 2022 12:00:19 -0700 (PDT) Received: by mail-qt1-x835.google.com with SMTP id hh9so8211142qtb.13 for ; Tue, 25 Oct 2022 12:00:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=XSGZJo1bl3SUQkhMNqs386T667b+Vn5p187f1yr+7xA=; b=E2HOl9ixsstjpF60K2tQ9pSVnZ4xuFcgpP4Amkng6w5crg3brzdjJzEm6hsZjD1ePV hz1tvFrOXBsiHoQoa/9iv/eZ0YY/kFCSGcNYLHKEe1d9zBD8StU4EVcfZhmamPZHcVP+ zTbTeTrlfoF8icVwrkWoJvXjopVL5OgyDYmrpuJokZMka+HcLR6YCVQIO7u+x9fdDXIV lfdvdes8cFgHIjuNfYALfdH1FRME9u6ufzs2GfSalBXDPsenHxz0ry96GiKzEZsdFoQs Sy5/z9778wRR/1Nw1ZTxA5oGgBnaVRnICCvTSFrlH0pS/ugoTs1wwPzokNLXAMXXbibe ADpQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=XSGZJo1bl3SUQkhMNqs386T667b+Vn5p187f1yr+7xA=; b=ueCjQXDHAVumC12eyyagN7Gds4Pi660aLhN2kdpTsQFCefZmEGU3sL3ZY2Vo9RYxUD piVbeJqOVdYOM36SdSwzFo1t7G55xooDc0vLhB5YHuKgGsRMXdLXq9b88hii8YuhgdPw n/083LPrzlVKJ/A0xTwS1OjzqpAUe7aRHngFSy8rgrxYQzEVdQ7ia+mn/7mvAaa/QflV LQAXBF6/sh2K7WOW2U3b5kzyrBUtn25GtAxhmDRNLJvs6YMPzIwXXL68UqmOBPb/GMvv FzcBtYiPsi0k7QmQEmuw8imYj58lbdEDAEvMnomBmjLarSovlYk6ewj7hRowvcLuh8xO GODw== X-Gm-Message-State: ACrzQf0FSY+tMZq3cU59CoMXk5KVN6/Lf4TQVkTpsDeT+dDe0CoFunYL qnxkavCuof6ZbOqTwjMVokHErg== X-Google-Smtp-Source: AMsMyM42TQk+Zo4mpoK4izG3t0mNi637t/x1i9/FW4VVIot9yp2nF07SBAVNC4YiTTlKvyTHaZNuDA== X-Received: by 2002:a05:622a:f:b0:39c:f320:f7a3 with SMTP id x15-20020a05622a000f00b0039cf320f7a3mr33350384qtw.101.1666724418798; Tue, 25 Oct 2022 12:00:18 -0700 (PDT) Received: from [192.168.1.11] ([64.57.193.93]) by smtp.gmail.com with ESMTPSA id x10-20020ac84a0a000000b00398ed306034sm1986707qtq.81.2022.10.25.12.00.16 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 25 Oct 2022 12:00:18 -0700 (PDT) Message-ID: <2fedd4a8-5b13-7049-7485-307e6d01f9f9@linaro.org> Date: Tue, 25 Oct 2022 15:00:16 -0400 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.4.0 Subject: Re: [RFC net-next 1/2] dt-bindings: net: dsa: add bindings for GSW Series switches Content-Language: en-US To: Andrew Lunn Cc: Camel Guo , "David S. Miller" , Eric Dumazet , Florian Fainelli , Jakub Kicinski , Krzysztof Kozlowski , Paolo Abeni , Rob Herring , Russell King , Vivien Didelot , Vladimir Oltean , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, Rob Herring , kernel@axis.com References: <20221025135243.4038706-1-camel.guo@axis.com> <20221025135243.4038706-2-camel.guo@axis.com> <16aac887-232a-7141-cc65-eab19c532592@linaro.org> From: Krzysztof Kozlowski In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 25/10/2022 11:01, Andrew Lunn wrote: >>> + - enum: >>> + - mxl,gsw145-mdio >> >> Why "mdio" suffix? > > I wondered about that as well. At some point in the future, there > could be an SPI version of this driver, and a UART version. Would they > all use the same compatible, and then context it used to determine the > correct binding? I think the kernel would be happy to do that, but i > don't know if the YAML tools can support that? In general the bus should not be encoded in the device compatible. On which bus this device sits, is determined from the parent, not from the device compatible. As you wrote the context is used to determine properties. There are few exceptions, though, but I think this is not a candidate for such. > >>> +examples: >>> + - | >>> + #include >>> + >>> + mdio { >> >> Hmmm... switch with MDIO is part of MDIO? > > Happens a lot. Nothing wrong with this. OK, everyday learning :) Best regards, Krzysztof