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;
From:   "Sit, Michael Wei Hong" <michael.wei.hong.sit@intel.com>
To:     Jakub Kicinski <kuba@kernel.org>
CC:     Andrew Lunn <andrew@lunn.ch>,
        Heiner Kallweit <hkallweit1@gmail.com>,
        Russell King <linux@armlinux.org.uk>,
        "David S . Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Paolo Abeni <pabeni@redhat.com>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "Looi, Hong Aun" <hong.aun.looi@intel.com>,
        "Voon, Weifeng" <weifeng.voon@intel.com>,
        "Tan, Tee Min" <tee.min.tan@intel.com>,
        "Zulkifli, Muhammad Husaini" <muhammad.husaini.zulkifli@intel.com>,
        "Gan, Yi Fang" <yi.fang.gan@intel.com>
Subject: RE: [PATCH net 1/1] net: phy: dp83867: Fix SGMII FIFO depth for non
 OF devices
Thread-Topic: [PATCH net 1/1] net: phy: dp83867: Fix SGMII FIFO depth for non
 OF devices
Thread-Index: AQHY895ZtWivL+GFVUe9kKNuDpdQc64120Sg
Date:   Wed, 9 Nov 2022 02:18:33 +0000
Message-ID: <DM5PR11MB1593619ECF5A94A92D5EA1B19D3E9@DM5PR11MB1593.namprd11.prod.outlook.com>
References: <20221108101218.612499-1-michael.wei.hong.sit@intel.com>
 <20221108175531.1a7c16a6@kernel.org>
In-Reply-To: <20221108175531.1a7c16a6@kernel.org>
Accept-Language: en-US
Content-Language: en-US
dlp-reaction: no-action
dlp-version: 11.6.500.17
dlp-product: dlpe-windows
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?paNOtpJHwNKdRNC682HJlSae8cfqZjJkuThDtcBv2kVG9yaO1UWFfpmH5sGx?=
 =?us-ascii?Q?I+nXgCnj4D5mkidm6hQdj9uGhwDtz1Kp+NaEg8Udkns+YHeCecRhebx2fTw/?=
 =?us-ascii?Q?Cfg5MRmaPyJr1LTjIycUqGvTj9GQR1SVAx62I8SJwQhRfhvf5GPU0td0H7Yw?=
 =?us-ascii?Q?Ye4s2Hyn0sAl5c7lLiiqpQ+cF5O+u2KXuzfqOpB2uoa1aRooKmStHwsWT+Kq?=
 =?us-ascii?Q?DNwBkn/p9aMD+Xz+AAz1qaI9ExUfwdQ65gXFpn4ncktTKZr4JxGaV9EzCU+S?=
 =?us-ascii?Q?uUNVFvWsjBjH+9Dnbq6howW+NtH5OAKf6ZFOPBAsMkl6x32xWWGvP6Fhnvmc?=
 =?us-ascii?Q?ks62k3/Xqh0NoeCC5JxbLjtB2T6RX7Qt6y562tM8CU3jXalkLKt2j/AmVBAV?=
 =?us-ascii?Q?DFwqDtUwKeB+WwQhDAt38bWMaAv3S5CLMVXYWpfBk84jUMQyRP/NN6u+6C8v?=
 =?us-ascii?Q?rfmMn9BYp6wTDLQr9/WvOAnpUY+QPjIQDV7k+8Tf03cjHQR6yCrKuQ1EJw8z?=
 =?us-ascii?Q?pHS95aoMKZ1HxIGQw5MIA/wV+bUr1iEfth2QQdSRVNunQGn84sr41+7sGxCf?=
 =?us-ascii?Q?q2mzA7gvKfkQ8/K16qIKS9PlzCjFNlVF/uxM0MBWMiKF1K9h/YC6aRBxCC8R?=
 =?us-ascii?Q?ZPOUUlm5yYCy/NfOLRUApPmeZy4klq87L1735y2w9BB7SbrjF4hSPfhqRMEb?=
 =?us-ascii?Q?Ti0bgrfnxEA3wfKERLxzXMMMqQ5NphwNKVNSoIFQhapXMeR4HHepUXLxO++n?=
 =?us-ascii?Q?s1E7GECyCfI9oDYuFQKe/zOecwSfQ228ZfU9s1TechfV7vppSISFvtpt2oPl?=
 =?us-ascii?Q?oe5pn++r2VouO7by1zErue205RIZXMKkho4GXdPr5yL2wj6N1udV0gCK0VL/?=
 =?us-ascii?Q?QDQLQ+M4efo3/AAqk3x+nReuZYk/jjAdJQB46yfanNLykyQ9ABWbYHWkPbg3?=
 =?us-ascii?Q?AQRP83EgTDxkqsn+tJ8MxfHi1fTAtwKRtRSvFijcI+qYDVeVQLXJXqubre/3?=
 =?us-ascii?Q?os8+4/0DsGyiu1d80i581LLeOJd/L2M0+/ZP9RHx4alwSQtggIzlOkacUT9A?=
 =?us-ascii?Q?PklhpgMKUDZavsCJ0FFnkr5hS+CSJXgVgzVt4UErvgZ/GhUF7BB5g561oRb3?=
 =?us-ascii?Q?TxjMJg6UZTZNbuxh+hleZJPmJLGyFtH2f6RjbGrsskAKh5Mxh1AjrL9JF9Am?=
 =?us-ascii?Q?w1ULir1Hb7cseK93V3bDJ/zQsNgYHJ2j3KkjjZNDoeIWLDDkrBCWk80nlALC?=
 =?us-ascii?Q?9fpMybmf+2YeAP9BbCZG/1gKdrqmhKdXoN+dwwYhYZ/H7635EbUgNkYjmV13?=
 =?us-ascii?Q?Lc4or8z9mQ7X/vsojPsCT6nTFS1uwPXjk//L0Kn9SpgSU5LIo48RXLPbvkn1?=
 =?us-ascii?Q?s9Zjmmy3xfYNsxEDeVB7KQ7EuDJSpiacUsSpdh4MSpL4vCWuQqZGjbVRk1IG?=
 =?us-ascii?Q?9LnIydz8SudGw/qIAWHzr3YWzg5nYUXy2LKJwtYmT3PcmQUSYlDqoQB3SIOW?=
 =?us-ascii?Q?tFdGRjeapQEWJqZsLY4Yb84Vf7RnwKT3/OJNteKfaf+tQ1s/Qlk3xpCVAWMg?=
 =?us-ascii?Q?+OQ2DE0KzLJQ0hrenZP3sK+esBg69s+SeTw/XmwO8vVcwvCgXB+hbp3qnRNj?=
 =?us-ascii?Q?Gw=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DM5PR11MB1593.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 84e582b2-189b-413a-d4bf-08dac1f8b34e
