netdev - RE: [Intel-wired-lan] [PATCH iwl-next] ice: unify PHY FW loading status handler for E800 devices

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <IA1PR11MB6219C60279624AE6843C5F5B92F6A@IA1PR11MB6219.namprd11.prod.outlook.com>
Date: Fri, 17 Oct 2025 08:45:50 +0000
From: "Nitka, Grzegorz" <grzegorz.nitka@...el.com>
To: Paul Menzel <pmenzel@...gen.mpg.de>
CC: "Loktionov, Aleksandr" <aleksandr.loktionov@...el.com>,
	"intel-wired-lan@...ts.osuosl.org" <intel-wired-lan@...ts.osuosl.org>,
	"netdev@...r.kernel.org" <netdev@...r.kernel.org>
Subject: RE: [Intel-wired-lan] [PATCH iwl-next] ice: unify PHY FW loading
 status handler for E800 devices

> -----Original Message-----
> From: Paul Menzel <pmenzel@...gen.mpg.de>
> Sent: Thursday, October 16, 2025 5:54 PM
> To: Nitka, Grzegorz <grzegorz.nitka@...el.com>
> Cc: Loktionov, Aleksandr <aleksandr.loktionov@...el.com>; intel-wired-
> lan@...ts.osuosl.org; netdev@...r.kernel.org
> Subject: Re: [Intel-wired-lan] [PATCH iwl-next] ice: unify PHY FW loading
> status handler for E800 devices
> 
> Dear Grzegorz,
> 
> 
> Thank you for your patch.
> 
> Am 14.10.25 um 10:46 schrieb Grzegorz Nitka:
> > Unify handling of PHY firmware load delays across all E800 family
> > devices. There is an existing mechanism to poll
> GL_MNG_FWSM_FW_LOADING_M
> > bit of GL_MNG_FWSM register in order to verify whether PHY FW loading
> > completed or not. Previously, this logic was limited to E827 variants
> > only.
> >
> > Also, inform a user of possible delay in initialization process, by
> > dumping informational message in dmesg log.
> 
> Paste the message here?
> 

Thanks Paul for your feedback. Your comment has been addressed in v2.
Kind Regards

Grzegorz

...

> >
> >   	hw->lane_num = ice_get_phy_lane_number(hw);
> 
> Reviewed-by: Paul Menzel <pmenzel@...gen.mpg.de>
> 
> 
> Kind regards,
> 
> Paul