Re: [PATCH] Input: ili210x - fix return value of is_visible function
From: Sven Van Asbroeck <hidden>
Date: 2020-02-10 15:07:47
Also in:
lkml
Luca, Good catch. That's what happens when I post a patch I can't test :) Reviewed-by: Sven Van Asbroeck <redacted> On Sun, Feb 9, 2020 at 9:58 AM Luca Weiss [off-list ref] wrote:
quoted hunk ↗ jump to hunk
The is_visible function expects the permissions associated with an attribute of the sysfs group or 0 if an attribute is not visible. Change the code to return the attribute permissions when the attribute should be visible which resolves the warning: Attribute calibrate: Invalid permissions 01 Fixes: cc12ba1872c6 ("Input: ili210x - optionally show calibrate sysfs attribute") Signed-off-by: Luca Weiss <redacted> --- drivers/input/touchscreen/ili210x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)diff --git a/drivers/input/touchscreen/ili210x.c b/drivers/input/touchscreen/ili210x.c index 37526baff8a6..199cf3daec10 100644 --- a/drivers/input/touchscreen/ili210x.c +++ b/drivers/input/touchscreen/ili210x.c@@ -351,7 +351,7 @@ static umode_t ili210x_calibrate_visible(struct kobject *kobj, struct i2c_client *client = to_i2c_client(dev); struct ili210x *priv = i2c_get_clientdata(client); - return priv->chip->has_calibrate_reg; + return priv->chip->has_calibrate_reg ? attr->mode : 0; } static const struct attribute_group ili210x_attr_group = { --2.25.0