On Thu, Mar 15, 2012 at 8:10 AM, Dan Carpenter <[email protected]> wrote: > This function returns a umode_t (unsigned short) instead of mode_t which > is an unsigned int on some architectures. Cleaning this up silences a > compile warning: > > drivers/platform/x86/samsung-laptop.c:1108:2: warning: initialization > from incompatible pointer type [enabled by default] > > Signed-off-by: Dan Carpenter <[email protected]> > > diff --git a/drivers/platform/x86/samsung-laptop.c > b/drivers/platform/x86/samsung-laptop.c > index 7d7109f..0550bc5 100644 > --- a/drivers/platform/x86/samsung-laptop.c > +++ b/drivers/platform/x86/samsung-laptop.c > @@ -1086,7 +1086,7 @@ static int __init samsung_backlight_init(struct > samsung_laptop *samsung) > return 0; > } > > -static mode_t samsung_sysfs_is_visible(struct kobject *kobj, > +static umode_t samsung_sysfs_is_visible(struct kobject *kobj, > struct attribute *attr, int idx) > { > struct device *dev = container_of(kobj, struct device, kobj);
Thanks for the patch. Acked-by: Corentin Chary <[email protected]> Matthew, could you grab that one too ? Thanks, -- Corentin Chary http://xf.iksaif.net -- To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