X-MS-Exchange-CrossTenant-originalarrivaltime: 09 Nov 2022 02:18:33.0528
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: VDqDQSNP2r0dKdoZvIJwqGESeyeMwylTtv5iVTGQkqYxIgZ/lis5wB5a6KkVAUuMLWiJ0yRbUrgQ/cAIYHhnJ9xp7dyTpNZqmRP8N30QfI8=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH8PR11MB7022
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,
        SPF_HELO_NONE,SPF_NONE 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: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


> -----Original Message-----
> From: Jakub Kicinski <kuba@kernel.org>
> Sent: Wednesday, November 9, 2022 9:56 AM
> To: Sit, Michael Wei Hong <michael.wei.hong.sit@intel.com>
> Cc: Andrew Lunn <andrew@lunn.ch>; Heiner Kallweit
> <hkallweit1@gmail.com>; Russell King <linux@armlinux.org.uk>;
> David S . Miller <davem@davemloft.net>; Eric Dumazet
> <edumazet@google.com>; Paolo Abeni <pabeni@redhat.com>;
> netdev@vger.kernel.org; linux-kernel@vger.kernel.org; Looi,
> Hong Aun <hong.aun.looi@intel.com>; Voon, Weifeng
> <weifeng.voon@intel.com>; Tan, Tee Min
> <tee.min.tan@intel.com>; Zulkifli, Muhammad Husaini
> <muhammad.husaini.zulkifli@intel.com>; Gan, Yi Fang
> <yi.fang.gan@intel.com>
> Subject: Re: [PATCH net 1/1] net: phy: dp83867: Fix SGMII FIFO
> depth for non OF devices
>=20
> On Tue,  8 Nov 2022 18:12:18 +0800 Michael Sit Wei Hong wrote:
> > Current driver code will read device tree node information, and
> set
> > default values if there is no info provided.
> >
> > This is not done in non-OF devices leading to SGMII fifo depths
> being
> > set to the smallest size.
> >
> > This patch sets the value to the default value of the PHY as
> stated in
> > the PHY datasheet.
>=20
> We need a Fixes tag, which commit should have contained this
> code?

Will add the Fixes tag in the next revision.
Thanks.