Re: [PATCH 0000/0003] drivers: hv: util

2012-05-12 Thread gre...@linuxfoundation.org
On Sat, May 12, 2012 at 08:41:06PM +, KY Srinivasan wrote: > > > > -Original Message- > > From: K. Y. Srinivasan [mailto:k...@microsoft.com] > > Sent: Saturday, May 12, 2012 4:44 PM > > To: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; > > devel@linuxdriverproject.org; vir

RE: [PATCH 0000/0003] drivers: hv: util

2012-05-12 Thread KY Srinivasan
> -Original Message- > From: K. Y. Srinivasan [mailto:k...@microsoft.com] > Sent: Saturday, May 12, 2012 4:44 PM > To: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; > devel@linuxdriverproject.org; virtualizat...@lists.osdl.org; oher...@suse.com > Cc: KY Srinivasan > Subject: [

[PATCH 2/2] Drivers: hv: util: Properly handle version negotiations.

2012-05-12 Thread K. Y. Srinivasan
The current version negotiation code is not "future proof". Fix this by allowing each service the flexibility to either specify the highest version it can support or it can support the highest version number the host is offering. Signed-off-by: K. Y. Srinivasan Reviewed-by: Haiyang Zhang --- dr

[PATCH 1/2] Drivers: hv: Get rid of an unnecessary check in vmbus_prep_negotiate_resp()

2012-05-12 Thread K. Y. Srinivasan
The vmbus_prep_negotiate_resp() is only invoked when we are negotiating the version; so the current check in vmbus_prep_negotiate_resp() is unnecessary. Get rid of it. Signed-off-by: K. Y. Srinivasan Reviewed-by: Haiyang Zhang --- drivers/hv/channel_mgmt.c | 39 +++

[PATCH 0000/0003] drivers: hv: util

2012-05-12 Thread K. Y. Srinivasan
Fix-up the version negotiation code for the util drivers. Regards, K. Y ___ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Re: [PATCH] staging: comedi: Add helper macro for comedi pci driver boilerplate

2012-05-12 Thread Ian Abbott
On 11/05/2012 22:59, gre...@linuxfoundation.org wrote: On Fri, May 11, 2012 at 01:26:37PM -0500, H Hartley Sweeten wrote: How about doing this in the register function: If (!pci_driver->name) pci_driver->name = comedi_driver->driver_name; This way all the drivers that s

Re: [PATCH] iio: Remove unused iio_device_put()

2012-05-12 Thread Jonathan Cameron
On 05/12/2012 03:33 PM, Greg Kroah-Hartman wrote: > On Sat, May 12, 2012 at 04:24:22PM +0200, Lars-Peter Clausen wrote: >> On 05/12/2012 04:18 PM, Greg Kroah-Hartman wrote: >>> On Fri, May 11, 2012 at 03:13:13PM +0200, Lars-Peter Clausen wrote: This function is currently unused and we do not h

Re: [PATCH] iio: Remove unused iio_device_put()

2012-05-12 Thread Greg Kroah-Hartman
On Sat, May 12, 2012 at 04:24:22PM +0200, Lars-Peter Clausen wrote: > On 05/12/2012 04:18 PM, Greg Kroah-Hartman wrote: > > On Fri, May 11, 2012 at 03:13:13PM +0200, Lars-Peter Clausen wrote: > >> This function is currently unused and we do not have a matching > >> iio_device_get() > >> function e

Re: [PATCH] iio: Remove unused iio_device_put()

2012-05-12 Thread Lars-Peter Clausen
On 05/12/2012 04:18 PM, Greg Kroah-Hartman wrote: > On Fri, May 11, 2012 at 03:13:13PM +0200, Lars-Peter Clausen wrote: >> This function is currently unused and we do not have a matching >> iio_device_get() >> function either, so just remove it. >> >> Signed-off-by: Lars-Peter Clausen >> Acked-by

Re: [PATCH] iio: Remove unused iio_device_put()

2012-05-12 Thread Greg Kroah-Hartman
On Fri, May 11, 2012 at 03:13:13PM +0200, Lars-Peter Clausen wrote: > This function is currently unused and we do not have a matching > iio_device_get() > function either, so just remove it. > > Signed-off-by: Lars-Peter Clausen > Acked-by: Jonathan Cameron No, I see a number of places it shou

[PATCH 22/23] staging:iio:resolver: Use dev_to_iio_dev()

2012-05-12 Thread Lars-Peter Clausen
Replace open-coded instances of getting a iio_dev struct from a device struct with dev_to_iio_dev(). Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Cameron --- drivers/staging/iio/resolver/ad2s1210.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) dif

