[PATCH] Staging: android: ion: fixed a kzalloc coding style issue.

2016-05-26 Thread Shubham Bansal
Fixed a coding style issue. Issue reported by checkpatch.pl. Signed-off-by: Shubham Bansal --- drivers/staging/android/ion/ion.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/android/ion/ion.c b/drivers/staging/android/ion/ion.c index 8536567..2217ccb 10064

Re: [RFC][PATCH 1/3] staging: ion: Move away from the DMA APIs for cache flushing

2016-05-26 Thread Liviu Dudau
On Wed, May 25, 2016 at 12:48:02PM -0700, Laura Abbott wrote: > > Ion is currently using the DMA APIs in non-compliant ways for cache > maintaince. The issue is Ion needs to do cache operations outside of > the regular DMA model. The Ion model matches more closely with the > DRM model which calls

Re: [RFC][PATCH 1/3] staging: ion: Move away from the DMA APIs for cache flushing

2016-05-26 Thread Russell King - ARM Linux
On Thu, May 26, 2016 at 10:58:35AM +0100, Liviu Dudau wrote: > On Wed, May 25, 2016 at 12:48:02PM -0700, Laura Abbott wrote: > > > > Ion is currently using the DMA APIs in non-compliant ways for cache > > maintaince. The issue is Ion needs to do cache operations outside of > > the regular DMA mode

Re: [RFC][PATCH 1/3] staging: ion: Move away from the DMA APIs for cache flushing

2016-05-26 Thread Christoph Hellwig
On Thu, May 26, 2016 at 11:59:22AM +0100, Russell King - ARM Linux wrote: > I _really_ hate seeing architecture internal functions being abused in > drivers - architecture internal functions are there to implement the > official kernel APIs and are not for drivers to poke about with. Exactly - if

Re: [PATCH 0/3] fix issues in prism2usb.c.

2016-05-26 Thread Luis de Bethencourt
On 26/05/16 04:22, Kevin McKinney wrote: > This patchset will address issues reported by checkpatch.pl tool > for file prism2usb.c. > > Kevin McKinney (3): > Staging: wlan-ng: fix alignment not matching. > Staging: wlan-ng: fix unnecessary parentheses. > Staging: wlan-ng: fix comparison to N

Re: [PATCH 1/2] staging: dgnc: remove redundant NULL check for brd

2016-05-26 Thread Luis de Bethencourt
On 26/05/16 05:56, DaeSeok Youn wrote: > 2016-05-26 6:48 GMT+09:00 Luis de Bethencourt : >> On 20/05/16 10:51, Daeseok Youn wrote: >>> the "brd" value cannot be NULL in dgnc_finalize_board_init(). >>> Because "brd" as a parameter of this function was already >>> checked for NULL. >>> >>> Signed-off

Re: [PATCH 0/3] fix issues in prism2usb.c.

2016-05-26 Thread Kevin
On Thu, May 26, 2016 at 01:24:28PM +0100, Luis de Bethencourt wrote: > On 26/05/16 04:22, Kevin McKinney wrote: > > This patchset will address issues reported by checkpatch.pl tool > > for file prism2usb.c. > > > > Kevin McKinney (3): > > Staging: wlan-ng: fix alignment not matching. > > Stagi

Re: [PATCH] Staging: android: ion: fixed a kzalloc coding style issue.

2016-05-26 Thread Luis de Bethencourt
On 26/05/16 09:01, Shubham Bansal wrote: > Fixed a coding style issue. Issue reported by checkpatch.pl. > > Signed-off-by: Shubham Bansal > --- > drivers/staging/android/ion/ion.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/staging/android/ion/ion.c > b/dri

Re: [PATCH 0/3] fix issues in prism2usb.c.

2016-05-26 Thread Luis de Bethencourt
On 26/05/16 13:32, Kevin wrote: > On Thu, May 26, 2016 at 01:24:28PM +0100, Luis de Bethencourt wrote: >> On 26/05/16 04:22, Kevin McKinney wrote: >>> This patchset will address issues reported by checkpatch.pl tool >>> for file prism2usb.c. >>> >>> Kevin McKinney (3): >>> Staging: wlan-ng: fix a

How should I use kernel-defined i2c structs in this driver

2016-05-26 Thread Andrey Utkin
Could anybody please give a hint - which kernel-defined i2c objects, and how many of them, I need to define and use to substitute these driver-defined functions i2c_read(), i2c_write() ? https://github.com/bluecherrydvr/linux/blob/release/tw5864/1.16/drivers/media/pci/tw5864/tw5864-config.c In a wo

How are you today??

2016-05-26 Thread Dr. Lena R. Finnanger
Hello, I need your assistance, My name is Lena Reenaas Finnanger, I am originally from Sweden but I am presently taking treatment due to my illness, I have been diagnosed with Oesophageal Cancer. It has defiled all forms of medical treatment and right now I have only about a few months to live

Response

2016-05-26 Thread Mr Daniel H.William
Hello ! accept my sincere apologies for my mode of contacting you out of the blues like this but something very urgent and important has come to my notice and I strongly believe it’s important we seek your consent for the mutual interest of all. I am Daniel H.William Auditor General of HSBC

[PATCH] fbtft: limit transfer length by spi device limit

2016-05-26 Thread Michal Suchanek
Some SPI controllers can transfer only small piece of data at a time. Since SPI core gained a function to get the maximum transfer length use it. Signed-off-by: Michal Suchanek --- Tested on sunxi spi with DMA enabled and disabled. Makes a visible speed difference and display works in either ca

