[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <190cec09-37ef-4649-9a69-7cab32b6d35b@intel.com>
Date: Wed, 21 Aug 2024 10:43:58 +0200
From: Przemek Kitszel <przemyslaw.kitszel@...el.com>
To: Maciej Fijalkowski <maciej.fijalkowski@...el.com>, Dawid Osuchowski
<dawid.osuchowski@...ux.intel.com>
CC: <intel-wired-lan@...ts.osuosl.org>, <netdev@...r.kernel.org>,
<larysa.zaremba@...el.com>, <kalesh-anakkur.purayil@...adcom.com>, "Jakub
Kicinski" <kuba@...nel.org>, Igor Bagnucki <igor.bagnucki@...el.com>
Subject: Re: [PATCH iwl-net v4] ice: Add netif_device_attach/detach into PF
reset flow
On 8/21/24 08:22, Maciej Fijalkowski wrote:
> On Tue, Aug 20, 2024 at 06:15:24PM +0200, Dawid Osuchowski wrote:
>> @@ -7591,6 +7594,7 @@ static void ice_rebuild(struct ice_pf *pf, enum ice_reset_req reset_type)
>> {
>> struct device *dev = ice_pf_to_dev(pf);
>> struct ice_hw *hw = &pf->hw;
>> + struct ice_vsi *vsi = ice_get_main_vsi(pf);
>
> we have an unwritten rule that is called 'reverse christmas tree' which
> requires us to have declarations of variables sorted from longest to
> shortest.
Hah, the rule is written down actually :P
https://docs.kernel.org/process/maintainer-netdev.html
Powered by blists - more mailing lists