From:   "Sit, Michael Wei Hong" <michael.wei.hong.sit@intel.com>
To:     Jakub Kicinski <kuba@kernel.org>
CC:     Giuseppe Cavallaro <peppe.cavallaro@st.com>,
        Alexandre Torgue <alexandre.torgue@foss.st.com>,
        Jose Abreu <joabreu@synopsys.com>,
        "David S . Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        "Paolo Abeni" <pabeni@redhat.com>,
        Maxime Coquelin <mcoquelin.stm32@gmail.com>,
        "Ong, Boon Leong" <boon.leong.ong@intel.com>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-stm32@st-md-mailman.stormreply.com" 
        <linux-stm32@st-md-mailman.stormreply.com>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.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>,
        "Lai, Peter Jun Ann" <peter.jun.ann.lai@intel.com>
Subject: RE: [PATCH net v2 1/2] net: stmmac: fix PHY handle parsing
Thread-Topic: [PATCH net v2 1/2] net: stmmac: fix PHY handle parsing
Thread-Index: AQHZWGNyiptgMBBkxEe3QmwqS1rIqK7+Om6w
Date:   Fri, 17 Mar 2023 02:10:41 +0000
Message-ID: <PH0PR11MB7587073E93849A1E43E3B1C49DBD9@PH0PR11MB7587.namprd11.prod.outlook.com>
References: <20230314070208.3703963-1-michael.wei.hong.sit@intel.com>
        <20230314070208.3703963-2-michael.wei.hong.sit@intel.com>
 <20230316170013.4681d0f3@kernel.org>
In-Reply-To: <20230316170013.4681d0f3@kernel.org>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?Z+5hfviHxR+wIm+QtzLXQWYyNwanHSKfWJh4dvbxJtrvN3U+D0xaVNFuLPcY?=
 =?us-ascii?Q?j8cB56y5yhPtK973AGE1/HRIsqhBFtu1HUPkTOcHTHwjVujFvRA2umsEIbmM?=
 =?us-ascii?Q?XAWPTtT/7/2YhJ8f8d8NHB2VSvY/xBmBS+wV/f1uglkytq+Zek8Hng87xaX1?=
 =?us-ascii?Q?2lVrmfI0F0vF9jPOcqq3JyiMWezWwwiM5deAXrpJjqqbZ0sB+OrkK/vdyU7E?=
 =?us-ascii?Q?v/cn8IcBeEJr4BFjVFJF7G2s4Fq3SOroZdG895A+njgMwkWUL22d1NVHzsOx?=
 =?us-ascii?Q?20/4o68Dc1o7NbTfJin5KfVTsEl9R3/tYcv+ZXZ1pk+lO1dylJdYOTO/XEeT?=
 =?us-ascii?Q?A2g7zAMPigIzL5XHDqCQT+YxUOzSB0nY6Of/hhZm1VosktbNr3ZHG1Ab6cIf?=
 =?us-ascii?Q?0ZG9ik3Jc3B30MJGuv60/sG9QApWKxb1CaEgdQjh1paLSm6wAm8HAk65kEyx?=
 =?us-ascii?Q?n2QWgyXSbjuh6H6U972f+OvMQMhfOLqUs75lfUboxPo1kAsUKGMH0zGVsAm4?=
 =?us-ascii?Q?894Kq6PADByMTsMq0coP9WmZSsQia5jq93W+Uc6OobcMaV6CQT39FgCfjHyW?=
 =?us-ascii?Q?yzrIXb7Q9bDQKScJ4Yb/iv9XjNAf4dlPMhRBruwmipSgscnrcG0+n5IGrdi6?=
 =?us-ascii?Q?ePJTliZAKu476nz8Dd3LJmktfNDPObUkFY0jcXyDOzWAM8fFYJKFJlTYFQiF?=
 =?us-ascii?Q?DLhVvJlVlVupaCXv488wctNF9pNEKcJ/nIE7hhZ2XTS++4KS4N3Bot9GgA3Y?=
 =?us-ascii?Q?EEF9fpBJEiF/SdrSGwHHMtoLax+NO7hSwcZAxDf48whEOgHaXK2obcDxWsi7?=
 =?us-ascii?Q?GBapWnUJ7Etqbwbb263DQ7rmZ5Qr1YCusJmGRwBdw0lSt54RYkZTkkEU1ASY?=
 =?us-ascii?Q?Ab+OuSp6SgXtv0Y19GW/VgOu1bzK92JI6Llx+FX8cQoygZFS9UuYbKdmHnLr?=
 =?us-ascii?Q?dqIxsdHgvgoLndKFw86E+APOgRs1dnXOsv1Brhx2YhX7hpZwcdjmH7tb1R5a?=
 =?us-ascii?Q?rud4n6hir4CtkRIKOb/aa+xYExG7cis5PCx+S7zeVXrrLx/NQIHNUECDgcgL?=
 =?us-ascii?Q?jhgfbxFYKLw80aN74uP63xto7aTCBjsDhTJH6tMODdMm5QGqn+mNfr4Pcxuf?=
 =?us-ascii?Q?Wlx7NWmuUJtUmQvlAxW279tU3fEjRLvqaNjLVltfdmz+1f9fZW5gLyDvlkJi?=
 =?us-ascii?Q?qzuxbsbP391TF6w9ErBy4SHVWCUiqLa8rClktzQXrB0S21VEKGRVRCLGXl1q?=
 =?us-ascii?Q?5vXagMKjPX04dHN0HGvBnKuw6Iqbm8zHjOXUnFCr6sWz6pdD7qpwlQloq94V?=
 =?us-ascii?Q?rezUSGm5qiWfcvWTWWdL2VDDNTph6n/Ys6UdAPNeCBkBduZIZX/yo1+R1U6V?=
 =?us-ascii?Q?P2kFOXRYqG95uAgf5UAG9FBVZdPHmmN0eRpqtDwX1yuOjTqBJ/gTkliZsqvu?=
 =?us-ascii?Q?jQAhNxyxqgipqi9sEDuYNjL+M292TkFFd3PcxTOYKY9Kfyx/F7ZkwIDPB/bd?=
 =?us-ascii?Q?cHaqDWO9jjSTqD7dQAJD3JERguUz1fPrd+4lFKvB3XTagsTFdjP3oThUEkhF?=
 =?us-ascii?Q?rL3TSniVPPRXKgzw8HWM5ZpcieRRxUK0lKicmhPVbREZXeXZdfZs1MnF1SJt?=
 =?us-ascii?Q?RQ=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: PH0PR11MB7587.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 99317b40-26b0-4fa8-67f5-08db268ccf57
