Fix checkpatch issues: "CHECK: Alignment should match open parenthesis".
Signed-off-by: Arushi Singhal
---
drivers/staging/rtl8712/mlme_linux.c| 4 ++--
drivers/staging/rtl8712/rtl8712_cmd.c | 2 +-
drivers/staging/rtl8712/rtl8712_efuse.c | 2 +-
3 files changed, 4 insertions(+), 4 deletio
This fixes a block statement which didnt end with */
Signed-off-by: Craig Inches
---
drivers/staging/lustre/include/linux/lnet/socklnd.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/lustre/include/linux/lnet/socklnd.h
b/drivers/staging/lustre/include/lin
This fixes multiple block statements found not to match
style as per checkpatch
Signed-off-by: Craig Inches
---
drivers/staging/lustre/include/linux/lnet/lnetst.h | 129 +
1 file changed, 81 insertions(+), 48 deletions(-)
diff --git a/drivers/staging/lustre/include/linux/lne
This patch fixes a warning generated by checkpatch for
a line over 80 characters.
Signed-off-by: Craig Inches
---
drivers/staging/lustre/include/linux/lnet/lib-lnet.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/lustre/include/linux/lnet/lib-lnet.h
b/dri
This patch fixes some multiline comment blocks which didnt conform
to the style guide, found by checkpatch.
Signed-off-by: Craig Inches
---
.../staging/lustre/include/linux/lnet/lib-types.h | 46 ++
1 file changed, 30 insertions(+), 16 deletions(-)
diff --git a/drivers/stag
This series fixes a few style issues in lustre. All where picked up
by checkpatch.
Craig Inches (4):
Staging: Luster: Clean up line over 80Char in lib-lnet.h
Staging: Lustre Fix up multiple Block Comments in lib-types.h
Staging: Lustre Fixing multiline block comments in lnetst.h
Staging:
Guenter Roeck / 2017-07-06T17:13-0700
> On 07/06/2017 03:54 PM, Thomas Gardner wrote:
> > The below warning is resolved by removing the cpu_to_le32() call. This
> > call was redundant; vdm_run_state_machine() ensures that SVDM responses
> > have the correct endianness before sending.
> >
> > typec
The below warning is resolved by removing the cpu_to_le32() call. This
call was redundant; vdm_run_state_machine() ensures that SVDM responses
have the correct endianness before sending.
typec/tcpm.c:1019:49: warning: incorrect type in assignment (different base
types)
typec/tcpm.c:1019:49:ex
On 07/06/2017 03:54 PM, Thomas Gardner wrote:
The below warning is resolved by removing the cpu_to_le32() call. This
call was redundant; vdm_run_state_machine() ensures that SVDM responses
have the correct endianness before sending.
typec/tcpm.c:1019:49: warning: incorrect type in assignment (di
There is an off-by-one bug here, which can cause host-to-guest write to stall.
When cur_write_sz == pending_sz, we shouldn't signal the host because it's
meaningless: the ring mustn't be 100% full.
But when cached_write_sz == pending_sz, we must signal the host.
Fixes: 433e19cf33d3 ("Drivers: hv
The return value from tsl2x7x_invoke_change() was not checked in most
places in the driver. This patch adds the proper error checks. The
return values inside tsl2x7x_invoke_change() are now checked by
this patch as well.
Previously, if there was an error turning the chip back on, then the
driver w
tsl2x7x_read_thresh() and tsl2x7x_write_thresh() currently assumes
that IIO_EV_INFO_VALUE is the only iio_event_info that will be
passed in. This patch refactors these two functions so that
additional iio_event_infos can be passed in. The functions are
renamed from tsl2x7x_{read,write}_thresh() to
The driver explicitly creates the in_illuminance0_integration_time sysfs
attribute outside the IIO core. This attribute is available in the IIO
core so this patches migrates the attribute to be created by
the iio_chan_spec.
Signed-off-by: Brian Masney
---
drivers/staging/iio/light/tsl2x7x.c | 68
tsl2x7x_i2c_read() would call i2c_smbus_write_byte() and
i2c_smbus_read_byte(). These two i2c functions can be replaced with a
single call to i2c_smbus_read_byte_data(). This patch removes the
tsl2x7x_i2c_read() function and replaces all occurrences with a call to
i2c_smbus_read_byte_data().
Signe
The calibration function calls i2c_smbus_write_byte() and
i2c_smbus_read_byte(). These two function calls are replaced with a
single call to i2c_smbus_read_byte_data() by this patch. This patch
also removes an unnecessary call that reads the CNTRL register
a second time. One of the error paths retu
This driver in some cases can busy wait for upwards of 15ms. Since the
kernel at this point is not running in atomic context, and is running in
process context, we can safely use usleep_range() instead. This patch
changes the two occurrences of mdelay() to usleep_range().
Signed-off-by: Brian Masn
Add device tree documentation for the tsl2x7x IIO driver.
Signed-off-by: Brian Masney
CC: Rob Herring
CC: Mark Rutland
CC: devicet...@vger.kernel.org
---
Documentation/devicetree/bindings/trivial-devices.txt | 10 ++
1 file changed, 10 insertions(+)
diff --git a/Documentation/devicetr
This begins my work to clean this driver up and eventually move it out
of staging. Driver changes were tested using a TSL2771 hooked up to a
Raspberry Pi 2.
Thanks to Jon Brenner at AMS/TAOS for loaning me some hardware samples
to test my driver changes.
Changes since v1:
- checkpatch cleanups (o
Add device tree support for the tsl2x7x IIO driver with no custom
properties. The device tree documentation is in a separate commit so
that the changes to trivial-devices.txt can go in via the device
tree subsystem.
Signed-off-by: Brian Masney
CC: Rob Herring
CC: Mark Rutland
CC: devicet...@vge
The TSL2X7X driver has a custom power_state sysfs attribute. Remove this
attribute since the runtime power management code provides a sysfs
attribute to control the power state of the device.
Signed-off-by: Brian Masney
---
drivers/staging/iio/light/tsl2x7x.c | 34 ---
The below warning is resolved by removing the cpu_to_le32() call. This
call was redundant; vdm_run_state_machine() ensures that SVDM responses
have the correct endianness before sending.
typec/tcpm.c:1019:49: warning: incorrect type in assignment (different base
types)
typec/tcpm.c:1019:49:ex
On 06/29/2017 10:50 PM, Dmitriy Cherkasov wrote:
After removing code which was permanently disabled with ifdefs, the
function ksocknal_csum() becomes just a wrapper for crc32_le(). Remove
this useless wrapper and instead call crc32_le() directly.
Any feedback on v4 please?
On Thu, Jun 29, 2017 at 01:03:47PM -0400, Brian Masney wrote:
> Add device tree support for the tsl2x7x IIO driver with no custom
> properties.
>
> Signed-off-by: Brian Masney
> CC: Rob Herring
> CC: Mark Rutland
> CC: devicet...@vger.kernel.org
> ---
> Documentation/devicetree/bindings/trivia
acpi_device_id are not supposed to change at runtime. All functions
working with acpi_device_id provided by work with
const acpi_device_id. So mark the non-const structs as const.
File size before:
textdata bss dec hex filename
107541360 4 121182f56
drivers/st
acpi_device_id are not supposed to change at runtime. All functions
working with acpi_device_id provided by work with
const acpi_device_id. So mark the non-const structs as const.
File size before:
textdata bss dec hex filename
151482640 8 177964584
drivers/st
acpi_device_id are not supposed to change at runtime. All functions
working with acpi_device_id provided by work with
const acpi_device_id. So mark the non-const structs as const.
File size before:
textdata bss dec hex filename
207293264 0 239935db9
drivers/st
acpi_device_id are not supposed to change at runtime. All functions
working with acpi_device_id provided by work with
const acpi_device_id. So mark the non-const structs as const.
File size before:
textdata bss dec hex filename
147711880 0 16651410b
drivers/s
acpi_device_id are not supposed to change at runtime. All functions
working with acpi_device_id provided by work with
const acpi_device_id. So mark the non-const structs as const.
File size before:
textdata bss dec hex filename
102971888 0 121852f99
drivers/s
acpi_device_id are not supposed to change at runtime. All functions
working with acpi_device_id provided by work with
const acpi_device_id. So mark the non-const structs as const.
File size before:
textdata bss dec hex filename
238048448 0 322527dfc
drivers/s
acpi_device_id are not supposed to change at runtime. All functions
working with acpi_device_id provided by work with
const acpi_device_id. So mark the non-const structs as const.
File size before:
textdata bss dec hex filename
124663120 8 155943cea
drivers/s
acpi_device_id are not supposed to change at runtime. All functions
working with acpi_device_id provided by work with
const acpi_device_id. So mark the non-const structs as const.
File size before:
textdata bss dec hex filename
53471920 2472911c7b
drivers/s
On 07/05/2017 07:57 PM, Thomas Gardner wrote:
Guenter Roeck / 2017-07-05T07:25-0700
On 07/05/2017 07:00 AM, Thomas Gardner wrote:
The warning below is resolved by casting the LHS to __le32.
typec/tcpm.c:1019:49: warning: incorrect type in assignment (different base
types)
typec/tcpm.c:1019:49
On Thu, Jul 06, 2017 at 07:29:32AM +0530, yash007 wrote:
> From: Yash Omer
>
Send your patch to yourself until you figure out the right format then
send it to the list.
regards,
dan carpenter
___
devel mailing list
de...@linuxdriverproject.org
http:/
From: Yash Omer
---
drivers/staging/dgnc/dgnc_neo.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/dgnc/dgnc_neo.c b/drivers/staging/dgnc/dgnc_neo.c
index 1943e66..0034ebe 100644
--- a/drivers/staging/dgnc/dgnc_neo.c
+++ b/drivers/staging/dgnc/dgnc
On Thu, Jul 6, 2017 at 9:13 AM, Jaya Durga wrote:
> Subject: Staging: lustre :lustre: include :lustre_compat.h: Prefer using the
> BIT macro
Don't overdo it ;-).
Subject: staging: lustre: lustre_compat.h: Prefer using the BIT macro
> Replace all instances of (1 << 27) with BIT(27) to fix
> che
Replace all instances of (1 << 27) with BIT(27) to fix
checkpatch check messages
Signed-off-by: Jaya Durga
---
drivers/staging/lustre/lustre/include/lustre_compat.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/lustre/lustre/include/lustre_compat.h
b/driver
From: Yash Omer
---
drivers/staging/dgnc/dgnc_neo.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/dgnc/dgnc_neo.c b/drivers/staging/dgnc/dgnc_neo.c
index 1943e66..0034ebe 100644
--- a/drivers/staging/dgnc/dgnc_neo.c
+++ b/drivers/staging/dgnc/dgnc
From: Yash Omer
---
drivers/staging/dgnc/dgnc_neo.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/dgnc/dgnc_neo.c b/drivers/staging/dgnc/dgnc_neo.c
index 1943e66..0034ebe 100644
--- a/drivers/staging/dgnc/dgnc_neo.c
+++ b/drivers/staging/dgnc/dgnc
38 matches
Mail list logo
