Corrected label name from err_destory_routes to err_destroy_routes.
Signed-off-by: Parav Pandit
---
drivers/staging/lustre/lnet/lnet/api-ni.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/lustre/lnet/lnet/api-ni.c
b/drivers/staging/lustre/lnet/lnet/
On Fri, Dec 02, 2016 at 08:13:49PM +0100, Fernando Apesteguia wrote:
> For the first lines of the patch, I opted to create a small function
> instead of breaking the the line in a weird way.
>
> The other changes are simple ones.
>
> Signed-off-by: Fernando Apesteguia
> ---
> drivers/staging/dg
Add parentheses around variable x for the readability purpose.
This warning was found using checkpatch.pl.
Signed-off-by: Tabrez khan
---
drivers/staging/media/bcm2048/radio-bcm2048.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/media/bcm2048/radio-bcm2048
On Sat, Dec 03, 2016 at 09:51:13AM +0100, Greg KH wrote:
> On Fri, Dec 02, 2016 at 08:13:49PM +0100, Fernando Apesteguia wrote:
> > For the first lines of the patch, I opted to create a small function
> > instead of breaking the the line in a weird way.
> >
> > The other changes are simple ones.
>
On Sat, Dec 03, 2016 at 10:56:54AM +0100, Fernando Apesteguia wrote:
> On Sat, Dec 03, 2016 at 09:51:13AM +0100, Greg KH wrote:
> > On Fri, Dec 02, 2016 at 08:13:49PM +0100, Fernando Apesteguia wrote:
> > > For the first lines of the patch, I opted to create a small function
> > > instead of breaki
From: ChinmayVS
Macros with multiple statements should be enclosed in a do - while loop
Signed-off-by: ChinmayVS
---
drivers/staging/greybus/loopback.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/greybus/loopback.c
b/drivers/staging/greybus/loopback.
On Sat, Dec 03, 2016 at 03:41:19PM +0530, Chinmay VS wrote:
> From: ChinmayVS
>
> Macros with multiple statements should be enclosed in a do - while loop
>
> Signed-off-by: ChinmayVS
> ---
> drivers/staging/greybus/loopback.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff
On Sat, Dec 3, 2016 at 3:52 PM, Greg KH wrote:
> On Sat, Dec 03, 2016 at 03:41:19PM +0530, Chinmay VS wrote:
>> From: ChinmayVS
>>
>> Macros with multiple statements should be enclosed in a do - while loop
>>
>> Signed-off-by: ChinmayVS
>> ---
>> drivers/staging/greybus/loopback.c | 4 +++-
>>
Fixed checkpatch.pl errors related to "space prohibited after that '*'
or '&'" in ks_wlan_net.c file.
Signed-off-by: Yamanappagouda Patil
---
drivers/staging/ks7010/ks_wlan_net.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/ks7010/ks_wlan_net.c
-ups-to-make-lustre_idl-h-a-proper-UAPI-header/20161203-112301
config: s390-allmodconfig (attached as .config)
compiler: s390x-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
reproduce:
wget
https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross
-O ~/bin
Fixed checkpatch.pl warnings 'Missing blank line after declaration'
in ks_wlan_net.c file.
Signed-off-by: Yamanappagouda Patil
---
drivers/staging/ks7010/ks_wlan_net.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/drivers/staging/ks7010/ks_wlan_net.c
b/drivers/staging/ks7010/ks_wla
From: Pan Bian
At the end of function lstcon_group_info(), "return 0" seems improper.
It may be better to return the value of rc.
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=188811
Signed-off-by: Pan Bian
---
drivers/staging/lustre/lnet/selftest/console.c | 2 +-
1 file changed, 1 i
From: Ping-Ke Shih
Add code to set the power mode bits in the descriptor. These bits are
valid in RTL8723BE, RTL8821AE, RTL8812AE, and RTL8192EE.
Signed-off-by: Ping-Ke Shih
Signed-off-by: Larry Finger
---
.../wireless/realtek/rtlwifi/btcoexist/rtl_btc.c | 13 +
.../wireless/realtek
From: Ping-Ke Shih
Bluetooth coexistence needs to be able to control the power-save
state. This patch turns the building of btcoexist back on.
Signed-off-by: Ping-Ke Shih
Signed-off-by: Larry Finger
---
drivers/net/wireless/realtek/rtlwifi/base.c| 8 +++
.../wireless/realtek/rtlwifi/
This set of patches implement a major rewrite of the BT coexistence
routines.
These patches must follow [PATCH 14/14] rtlwifi: Add work queue for c2h cmd.
Signed-off-by: Ping-Ke Shih
Signed-off-by: Larry Finger
Ping-Ke Shih (7)
rtlwifi: btcoexist: Update routines for RTL8192EE
rtlwifi: btc
From: K. Y. Srinivasan
Fixes to handle CPU online/offline. Also included is a new uio
driver for Hyper-V.
V2: Updated commit logs (Greg KH)
Re-implemented the API to detect if it is a vmbus device
Alex Fluter (1):
Tools: hv: kvp: configurable external scripts path
Haiyang
From: Stephen Hemminger
This patch adds sysfs interface to dynamically bind new UUID values
to existing VMBus device. This is useful for generic UIO driver to
act similar to uio_pci_generic.
Signed-off-by: Stephen Hemminger
Signed-off-by: K. Y. Srinivasan
---
drivers/hv/vmbus_drv.c | 174 +++
From: Haiyang Zhang
Changed it to HV_UNKNOWN
Signed-off-by: Haiyang Zhang
Signed-off-by: K. Y. Srinivasan
---
drivers/hv/channel_mgmt.c |6 +++---
include/linux/hyperv.h|2 +-
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/hv/channel_mgmt.c b/drivers/hv/cha
From: Vitaly Kuznetsov
It may happen that not all CPUs are online when we do hv_synic_alloc() and
in case more CPUs come online later we may try accessing these allocated
structures.
Signed-off-by: Vitaly Kuznetsov
Signed-off-by: K. Y. Srinivasan
---
drivers/hv/hv.c |4 ++--
1 files chang
From: Stephen Hemminger
This is a new driver to enable userspace networking on VMBus.
It is based largely on the similar driver that already exists
for PCI, and earlier work done by Brocade to support DPDK.
Signed-off-by: Stephen Hemminger
Signed-off-by: K. Y. Srinivasan
---
MAINTAINERS
From: Vitaly Kuznetsov
Since commit e513229b4c38 ("Drivers: hv: vmbus: prevent cpu offlining on
newer hypervisors") cpu offlining was disabled. It is still true that we
can't offline CPUs which have VMBus channels bound to them but we may have
'free' CPUs (e.v. we booted with maxcpus= parameter a
From: Vitaly Kuznetsov
It may happen that secondary CPUs are still alive and resetting
hv_context.tsc_page will cause a consequent crash in read_hv_clock_tsc()
as we don't check for it being not NULL there. It is safe as we're not
freeing this page anyways.
Signed-off-by: Vitaly Kuznetsov
Signe
From: Vitaly Kuznetsov
DoS protection conditions were altered in WS2016 and now it's easy to get
-EAGAIN returned from vmbus_post_msg() (e.g. when we try changing MTU on a
netvsc device in a loop). All vmbus_post_msg() callers don't retry the
operation and we usually end up with a non-functional
From: Haiyang Zhang
On Hyper-V, every VF interface has a corresponding synthetic
interface managed by netvsc that share the same MAC
address. netvsc registers for netdev events to manage this
association. Currently we use the MAC address to manage this
association but going forward, we want to us
From: Alex Fluter
error when running hypervkvpd:
$ sudo ./hv_kvp_daemon -n
sh: hv_get_dns_info: command not found
sh: hv_get_dhcp_info: command not found
sh: hv_get_dns_info: command not found
sh: hv_get_dhcp_info: command not found
The external scripts are not installed in system path,
adding
From: K. Y. Srinivasan
Enhance the rescind callback functionality by permitting the passing of an
opaque
pointer. This functionality will be used by vmbus device drivers to implement
rescind related cleanup more efficiently.
Signed-off-by: K. Y. Srinivasan
---
drivers/hv/channel_mgmt.c | 11
From: Vitaly Kuznetsov
"kernel BUG at drivers/hv/channel_mgmt.c:350!" is observed when hv_vmbus
module is unloaded. BUG_ON() was introduced in commit 85d9aa705184
("Drivers: hv: vmbus: add an API vmbus_hvsock_device_unregister()") as
vmbus_free_channels() codepath was apparently forgotten.
Fixes
From: Vitaly Kuznetsov
To make it possible to online/offline CPUs switch to cpuhp infrastructure
for doing hv_synic_init()/hv_synic_cleanup().
Signed-off-by: Vitaly Kuznetsov
Signed-off-by: K. Y. Srinivasan
---
drivers/hv/hv.c | 15 +++
drivers/hv/hyperv_vmbus.h |4
From: Vitaly Kuznetsov
Initializing hv_context.percpu_list in hv_synic_alloc() helps to prevent a
crash in percpu_channel_enq() when not all CPUs were online during
initialization and it naturally belongs there.
Signed-off-by: Vitaly Kuznetsov
Signed-off-by: K. Y. Srinivasan
---
drivers/hv/hv
From: Vitaly Kuznetsov
To get prepared to CPU offlining support we need co change the way how we
unbind clockevent devices. As one CPU may go online/offline multiple times
we need to bind it in hv_synic_init() and unbind it in hv_synic_cleanup().
There is an additional corner case: when we unload
From: K. Y. Srinivasan
After the channel is rescinded, the host does not read from the rescinded
channel.
Fail writes to a channel that has already been rescinded.
Signed-off-by: K. Y. Srinivasan
---
drivers/hv/ring_buffer.c |3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff -
Hi ChinmayVS,
[auto build test ERROR on staging/staging-testing]
[also build test ERROR on v4.9-rc7 next-20161202]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://github.com/0day-ci/linux/commits/Chinmay-VS/staging-greybus-Fix-m
The vme_driver structure currently has a "shutdown" entry. This entry is
never used, it lacks the correct parameter (it should be providing a
pointer to the relevant vme_dev struct to even *look* usable), the VME
subsystem currently doesn't provide support for shutdown functions and no
in-tree driv
Hi Lino,
[auto build test ERROR on net-next/master]
url:
https://github.com/0day-ci/linux/commits/Lino-Sanfilippo/net-ethernet-slicoss-add-slicoss-gigabit-ethernet-driver/20161126-202438
config: sparc64-allyesconfig (attached as .config)
compiler: sparc64-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1
There are two callers to the function isl29028_set_als_ir_mode() and
both instances use a nested if statement to only change the chip state
if it is not in the proper mode. This patch moves this check into the
isl29028_set_als_ir_mode() function to remove the nested if
statements.
Signed-off-by: B
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 parameter from that function.
Signed-off-by: Brian Masney
---
drivers/staging/iio/light/isl29028.c | 10 --
1
The alignment of the #defines at the top of the file is not consistent.
This changes all of the defines to use consistent alignment to improve
the code readability.
Signed-off-by: Brian Masney
---
drivers/staging/iio/light/isl29028.c | 42 ++--
1 file changed, 21
This patch adds power management support to the isl29028 driver.
Signed-off-by: Brian Masney
---
drivers/staging/iio/light/isl29028.c | 59
1 file changed, 59 insertions(+)
diff --git a/drivers/staging/iio/light/isl29028.c
b/drivers/staging/iio/light/isl290
This is my first round of cleanups to the isl29028 light driver in
preparation for moving the driver out of staging. The main feature
introduced by this patch set is support for power management. The
rest of the patches are minor improvements, mostly style.
I have one of these devices from
https:/
The alignment of the variables in the struct isl29028_chip is not
consistent. This changes all of the variables to use consistent
alignment to improve the code readability.
Signed-off-by: Brian Masney
---
drivers/staging/iio/light/isl29028.c | 10 +-
1 file changed, 5 insertions(+), 5 de
isl29028_chip_init() contains the device driver defaults and
two I2C calls that detect the presence of the chip. This patch
moves these into isl29028_probe() so that this function can
be used by the power management runtinme in a followup patch. This
patch also renames isl29028_chip_init() to
isl29
When isl29028_set_proxim_sampling() fails, it was up to both callers to
log the failure message. This patch moves the logging into
isl29028_set_proxim_sampling() to reduce the overall amount of code in
the driver.
Signed-off-by: Brian Masney
---
drivers/staging/iio/light/isl29028.c | 28
Two separate calls to regmap_update_bits() in isl29028_set_als_scale()
and isl29028_set_als_ir_mode() did not have their function arguments
on the next line aligned correctly to the open parenthesis. This patch
corrects the alignment.
Signed-off-by: Brian Masney
---
drivers/staging/iio/light/isl
If the call to devm_iio_device_alloc() fails, then isl29028_probe()
logs a message saying that memory cannot be allocated. The user's system
most likely has larger issues at this point. This patch removes that
error message since the error code is passed on and the message is not
necessary.
Signed
The isl29028 staging driver contains a legacy binding of "isl,isl29028"
that is marked as legacy and to not use. There are no in tree references
to that binding. This patch removes the legacy binding in preparation
for moving the driver out of staging. Otherwise, we will have to support
this bindin
Fixed warning found by make W=2 to reduce the amount of build noise:
warning: comparison between signed and unsigned integer expressions
[-Wsign-compare]
Signed-off-by: Brian Masney
---
drivers/staging/iio/light/isl29028.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/driv
When isl29028_set_als_scale() fails, it was up to both callers to log
the failure message. This patch moves the logging into
isl29028_set_als_scale() to reduce the overall amount of code in the
driver.
Signed-off-by: Brian Masney
---
drivers/staging/iio/light/isl29028.c | 23 +---
The #define ISL29028_DEV_ATTR was not used so this patch removes the
unnecessary code.
Signed-off-by: Brian Masney
---
drivers/staging/iio/light/isl29028.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/iio/light/isl29028.c
b/drivers/staging/iio/light/isl29028.c
index 8f9295
Add and remove newlines to improve code readability in preparation for
moving the driver out of staging.
Signed-off-by: Brian Masney
---
drivers/staging/iio/light/isl29028.c | 31 ++-
1 file changed, 22 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/iio/l
If the call to isl29028_chip_init_and_power_on() in isl29028_probe()
fails, then isl29028_probe() will log an error message. All of the
error paths in that call path already have error logging in place. This
patch removes the unnecessary logging.
Signed-off-by: Brian Masney
---
drivers/staging/i
isl29028_proxim_get() checks to see if the promixity needs to be
enabled on the chip and then calls isl29028_read_proxim(). There
are no other callers of isl29028_read_proxim(). The naming between
these two functions can be confusing so this patch combines the
two to avoid the confusion.
Signed-of
The wording and style of the different error messages was not
consistent. This patches makes the wording and style consistent
throughout the driver.
Signed-off-by: Brian Masney
---
drivers/staging/iio/light/isl29028.c | 56
1 file changed, 32 insertions(+), 2
isl29028_write_raw() contains unnecessary parenthesis when checking to
see if the passed in lux scale is valid. This patch removes the
unnecessary parenthesis.
Signed-off-by: Brian Masney
---
drivers/staging/iio/light/isl29028.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a
The three info_mask_separate members OR several BIT(xxx) fields
together. This patch changes the column alignment of these fields to be
aligned at the same column to improve the overall code readability. It
also moves the { for the next channel to the next line to improve code
readability.
Signed-
54 matches
Mail list logo
