On Tue, 2012-07-03 at 15:24 +, KY Srinivasan wrote:
>
> > -Original Message-
> > From: Ben Hutchings [mailto:b...@decadent.org.uk]
> > Sent: Monday, July 02, 2012 3:57 PM
> > To: KY Srinivasan
> > Cc: Olaf Hering; Greg KH; a...@canonical.com; devel@linuxdriverproject.org;
> > virtualiz
On Fri, 20 Jul 2012 17:03:59 +0200, richard -rw- weinberger said:
> On Fri, Jul 20, 2012 at 4:00 PM, KY Srinivasan wrote:
> > Thank you for your interest in fixing this problem. When we decide to
> > change this
> > ID, we will conform to the MSFT guidelines on constructing this guest ID.
> >
>
>
Now checkpatch clean.
$ find drivers/staging/vme -name "*.[ch]"|xargs ./scripts/checkpatch.pl \
-f --terse --nosummary|cut -f3- -d":"|sort |uniq -c|sort -n
1 ERROR: trailing whitespace
2 WARNING: Prefer pr_debug(... to printk(KERN_DEBUG, ...
5 WARNING: Prefer pr_info(... to pr
the code under _init and _exit does platform_driver_register and
platform_driver_unregister respectively only,
so its better to use the module_platform_driver than just replicating
the module_platform_driver's implementation
Signed-off-by: Devendra Naga
---
drivers/staging/tidspbridge/rmgr/drv
the code which under _init and _exit does only the platform_driver_register
and platform_driver_unregister, and nothing else,
so its better to use the module_platform_driver macro rather duplicating
its implementation
Signed-off-by: Devendra Naga
---
drivers/staging/iio/iio_hwmon.c | 12 +
we do a check of firmware pointer against the NULL value before we give
it to the release_firmware.
as release_firmware is actually having a check against NULL, its not needed
here.
Signed-off-by: Devendra Naga
---
drivers/staging/csr/firmware.c |4 +---
1 file changed, 1 insertion(+), 3 de
the following coding style problems were fixed:
1. move function return type from top to the starting of the function
2. remove all the spaces at the start of a line and used tabs
Signed-off-by: Devendra Naga
---
drivers/staging/csr/sdio_mmc.c | 43 +++-
1
the following warnings were fixed:
1. removed all spaces before the start of a line and used tabs
2. removed the braces around the if as it encloses only a single statement
Signed-off-by: Devendra Naga
---
drivers/staging/csr/csr_time.c | 20 +---
1 file changed, 9 insertions(+
