Re: [PATCH v2 02/17] Staging: quickstart: Prefix remaining functions names with quickstart_

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jan 11, 2012 at 11:22:37PM +0100, Szymon Janc wrote:
> Signed-off-by: Szymon Janc <szymon@xxxxxxxxxxx>
> ---
>  drivers/staging/quickstart/quickstart.c |   18 +++++++++---------
>  1 files changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/drivers/staging/quickstart/quickstart.c b/drivers/staging/quickstart/quickstart.c
> index d03b89d..adce56b 100644
> --- a/drivers/staging/quickstart/quickstart.c
> +++ b/drivers/staging/quickstart/quickstart.c
> @@ -92,19 +92,19 @@ static struct acpi_driver quickstart_acpi_driver = {
>  struct input_dev *quickstart_input;
>  
>  /* Platform driver structs */
> -static ssize_t buttons_show(struct device *dev,
> +static ssize_t quickstart_buttons_show(struct device *dev,
>  					struct device_attribute *attr,
>  					char *buf);

I'll take this, but you do know there is no reason for this, as these
are static functions.  I'd recommend keeping them as short as possible,
but hey, it's not my code...

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux