On Thu, May 29, 2014 at 11:03 PM, Paul Bolle wrote:
> On Thu, 2014-05-29 at 22:53 +0900, Magnus Damm wrote:
>> --- 0011/drivers/staging/board/Kconfig
>> +++ work/drivers/staging/board/Kconfig2014-05-29 22:32:39.0 +0900
>> @@ -1,5 +1,6 @@
>> config STAGING_BOARD
>> boolean "Stagin
On Wed, May 28, 2014 at 02:30:27PM -0700, Greg KH wrote:
> On Wed, May 28, 2014 at 06:52:51PM +1000, Scott Weir wrote:
> > This patch corrects coding style issue:
> > WARNING: Missing a black line after declarations
> >
> > Signed-off-by: Scott Weir
> > ---
> > drivers/staging/android/sync_debug
This board supports an interrupt that can be generated by an AND/OR
combination of 16 of the input channels.
Create a separate subdevice to handle this interrupt.
Signed-off-by: Chase Southwood
Cc: Ian Abbott
Cc: H Hartley Sweeten
---
drivers/staging/comedi/drivers/addi_apci_1564.c | 16 +
The board supported by this driver can generate an interrupt based
on the state of input channels 0-15.
The apci1564_di_config() function is used to configure which
inputs are used to generate the interrupt. Currently this function
is broken since it does not follow the comedi API for insn_config
Move the function apci1564_interrupt() from hwdrv_apci1564.c to
addi_apci_1564.c. On moving, for now just strip out all of the
code for interrupts that the driver does not yet support at this
time.
Rename the variable ui_InterruptStatus_1564 to ctrl, and change the return
from IRQ_RETVAL(1) to IR
The addi-data drivers use send_sig() to let the user know when an
interrupt has occurred. The "standard" way to do this in the comedi
subsystem is to have a subdevice that supports asynchronous commands
and use comedi_event() to signal the user.
Remove the send_sig() usage in this driver.
Signed-
The addi_private struct defined in addi-data/addi_common.h is very bloated
and contains many fields which addi_apci_1564 does not require. In the
interest of eventually removing this driver's dependency on
addi_common.h, we can create a private data struct specifically for
addi_apci_1564 containin
This member of the private data struct is only set at one location in the
entire driver, and then never even used for anything. Let's just remove
its use.
Signed-off-by: Chase Southwood
Cc: Ian Abbott
Cc: H Hartley Sweeeten
---
drivers/staging/comedi/drivers/addi-data/hwdrv_apci1564.c | 5 ---
This patchset introduces a new private data struct for this driver, adds
all of the code required to support Change-of-State interrupts for the
digital input subsystem, and finally focuses and fixes
apci1564_interrupt() to service this type of interrupt correctly.
Chase Southwood (6):
staging: c
Hi Greg,
On Sat, May 24, 2014 at 5:23 PM, Chase Southwood
wrote:
> This patchset adds the required subdevice for supporting DI COS interrupts,
> as well as introducing a driver-specific private data struct that will
> make the COS interrupt operations much more straightforward and clean.
>
> Chas
Remove some unnecessary parenthesis as recommended by checkpatch.pl.
Signed-off-by: Sam Bobroff
---
drivers/staging/rts5139/rts51x_scsi.c |8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/rts5139/rts51x_scsi.c
b/drivers/staging/rts5139/rts51x_scsi.c
Now that the pages returned from the pool are compound pages, we do not
need to pass the order information to free_buffer_page().
Signed-off-by: Heesub Shin
Reviewed-by: Mitchel Humpherys
Tested-by: John Stultz
---
drivers/staging/android/ion/ion_system_heap.c | 9 -
1 file changed, 4
ION system heap creates a temporary list of pages to build
scatter/gather table, introducing an internal data type, page_info. Now
that the order field has been removed from it, we do not need to depend
on such data type anymore.
Signed-off-by: Heesub Shin
Reviewed-by: Mitchel Humpherys
Tested-b
ION system heap uses an internal data structure, struct page_info, for
tracking down the meta information of the pages allocated from the pool.
Now that the pool returns compound pages, we don't need to store page
order in struct page_info.
Signed-off-by: Heesub Shin
Reviewed-by: Mitchel Humphery
struct ion_system_heap has an array for storing pointers to page pools
and it is allocated separately from the containing structure. There is
no point in allocating those two small objects individually, bothering
slab allocator. Using a variable length array simplifies code lines and
reduces overhe
On 05/30/2014 05:42 AM, Greg Kroah-Hartman wrote:
On Wed, May 28, 2014 at 03:52:52PM +0900, Heesub Shin wrote:
For aesthetics and readability, rename goto labels, remove
useless code lines, and clarify function return type.
Signed-off-by: Heesub Shin
Reviewed-by: Mitchel Humpherys
Tested-by:
Hi abdoulaye,
On Fri, 30 May 2014 01:16:22 +0200 abdoulaye berthe wrote:
>
> The aim of this patch is to make gpiochip_remove() behavior consistent,
> especially when issuing a remove request while the chipio chip is
> still requested. A patch has been submitted to change the return value of
> gp
On Thu, May 29, 2014 at 10:27:30PM +0900, Magnus Damm wrote:
> Hi Dan,
>
> On Thu, May 29, 2014 at 10:20 PM, Dan Carpenter
> wrote:
> > On Thu, May 29, 2014 at 10:17:32PM +0900, Magnus Damm wrote:
> >> --- /dev/null
> >> +++ work/drivers/staging/board/Kconfig2014-05-29
> >> 21:40:41.
On Thu, May 29, 2014 at 11:54:52PM +0200, abdoulaye berthe wrote:
> Signed-off-by: abdoulaye berthe
Why?
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
On 05/29/2014 02:54 PM, abdoulaye berthe wrote:
Did you forget a changelog explaining why this is either needed, or even
a good idea? I joined the conversation late and don't know why you are
doing this.
Thanks,
David Daney
Signed-off-by: abdoulaye berthe
---
arch/arm/common/scoop.c
On Fri, May 30, 2014 at 12:41:15AM +0300, Dan Carpenter wrote:
> On Thu, May 29, 2014 at 02:00:51PM -0700, Greg KH wrote:
> > On Thu, May 29, 2014 at 01:43:01PM -0700, Chaitanya Hazarey wrote:
> > > I have 1/5 through to 5/5. Will resend rightaway with slightly changed
> > > subjects.
> >
> > What
Ok will be quite for till the last ones get accepted.
Sorry for the bother !
Chaitanya
On Thu, May 29, 2014 at 2:41 PM, Dan Carpenter wrote:
> On Thu, May 29, 2014 at 02:00:51PM -0700, Greg KH wrote:
>> On Thu, May 29, 2014 at 01:43:01PM -0700, Chaitanya Hazarey wrote:
>> > I have 1/5 through t
I have reviewed this patchset.
Reviewed-by: Dan Carpenter
Btw, I wish you would fold some of these patches together next time.
For example, patches 3-6 are very closely related and they should have
just been one thing.
Or the times where the change log says, "I am going to delete function
frob_
On Thu, May 29, 2014 at 02:00:51PM -0700, Greg KH wrote:
> On Thu, May 29, 2014 at 01:43:01PM -0700, Chaitanya Hazarey wrote:
> > I have 1/5 through to 5/5. Will resend rightaway with slightly changed
> > subjects.
>
> What does that have to do with patch 2/2 here? You sent a 1/2 patch, so
> obvi
A: No.
Q: Should I include quotations after my reply?
http://daringfireball.net/2007/07/on_top
On Thu, May 29, 2014 at 01:43:01PM -0700, Chaitanya Hazarey wrote:
> I have 1/5 through to 5/5. Will resend rightaway with slightly changed
> subjects.
What does that have to do with patch 2/2 here?
On Thu, May 29, 2014 at 09:40:39AM +0300, Dan Carpenter wrote:
> On Thu, May 29, 2014 at 09:17:09AM +0900, DaeSeok Youn wrote:
> > Hi, Dan.
> >
> > 2014-05-28 19:11 GMT+09:00 Dan Carpenter :
> > > On Wed, May 28, 2014 at 06:29:38PM +0900, DaeSeok Youn wrote:
> > >> > In your patch it has:
> > >> >
To address the error -
ERROR: do not use C99 // comments
Removed all C99 comments.
Signed-off-by: Chaitanya Hazarey
---
drivers/staging/rtl8192u/r8192U_wx.c | 130 +--
1 file changed, 65 insertions(+), 65 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192
Removed dead code from the file.
Signed-off-by: Chaitanya Hazarey
---
drivers/staging/rtl8192u/r8192U_wx.c | 18 +-
1 file changed, 1 insertion(+), 17 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U_wx.c
b/drivers/staging/rtl8192u/r8192U_wx.c
index 09b1e81..ceb06d8 10
On Thu, May 29, 2014 at 09:43:27AM -0700, Chaitanya Hazarey wrote:
> Ok I will edit PATCH[5/5] and resend.
Sorry, I didn't mean that. It's a very minor thing and anyway you
didn't introduce it. I see you have already fixed this, but you could
have just fixed it up in a later patch.
I am always
Added a space around '|' to address:
ERROR: need consistent spacing around '|' (ctx:VxW)
Signed-off-by: Chaitanya Hazarey
---
drivers/staging/rtl8192u/r8192U_wx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8192u/r8192U_wx.c
b/drivers/staging/rtl8192u
Fixed a misplaced brace in the r8192_wx_set_scan_type function.
Signed-off-by: Chaitanya Hazarey
---
drivers/staging/rtl8192u/r8192U_wx.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8192u/r8192U_wx.c
b/drivers/staging/rtl8192u/r8192U_wx.c
index 1af7c
Removed dead code, commented out printks and DMESG.
Signed-off-by: Chaitanya Hazarey
---
drivers/staging/rtl8192u/r8192U_wx.c | 10 --
1 file changed, 10 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U_wx.c
b/drivers/staging/rtl8192u/r8192U_wx.c
index 675a12d..7dc2a96 100644
I have 1/5 through to 5/5. Will resend rightaway with slightly changed
subjects.
Chaitanya
On Thu, May 29, 2014 at 1:43 PM, Greg KH wrote:
> On Wed, May 28, 2014 at 04:23:47PM -0700, Chaitanya Hazarey wrote:
>> Fixed a misplaced brace in a function
>>
>> Signed-off-by: Chaitanya Hazarey
>> ---
On Wed, May 28, 2014 at 04:23:47PM -0700, Chaitanya Hazarey wrote:
> Fixed a misplaced brace in a function
>
> Signed-off-by: Chaitanya Hazarey
> ---
> drivers/staging/rtl8192u/r8192U_wx.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
Where is patch 2/2 ?
__
On Wed, May 28, 2014 at 03:52:52PM +0900, Heesub Shin wrote:
> For aesthetics and readability, rename goto labels, remove
> useless code lines, and clarify function return type.
>
> Signed-off-by: Heesub Shin
> Reviewed-by: Mitchel Humpherys
> Tested-by: John Stultz
Not all of these would appl
This macro is used to create the private data structure that is used by
the ni_atmio, ni_mio_cs, and ni_pcimio drivers. These drivers all include
the ni_mio_common.c source to provide most of the driver functionality.
The only driver specific information needed to convert the macro into a
proper s
The {read,write}[bwl] macros used to access the registers in the ni_atmio,
ni_mio_cs, and ni_pcimio drivers and the included ni_mio_common.c file all
rely on a local variable having a specific name. They also require some of
the ni_mio_common code to need a __maybe_unused tag on the devpriv local
v
Remove all the unsed members from the private data for the ni mio drivers.
The ao0p and ao1p members are only used in the ni_ao_reset() function and
the code is commented out. Remove them, and the commented out code, as well.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartma
The comedi drivers that use the ni_mio_common.c source share a private data
struct. This struct is defined in ni_stc.h as a macro so that the real driver
can add additional members to the struct. These additional members are not
actually needed since they are either unused or can be found when need
driver doesn't need usb_vendor_req.h.
Signed-off-by: navin patidar
---
drivers/staging/rtl8188eu/include/usb_osintf.h |1 -
drivers/staging/rtl8188eu/include/usb_vendor_req.h | 52
drivers/staging/rtl8188eu/os_dep/usb_intf.c|1 -
3 files changed, 54 del
Signed-off-by: navin patidar
---
drivers/staging/rtl8188eu/include/usb_osintf.h |3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/staging/rtl8188eu/include/usb_osintf.h
b/drivers/staging/rtl8188eu/include/usb_osintf.h
index 9de99ca..2dce3bf 100644
--- a/drivers/staging/rtl8188eu/i
value stored in variable last_tx_complete_time isn't being used
by driver.
Signed-off-by: navin patidar
---
drivers/staging/rtl8188eu/core/rtw_sreset.c |1 -
drivers/staging/rtl8188eu/include/rtw_sreset.h |1 -
drivers/staging/rtl8188eu/os_dep/usb_ops_linux.c |5 -
3 files
value stored in variable last_tx_time isn't being used
by driver.
Signed-off-by: navin patidar
---
drivers/staging/rtl8188eu/core/rtw_sreset.c |1 -
drivers/staging/rtl8188eu/include/rtw_sreset.h |1 -
drivers/staging/rtl8188eu/os_dep/usb_ops_linux.c |6 +-
3 files changed
Driver doesn't need Hal8188EReg.h file.
Signed-off-by: navin patidar
---
drivers/staging/rtl8188eu/include/Hal8188EReg.h | 46 ---
drivers/staging/rtl8188eu/include/odm_precomp.h |1 -
2 files changed, 47 deletions(-)
delete mode 100644 drivers/staging/rtl8188eu/includ
Remove rtl8188e_silentreset_for_specific_platform() and function pointer
->silentreset pointing to it.
Signed-off-by: navin patidar
---
drivers/staging/rtl8188eu/hal/rtl8188e_hal_init.c |1 -
drivers/staging/rtl8188eu/hal/rtl8188e_sreset.c |3 ---
drivers/staging/rtl8188eu/include/
rtw_hal_sreset_reset() is a wrapper to call
rtl8188e_silentreset_for_specific_platform(), which has empty defination.
Signed-off-by: navin patidar
---
drivers/staging/rtl8188eu/core/rtw_mlme.c |2 --
drivers/staging/rtl8188eu/hal/hal_intf.c |6 --
drivers/staging/rtl8188eu
Signed-off-by: navin patidar
---
drivers/staging/rtl8188eu/Makefile |1 -
drivers/staging/rtl8188eu/hal/rtl8188e_sreset.c| 25 -
drivers/staging/rtl8188eu/include/rtl8188e_hal.h |3 +--
.../staging/rtl8188eu/include/rtl8188e_sreset.h| 28 -
Remove rtl8188e_sreset_xmit_status_check() and function pointer
->sreset_xmit_status_check pointing to it.
Signed-off-by: navin patidar
---
drivers/staging/rtl8188eu/hal/rtl8188e_hal_init.c |1 -
drivers/staging/rtl8188eu/hal/rtl8188e_sreset.c| 35
drivers/staging
Remove rtl8188e_sreset_linked_status_check() and function pointer
->sreset_linked_status_check pointing to it.
Signed-off-by: navin patidar
---
drivers/staging/rtl8188eu/hal/rtl8188e_hal_init.c |1 -
drivers/staging/rtl8188eu/hal/rtl8188e_sreset.c | 17 -
drivers/stag
rtw_hal_sreset_xmit_status_check() is a wrapper to call
rtl8188e_sreset_xmit_status_check(), which calls
rtl8188e_silentreset_for_specific_platform() in case of transmission has
stopped to do a silent reset and restore the transmission but
rtl8188e_silentreset_for_specific_platform() has empty defi
rtw_hal_sreset_linked_status_check() is a wrapper to call
rtl8188e_sreset_linked_status_check(), which doesn't do anything
useful.
rtl8188e_sreset_linked_status_check() will also be removed later.
Signed-off-by: navin patidar
---
drivers/staging/rtl8188eu/core/rtw_mlme_ext.c |2 --
drivers/
Signed-off-by: navin patidar
---
drivers/staging/rtl8188eu/core/rtw_sreset.c|9 -
drivers/staging/rtl8188eu/include/rtw_sreset.h |1 -
2 files changed, 10 deletions(-)
diff --git a/drivers/staging/rtl8188eu/core/rtw_sreset.c
b/drivers/staging/rtl8188eu/core/rtw_sreset.c
inde
Signed-off-by: navin patidar
---
drivers/staging/rtl8188eu/hal/hal_intf.c |6 --
drivers/staging/rtl8188eu/include/hal_intf.h |1 -
2 files changed, 7 deletions(-)
diff --git a/drivers/staging/rtl8188eu/hal/hal_intf.c
b/drivers/staging/rtl8188eu/hal/hal_intf.c
index d0ac4a1..699
Signed-off-by: navin patidar
---
drivers/staging/rtl8188eu/hal/rtl8188e_hal_init.c |1 -
drivers/staging/rtl8188eu/include/hal_intf.h |1 -
2 files changed, 2 deletions(-)
diff --git a/drivers/staging/rtl8188eu/hal/rtl8188e_hal_init.c
b/drivers/staging/rtl8188eu/hal/rtl8188e_hal_in
We can replace rtw_hal_sreset_reset_value() with rtw_hal_sreset_init() because
both function call sreset_reset_value() and sreset_init_value() respectively at
the end of code path, and function defination of sreset_reset_value() and
sreset_init_value() is identical.
Signed-off-by: navin patidar
-
psrtpriv->silent_reset_inprogress is always false.
Signed-off-by: navin patidar
---
drivers/staging/rtl8188eu/core/rtw_sreset.c|4
drivers/staging/rtl8188eu/include/rtw_sreset.h |1 -
2 files changed, 5 deletions(-)
diff --git a/drivers/staging/rtl8188eu/core/rtw_sreset.c
b/dr
silentreset_mutex initialized but not being used.
Signed-off-by: navin patidar
---
drivers/staging/rtl8188eu/core/rtw_sreset.c|1 -
drivers/staging/rtl8188eu/include/rtw_sreset.h |1 -
2 files changed, 2 deletions(-)
diff --git a/drivers/staging/rtl8188eu/core/rtw_sreset.c
b/driver
Add an (*insn_config) to the counter subdevice to allow the user to select
the clock source for the counters using the INSN_CONFIG_SET_CLOCK_SRC
instruction. The current selection can be queried with the instruction
INSN_CONFIG_GET_CLOCK_SRC.
Also, handle the INSN_CONFIG_RESET instruction to reset
This addes support to comedi for a couple more Diamond Systems PC/104 boards.
The IR104-PBF board is a PC/104 module with 20 optoisolated inputs and 20 relay
outputs.
The ONYX-MM-DIO board is a PC/104 module with two 8255 chips providing 48
digital I/O channels.
H Hartley Sweeten (2):
staging
The Diamond Systems IR104-PBF board is a PC/104 module with 20
optoisolated inputs and 20 relay outputs.
This board can be supported by the pcl730 driver.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/Kconfig | 1 +
drivers/staging
The Diamond Systems ONYX-MM-DIO board is a PC/104 module with two 8255
chips providing 48 digital I/O channels.
This board can be supported by the pcl724 driver.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/Kconfig | 15 ---
Introduce a helper function to return the number of bytes that are
ready to read/write from/to the comedi_async buffer. The "write to"
use doesn't really make much sense but is handled for completeness.
Use the helper function in the comedi drivers that currently do the
calculation as part of the
The functions pci9111_trigger_source_set(), pci9111_pretrigger_set(),
and pci9111_autoscan_set() are all used to select the A/D trigger type.
They all do a read/mask/set/write of the A/D Trigger Mode Control register.
Simplify the code by removing these helper functions and combining all
the trigg
A couple more cleanup patches for the adl_pci9111 driver.
H Hartley Sweeten (3):
staging: comedi: adl_pci9111: remove PCI9111_DRIVER_NAME define
staging: comedi: adl_pci9111: remove PCI9111_HR_DEVICE_ID define
staging: comedi: adl_pci9111: simplify A/D trigger selection code
drivers/stagin
This define is only used in a comedi_error() message. The addition of
the driver name to the message is not necessary. Remove the define.
For aesthetics, convert the comedi_error() into a dev_dbg().
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/com
This define is only used in the pci_device_id table and doesn't add
any additional clarity to the code. Remove the define and just open
code the value.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/adl_pci9111.c | 4 +---
1 file chang
These defines don't add any additional clarity to the addi_data drivers.
Just remove them.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
.../staging/comedi/drivers/addi-data/addi_common.h | 7 --
.../comedi/drivers/addi-data/hwdrv_apci035.c | 9 +-
.../co
Many of the ADDI-DATA drivers have been separated from the "addi_common"
code and cleaned up. Because of this cleanup there are some unnecessary
members hanging around in the private data struct and a number of defines
that don't add any sigificant clarity to the remaining drivers in the
addi-data
These defines don't add any additional clarity to the addi_data drivers.
They are also only used in the addi_apci_3120 driver. Just remove them.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/addi-data/addi_common.h | 3 ---
drivers/st
These macros don't add any additional clarity to the addi_data drivers.
Just remove them.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/addi-data/addi_common.h | 3 ---
.../staging/comedi/drivers/addi-data/hwdrv_apci3120.c| 16
This member is set but never used. Just remove it.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/addi-data/addi_common.c | 10 +++---
drivers/staging/comedi/drivers/addi-data/addi_common.h | 1 -
drivers/staging/comedi/drivers/ad
These defines don't add any additional clarity to the addi_data drivers.
Just remove them.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/addi-data/addi_common.h | 5 -
drivers/staging/comedi/drivers/addi_apci_035.c | 2 +-
This define is not used. Remove it.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/addi-data/addi_common.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/comedi/drivers/addi-data/addi_common.h
b/drivers/staging/com
On Thursday, May 29, 2014 2:58 AM, Ian Abbott wrote:
> On 2014-05-29 00:26, H Hartley Sweeten wrote:
>> Remove all the forward declarations and do a bit of cleanup to this
>> file.
[snip]
>> drivers/staging/comedi/drivers/ni_mio_common.c | 4536
>>
>> 1 file changed,
On Thursday, May 29, 2014 4:16 AM, Ian Abbott wrote:
>
> Reformat the comment describing this comedi driver to use the usual
> block comment format.
>
> Also remove reference to digital I/O emulating an 8255, because it
> doesn't, and remove "DIO only" from the "Description:" line as it also
> supp
On Thursday, May 29, 2014 5:29 AM, Ian Abbott wrote:
> From: Fred Brooks
>
> Add support for switching the input range and the single-ended/
> differential input mode for the AI subdevice. We needed to clear the
> FIFO of data before the conversion to handle card mode switching
> glitches.
>
> [
Added a space around '|' to address:
ERROR: need consistent spacing around '|' (ctx:VxW)
Signed-off-by: Chaitanya Hazarey
---
drivers/staging/rtl8192u/r8192U_wx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8192u/r8192U_wx.c
b/drivers/staging/rtl8192u
Removed dead code, commented out printks and DMESG.
Signed-off-by: Chaitanya Hazarey
---
drivers/staging/rtl8192u/r8192U_wx.c | 10 --
1 file changed, 10 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U_wx.c
b/drivers/staging/rtl8192u/r8192U_wx.c
index 675a12d..7dc2a96 100644
Removed dead code from the file.
Signed-off-by: Chaitanya Hazarey
---
drivers/staging/rtl8192u/r8192U_wx.c | 18 +-
1 file changed, 1 insertion(+), 17 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U_wx.c
b/drivers/staging/rtl8192u/r8192U_wx.c
index 09b1e81..ceb06d8 10
Fixed a misplaced brace in the r8192_wx_set_scan_type function.
Signed-off-by: Chaitanya Hazarey
---
drivers/staging/rtl8192u/r8192U_wx.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8192u/r8192U_wx.c
b/drivers/staging/rtl8192u/r8192U_wx.c
index 1af7c
To address the error -
ERROR: do not use C99 // comments
Removed all C99 comments.
Signed-off-by: Chaitanya Hazarey
---
drivers/staging/rtl8192u/r8192U_wx.c | 130 +--
1 file changed, 65 insertions(+), 65 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192
Ok I will edit PATCH[5/5] and resend.
On Thu, May 29, 2014 at 2:54 AM, Dan Carpenter wrote:
> On Wed, May 28, 2014 at 09:49:21PM -0700, Chaitanya Hazarey wrote:
>> static iw_handler r8192_private_handler[] = {
>> - /* r8192_wx_set_monitor, */ /* SIOCIWFIRSTPRIV */
>> r8192_wx_set_crcmo
On Thu, May 29, 2014 at 04:01:27PM +0900, Daeseok Youn wrote:
> This reverts commit 0ade4a34fd439d62df46937e8f3e584eb0879579.
>
> This patch removes kfree for serial_ports in dgap_tty_register_ports()
> if the "brd->printer_ports" allocation fails and serial_ports may be
> freed by calling dgap_tt
On Thu, 2014-05-29 at 22:53 +0900, Magnus Damm wrote:
> --- 0011/drivers/staging/board/Kconfig
> +++ work/drivers/staging/board/Kconfig2014-05-29 22:32:39.0 +0900
> @@ -1,5 +1,6 @@
> config STAGING_BOARD
> boolean "Staging Board Support"
> +depends on OF && OF_ADDRESS
I have
On Thu, May 29, 2014 at 10:27 PM, Magnus Damm wrote:
> Hi Dan,
>
> On Thu, May 29, 2014 at 10:20 PM, Dan Carpenter
> wrote:
>> On Thu, May 29, 2014 at 10:17:32PM +0900, Magnus Damm wrote:
>>> --- /dev/null
>>> +++ work/drivers/staging/board/Kconfig2014-05-29 21:40:41.0
>>> +0900
Hi Dan,
On Thu, May 29, 2014 at 10:20 PM, Dan Carpenter
wrote:
> On Thu, May 29, 2014 at 10:17:32PM +0900, Magnus Damm wrote:
>> --- /dev/null
>> +++ work/drivers/staging/board/Kconfig2014-05-29 21:40:41.0
>> +0900
>> @@ -0,0 +1,7 @@
>> +config STAGING_BOARD
>> + boolean "Sta
On Thu, May 29, 2014 at 10:17:32PM +0900, Magnus Damm wrote:
> --- /dev/null
> +++ work/drivers/staging/board/Kconfig2014-05-29 21:40:41.0
> +0900
> @@ -0,0 +1,7 @@
> +config STAGING_BOARD
> + boolean "Staging Board Support"
> + help
> + Select to enable per-board sta
From: Magnus Damm
Add a TODO file for emxx_udc to show what is left to do.
Signed-off-by: Magnus Damm
---
Changes since V1:
- New patch
drivers/staging/emxx_udc/TODO |4
1 file changed, 4 insertions(+)
--- /dev/null
+++ work/drivers/staging/emxx_udc/TODO 2014-05-29 18:08:58.
From: Magnus Damm
Add staging board base support to allow continuous upstream
in-tree development and integration of platform devices.
Helps developers integrate devices as platform devices for
device drivers that only provide platform device bindings.
This in turn allows for incremental develop
From: Magnus Damm
Add staging board support for the KZM9D board and add
an emxx_udc platform device to allow in-tree continous
development of the driver on the KZM9D board.
When DT bindings are ready for the emxx_udc driver then
the platform device in the KZM9D staging board code can
easily be
From: Magnus Damm
Adjust the emxx_udc driver to make use of the standard
driver model to pass I/O memory and IRQ as resources
instead of hard coding those things in the driver.
Needs more work - the VBUS signal is yet not handled.
Signed-off-by: Magnus Damm
---
Changes since V1:
- None
dr
staging: Emma Mobile USB driver and KZM9D board code V2
[PATCH v2 01/05] staging: emxx_udc: Add Emma Mobile USB Gadget driver
[PATCH v2 02/05] staging: emxx_udc: I/O memory and IRQ resource support
[PATCH v2 03/05] staging: emxx_udc: Add TODO file
[PATCH v2 04/05] staging: board: Initial board sta
From: Fred Brooks
Add support for switching the input range and the single-ended/
differential input mode for the AI subdevice. We needed to clear the
FIFO of data before the conversion to handle card mode switching
glitches.
[ Minor whitespace fixes and driver comment reformatting. - Ian ]
S
On 2014/05/29 12:51 PM, Dan Carpenter wrote:
> These days, I don't barely look at comedi changes at all because you
> guys do such a great job. I just had a process/git comment on this one.
> Normally, these changelogs look like:
>
> From: Fred Brooks
>
> Add support for switching the input ran
These days, I don't barely look at comedi changes at all because you
guys do such a great job. I just had a process/git comment on this one.
Normally, these changelogs look like:
From: Fred Brooks
Add support for switching the input range and the
single-ended/differential input mode for the AI
Casting value returned by kzalloc is useless.
Signed-off-by: Sachin Kamat
---
drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 27 +
1 file changed, 10 insertions(+), 17 deletions(-)
diff --git a/drivers/staging/rtl8723au/core/rtw_mlme_ext.c
b/drivers/staging/rtl8723au
Casting value returned by kzalloc is useless.
Signed-off-by: Sachin Kamat
---
drivers/staging/rtl8723au/core/rtw_mlme.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8723au/core/rtw_mlme.c
b/drivers/staging/rtl8723au/core/rtw_mlme.c
index 7170258d2601
Casting value returned by kzalloc is useless.
Signed-off-by: Sachin Kamat
---
drivers/staging/bcm/Bcmchar.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/bcm/Bcmchar.c b/drivers/staging/bcm/Bcmchar.c
index 606d5f5e9216..c1e01f7d64ba 100644
--- a/drivers/s
Reformat the comment describing this comedi driver to use the usual
block comment format.
Also remove reference to digital I/O emulating an 8255, because it
doesn't, and remove "DIO only" from the "Description:" line as it also
supports analog inputs.
Signed-off-by: Ian Abbott
---
drivers/stagi
I received a patch from the original author of the driver, Fred Brooks,
to add support for switching the input range and the
single-ended/differential input mode for the AI subdevice.
I used the patch as-is apart from minor whitespace fixes and
reformatting the changes to the driver description co
1 - 100 of 111 matches
Mail list logo
