From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Google-Smtp-Source: AB8JxZrlQK+V6ah/5rpG0PSeGxr/2cSqoXPrEoJy3KrnsiUCe9YIKSj/xHleXBMEC8xTa+qcDC2X ARC-Seal: i=1; a=rsa-sha256; t=1525390668; cv=none; d=google.com; s=arc-20160816; b=IUR0xcBeRp/pQFy3fmU00QfBrb6VcNpL0MKGBJtOzlFJvWpyTWdFF0UULL1hDQU6DJ ZYsvhtkSAM4apX/sxvvTJUxe9X8CUp4X9xNjVMKUsUWeB3bwvykB0kgPdwP/SCoQQ9fm IURCeWhglTefEVwQ11xYFT/O08FSYxrnaSrkohVy8YQ2XlJAwQyz+hAOzHcJSoF18+Bv Ck8n3tC4ZhhDxATDETE4ppykbhie+6sNeE5ISelQ1k14iRNamVW+vSxKdgIP5uCipwQO oSlL7y8I4LV8eabcFheYLHN0R76e+q4b6QQ0+RjBIFioMmjdqkAO8liUZ108269c8GJQ LAQg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=user-agent:in-reply-to:content-disposition:mime-version:references :message-id:subject:cc:to:from:date:arc-authentication-results; bh=7R8tLY0R8FNiwC0onWvkOChtMQO5rJzXvJfGvdZuM7Y=; b=hUw9v7uDfw4uSLI11eYA9Fv+9ASIdND70H8OMxwyNlFEfKAkRW8anCJB6g7VHVKiiy R8n8QBUUhPERRqkLEUa3z5DeNEgUcNWSOzf3zmZvq0U9Z0xFRt2O4XQ5MTZqvUGAz/XY D5FwES78m8lRHYgbJ8QONhlQK82c3B/i4vhEHmfj30jiLvqVkjzvW9SK08cwlkQuHzxV xbvRTW+D6yH8066Qf2K8UwStGIfvvfkKq7h3Xv/3l31uiyev3caIMtLwolmeHMxJZMuy s8qScaIW5iCN7YpKvGZiXS5YaD6e2pT+fp4HC8noKpeKGdrroEcoFKR4HUJWfgKfGTpd kUIA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of lurodriguez@suse.de designates 195.135.220.15 as permitted sender) smtp.mailfrom=lurodriguez@suse.de; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Authentication-Results: mx.google.com; spf=pass (google.com: domain of lurodriguez@suse.de designates 195.135.220.15 as permitted sender) smtp.mailfrom=lurodriguez@suse.de; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Date: Thu, 3 May 2018 23:37:46 +0000 From: "Luis R. Rodriguez" To: Andres Rodriguez Cc: linux-kernel@vger.kernel.org, gregkh@linuxfoundation.org, mcgrof@kernel.org, alexdeucher@gmail.com, christian.koenig@amd.com, kvalo@codeaurora.org, arend.vanspriel@broadcom.com, linux-wireless@vger.kernel.org, ath10k@lists.infradead.org, hdegoede@redhat.com, Kees Cook Subject: Re: [PATCH 4/9] firmware: use () to terminate kernel-doc function names Message-ID: <20180503233746.GV27853@wotan.suse.de> References: <20180423201205.20533-1-andresx7@gmail.com> <20180423201205.20533-5-andresx7@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180423201205.20533-5-andresx7@gmail.com> User-Agent: Mutt/1.6.0 (2016-04-01) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: =?utf-8?q?1598569148476307010?= X-GMAIL-MSGID: =?utf-8?q?1599488045939142572?= X-Mailing-List: linux-kernel@vger.kernel.org List-ID: On Mon, Apr 23, 2018 at 04:12:00PM -0400, Andres Rodriguez wrote: > The kernel-doc spec dictates a function name ends in (). > > Signed-off-by: Andres Rodriguez > Acked-by: Randy Dunlap 0-day never got back to me about my full sweep API rename so I think your changes are best to go in first, I'll do some small changes to account for this and resubmit to Greg once 0-day gives me its blessing. Luis > --- > drivers/base/firmware_loader/fallback.c | 8 ++++---- > drivers/base/firmware_loader/main.c | 22 +++++++++++----------- > 2 files changed, 15 insertions(+), 15 deletions(-) > > diff --git a/drivers/base/firmware_loader/fallback.c b/drivers/base/firmware_loader/fallback.c > index ecc49a619298..e75928458489 100644 > --- a/drivers/base/firmware_loader/fallback.c > +++ b/drivers/base/firmware_loader/fallback.c > @@ -124,7 +124,7 @@ static ssize_t timeout_show(struct class *class, struct class_attribute *attr, > } > > /** > - * firmware_timeout_store - set number of seconds to wait for firmware > + * firmware_timeout_store() - set number of seconds to wait for firmware > * @class: device class pointer > * @attr: device attribute pointer > * @buf: buffer to scan for timeout value > @@ -238,7 +238,7 @@ static int map_fw_priv_pages(struct fw_priv *fw_priv) > } > > /** > - * firmware_loading_store - set value in the 'loading' control file > + * firmware_loading_store() - set value in the 'loading' control file > * @dev: device pointer > * @attr: device attribute pointer > * @buf: buffer to scan for loading control value > @@ -430,7 +430,7 @@ static int fw_realloc_pages(struct fw_sysfs *fw_sysfs, int min_size) > } > > /** > - * firmware_data_write - write method for firmware > + * firmware_data_write() - write method for firmware > * @filp: open sysfs file > * @kobj: kobject for the device > * @bin_attr: bin_attr structure > @@ -536,7 +536,7 @@ fw_create_instance(struct firmware *firmware, const char *fw_name, > } > > /** > - * fw_load_sysfs_fallback - load a firmware via the sysfs fallback mechanism > + * fw_load_sysfs_fallback() - load a firmware via the sysfs fallback mechanism > * @fw_sysfs: firmware sysfs information for the firmware to load > * @opt_flags: flags of options, FW_OPT_* > * @timeout: timeout to wait for the load > diff --git a/drivers/base/firmware_loader/main.c b/drivers/base/firmware_loader/main.c > index 5baadad3012d..d028cd3257f7 100644 > --- a/drivers/base/firmware_loader/main.c > +++ b/drivers/base/firmware_loader/main.c > @@ -597,7 +597,7 @@ _firmware_request(const struct firmware **firmware_p, const char *name, > } > > /** > - * firmware_request: - send firmware request and wait for it > + * firmware_request() - send firmware request and wait for it > * @firmware_p: pointer to firmware image > * @name: name of firmware file > * @device: device for which firmware is being loaded > @@ -632,7 +632,7 @@ firmware_request(const struct firmware **firmware_p, const char *name, > EXPORT_SYMBOL(firmware_request); > > /** > - * firmware_request_direct: - load firmware directly without usermode helper > + * firmware_request_direct() - load firmware directly without usermode helper > * @firmware_p: pointer to firmware image > * @name: name of firmware file > * @device: device for which firmware is being loaded > @@ -657,7 +657,7 @@ int firmware_request_direct(const struct firmware **firmware_p, > EXPORT_SYMBOL_GPL(firmware_request_direct); > > /** > - * firmware_request_cache: - cache firmware for suspend so resume can use it > + * firmware_request_cache() - cache firmware for suspend so resume can use it > * @name: name of firmware file > * @device: device for which firmware should be cached for > * > @@ -681,7 +681,7 @@ int firmware_request_cache(struct device *device, const char *name) > EXPORT_SYMBOL_GPL(firmware_request_cache); > > /** > - * firmware_request_into_buf - load firmware into a previously allocated buffer > + * firmware_request_into_buf() - load firmware into a previously allocated buffer > * @firmware_p: pointer to firmware image > * @name: name of firmware file > * @device: device for which firmware is being loaded and DMA region allocated > @@ -713,7 +713,7 @@ firmware_request_into_buf(const struct firmware **firmware_p, const char *name, > EXPORT_SYMBOL(firmware_request_into_buf); > > /** > - * firmware_release: - release the resource associated with a firmware image > + * firmware_release() - release the resource associated with a firmware image > * @fw: firmware resource to release > **/ > void firmware_release(const struct firmware *fw) > @@ -755,7 +755,7 @@ static void firmware_request_work_func(struct work_struct *work) > } > > /** > - * firmware_request_nowait - asynchronous version of firmware_request > + * firmware_request_nowait() - asynchronous version of firmware_request > * @module: module requesting the firmware > * @uevent: sends uevent to copy the firmware image if this flag > * is non-zero else the firmware copy must be done manually. > @@ -824,7 +824,7 @@ EXPORT_SYMBOL(firmware_request_nowait); > static ASYNC_DOMAIN_EXCLUSIVE(fw_cache_domain); > > /** > - * cache_firmware - cache one firmware image in kernel memory space > + * cache_firmware() - cache one firmware image in kernel memory space > * @fw_name: the firmware image name > * > * Cache firmware in kernel memory so that drivers can use it when > @@ -866,7 +866,7 @@ static struct fw_priv *lookup_fw_priv(const char *fw_name) > } > > /** > - * uncache_firmware - remove one cached firmware image > + * uncache_firmware() - remove one cached firmware image > * @fw_name: the firmware image name > * > * Uncache one firmware image which has been cached successfully > @@ -1042,7 +1042,7 @@ static void __device_uncache_fw_images(void) > } > > /** > - * device_cache_fw_images - cache devices' firmware > + * device_cache_fw_images() - cache devices' firmware > * > * If one device called firmware_request or its nowait version > * successfully before, the firmware names are recored into the > @@ -1075,7 +1075,7 @@ static void device_cache_fw_images(void) > } > > /** > - * device_uncache_fw_images - uncache devices' firmware > + * device_uncache_fw_images() - uncache devices' firmware > * > * uncache all firmwares which have been cached successfully > * by device_uncache_fw_images earlier > @@ -1092,7 +1092,7 @@ static void device_uncache_fw_images_work(struct work_struct *work) > } > > /** > - * device_uncache_fw_images_delay - uncache devices firmwares > + * device_uncache_fw_images_delay() - uncache devices firmwares > * @delay: number of milliseconds to delay uncache device firmwares > * > * uncache all devices's firmwares which has been cached successfully > -- > 2.14.1 > > -- Do not panic