[PATCH 1/1] staging: hv: Fix Kconfig dependency of hv_blkvsc

2010-07-23 Thread Haiyang Zhang
From: Haiyang Zhang Subject: staging: hv: Fix Kconfig dependency of hv_blkvsc LBDAF is not available nor necessary on 64BIT kernel. This patch fixed the dependency for hv_blkvsc module on 64BIT kernel. Thanks vrataj2 [vrat...@comcast.net] for reporting this problem. Signed-off-by: Haiyang Zhang

[PATCH v2] Staging: dt3155: make local variables static

2010-07-23 Thread H Hartley Sweeten
The symbols dt3155_dev_open, unique_tag, and allocatorAddr are only used in the file dt3155_drv.c and should be static. Also, as pointed out by Joe Perches, static variables do not need initialization so remove them. While here, also move the definition for allocatorAddr so that all the variables a

Re: [PATCH] Staging: dt3155: make local variables static

2010-07-23 Thread Joe Perches
On Fri, 2010-07-23 at 11:45 -0700, H Hartley Sweeten wrote: > The symbols dt3155_dev_open, unique_tag, and allocatorAddr are only used in > the file dt3155_drv.c and should be static. > diff --git a/drivers/staging/dt3155/dt3155_drv.c > b/drivers/staging/dt3155/dt3155_drv.c > index 9e7b880..49924d

[PATCH] Staging: dt3155: make local functions static

2010-07-23 Thread H Hartley Sweeten
The functions adjust_4MB and allocate_buffers are only called locally in dt3155_isr.c and should be static. Signed-off-by: H Hartley Sweeten Cc: Greg Kroah-Hartman Cc: Scott Smedley --- diff --git a/drivers/staging/dt3155/dt3155_isr.c b/drivers/staging/dt3155/dt3155_isr.c index 549186f..1e91

[PATCH] Staging: dt3155: make local variables static

2010-07-23 Thread H Hartley Sweeten
The symbols dt3155_dev_open, unique_tag, and allocatorAddr are only used in the file dt3155_drv.c and should be static. Signed-off-by: H Hartley Sweeten Cc: Greg Kroah-Hartman Cc: Scott Smedley --- diff --git a/drivers/staging/dt3155/dt3155_drv.c b/drivers/staging/dt3155/dt3155_drv.c index 9

[PATCH] Staging: dt3155: make module_{init/exit} functions static

2010-07-23 Thread H Hartley Sweeten
The module_init() and module_exit() functions should be static and marked with __init and __exit. Signed-off-by: H Hartley Sweeten Cc: Greg Kroah-Hartman Cc: Scott Smedley --- diff --git a/drivers/staging/dt3155/dt3155_drv.c b/drivers/staging/dt3155/dt3155_drv.c index 9e7b880..7316996 100644

Re: [PATCH 0/3] drivers:staging:ti-st: patches

2010-07-23 Thread Randy Dunlap
On 07/22/10 21:56, Pavan Savoy wrote: > Randy, > > On Fri, Jul 23, 2010 at 1:23 AM, Randy Dunlap wrote: >> On Thu, 22 Jul 2010 05:32:04 -0500 pavan_sa...@ti.com wrote: >> >>> From: Pavan Savoy >>> >>> The following patches cleanup bit of a mess and also adds functionality to >>> protocol driver

Re: [PATCH] Staging: comedi: comedidev: Fixed a macro definition code style issue

2010-07-23 Thread Greg KH
On Fri, Jul 23, 2010 at 12:36:44PM +0200, Jiri Slaby wrote: > On 07/23/2010 11:17 AM, Michael Jensen wrote: > > On Fri, Jul 23, 2010 at 6:53 PM, Jiri Slaby wrote: > >> > > > >> Hmm, does this compile? > >> > > I did build the comedi directory (as per Greg's "Write and submit your > > first kernel

Re: [PATCH] Staging: comedi: comedidev: Fixed a macro definition code style issue

2010-07-23 Thread Jiri Slaby
On 07/23/2010 11:17 AM, Michael Jensen wrote: > On Fri, Jul 23, 2010 at 6:53 PM, Jiri Slaby wrote: >> > >> Hmm, does this compile? >> > I did build the comedi directory (as per Greg's "Write and submit your > first kernel patch" video), and didn't get any errors (once I'd done > the configuration

Re: [PATCH] Staging: comedi: comedidev: Fixed a macro definition code style issue

2010-07-23 Thread Michael Jensen
On Fri, Jul 23, 2010 at 6:53 PM, Jiri Slaby wrote: > > Hmm, does this compile? > > -- > js I did build the comedi directory (as per Greg's "Write and submit your first kernel patch" video), and didn't get any errors (once I'd done the configuration)... Did I miss something? [apologies, resendi

Re: [PATCH] Staging: comedi: comedidev: Fixed a macro definition code style issue

2010-07-23 Thread Jiri Slaby
On 07/23/2010 08:11 AM, Michael Jensen wrote: > Fixed a code style issue. > > Signed-off-by: Michael Jensen > --- > drivers/staging/comedi/comedidev.h |7 --- > 1 files changed, 4 insertions(+), 3 deletions(-) > > diff --git a/drivers/staging/comedi/comedidev.h > b/drivers/staging/come