linux-kernel - Re: [PATCH] mfd: lm3533: fix return type of attribute is

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <20120518111804.GA20467@localhost>
Date:	Fri, 18 May 2012 13:18:04 +0200
From:	Johan Hovold <jhovold@...il.com>
To:	Samuel Ortiz <sameo@...ux.intel.com>
Cc:	linux-kernel@...r.kernel.org,
	Andrew Morton <akpm@...ux-foundation.org>,
	Johan Hovold <jhovold@...il.com>
Subject: Re: [PATCH] mfd: lm3533: fix return type of attribute is_visible

Hi Samuel,

On Fri, May 11, 2012 at 10:40:41AM +0200, Johan Hovold wrote:
> Since commit 587a1f1659 ("switch ->is_visible() to returning umode_t")
> the return type of is_visible is umode_t rather than mode_t.
> 
> This silences a compiler warning on some architectures where these types
> are not compatible.
> 
> Reported-by: Andrew Morton <akpm@...ux-foundation.org>
> Signed-off-by: Johan Hovold <jhovold@...il.com>
> ---

Apparently Dan Carpenter sent a fix for this compiler warning the day
after I submitted mine. Dan's patch, however, only fixed the return type
and not the type of the mode variable.

Could you consider applying this patch one on top of Dan's (which you
applied to mfd-next on Monday)?

Thanks,
Johan

>  drivers/mfd/lm3533-core.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/mfd/lm3533-core.c b/drivers/mfd/lm3533-core.c
> index d11c891..5770bea 100644
> --- a/drivers/mfd/lm3533-core.c
> +++ b/drivers/mfd/lm3533-core.c
> @@ -359,7 +359,7 @@ static struct attribute *lm3533_attributes[] = {
>  #define to_dev_attr(_attr) \
>  	container_of(_attr, struct device_attribute, attr)
>  
> -static mode_t lm3533_attr_is_visible(struct kobject *kobj,
> +static umode_t lm3533_attr_is_visible(struct kobject *kobj,
>  					     struct attribute *attr, int n)
>  {
>  	struct device *dev = container_of(kobj, struct device, kobj);
> @@ -367,7 +367,7 @@ static mode_t lm3533_attr_is_visible(struct kobject *kobj,
>  	struct device_attribute *dattr = to_dev_attr(attr);
>  	struct lm3533_device_attribute *lattr = to_lm3533_dev_attr(dattr);
>  	enum lm3533_attribute_type type = lattr->type;
> -	mode_t mode = attr->mode;
> +	umode_t mode = attr->mode;
>  
>  	if (!lm3533->have_backlights && type == LM3533_ATTR_TYPE_BACKLIGHT)
>  		mode = 0;
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/