On Fri, Sep 20, 2013 at 04:45:08PM +0200, Iker Pedrosa wrote:
> On Tue, 17 Sep 2013 12:43:26 +0300
> Dan Carpenter wrote:
>
> > On Tue, Sep 17, 2013 at 02:29:22PM +0530, Adil Mujeeb wrote:
> > > Hi,
> > >
> > >
> > > >- PowerData = (1 << 31) | (0 << 30) | (24 << 24) |
> > > BitReverse(w89
Need use 'ccflags-y' instead of EXTRA_CFLAGS, or compiling issue will
be occurred with "EXTRA_CFLAGS=-W".
Also can reference scripts/checkpatch.pl (1755..1766) to know about it.
Signed-off-by: Chen Gang
---
drivers/staging/dgap/Makefile |2 +-
1 files changed, 1 insertions(+), 1 deletions(
On Sat, Sep 21, 2013 at 07:12:36PM +0800, Chen Gang wrote:
> Need use 'ccflags-y' instead of EXTRA_CFLAGS, or compiling issue will
> be occurred with "EXTRA_CFLAGS=-W".
>
Just add it to drivers/staging/dgap/dgap_driver.c and delete it from the
Makefile.
regards,
dan carpenter
__
On 09/19/13 13:22, Juergen Beisert wrote:
> For battery driven systems it is a very bad idea to collect the touchscreen
> data within a kernel busy loop.
>
> This change uses the features of the hardware to delay and accumulate samples
> in
> hardware to avoid a high interrupt and CPU load.
>
>
On 09/21/2013 07:22 PM, Dan Carpenter wrote:
> On Sat, Sep 21, 2013 at 07:12:36PM +0800, Chen Gang wrote:
>> Need use 'ccflags-y' instead of EXTRA_CFLAGS, or compiling issue will
>> be occurred with "EXTRA_CFLAGS=-W".
>>
>
> Just add it to drivers/staging/dgap/dgap_driver.c and delete it from the
The device tree list has moved, so I've changed the cc.
Few comments inline. Basically I'd go for longer more
descriptive names when the abreviation isn't a really
well known common one.
Note I'm not all that familiar with device tree conventions
so may be barking up the wrong tree ;)
On 09/19/
Normally, the macros from command line are system specific macros (e.g
__linux, __KERNEL__ ...), and module own macros are usually defined in
their header files.
DG_NAME and DG_PART are driver 'dgap' owned macros which are used by
multiple files within driver, and need be defined in the driver mai
On Fri, Sep 20, 2013 at 04:42:08PM +0100, Ben Hutchings wrote:
> On Thu, 2013-09-19 at 22:43 +0100, Russell King wrote:
> > + if (dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(64)) ||
> > + dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(32))) {
>
> Surely we want && here.
Good ca
On Fri, Sep 20, 2013 at 07:26:27PM +0200, Heiko Stübner wrote:
> Am Donnerstag, 19. September 2013, 23:49:01 schrieb Russell King:
> > The DMA API requires drivers to call the appropriate dma_set_mask()
> > functions before doing any DMA mapping. Add this required call to
> > the AMBA PL08x driver
The function cmpk_message_handle_tx() is called only in r8192U_dm.c in two
places. The first call resides outside an #ifdef RTL8192U guard, and since
RTL8192U is defined this call can be removed. At the other site this function
is called, there is no check on its return value. Since cmpk_message_ha
This patch removes dm_gpio_change_rf_callback() because it is not called
anywhere and it resides inside an #ifdef RTL8192E guard while RTL8192E
is not defined.
Signed-off-by: Xenia Ragiadakou
---
drivers/staging/rtl8192u/r8192U_dm.c | 73
1 file changed, 73 d
This patch removes the unused code inside #ifdef RTL8190P header guard,
since RTL8190P is not defined.
Signed-off-by: Xenia Ragiadakou
---
drivers/staging/rtl8192u/r8190_rtl8256.h | 4
1 file changed, 4 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8190_rtl8256.h
b/drivers/staging/r
This patch removes the unused code that resides inside #ifdef RTL8190P since
RTL8190P is not defined.
Signed-off-by: Xenia Ragiadakou
---
drivers/staging/rtl8192u/r819xU_firmware.c | 15 ---
1 file changed, 15 deletions(-)
diff --git a/drivers/staging/rtl8192u/r819xU_firmware.c
b/d
This patch removes the unused code that resides inside #ifdef RTL8190P
header guard since RTL8190P is not defined.
Signed-off-by: Xenia Ragiadakou
---
drivers/staging/rtl8192u/r8192U_dm.c | 137 +++
1 file changed, 12 insertions(+), 125 deletions(-)
diff --git a/
This patch removes the code that resides outside #ifdef RTL8192U,
since RTL8192U is defined in r8192U.h and removes, also, the header
guard itself.
Signed-off-by: Xenia Ragiadakou
---
drivers/staging/rtl8192u/r819xU_firmware.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/drivers/stag
This patch removes #ifdef RTL8192U header guard and the code that falls
outside it, since RTL8192U is defined.
Signed-off-by: Xenia Ragiadakou
---
drivers/staging/rtl8192u/r8192U.h | 4
1 file changed, 4 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U.h
b/drivers/staging/rtl8192
This patch adds a check whether skb allocation, in fw_download_code(), was
successful.
Signed-off-by: Xenia Ragiadakou
---
drivers/staging/rtl8192u/r819xU_firmware.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/staging/rtl8192u/r819xU_firmware.c
b/drivers/staging/rtl8192u/r819x
The file r819xU_cmdpkt.c includes header r8192U.h which defines RTL8192U.
This patch removes from cmpk_message_handle_tx() the part of the code that
is never used because it resides outside the header guard #ifdef RTL8192U.
Signed-off-by: Xenia Ragiadakou
---
drivers/staging/rtl8192u/r819xU_cmdp
This patch removes #ifdef RTL8192U header guards from r8192U_dm.c, since
RTL8192U is defined in the included r8192U.h header.
Signed-off-by: Xenia Ragiadakou
---
drivers/staging/rtl8192u/r8192U_dm.c | 4
1 file changed, 4 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U_dm.c
b/dr
This patch removes dm_check_rfctrl_gpio() because it does nothing when
RTL8192U is defined.
Signed-off-by: Xenia Ragiadakou
---
drivers/staging/rtl8192u/r8192U_dm.c | 42
1 file changed, 42 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U_dm.c
b/dr
This patch removes #ifndef RTL8190P header guard since RTL8190P is not
defined anywhere in rtl8192u code.
Signed-off-by: Xenia Ragiadakou
---
drivers/staging/rtl8192u/r8192U_dm.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U_dm.c
b/drivers/staging/rtl8
This patch removes the unused code inside #ifndef RTL8192U header guard
and the guard itself, since RTL8192U is defined.
Signed-off-by: Xenia Ragiadakou
---
drivers/staging/rtl8192u/r8192U_dm.c | 8
1 file changed, 8 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U_dm.c
b/dri
This patch removes the unused code inside #ifdef RTL8190P and #ifdef RTL8192E
guards since RTL8190P and RTL8192E are not defined.
Signed-off-by: Xenia Ragiadakou
---
drivers/staging/rtl8192u/r819xU_phy.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/drivers/staging/rtl8192u/r819xU_phy
The variable execute_shutdown is only assigned the values true and
false. Change its type to bool.
The simplified semantic patch that find this problem is as
follows (http://coccinelle.lip6.fr/):
@exists@
type T;
identifier b;
@@
- T
+ bool
b = ...;
... when any
b = \(true\|false\)
Signed-
The variable ownbit_flag is only assigned the values true and false.
Change its type to bool.
The simplified semantic patch that find this problem is as
follows (http://coccinelle.lip6.fr/):
@exists@
type T;
identifier b;
@@
- T
+ bool
b = ...;
... when any
b = \(true\|false\)
Signed-off-b
The variables bMultiBand and bKeyTableFull are only assigned the
values true and false. Change its type to bool.
The simplified semantic patch that find this problem is as
follows (http://coccinelle.lip6.fr/):
@exists@
type T;
identifier b;
@@
- T
+ bool
b = ...;
... when any
b = \(true\|fa
The variable Trigger is only assigned the values true and false.
Change its type to bool.
The simplified semantic patch that find this problem is as
follows (http://coccinelle.lip6.fr/):
@exists@
type T;
identifier b;
@@
- T
+ bool
b = ...;
... when any
b = \(true\|false\)
Signed-off-by: P
On Sun, Sep 22, 2013 at 12:27:47AM +0200, Peter Senna Tschudin wrote:
> The variable ownbit_flag is only assigned the values true and false.
> Change its type to bool.
>
> The simplified semantic patch that find this problem is as
> follows (http://coccinelle.lip6.fr/):
>
> @exists@
> type T;
> i
Mike,
> We don't need to go through staging. If the dm-writeboost target is
> designed well and provides a tangible benefit it doesn't need
> wide-spread users as justification for going in. The users will come if
> it is implemented well.
OK.
The benefit of introducing writeboost will be docume
29 matches
Mail list logo
