]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
e1000e: Minor flow correction in e1000_shutdown function
authorVitaly Lifshits <vitaly.lifshits@intel.com>
Fri, 1 Mar 2024 18:48:05 +0000 (10:48 -0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 10 Apr 2024 14:35:56 +0000 (16:35 +0200)
commiteb96a5c025533ba42eb2a16559e6a4e90d4dfe9e
tree8eca50122ce1373af7bd61e82672c6b67ece705a
parent1d16cd91cd319d5bf6230c8493feb56a61e486a1
e1000e: Minor flow correction in e1000_shutdown function

[ Upstream commit 662200e324daebe6859c1f0f3ea1538b0561425a ]

Add curly braces to avoid entering to an if statement where it is not
always required in e1000_shutdown function.
This improves code readability and might prevent non-deterministic
behaviour in the future.

Signed-off-by: Vitaly Lifshits <vitaly.lifshits@intel.com>
Tested-by: Naama Meir <naamax.meir@linux.intel.com>
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
Link: https://lore.kernel.org/r/20240301184806.2634508-5-anthony.l.nguyen@intel.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Stable-dep-of: 861e8086029e ("e1000e: move force SMBUS from enable ulp function to avoid PHY loss issue")
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/net/ethernet/intel/e1000e/netdev.c