[PATCH] Staging: comedi: ni_atmio: fixed spacing and comment style issues

2016-05-26 Thread Steven Laabs
Fixed coding style issues. Signed-off-by: Steven Laabs --- drivers/staging/comedi/drivers/ni_atmio.c | 180 +++--- 1 file changed, 91 insertions(+), 89 deletions(-) diff --git a/drivers/staging/comedi/drivers/ni_atmio.c b/drivers/staging/comedi/drivers/ni_atmio.c index

Re: How should I use kernel-defined i2c structs in this driver

2016-05-26 Thread Antti Palosaari
On 05/26/2016 04:59 PM, Andrey Utkin wrote: Could anybody please give a hint - which kernel-defined i2c objects, and how many of them, I need to define and use to substitute these driver-defined functions i2c_read(), i2c_write() ? https://github.com/bluecherrydvr/linux/blob/release/tw5864/1.16/dr

Re: [PATCH 1/2] staging: dgnc: remove redundant NULL check for brd

2016-05-26 Thread DaeSeok Youn
2016-05-26 21:29 GMT+09:00 Luis de Bethencourt : > On 26/05/16 05:56, DaeSeok Youn wrote: >> 2016-05-26 6:48 GMT+09:00 Luis de Bethencourt : >>> On 20/05/16 10:51, Daeseok Youn wrote: the "brd" value cannot be NULL in dgnc_finalize_board_init(). Because "brd" as a parameter of this functi

[PATCH 1/2 v2] staging: dgnc: remove redundant NULL check for brd

2016-05-26 Thread Daeseok Youn
the "brd" value cannot be NULL in dgnc_finalize_board_init(). Because "brd" as a parameter of this function was already checked for NULL. the dgnc_finalize_board_init() as a static function was called only from dgnc_found_board() function and brd->magic value was assigned once in dgnc_found_board(

[PATCH 2/2 V2] staging: dgnc: remove redundant null check in

2016-05-26 Thread Daeseok Youn
the "brd" was already checked for NULL before calling dgnc_do_remap(). the dgnc_do_remap() function was called only from the dgnc_found_board() and the DGNC_BOARD_MAGIC value was assigned to "brd->magic" in dgcn_found_board(). So it doesn't need to check about magic value. Signed-off-by: Daeseok

[PATCH 1/7] staging: dgnc: remove useless variable 'intr_count'

2016-05-26 Thread Daeseok Youn
The 'intr_count' variable was used only for increasing. So the 'intr_count' variable is not useful for this driver. Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_cls.c| 2 -- drivers/staging/dgnc/dgnc_driver.h | 1 - drivers/staging/dgnc/dgnc_neo.c| 2 -- 3 files changed, 5 d

[PATCH 2/7] staging: dgnc: remove unused variable 'intr_modem'

2016-05-26 Thread Daeseok Youn
The 'intr_modem' variable was used only for increasing. So the 'intr_modem' variable is not useful for this driver. Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_cls.c| 1 - drivers/staging/dgnc/dgnc_driver.h | 1 - drivers/staging/dgnc/dgnc_neo.c| 1 - 3 files changed, 3 del

[PATCH 4/7] staging: dgnc: remove useless variable 'intr_rx'

2016-05-26 Thread Daeseok Youn
The 'intr_rx' variable was used only for increasing. So the 'intr_rx' variable is not useful for this driver. Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_cls.c| 1 - drivers/staging/dgnc/dgnc_driver.h | 1 - drivers/staging/dgnc/dgnc_neo.c| 2 -- 3 files changed, 4 deletion

[PATCH 3/7] staging: dgnc: remove useless variable 'intr_tx'

2016-05-26 Thread Daeseok Youn
The 'intr_tx' variable was used only for increasing. So the 'intr_tx' variable is not useful for this driver. Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_cls.c| 1 - drivers/staging/dgnc/dgnc_driver.h | 1 - drivers/staging/dgnc/dgnc_neo.c| 3 --- 3 files changed, 5 deletio

[PATCH 5/7] staging: dgnc: remove useless variable 'ch_intr_modem'

2016-05-26 Thread Daeseok Youn
The 'ch_intr_modem' variable was used only for increasing. So the 'ch_intr_modem' variable is not useful for this driver. Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_cls.c| 9 - drivers/staging/dgnc/dgnc_driver.h | 1 - drivers/staging/dgnc/dgnc_neo.c| 1 - 3 files

[PATCH 7/7] staging: dgnc: remove useless variable 'ch_intr_rx'

2016-05-26 Thread Daeseok Youn
The 'ch_intr_rx' variable was used only for increasing. So the 'ch_intr_rx' variable is not useful for this driver. Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_cls.c| 1 - drivers/staging/dgnc/dgnc_driver.h | 2 -- drivers/staging/dgnc/dgnc_neo.c| 2 -- 3 files changed, 5 d

[PATCH 6/7] staging: dgnc: remove useless variable 'ch_intr_tx'

2016-05-26 Thread Daeseok Youn
The 'ch_intr_tx' variable was used only for increasing. So the 'ch_intr_tx' variable is not useful for this driver. Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_cls.c| 1 - drivers/staging/dgnc/dgnc_driver.h | 1 - drivers/staging/dgnc/dgnc_neo.c| 3 --- 3 files changed, 5 d