Hi,
2014-10-13 12:25 GMT+09:00 Greg KH :
> On Mon, Oct 13, 2014 at 11:34:25AM +0900, Daeseok Youn wrote:
>> Re-arrange the functions for removing forward declarations.
>>
>> Signed-off-by: Daeseok Youn
>> ---
>> This patch has too many changes for re-arranging the functions.
>> So I wonder that I
This fixes space related ERROR reports by checkpatch.pl
Generated by $ git ls-files "drivers/staging/rtl8188eu/*.[ch]" | \
xargs ./scripts/checkpatch.pl -f --fix-inplace --strict --types=SPACING
Already checked by text comparasion
$git diff -w
and binary comparasion of r8188eu.ko
$objdiff diff
On 12/10/14 13:23, Rickard Strandqvist wrote:
Changed from using strncat to strlcat to simplify the code
Signed-off-by: Rickard Strandqvist
---
drivers/staging/comedi/drivers/comedi_bond.c |6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/comedi/dri
On 10/10/14 20:32, Bernd Porr wrote:
I've updated my contact details of the driver. I've also tested it
thoroughly and it works perfectly. I've changed the status to stable.
Signed-off-by: Bernd Porr
---
drivers/staging/comedi/drivers/usbduxsigma.c | 8
1 file changed, 4 insertions(
On 10/10/14 20:34, Bernd Porr wrote:
I've updated my contact details and removed obsolete comments.
Signed-off-by: Bernd Porr
---
drivers/staging/comedi/drivers/usbdux.c | 59 -
1 file changed, 28 insertions(+), 31 deletions(-)
Reviewed-by: Ian Abbott
--
-
On 10/10/14 20:35, Bernd Porr wrote:
Updated the range of years, e-mail and added driver desription as
usually done in comedi.
Signed-off-by: Bernd Porr
---
drivers/staging/comedi/drivers/usbduxfast.c | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
Reviewed-by: Ian Abbott
On Mon, 2014-10-13 at 16:47 +0800, Jia He wrote:
> This fixes space related ERROR reports by checkpatch.pl
> Generated by $ git ls-files "drivers/staging/rtl8188eu/*.[ch]" | \
> xargs ./scripts/checkpatch.pl -f --fix-inplace --strict --types=SPACING
checkpatch isn't perfect.
Always verify the o
The Comedi driver comment mentions a couple of manual configuration
options, but manual configuration via the COMEDI_DEVCONFIG ioctl and
comedi "attach" handler is no longer supported by this driver. Update
the comment appropriately.
Signed-off-by: Ian Abbott
---
drivers/staging/comedi/drivers/
Fix up the block comment style and add support for the PCIe-DAS1602/16.
This is the PCI-Express version of the PCIM-DAS1602/16 and is software
compatible with it, apart from a different PCI device ID.
1) staging: comedi: cb_pcimdas: use preferred block comment style
2) staging: comedi: cb_pcimdas:
Use the preferred block comment style for the copyright and Comedi
driver description header comments.
Signed-off-by: Ian Abbott
---
drivers/staging/comedi/drivers/cb_pcimdas.c | 71 +++--
1 file changed, 36 insertions(+), 35 deletions(-)
diff --git a/drivers/staging/com
The Measurement Computing PCIe-DAS1602/16 is a PCI-Express version of
the PCIM-DAS1602/16, but has a different PCI device ID. Add support
for this device and update the driver description, Kconfig description
and MODULE_DESCRIPTION.
Thanks to Christoph Langbein for supplying the PCI device ID.
S
On 10/12/14 23:55, Rickard Strandqvist wrote:
> Changed from using strncat with strlcat to avoid overwrite the max size.
> Take the opportunity to change a snprint to strlcpy.
>
> Signed-off-by: Rickard Strandqvist
> ---
> drivers/staging/lustre/lustre/ptlrpc/sec.c |5 ++---
> 1 file changed
On Mon, 2014-10-13 at 17:01 +0900, DaeSeok Youn wrote:
> Hi,
>
> 2014-10-13 12:25 GMT+09:00 Greg KH :
> > On Mon, Oct 13, 2014 at 11:34:25AM +0900, Daeseok Youn wrote:
> >> Re-arrange the functions for removing forward declarations.
> >>
> >> Signed-off-by: Daeseok Youn
> >> ---
> >> This patch h
Introduce COMEDI_CB_ERROR_MASK and COMEDI_CB_CANCEL_MASK to clarify the
async->events that indicate errors and cancel an async command.
Use the new defines to tidy up and clarify the code.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/comedi
The COMEDI_CB_* defines are the comedi_async "events" that the drivers set
to let the core detect the state of running async commands. These "events"
are only relevant to the kernel modules and should not be exposed to
userspace in the comedi.h user API header.
Move the defines to comedidev.h to a
From: Devendra Naga
The class_create function can return a pointer with an error code
stuffed inside it if it fails. Handled it to make sure
device_register doesn't get an invalid pointer.
Also removed assignment of rc to 0 in the beginning of function
because it gets changed after the call to r
The async event defines, COMEDI_CB_*, are only used in the kernel. They should
not be exposed to userspace in the comedi.h uapi header. Move the defines to
the comedidev.h kernel header and tidy up the documentation.
Also, introduce some mask defines to clarify the events that indicate error
condi
2014-10-13 11:14 GMT+02:00 Ian Abbott :
> On 12/10/14 13:23, Rickard Strandqvist wrote:
>>
>> Changed from using strncat to strlcat to simplify the code
>>
>> Signed-off-by: Rickard Strandqvist
>>
>> ---
>> drivers/staging/comedi/drivers/comedi_bond.c |6 ++
>> 1 file changed, 2 inserti
The idea was to reduce the length of the line (to 80 characters) by renaming
this function.
But, now reviewing the result I can see that the name of the function has lost
meaning.
Please ignore this patch.
-Erich
12.10.2014, 21:28, "Greg KH" :
> On Sun, Oct 12, 2014 at 09:29:32PM -0500, Eric
Hi,
2014-10-13 23:56 GMT+09:00 Joe Perches :
> On Mon, 2014-10-13 at 17:01 +0900, DaeSeok Youn wrote:
>> Hi,
>>
>> 2014-10-13 12:25 GMT+09:00 Greg KH :
>> > On Mon, Oct 13, 2014 at 11:34:25AM +0900, Daeseok Youn wrote:
>> >> Re-arrange the functions for removing forward declarations.
>> >>
>> >> S
On Mon, Oct 13, 2014 at 07:56:38AM -0700, Joe Perches wrote:
> On Mon, 2014-10-13 at 17:01 +0900, DaeSeok Youn wrote:
> > Hi,
> >
> > 2014-10-13 12:25 GMT+09:00 Greg KH :
> > > On Mon, Oct 13, 2014 at 11:34:25AM +0900, Daeseok Youn wrote:
> > >> Re-arrange the functions for removing forward declar
On Tue, 2014-10-14 at 04:04 +0200, Greg KH wrote:
> On Mon, Oct 13, 2014 at 07:56:38AM -0700, Joe Perches wrote:
> > On Mon, 2014-10-13 at 17:01 +0900, DaeSeok Youn wrote:
> > > 2014-10-13 12:25 GMT+09:00 Greg KH :
> > > > On Mon, Oct 13, 2014 at 11:34:25AM +0900, Daeseok Youn wrote:
> > > >> Re-ar
This fixes space related ERROR reports by checkpatch.pl
Generated by $ git ls-files "drivers/staging/rtl8188eu/*.[ch]" | \
xargs ./scripts/checkpatch.pl -f --fix-inplace --strict --types=SPACING
Already checked by text comparasion
$git diff -w
and binary comparasion of r8188eu.ko
$objdiff diff
From: Guenter Roeck
> arch/m32r/kernel/process.c | 8
Acked-by: Hirokazu Takata
Thank you.
-- takata
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Sorry,seems git ate my one line words after mail subject:
>From v1: suggested by j...@perches.com, fixed some points not checked
or not correctly checked by checkpatch.pl
On Tue, 14 Oct 2014 11:28:32 +0800 from hejia...@gmail.com wrote:
> This fixes space related ERROR reports by checkpatch.pl
>
25 matches
Mail list logo
