TWxSoftware/twx-linux
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

platform/x86: ideapad-laptop: Make the scope_guard() clear of its scope

Browse Source 
[ Upstream commit a093cb667c ]

First of all, it's a bit counterintuitive to have something like

	int err;
	...
	scoped_guard(...)
		err = foo(...);
	if (err)
		return err;

Second, with a particular kernel configuration and compiler version in
one of such cases the objtool is not happy:

  ideapad-laptop.o: warning: objtool: .text.fan_mode_show: unexpected end of section

I'm not an expert on all this, but the theory is that compiler and
linker in this case can't understand that 'result' variable will be
always initialized as long as no error has been returned. Assigning
'result' to a dummy value helps with this. Note, that fixing the
scoped_guard() scope (as per above) does not make issue gone.

That said, assign dummy value and make the scope_guard() clear of its scope.
For the sake of consistency do it in the entire file.

Fixes: 7cc06e7294 ("platform/x86: ideapad-laptop: add a mutex to synchronize VPC commands")
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202408290219.BrPO8twi-lkp@intel.com/
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20240829165105.1609180-1-andriy.shevchenko@linux.intel.com
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
Andy Shevchenko
2024-08-29 19:50:32 +03:00
committed by Greg Kroah-Hartman
parent 849a6842ad
commit 5ce111f71a
1 changed files with 27 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files
drivers/platform/x86/ideapad-laptop.c
+27 -21
View File
@@ -419,13 +419,14 @@ static ssize_t camera_power_show(struct device *dev,
char *buf)
{
struct ideapad_private *priv = dev_get_drvdata(dev);
unsigned long result;
unsigned long result = 0;
int err;
scoped_guard(mutex, &priv->vpc_mutex)
scoped_guard(mutex, &priv->vpc_mutex) {
err = read_ec_data(priv->adev->handle, VPCCMD_R_CAMERA, &result);
if (err)
return err;
if (err)
return err;
}
return sysfs_emit(buf, "%d\n", !!result);
}
@@ -442,10 +443,11 @@ static ssize_t camera_power_store(struct device *dev,
if (err)
return err;
scoped_guard(mutex, &priv->vpc_mutex)
scoped_guard(mutex, &priv->vpc_mutex) {
err = write_ec_cmd(priv->adev->handle, VPCCMD_W_CAMERA, state);
if (err)
return err;
if (err)
return err;
}
return count;
}
@@ -493,13 +495,14 @@ static ssize_t fan_mode_show(struct device *dev,
char *buf)
{
struct ideapad_private *priv = dev_get_drvdata(dev);
unsigned long result;
unsigned long result = 0;
int err;
scoped_guard(mutex, &priv->vpc_mutex)
scoped_guard(mutex, &priv->vpc_mutex) {
err = read_ec_data(priv->adev->handle, VPCCMD_R_FAN, &result);
if (err)
return err;
if (err)
return err;
}
return sysfs_emit(buf, "%lu\n", result);
}
@@ -519,10 +522,11 @@ static ssize_t fan_mode_store(struct device *dev,
if (state > 4 || state == 3)
return -EINVAL;
scoped_guard(mutex, &priv->vpc_mutex)
scoped_guard(mutex, &priv->vpc_mutex) {
err = write_ec_cmd(priv->adev->handle, VPCCMD_W_FAN, state);
if (err)
return err;
if (err)
return err;
}
return count;
}
@@ -602,13 +606,14 @@ static ssize_t touchpad_show(struct device *dev,
char *buf)
{
struct ideapad_private *priv = dev_get_drvdata(dev);
unsigned long result;
unsigned long result = 0;
int err;
scoped_guard(mutex, &priv->vpc_mutex)
scoped_guard(mutex, &priv->vpc_mutex) {
err = read_ec_data(priv->adev->handle, VPCCMD_R_TOUCHPAD, &result);
if (err)
return err;
if (err)
return err;
}
priv->r_touchpad_val = result;
@@ -627,10 +632,11 @@ static ssize_t touchpad_store(struct device *dev,
if (err)
return err;
scoped_guard(mutex, &priv->vpc_mutex)
scoped_guard(mutex, &priv->vpc_mutex) {
err = write_ec_cmd(priv->adev->handle, VPCCMD_W_TOUCHPAD, state);
if (err)
return err;
if (err)
return err;
}
priv->r_touchpad_val = state;