On Sun, Jun 29, 2014 at 1:20 AM, Joe Perches <joe@xxxxxxxxxxx> wrote: > The DEFINE_SIMPLE_ATTRIBUTE macro should not end in a ; > Fix the one use in the kernel tree that did not have a semicolon. > > Signed-off-by: Joe Perches <joe@xxxxxxxxxxx> Acked-by: Luca Tettamanti <kronos.it@xxxxxxxxx> For my driver only ;-) Luca > --- > drivers/hwmon/asus_atk0110.c | 2 +- > include/linux/fs.h | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/hwmon/asus_atk0110.c b/drivers/hwmon/asus_atk0110.c > index ae208f6..cccef87 100644 > --- a/drivers/hwmon/asus_atk0110.c > +++ b/drivers/hwmon/asus_atk0110.c > @@ -688,7 +688,7 @@ static int atk_debugfs_gitm_get(void *p, u64 *val) > DEFINE_SIMPLE_ATTRIBUTE(atk_debugfs_gitm, > atk_debugfs_gitm_get, > NULL, > - "0x%08llx\n") > + "0x%08llx\n"); > > static int atk_acpi_print(char *buf, size_t sz, union acpi_object *obj) > { > diff --git a/include/linux/fs.h b/include/linux/fs.h > index 8a9e7f8..e78e9ac 100644 > --- a/include/linux/fs.h > +++ b/include/linux/fs.h > @@ -2701,7 +2701,7 @@ static const struct file_operations __fops = { \ > .read = simple_attr_read, \ > .write = simple_attr_write, \ > .llseek = generic_file_llseek, \ > -}; > +} > > static inline __printf(1, 2) > void __simple_attr_check_format(const char *fmt, ...) > -- > 1.8.1.2.459.gbcd45b4.dirty > -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html