Move EXPORT_SYMBOL annotations so they follow immediately after the
closing function brace line.
Signed-off-by: Daniel Mack <[email protected]>
---
drivers/base/firmware_class.c | 11 +++++------
1 file changed, 5 insertions(+), 6 deletions(-)
diff --git a/drivers/base/firmware_class.c b/drivers/base/firmware_class.c
index 4b1f926..4d9643f 100644
--- a/drivers/base/firmware_class.c
+++ b/drivers/base/firmware_class.c
@@ -1079,6 +1079,7 @@ request_firmware(const struct firmware **firmware_p, const char *name,
{
return _request_firmware(firmware_p, name, device, true, false);
}
+EXPORT_SYMBOL(request_firmware);
/**
* release_firmware: - release the resource associated with a firmware image
@@ -1092,6 +1093,7 @@ void release_firmware(const struct firmware *fw)
kfree(fw);
}
}
+EXPORT_SYMBOL(release_firmware);
/* Async support */
struct firmware_work {
@@ -1172,6 +1174,7 @@ request_firmware_nowait(
schedule_work(&fw_work->work);
return 0;
}
+EXPORT_SYMBOL(request_firmware_nowait);
/**
* cache_firmware - cache one firmware image in kernel memory space
@@ -1202,6 +1205,7 @@ int cache_firmware(const char *fw_name)
return ret;
}
+EXPORT_SYMBOL_GPL(cache_firmware);
/**
* uncache_firmware - remove one cached firmware image
@@ -1232,6 +1236,7 @@ int uncache_firmware(const char *fw_name)
return -EINVAL;
}
+EXPORT_SYMBOL_GPL(uncache_firmware);
#ifdef CONFIG_PM_SLEEP
static ASYNC_DOMAIN_EXCLUSIVE(fw_cache_domain);
@@ -1536,9 +1541,3 @@ static void __exit firmware_class_exit(void)
fs_initcall(firmware_class_init);
module_exit(firmware_class_exit);
-
-EXPORT_SYMBOL(release_firmware);
-EXPORT_SYMBOL(request_firmware);
-EXPORT_SYMBOL(request_firmware_nowait);
-EXPORT_SYMBOL_GPL(cache_firmware);
-EXPORT_SYMBOL_GPL(uncache_firmware);
--
1.8.1.4
At Thu, 23 May 2013 22:17:18 +0200,
Daniel Mack wrote:
>
> Move EXPORT_SYMBOL annotations so they follow immediately after the
> closing function brace line.
>
> Signed-off-by: Daniel Mack <[email protected]>
Acked-by: Takashi Iwai <[email protected]>
thanks,
Takashi
> ---
> drivers/base/firmware_class.c | 11 +++++------
> 1 file changed, 5 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/base/firmware_class.c b/drivers/base/firmware_class.c
> index 4b1f926..4d9643f 100644
> --- a/drivers/base/firmware_class.c
> +++ b/drivers/base/firmware_class.c
> @@ -1079,6 +1079,7 @@ request_firmware(const struct firmware **firmware_p, const char *name,
> {
> return _request_firmware(firmware_p, name, device, true, false);
> }
> +EXPORT_SYMBOL(request_firmware);
>
> /**
> * release_firmware: - release the resource associated with a firmware image
> @@ -1092,6 +1093,7 @@ void release_firmware(const struct firmware *fw)
> kfree(fw);
> }
> }
> +EXPORT_SYMBOL(release_firmware);
>
> /* Async support */
> struct firmware_work {
> @@ -1172,6 +1174,7 @@ request_firmware_nowait(
> schedule_work(&fw_work->work);
> return 0;
> }
> +EXPORT_SYMBOL(request_firmware_nowait);
>
> /**
> * cache_firmware - cache one firmware image in kernel memory space
> @@ -1202,6 +1205,7 @@ int cache_firmware(const char *fw_name)
>
> return ret;
> }
> +EXPORT_SYMBOL_GPL(cache_firmware);
>
> /**
> * uncache_firmware - remove one cached firmware image
> @@ -1232,6 +1236,7 @@ int uncache_firmware(const char *fw_name)
>
> return -EINVAL;
> }
> +EXPORT_SYMBOL_GPL(uncache_firmware);
>
> #ifdef CONFIG_PM_SLEEP
> static ASYNC_DOMAIN_EXCLUSIVE(fw_cache_domain);
> @@ -1536,9 +1541,3 @@ static void __exit firmware_class_exit(void)
>
> fs_initcall(firmware_class_init);
> module_exit(firmware_class_exit);
> -
> -EXPORT_SYMBOL(release_firmware);
> -EXPORT_SYMBOL(request_firmware);
> -EXPORT_SYMBOL(request_firmware_nowait);
> -EXPORT_SYMBOL_GPL(cache_firmware);
> -EXPORT_SYMBOL_GPL(uncache_firmware);
> --
> 1.8.1.4
>
On 27.05.2013 10:35, Takashi Iwai wrote:
> At Thu, 23 May 2013 22:17:18 +0200,
> Daniel Mack wrote:
>>
>> Move EXPORT_SYMBOL annotations so they follow immediately after the
>> closing function brace line.
>>
>> Signed-off-by: Daniel Mack <[email protected]>
>
> Acked-by: Takashi Iwai <[email protected]>
Thanks. Greg, could you take this through your tree?
Daniel
>
>
> thanks,
>
> Takashi
>
>> ---
>> drivers/base/firmware_class.c | 11 +++++------
>> 1 file changed, 5 insertions(+), 6 deletions(-)
>>
>> diff --git a/drivers/base/firmware_class.c b/drivers/base/firmware_class.c
>> index 4b1f926..4d9643f 100644
>> --- a/drivers/base/firmware_class.c
>> +++ b/drivers/base/firmware_class.c
>> @@ -1079,6 +1079,7 @@ request_firmware(const struct firmware **firmware_p, const char *name,
>> {
>> return _request_firmware(firmware_p, name, device, true, false);
>> }
>> +EXPORT_SYMBOL(request_firmware);
>>
>> /**
>> * release_firmware: - release the resource associated with a firmware image
>> @@ -1092,6 +1093,7 @@ void release_firmware(const struct firmware *fw)
>> kfree(fw);
>> }
>> }
>> +EXPORT_SYMBOL(release_firmware);
>>
>> /* Async support */
>> struct firmware_work {
>> @@ -1172,6 +1174,7 @@ request_firmware_nowait(
>> schedule_work(&fw_work->work);
>> return 0;
>> }
>> +EXPORT_SYMBOL(request_firmware_nowait);
>>
>> /**
>> * cache_firmware - cache one firmware image in kernel memory space
>> @@ -1202,6 +1205,7 @@ int cache_firmware(const char *fw_name)
>>
>> return ret;
>> }
>> +EXPORT_SYMBOL_GPL(cache_firmware);
>>
>> /**
>> * uncache_firmware - remove one cached firmware image
>> @@ -1232,6 +1236,7 @@ int uncache_firmware(const char *fw_name)
>>
>> return -EINVAL;
>> }
>> +EXPORT_SYMBOL_GPL(uncache_firmware);
>>
>> #ifdef CONFIG_PM_SLEEP
>> static ASYNC_DOMAIN_EXCLUSIVE(fw_cache_domain);
>> @@ -1536,9 +1541,3 @@ static void __exit firmware_class_exit(void)
>>
>> fs_initcall(firmware_class_init);
>> module_exit(firmware_class_exit);
>> -
>> -EXPORT_SYMBOL(release_firmware);
>> -EXPORT_SYMBOL(request_firmware);
>> -EXPORT_SYMBOL(request_firmware_nowait);
>> -EXPORT_SYMBOL_GPL(cache_firmware);
>> -EXPORT_SYMBOL_GPL(uncache_firmware);
>> --
>> 1.8.1.4
>>
On Tue, May 28, 2013 at 08:30:45PM +0200, Daniel Mack wrote:
> On 27.05.2013 10:35, Takashi Iwai wrote:
> > At Thu, 23 May 2013 22:17:18 +0200,
> > Daniel Mack wrote:
> >>
> >> Move EXPORT_SYMBOL annotations so they follow immediately after the
> >> closing function brace line.
> >>
> >> Signed-off-by: Daniel Mack <[email protected]>
> >
> > Acked-by: Takashi Iwai <[email protected]>
>
> Thanks. Greg, could you take this through your tree?
Yes, will do, give me some time to catch up on patches after returning
from LinuxCon Japan this week.
greg k-h