All of lore.kernel.org
 help / color / mirror / Atom feed
From: Daniel Mack <zonque@gmail.com>
To: Takashi Iwai <tiwai@suse.de>, gregkh@linuxfoundation.org
Cc: linux-kernel@vger.kernel.org, ming.lei@canonical.com
Subject: Re: [PATCH] firmware: move EXPORT_SYMBOL annotations
Date: Tue, 28 May 2013 20:30:45 +0200	[thread overview]
Message-ID: <51A4F7D5.9070107@gmail.com> (raw)
In-Reply-To: <s5h38t8stua.wl%tiwai@suse.de>

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 <zonque@gmail.com>
> 
>   Acked-by: Takashi Iwai <tiwai@suse.de>

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
>>


  reply	other threads:[~2013-05-28 18:30 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-23 20:17 [PATCH] firmware: move EXPORT_SYMBOL annotations Daniel Mack
2013-05-27  8:35 ` Takashi Iwai
2013-05-28 18:30   ` Daniel Mack [this message]
2013-05-28 19:27     ` Greg KH

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=51A4F7D5.9070107@gmail.com \
    --to=zonque@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=ming.lei@canonical.com \
    --cc=tiwai@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.