[PATCH 19/23] staging:iio:imu: Use dev_to_iio_dev()

2012-05-12 Thread Lars-Peter Clausen
Replace open-coded instances of getting a iio_dev struct from a device struct with dev_to_iio_dev(). Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Cameron --- drivers/staging/iio/imu/adis16400_core.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/sta

[PATCH 18/23] staging:iio:impedance-analyzer: Use dev_to_iio_dev()

2012-05-12 Thread Lars-Peter Clausen
Replace open-coded instances of getting a iio_dev struct from a device struct with dev_to_iio_dev(). Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Cameron --- drivers/staging/iio/impedance-analyzer/ad5933.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/dr

[PATCH 11/23] staging:iio:adc: Use dev_to_iio_dev()

2012-05-12 Thread Lars-Peter Clausen
Replace open-coded instances of getting a iio_dev struct from a device struct with dev_to_iio_dev(). Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Cameron --- drivers/staging/iio/adc/ad7192.c | 12 ++-- drivers/staging/iio/adc/ad7280a.c | 12 ++-- drivers/s

[PATCH 20/23] staging:iio:light: Use dev_to_iio_dev()

2012-05-12 Thread Lars-Peter Clausen
Replace open-coded instances of getting a iio_dev struct from a device struct with dev_to_iio_dev(). Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Cameron --- drivers/staging/iio/light/isl29018.c | 12 - drivers/staging/iio/light/tsl2583.c | 26 +-- d

[PATCH 15/23] staging:iio:dac: Use dev_to_iio_dev()

2012-05-12 Thread Lars-Peter Clausen
Replace open-coded instances of getting a iio_dev struct from a device struct with dev_to_iio_dev(). Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Cameron --- drivers/staging/iio/dac/ad5360.c |4 ++-- drivers/staging/iio/dac/ad5380.c |8 drivers/staging/iio/dac

[PATCH 21/23] staging:iio:magnetometer: Use dev_to_iio_dev()

2012-05-12 Thread Lars-Peter Clausen
Replace open-coded instances of getting a iio_dev struct from a device struct with dev_to_iio_dev(). Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Cameron --- drivers/staging/iio/magnetometer/ak8975.c |4 ++-- drivers/staging/iio/magnetometer/hmc5843.c | 16 2 fil

[PATCH 23/23] staging:iio:meter: Use dev_to_iio_dev()

2012-05-12 Thread Lars-Peter Clausen
Replace open-coded instances of getting a iio_dev struct from a device struct with dev_to_iio_dev(). Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Cameron --- drivers/staging/iio/meter/ade7753.c | 12 ++-- drivers/staging/iio/meter/ade7754.c | 12 ++-- drive

[PATCH 14/23] staging:iio:cdc: Use dev_to_iio_dev()

2012-05-12 Thread Lars-Peter Clausen
Replace open-coded instances of getting a iio_dev struct from a device struct with dev_to_iio_dev(). Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Cameron --- drivers/staging/iio/cdc/ad7150.c |4 ++-- drivers/staging/iio/cdc/ad7152.c |6 +++--- drivers/staging/iio/cdc/ad7746.c |

[PATCH 13/23] staging:iio:addac: Use dev_to_iio_dev()

2012-05-12 Thread Lars-Peter Clausen
Replace open-coded instances of getting a iio_dev struct from a device struct with dev_to_iio_dev(). Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Cameron --- drivers/staging/iio/addac/adt7316.c | 140 +-- 1 file changed, 70 insertions(+), 70 deletions(-)

[PATCH 17/23] staging:iio:gyro: Use dev_to_iio_dev()

2012-05-12 Thread Lars-Peter Clausen
Replace open-coded instances of getting a iio_dev struct from a device struct with dev_to_iio_dev(). Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Cameron --- drivers/staging/iio/gyro/adis16260_core.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/

[PATCH 16/23] staging:iio:frequency: Use dev_to_iio_dev()

