Removed macros not in use.
Signed-off-by: Kartikey Singh
---
drivers/staging/rtl8188eu/include/wifi.h | 12 +++-
1 file changed, 3 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/rtl8188eu/include/wifi.h
b/drivers/staging/rtl8188eu/include/wifi.h
index 9e08e68..9c9c334 10064
Changed file permissions to octal.
Found with checkpatch.
Signed-off-by: Derek Robson
---
drivers/staging/rtl8192e/rtl8192e/rtl_core.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/rtl8192e/rtl8192e/rtl_core.c
b/drivers/staging/rtl8192e/rtl8192e/rtl_c
Change file permissions to octal style.
Found using checkpatch.
Signed-off-by: Derek Robson
---
drivers/staging/media/davinci_vpfe/vpfe_mc_capture.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/media/davinci_vpfe/vpfe_mc_capture.c
b/drivers/staging/media/d
This change fixes a checkpatch error for "that open brace { should be
on the previous line" as well as a related spacing warning.
Signed-off-by: David Wittman
---
drivers/staging/ks7010/ks_wlan_net.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/ks7010/
(sorry sending again w/o html)
On 01/13/2017 07:20 AM, Philipp Zabel wrote:
Am Freitag, den 06.01.2017, 18:11 -0800 schrieb Steve Longerbeam:
+For image capture, the IPU contains the following internal subunits:
+
+- Image DMA Controller (IDMAC)
+- Camera Serial Interface (CSI)
+- Image Conver
On Sun, Dec 04, 2016 at 11:16:04AM +, Jonathan Cameron wrote:
> On 04/12/16 02:19, Brian Masney wrote:
> > isl29028_enable_proximity() has a boolean argument named enable. This
> > function is only called once and the enable flag is set to true in that
> > call. This patch removes the enable pa
On 01/14/2017 10:40 PM, Greg KH wrote:
On Sun, Jan 15, 2017 at 12:44:41AM +0530, Kartikey singh wrote:
Even better yet, remove it and rebuild the driver and see if it
breaks :)
Only if the code is not between #ifn?def / #endif...
___
devel mailing lis
On Sat, Jan 14, 2017 at 07:38:01PM +0100, Greg KH wrote:
> On Sat, Jan 14, 2017 at 11:53:36PM +0530, Kartikey Singh wrote:
> > do while loop removed from single statement macro
> >
> > Signed-off-by: Kartikey Singh
> > ---
> > drivers/staging/rtl8188eu/include/wifi.h | 4 +---
> > 1 file changed
On Sun, Jan 15, 2017 at 12:44:41AM +0530, Kartikey singh wrote:
> I think that macro is required. Anyway I won't have trouble cleaning it up if
> you say so.
I don't see it being used anywhere, do you? Use 'git grep' to search
for it. Even better yet, remove it and rebuild the driver and see if
On Sat, Jan 14, 2017 at 11:47:56PM +0530, Kartikey Singh wrote:
> fixed a trailing semicolon after macro definition
>
> Signed-off-by: Kartikey Singh
> ---
> drivers/staging/rtl8188eu/include/basic_types.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/
On Sat, Jan 14, 2017 at 11:53:36PM +0530, Kartikey Singh wrote:
> do while loop removed from single statement macro
>
> Signed-off-by: Kartikey Singh
> ---
> drivers/staging/rtl8188eu/include/wifi.h | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/staging/rtl8
do while loop removed from single statement macro
Signed-off-by: Kartikey Singh
---
drivers/staging/rtl8188eu/include/wifi.h | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/staging/rtl8188eu/include/wifi.h
b/drivers/staging/rtl8188eu/include/wifi.h
index 9e08e68..
fixed a trailing semicolon after macro definition
Signed-off-by: Kartikey Singh
---
drivers/staging/rtl8188eu/include/basic_types.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8188eu/include/basic_types.h
b/drivers/staging/rtl8188eu/include/basic_types
mixer control->info call back function checks for -ve values to rebase
min and max values. However, le32 variable is used to fetch values from
GB module FW. Thus -ve value checking is not required. Fix this!!
Signed-off-by: Vaibhav Agarwal
---
drivers/staging/greybus/audio_topology.c | 12 --
From: Vaibhav Agarwal
Uninitialized variable sig_bits was used while configuring stream params
for codec module. These params are used to configure PCM settings for
APBridgeA.
Usually, this is dependent on codec capability and thus populated via
codec dai_driver definition. In our case, it is fi
From: Vaibhav Agarwal
Proper byte order was completely disregarded for multi byte data shared
between AP and module (and APB1). Fix this.
Signed-off-by: Vaibhav Agarwal
Signed-off-by: Vaibhav Agarwal
---
drivers/staging/greybus/audio_module.c | 2 +-
drivers/staging/greybus/audio_topology.
This patch series include following cleanup changes in GB Audio driver.
- Avoid unnecessary checks for le32 variables
- Initialize sig_bits before configuring hw_params
- Remove junk codec register mapping
- Ensure proper byte ordering
Next task is to enable build for GB codec driver. However this
From: Vaibhav Agarwal
Dummy codec register were initially added while populating dummy codec
mixer controls until module topology parser was available. Now, these
dummy registers are nowhere used and thus can be safely removed.
Since ASoC framework requires a valid callback for both read & write
'cb' and 'sgb' were assigned __le values but were not marked as such,
this fixes the following sparse warnings:
drivers/staging/rts5208/rtsx_transport.c:220:34: warning: incorrect type in
assignment (different base types)
drivers/staging/rts5208/rtsx_transport.c:220:34:expected unsigned int
On Fri, Jan 13, 2017 at 11:45:21AM -0800, Stephen Hemminger wrote:
> Here is an update with cleanups based on recent changes.
>
> The following changes since commit b0f2d7d546d37697d3f50753904f6f0c549b62bc:
>
> VME: Remove node entry from vme_driver (2017-01-11 09:21:41 +0100)
>
> are availabl
20 matches
Mail list logo
