diff options
author | Wolfram Sang <w.sang@pengutronix.de> | 2010-03-14 20:29:41 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2010-03-14 23:28:39 -0400 |
commit | f937331b3f92cb2f67bc81baa1b8cc5198c439e5 (patch) | |
tree | e14edb4d6e31ef48f07145e6f3e4016c6075f4d8 | |
parent | 181427a7e01beab76c789414334375839f026128 (diff) |
init dynamic bin_attribute structures
Commit 6992f5334995af474c2b58d010d08bc597f0f2fe ("sysfs: Use one lockdep
class per sysfs attribute.") 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>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | arch/mips/txx9/generic/setup.c | 1 | ||||
-rw-r--r-- | drivers/misc/eeprom/at25.c | 1 | ||||
-rw-r--r-- | drivers/rtc/rtc-ds1742.c | 1 |
3 files changed, 3 insertions, 0 deletions
diff --git a/arch/mips/txx9/generic/setup.c b/arch/mips/txx9/generic/setup.c index 7174d830dd05..95184a0a1ae6 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) | |||
956 | if (!dev->base) | 956 | if (!dev->base) |
957 | goto exit; | 957 | goto exit; |
958 | dev->dev.cls = &txx9_sramc_sysdev_class; | 958 | dev->dev.cls = &txx9_sramc_sysdev_class; |
959 | sysfs_bin_attr_init(&dev->bindata_attr); | ||
959 | dev->bindata_attr.attr.name = "bindata"; | 960 | dev->bindata_attr.attr.name = "bindata"; |
960 | dev->bindata_attr.attr.mode = S_IRUSR | S_IWUSR; | 961 | dev->bindata_attr.attr.mode = S_IRUSR | S_IWUSR; |
961 | dev->bindata_attr.read = txx9_sram_read; | 962 | dev->bindata_attr.read = txx9_sram_read; |
diff --git a/drivers/misc/eeprom/at25.c b/drivers/misc/eeprom/at25.c index d902d81dde39..d194212a41f6 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) | |||
347 | * that's sensitive for read and/or write, like ethernet addresses, | 347 | * that's sensitive for read and/or write, like ethernet addresses, |
348 | * security codes, board-specific manufacturing calibrations, etc. | 348 | * security codes, board-specific manufacturing calibrations, etc. |
349 | */ | 349 | */ |
350 | sysfs_bin_attr_init(&at25->bin); | ||
350 | at25->bin.attr.name = "eeprom"; | 351 | at25->bin.attr.name = "eeprom"; |
351 | at25->bin.attr.mode = S_IRUSR; | 352 | at25->bin.attr.mode = S_IRUSR; |
352 | at25->bin.read = at25_bin_read; | 353 | at25->bin.read = at25_bin_read; |
diff --git a/drivers/rtc/rtc-ds1742.c b/drivers/rtc/rtc-ds1742.c index a1273360a44e..cad9ceb89baf 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) | |||
184 | pdata->size_nvram = pdata->size - RTC_SIZE; | 184 | pdata->size_nvram = pdata->size - RTC_SIZE; |
185 | pdata->ioaddr_rtc = ioaddr + pdata->size_nvram; | 185 | pdata->ioaddr_rtc = ioaddr + pdata->size_nvram; |
186 | 186 | ||
187 | sysfs_bin_attr_init(&pdata->nvram_attr); | ||
187 | pdata->nvram_attr.attr.name = "nvram"; | 188 | pdata->nvram_attr.attr.name = "nvram"; |
188 | pdata->nvram_attr.attr.mode = S_IRUGO | S_IWUSR; | 189 | pdata->nvram_attr.attr.mode = S_IRUGO | S_IWUSR; |
189 | pdata->nvram_attr.read = ds1742_nvram_read; | 190 | pdata->nvram_attr.read = ds1742_nvram_read; |