| Message ID | 1268612981-9009-1-git-send-email-w.sang@pengutronix.de |
|---|---|
| State | Accepted |
| Headers | show |
On Mon, 15 Mar 2010, Wolfram Sang wrote: > Commit 6992f5334995af474c2b58d010d08bc597f0f2fe introduced this requirement. > First, at25 was fixed manually. Then, other occurences were found with > coccinelle and the following semantic patch. Results were reviewed and fixed > up: > > @ init @ > identifier struct_name, bin; > @@ > > struct struct_name { > ... > struct bin_attribute bin; > ... > }; > > @ main extends init @ > expression E; > statement S; > identifier name, err; > @@ > > ( > struct struct_name *name; > | > - struct struct_name *name = NULL; > + struct struct_name *name; > ) > ... > ( > sysfs_bin_attr_init(&name->bin); > | > + sysfs_bin_attr_init(&name->bin); > if (sysfs_create_bin_file(E, &name->bin)) > S > | > + sysfs_bin_attr_init(&name->bin); > err = sysfs_create_bin_file(E, &name->bin); > ) > > Signed-off-by: Wolfram Sang <w.sang@pengutronix.de> > Cc: Eric W. Biederman <ebiederm@xmission.com> > Cc: Linus Torvalds <torvalds@linux-foundation.org> > --- > arch/mips/txx9/generic/setup.c | 1 + > drivers/misc/eeprom/at25.c | 1 + > drivers/rtc/rtc-ds1742.c | 1 + > 3 files changed, 3 insertions(+), 0 deletions(-) I don't understand how cocinelle works, but the resulting patch seems to be incomplete. The fix is needed at least in firmware loader (for which Greg has already queued patch), in ACPI thermal driver [1], etc). [1] http://lkml.indiana.edu/hypermail/linux/kernel/1003.1/02680.html
> I don't understand how cocinelle works, but the resulting patch seems to > be incomplete. Okay, will investigate, thanks for the pointer. > The fix is needed at least in firmware loader (for which Greg has already > queued patch), in ACPI thermal driver [1], etc). Firmware loader should be handled differently, see other mails in the originating thread, so I skipped it intentionally. at24 also needed a fix, but that one was just applied via the i2c-tree, so I skipped it, too. Will check the ACPI-case! Regards, Wolfram
On Mon, 15 Mar 2010, Jiri Kosina wrote: > > I don't understand how cocinelle works, but the resulting patch seems to > be incomplete. > > The fix is needed at least in firmware loader (for which Greg has already > queued patch), in ACPI thermal driver [1], etc). The firmware loader should be done differently: http://patchwork.kernel.org/patch/84814/ and I suspect that Wolfram may have skipped the thermal one as having been done by another patch (that I didn't merge, due to having questions about the whole issue, but that I don't have any objections to) But maybe there is also some limitation to the coccinelle rule too. Linus
diff --git a/arch/mips/txx9/generic/setup.c b/arch/mips/txx9/generic/setup.c index 7174d83..95184a0 100644 --- a/arch/mips/txx9/generic/setup.c +++ b/arch/mips/txx9/generic/setup.c @@ -956,6 +956,7 @@ void __init txx9_sramc_init(struct resource *r) if (!dev->base) goto exit; dev->dev.cls = &txx9_sramc_sysdev_class; + sysfs_bin_attr_init(&dev->bindata_attr); dev->bindata_attr.attr.name = "bindata"; dev->bindata_attr.attr.mode = S_IRUSR | S_IWUSR; dev->bindata_attr.read = txx9_sram_read; diff --git a/drivers/misc/eeprom/at25.c b/drivers/misc/eeprom/at25.c index d902d81..d194212 100644 --- a/drivers/misc/eeprom/at25.c +++ b/drivers/misc/eeprom/at25.c @@ -347,6 +347,7 @@ static int at25_probe(struct spi_device *spi) * that's sensitive for read and/or write, like ethernet addresses, * security codes, board-specific manufacturing calibrations, etc. */ + sysfs_bin_attr_init(&at25->bin); at25->bin.attr.name = "eeprom"; at25->bin.attr.mode = S_IRUSR; at25->bin.read = at25_bin_read; diff --git a/drivers/rtc/rtc-ds1742.c b/drivers/rtc/rtc-ds1742.c index a127336..cad9ceb 100644 --- a/drivers/rtc/rtc-ds1742.c +++ b/drivers/rtc/rtc-ds1742.c @@ -184,6 +184,7 @@ static int __devinit ds1742_rtc_probe(struct platform_device *pdev) pdata->size_nvram = pdata->size - RTC_SIZE; pdata->ioaddr_rtc = ioaddr + pdata->size_nvram; + sysfs_bin_attr_init(&pdata->nvram_attr); pdata->nvram_attr.attr.name = "nvram"; pdata->nvram_attr.attr.mode = S_IRUGO | S_IWUSR; pdata->nvram_attr.read = ds1742_nvram_read;
Commit 6992f5334995af474c2b58d010d08bc597f0f2fe introduced this requirement. First, at25 was fixed manually. Then, other occurences were found with coccinelle and the following semantic patch. Results were reviewed and fixed up: @ init @ identifier struct_name, bin; @@ struct struct_name { ... struct bin_attribute bin; ... }; @ main extends init @ expression E; statement S; identifier name, err; @@ ( struct struct_name *name; | - struct struct_name *name = NULL; + struct struct_name *name; ) ... ( sysfs_bin_attr_init(&name->bin); | + sysfs_bin_attr_init(&name->bin); if (sysfs_create_bin_file(E, &name->bin)) S | + sysfs_bin_attr_init(&name->bin); err = sysfs_create_bin_file(E, &name->bin); ) Signed-off-by: Wolfram Sang <w.sang@pengutronix.de> Cc: Eric W. Biederman <ebiederm@xmission.com> Cc: Linus Torvalds <torvalds@linux-foundation.org> --- arch/mips/txx9/generic/setup.c | 1 + drivers/misc/eeprom/at25.c | 1 + drivers/rtc/rtc-ds1742.c | 1 + 3 files changed, 3 insertions(+), 0 deletions(-)