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=-0.6 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no 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 2BBD7C433DF for ; Mon, 25 May 2020 13:57:40 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 09F5F207C3 for ; Mon, 25 May 2020 13:57:40 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (1024-bit key) header.d=lunn.ch header.i=@lunn.ch header.b="cJD9+lcJ" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2403932AbgEYN5i (ORCPT ); Mon, 25 May 2020 09:57:38 -0400 Received: from vps0.lunn.ch ([185.16.172.187]:48160 "EHLO vps0.lunn.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2403805AbgEYN5h (ORCPT ); Mon, 25 May 2020 09:57:37 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lunn.ch; s=20171124; h=In-Reply-To:Content-Type:MIME-Version:References:Message-ID: Subject:Cc:To:From:Date:Sender:Reply-To:Content-Transfer-Encoding:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=GaU25l9pyO7K3XI0B24cqiTiH7sFU/aybUfiy90S9MQ=; b=cJD9+lcJpMXmAgH2bxGoTpihrb CesVhtmmnWKMcOJKY8VMvm1FouO5Sxl/A8p19o3l5wXlkeRw8bngaVqik2FyWlv61n3fjIlLJjCtX xNxr6p9XGYXxOOMnF0FlJRbzCCcRHKhXTAPQl/+6jGRRoj9HJ1h98BIZRk/uQvdJ1yPw=; Received: from andrew by vps0.lunn.ch with local (Exim 4.93) (envelope-from ) id 1jdDbc-003C5d-CK; Mon, 25 May 2020 15:57:28 +0200 Date: Mon, 25 May 2020 15:57:28 +0200 From: Andrew Lunn To: Pavel Machek Cc: Florian Fainelli , Martin Blumenstingl , robh+dt@kernel.org, linux-amlogic@lists.infradead.org, devicetree@vger.kernel.org, jianxin.pan@amlogic.com, davem@davemloft.net, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v3 1/8] dt-bindings: net: meson-dwmac: Add the amlogic,rx-delay-ns property Message-ID: <20200525135728.GE752669@lunn.ch> References: <20200512211103.530674-1-martin.blumenstingl@googlemail.com> <20200512211103.530674-2-martin.blumenstingl@googlemail.com> <20200524212843.GF1192@bug> <20200525090718.GB16796@amd> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200525090718.GB16796@amd> Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org > > standardizing on rx-delay-ps and tx-delay-ps would make sense since that > > is the lowest resolution and the property would be correctly named with > > an unit in the name. > > Seems like similar patch is already being reviewed from Dan Murphy (?) > from TI. Dan is working on the PHY side. But there is probably code which can be shared. One question to consider, do we want the same properties names for MAC and PHY, or do we want to make them different, to avoid confusion? Andrew