X-MS-Exchange-CrossTenant-originalarrivaltime: 17 Mar 2023 02:10:41.8477
 (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: TW5u+3jxVOj1rTOsi4dO28/p/dGTjnPyrcweiuZp0OlupY8rxFKCc7Bq4ZVIF7axda/B/rKSM6KFCMqyydIXm69IJe3Lw82Q2EXpglaMTSg=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY5PR11MB6091
X-OriginatorOrg: intel.com
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: Friday, March 17, 2023 8:00 AM
> To: Sit, Michael Wei Hong <michael.wei.hong.sit@intel.com>
> Cc: Giuseppe Cavallaro <peppe.cavallaro@st.com>; Alexandre
> Torgue <alexandre.torgue@foss.st.com>; Jose Abreu
> <joabreu@synopsys.com>; David S . Miller
> <davem@davemloft.net>; Eric Dumazet
> <edumazet@google.com>; Paolo Abeni <pabeni@redhat.com>;
> Maxime Coquelin <mcoquelin.stm32@gmail.com>; Ong, Boon
> Leong <boon.leong.ong@intel.com>; netdev@vger.kernel.org;
> linux-stm32@st-md-mailman.stormreply.com; linux-arm-
> kernel@lists.infradead.org; linux-kernel@vger.kernel.org; Looi,
> Hong Aun <hong.aun.looi@intel.com>; Voon, Weifeng
> <weifeng.voon@intel.com>; Lai, Peter Jun Ann
> <peter.jun.ann.lai@intel.com>
> Subject: Re: [PATCH net v2 1/2] net: stmmac: fix PHY handle
> parsing
>=20
> On Tue, 14 Mar 2023 15:02:07 +0800 Michael Sit Wei Hong wrote:
> > +		fixed_node =3D
> fwnode_get_named_child_node(fwnode, "fixed-link");
> > +		fwnode_handle_put(fixed_node);
>=20
> fwnode_property_present() ?
Good suggestion, will modify and submit in next revision.