On Wed, Jan 18, 2017 at 02:57:13PM -0800, Stephen Hemminger wrote:
> On Sat, 14 Jan 2017 14:16:07 +0100
> Greg KH wrote:
>
> > 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
On Thu, 2017-01-19 at 02:01 +0300, Dan Carpenter wrote:
> On Wed, Jan 18, 2017 at 07:04:46AM -0800, Michael Zoran wrote:
> > Add the top level compat ioctl handler as a placeholder
> > for adding additional handlers.
> >
> > The ioctls are first filtered and forwarded to the
> > regular ioctl hand
On Wed, Jan 18, 2017 at 04:44:32PM -0700, k...@exchange.microsoft.com wrote:
> From: K. Y. Srinivasan
"interesting" subject :(
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-deve
checkpatch.pl warning:
Symbolic permissions are not preferred. Consider using octal permissions.
Signed-off-by: Igor Pylypiv
---
drivers/staging/greybus/tools/loopback_test.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/greybus/tools/loopback_test.c
b/dr
On 01/14/2017 02:42 PM, Steve Longerbeam wrote:
+/* parse inputs property from a sensor node */
+static void of_parse_sensor_inputs(struct imx_media_dev *imxmd,
+ struct imx_media_subdev *sensor,
+ struct device_node *sensor_np
On Sat, 14 Jan 2017 14:16:07 +0100
Greg KH wrote:
> 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 ent
On Wed, Jan 18, 2017 at 03:28:58PM -0500, Cathy Avery wrote:
> Enable FC lightweight host option so that the luns exposed by
> the driver may be manually scanned.
>
> Signed-off-by: Cathy Avery
> ---
> drivers/scsi/storvsc_drv.c | 6 +-
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> d
On Wed, Jan 18, 2017 at 12:56:29PM +0300, Dan Carpenter wrote:
> On Tue, Jan 17, 2017 at 07:00:28PM -0800, Alison Schofield wrote:
> > Using iio_trigger_put() to free a trigger leads to release of
> > a resource we never held. Replace with iio_trigger_free().
>
> They're basically the same except
On Wed, Jan 18, 2017 at 07:04:46AM -0800, Michael Zoran wrote:
> Add the top level compat ioctl handler as a placeholder
> for adding additional handlers.
>
> The ioctls are first filtered and forwarded to the
> regular ioctl handler if the ioctl does not require
> any extra compatibility processi
From: Tobias Klauser
Date: Wed, 18 Jan 2017 17:45:01 +0100
> The network stack no longer uses the last_rx member of struct net_device
> since the bonding driver switched to use its own private last_rx in
> commit 9f242738376d ("bonding: use last_arp_rx in slave_last_rx()").
>
> However, some dri
The patch provides a means to offer a lightweight option to the
current FC transport class. The new option is selected by a
driver when it indicates it wants the lightweight
transport via fc_function_template.
Signed-off-by: Cathy Avery
---
drivers/scsi/scsi_transport_fc.c | 125
Enable FC lightweight host option so that the luns exposed by
the driver may be manually scanned.
Signed-off-by: Cathy Avery
---
drivers/scsi/storvsc_drv.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c
index 888e1
Currently virtual FC hosts or lightweight hosts are not able to be
manually scanned via sysfs due to the fact that they do not
contain the complete characteristic set associated with a normal FC host
( missing rports, vports, etc ) and are not consistent with the current FC
transport model.
Patch
On Wed, Jan 18, 2017 at 10:51:49PM +0530, Vaibhav Agarwal wrote:
> 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 by
Tobias Klauser wrote:
>The network stack no longer uses the last_rx member of struct net_device
>since the bonding driver switched to use its own private last_rx in
>commit 9f242738376d ("bonding: use last_arp_rx in slave_last_rx()").
>
>However, some drivers still (ab)use the field for their own
On Wed, 2017-01-18 at 18:23 +0100, Greg KH wrote:
> On Wed, Jan 18, 2017 at 09:18:43AM -0800, Michael Zoran wrote:
> > On Wed, 2017-01-18 at 18:13 +0100, Greg KH wrote:
> > > On Wed, Jan 18, 2017 at 07:04:45AM -0800, Michael Zoran wrote:
> > > > This change adds the compatibility data structures fo
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
On Wed, 2017-01-18 at 17:45 +0100, Tobias Klauser wrote:
> The network stack no longer uses the last_rx member of struct net_device
> since the bonding driver switched to use its own private last_rx in
> commit 9f242738376d ("bonding: use last_arp_rx in slave_last_rx()").
>
> However, some drivers
On Wed, Jan 18, 2017 at 09:18:43AM -0800, Michael Zoran wrote:
> On Wed, 2017-01-18 at 18:13 +0100, Greg KH wrote:
> > On Wed, Jan 18, 2017 at 07:04:45AM -0800, Michael Zoran wrote:
> > > This change adds the compatibility data structures for
> > > compatibility
> > > ioctls and defines compatibili
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 negative value checking is not required. Fix this!!
Signed-off-by: Vaibhav Agarwal
---
drivers/staging/greybus/audio_topology.c | 8 ++
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.
On Wed, Jan 18, 2017 at 11:53:07AM +0100, Dhananjay Balan wrote:
> Split line at boolean operator.
>
> Error was reported by checkpatch.pl as
> WARNING: Avoid multiple line dereference - prefer 'mc_msi_domain->host_data'
>
> Signed-off-by: Dhananjay Balan
> ---
> drivers/staging/fsl-mc/bus/irq-
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
On Wed, 2017-01-18 at 18:13 +0100, Greg KH wrote:
> On Wed, Jan 18, 2017 at 07:04:45AM -0800, Michael Zoran wrote:
> > This change adds the compatibility data structures for
> > compatibility
> > ioctls and defines compatibility ioctl definitions for the ioctls
> > that have changed.
> >
> > Signe
On Wed, Jan 18, 2017 at 07:04:45AM -0800, Michael Zoran wrote:
> This change adds the compatibility data structures for compatibility
> ioctls and defines compatibility ioctl definitions for the ioctls
> that have changed.
>
> Signed-off-by: Michael Zoran
> ---
> .../interface/vchiq_arm/vchiq_io
On Tue, Jan 17, 2017 at 11:22 PM, Mark Greer wrote:
> On Tue, Jan 17, 2017 at 08:19:30PM +0530, Vaibhav Agarwal wrote:
>> 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
The network stack no longer uses the last_rx member of struct net_device
since the bonding driver switched to use its own private last_rx in
commit 9f242738376d ("bonding: use last_arp_rx in slave_last_rx()").
However, some drivers still (ab)use the field for their own purposes and
some driver jus
On Tue, Jan 17, 2017 at 11:04 PM, Mark Greer wrote:
> On Tue, Jan 17, 2017 at 08:19:29PM +0530, Vaibhav Agarwal wrote:
>> From: Vaibhav Agarwal
>>
>> Dummy codec register were initially added while populating dummy codec
>> mixer controls until module topology parser was available. Now, these
>>
On Tue, Jan 17, 2017 at 10:56 PM, Mark Greer wrote:
> Hi Vaibhav.
>
> On Tue, Jan 17, 2017 at 08:19:27PM +0530, Vaibhav Agarwal wrote:
>> 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 F
This change adds the compatibility data structures for compatibility
ioctls and defines compatibility ioctl definitions for the ioctls
that have changed.
Signed-off-by: Michael Zoran
---
.../interface/vchiq_arm/vchiq_ioctl.h | 96 ++
1 file changed, 96 insertions
Add compat handler for "dump mem" ioctl.
Signed-off-by: Michael Zoran
---
.../staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 14 ++
1 file changed, 14 insertions(+)
diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
b/drivers/staging/vc04_services/
Add compat handler for "create service" ioctl and move
parts in common with the regular ioctl to vchiq_ioctl_create_service
Signed-off-by: Michael Zoran
---
.../vc04_services/interface/vchiq_arm/vchiq_arm.c | 180 +
1 file changed, 112 insertions(+), 68 deletions(-)
diff --
Add the top level compat ioctl handler as a placeholder
for adding additional handlers.
The ioctls are first filtered and forwarded to the
regular ioctl handler if the ioctl does not require
any extra compatibility processing.
Signed-off-by: Michael Zoran
---
.../vc04_services/interface/vchiq_a
Add compat handler for "await_completion" ioctl and move
parts in common with the regular ioctl to vchiq_ioctl_await_completion
Signed-off-by: Michael Zoran
---
.../vc04_services/interface/vchiq_arm/vchiq_arm.c | 398 ++---
1 file changed, 267 insertions(+), 131 deletions(-)
di
Add compat handler for "queue bulk" ioctls and move
parts in common with the regular ioctls to vchiq_ioctl_queue_bulk
Signed-off-by: Michael Zoran
---
.../vc04_services/interface/vchiq_arm/vchiq_arm.c | 206 ++---
1 file changed, 141 insertions(+), 65 deletions(-)
diff --git a/
Add compat handler for "dequeue messagen" ioctl and move
parts in common with the regular ioctl to vchiq_ioctl_dequeue_message.
Signed-off-by: Michael Zoran
---
.../vc04_services/interface/vchiq_arm/vchiq_arm.c | 168 -
1 file changed, 100 insertions(+), 68 deletions(-)
dif
These set of changes add compat ioctls for vc04_services. When possible
and practical common code between the native ioctl and the compat ioctl
has been moved into a common function.
This set includes one ioctl per patch with the first two patchs being
common changes. No attempt is being made to
Add compat handler for "queue message" ioctl.
Signed-off-by: Michael Zoran
---
.../vc04_services/interface/vchiq_arm/vchiq_arm.c | 36 ++
1 file changed, 36 insertions(+)
diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
b/drivers/staging/vc04_serv
Add compat handler for "get config" ioctl.
Signed-off-by: Michael Zoran
---
.../vc04_services/interface/vchiq_arm/vchiq_arm.c | 32 ++
1 file changed, 32 insertions(+)
diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
b/drivers/staging/vc04_service
Split line at boolean operator.
Error was reported by checkpatch.pl as
WARNING: Avoid multiple line dereference - prefer 'mc_msi_domain->host_data'
Signed-off-by: Dhananjay Balan
---
drivers/staging/fsl-mc/bus/irq-gic-v3-its-fsl-mc-msi.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
On Wed, Jan 18, 2017 at 11:53:07AM +0100, Dhananjay Balan wrote:
> Split line at boolean operator.
>
> Error was reported by checkpatch.pl as
> WARNING: Avoid multiple line dereference - prefer 'mc_msi_domain->host_data'
>
> Signed-off-by: Dhananjay Balan
> ---
> drivers/staging/fsl-mc/bus/irq-
On Tue, Jan 17, 2017 at 07:00:28PM -0800, Alison Schofield wrote:
> Using iio_trigger_put() to free a trigger leads to release of
> a resource we never held. Replace with iio_trigger_free().
They're basically the same except iio_trigger_put() puts the module and
the device and free only puts the
43 matches
Mail list logo