2012-05-12 Thread Lars-Peter Clausen
Replace open-coded instances of getting a iio_dev struct from a device struct with dev_to_iio_dev(). Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Cameron --- drivers/staging/iio/frequency/ad5930.c |2 +- drivers/staging/iio/frequency/ad9832.c |2 +- drivers/staging/iio/frequency

[PATCH 12/23] staging:iio:accel: Use dev_to_iio_dev()

2012-05-12 Thread Lars-Peter Clausen
Replace open-coded instances of getting a iio_dev struct from a device struct with dev_to_iio_dev(). Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Cameron --- drivers/staging/iio/accel/adis16201_core.c |2 +- drivers/staging/iio/accel/adis16203_core.c |2 +- drivers/staging/iio/a

[PATCH 10/23] staging:iio:adis16400_ring: Pass IIO device directly

2012-05-12 Thread Lars-Peter Clausen
When calling adis16350_spi_read_all and adis16400_spi_read_burst we pass the device struct of embedded in the IIO device only to look up the IIO device from the device struct again right away. This patch changes the code to pass the IIO device directly. Signed-off-by: Lars-Peter Clausen Acked-by:

[PATCH 09/23] staging:iio:ade7758_spi_read_burst: Pass IIO device directly

2012-05-12 Thread Lars-Peter Clausen
When calling ade7758_spi_read_burst we pass the device struct of embedded in the IIO device only to look up the IIO device from the device struct again right away. This patch changes the code to pass the IIO device directly. Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Cameron --- drive

[PATCH 07/23] staging:iio:__lis3l02dq_write_data_ready_config: Pass IIO device directly

2012-05-12 Thread Lars-Peter Clausen
When calling __lis3l02dq_write_data_ready_config we pass the device struct of embedded in the IIO device only to look up the IIO device from the device struct again right away. This patch changes the code to pass the IIO device directly. Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Camero

[PATCH 08/23] staging:iio:adis16260_read_ring_data: Pass IIO device directly

2012-05-12 Thread Lars-Peter Clausen
When calling adis16260_read_ring_data we pass the device struct of embedded in the IIO device only to look up the IIO device from the device struct again right away. This patch changes the code to pass the IIO device directly. Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Cameron --- dri

[PATCH 06/23] staging:iio:adis16240_read_ring_data: Pass IIO device directly

2012-05-12 Thread Lars-Peter Clausen
When calling adis16240_read_ring_data we pass the device struct of embedded in the IIO device only to look up the IIO device from the device struct again right away. This patch changes the code to pass the IIO device directly. Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Cameron --- dri

[PATCH 01/23] iio: Add dev_to_iio_dev() helper function

2012-05-12 Thread Lars-Peter Clausen
This patch adds a helper function for retriving a iio_dev struct from a device struct. Currently we open-code this in two different ways. One is using dev_get_drvdata on the device and the other is using container_of. The new helper function uses the container_of solution as it creates slightly sma

[PATCH 02/23] iio: Use dev_to_iio_dev()

2012-05-12 Thread Lars-Peter Clausen
Replace open-coded instances of getting a iio_dev struct from a device struct with dev_to_iio_dev(). Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Cameron --- drivers/iio/industrialio-buffer.c | 16 drivers/iio/industrialio-core.c| 12 ++-- drivers/iio/i

[PATCH 05/23] staging:iio:adis16209_read_ring_data: Pass IIO device directly

2012-05-12 Thread Lars-Peter Clausen
When calling adis16209_read_ring_data we pass the device struct of embedded in the IIO device only to look up the IIO device from the device struct again right away. This patch changes the code to pass the IIO device directly. Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Cameron --- dri

[PATCH 03/23] staging:iio:adis16203_read_ring_data: Pass IIO device directly

2012-05-12 Thread Lars-Peter Clausen
When calling adis16203_read_ring_data we pass the device struct of embedded in the IIO device only to look up the IIO device from the device struct again right away. This patch changes the code to pass the IIO device directly. Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Cameron --- dri

[PATCH 04/23] staging:iio:adis16204_read_ring_data: Pass IIO device directly

2012-05-12 Thread Lars-Peter Clausen
When calling adis16204_read_ring_data we pass the device struct of embedded in the IIO device only to look up the IIO device from the device struct again right away. This patch changes the code to pass the IIO device directly. Signed-off-by: Lars-Peter Clausen Acked-by: Jonathan Cameron --- dri