Received: by 2002:a05:6358:9144:b0:117:f937:c515 with SMTP id r4csp7686718rwr; Wed, 10 May 2023 11:08:26 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ6jOqPN5D4C9VcHnrn0nPWaIBClF71gb67p5KjdRkbIwVDOnS8UyvDYVy/PrGNAlMIsEbHg X-Received: by 2002:a05:6870:5143:b0:18e:ccd1:16f7 with SMTP id z3-20020a056870514300b0018eccd116f7mr9950924oak.57.1683742105903; Wed, 10 May 2023 11:08:25 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1683742105; cv=none; d=google.com; s=arc-20160816; b=vn8enbHMgDpq5AbilxBhWuQB7/X9N577bIR6yfXaifJ5C4ePgnnAkH/xenJxtNFI/p M85/vHBxVFJy5SnUa993pvua+9Fy/FS0kUf1IyEoqFVtHze9VF7GFmzy6HUJcHKi179S dZXE4W+LAB21UNFxostURCsfyPeoDhkagK4JYrunmtLzltYJNgw5PO/MhfDlGIWaSi6R IOlipXbc4Poh2VKw5dl9MKp3ME1Z76N23J4w8xXOFE6OYOoTXWn7SJayjxjHEEjBy6i/ iuUwsgW6Lx7ItQc6H3kezKf9aVrOyZAfyWbfPjxvIbIX69eWDn9G08DGAMiYkc7yjFqr yPTw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-transfer-encoding :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=/5SzyOquzV31n1MIogwU0nPMAcBN0JZCHqLz6cmHYk0=; b=nSG5lDb1uBWxnMvqUXh5SfFx4UoPXmpFtGAuUsgf7QnsNpuxjC/DiFD99PZ4ffslrY mCEB6zvrf4i7qB7ws9j7FfRX+Gv7PBdRn8cLbRzGPZuKqKqvSBMr5tHct2B5qA32/1VJ 3mZukEc2LCVEJMYzHVA0ndExoPEs7WZh6pTyicE5jv1Bv2y3xxIziToomZqpeOysvLmX oR99uVi0ujAwDXke3zTDEO1NQhKV/T45fogNUeX9I/nWVm9jLWasS0/TO4+ixrHXN0vx MEyIIurrfK97CE3ksBET+Q0QgmlPqz8/9FqsraGkkXT+wXQWQTwdQOSVtgkJDE3kj06a sckg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id m27-20020a056870a11b00b001924ff27a30si9603374oae.227.2023.05.10.11.08.12; Wed, 10 May 2023 11:08:25 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235927AbjEJR40 (ORCPT + 99 others); Wed, 10 May 2023 13:56:26 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45964 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229773AbjEJR4Z (ORCPT ); Wed, 10 May 2023 13:56:25 -0400 Received: from smtp.dudau.co.uk (dliviu.plus.com [80.229.23.120]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 27E7D4200; Wed, 10 May 2023 10:56:24 -0700 (PDT) Received: from mail.dudau.co.uk (bart.dudau.co.uk [192.168.14.2]) by smtp.dudau.co.uk (Postfix) with SMTP id BC40E41D13BF; Wed, 10 May 2023 18:56:21 +0100 (BST) Received: by mail.dudau.co.uk (sSMTP sendmail emulation); Wed, 10 May 2023 18:56:21 +0100 Date: Wed, 10 May 2023 18:56:21 +0100 From: Liviu Dudau To: =?utf-8?B?QXLEsW7DpyDDnE5BTA==?= Cc: Thomas Bogendoerfer , Paul Burton , Rob Herring , Sergio Paracuellos , linux-mips@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH] mips: dts: ralink: Clarify usage of MT7621 ethernet phy arguments Message-ID: References: <20230509200032.308934-1-liviu@dudau.co.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,SPF_HELO_NONE, SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Arınç, On Wed, May 10, 2023 at 02:59:44PM +0200, Arınç ÜNAL wrote: > On 9.05.2023 22:00, Liviu Dudau wrote: > > The device tree uses numbers as arguments to the phys property that are > > confusing for newcomers. Define names for the values and use them in the > > device tree. > > > > Signed-off-by: Liviu Dudau > > You should document this on > Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml instead of > doing this. Under the phys property, add 'description:' and explain this. There is already some sort of explanation under Documentation/devicetree/bindings/phy/mediatek,mt7621-pci-phy.yaml, so I'm not sure what I'm improving by adding new text in the /pci/ section. Maybe I haven't explained properly in the commit message, this is meant to give a name to the 1 and 0 values used in the device tree, not to clarify any perceived missing documentation. > > Arınç Best regards, Liviu -- Everyone who uses computers frequently has had, from time to time, a mad desire to attack the precocious abacus with an axe. -- John D. Clark, Ignition!