Re: [PATCH V5 7/7] dt-bindings: add document for dw-hdmi

2014-11-10 Thread Zubair Lutfullah Kakakhel
Nice work. This patch should move the binding from Documentation/devicetree/bindings/staging/imx-drm/hdmi.txt to the location instead of making a new file and leaving the old one in place. And use git format-patch -M to highlight any changes. Regards ZubairLK On 08/11/14 05:32, Andy Yan

Re: [PATCH V5 3/7] dw-hdmi: make checkpatch happy

2014-11-10 Thread Zubair Lutfullah Kakakhel
Hi Andy, In 3.18 rc4, I cannot find checkpatch errors in imx-hdmi.c Have these errors come during the previous 2 patches. If yes, then these changes need to be squashed into the previous patches. No patch should add a checkpatch error and then fix it in a later patch. Regards ZubairLK On

Re: [PATCH V5 3/7] dw-hdmi: make checkpatch happy

2014-11-10 Thread Andy Yan
Hi ZubairLK: On 2014年11月10日 17:22, Zubair Lutfullah Kakakhel wrote: Hi Andy, In 3.18 rc4, I cannot find checkpatch errors in imx-hdmi.c Have these errors come during the previous 2 patches. If yes, then these changes need to be squashed into the previous patches. No patch should add a

Re: [PATCH V5 3/7] dw-hdmi: make checkpatch happy

2014-11-10 Thread Andy Yan
Hi ZubairLK: On 2014年11月10日 17:22, Zubair Lutfullah Kakakhel wrote: Hi Andy, In 3.18 rc4, I cannot find checkpatch errors in imx-hdmi.c Have these errors come during the previous 2 patches. If yes, then these changes need to be squashed into the previous patches. No patch should add a

Re: [PATCH V5 7/7] dt-bindings: add document for dw-hdmi

2014-11-10 Thread Zubair Lutfullah Kakakhel
On 10/11/14 09:44, Andy Yan wrote: Hi ZubairLK On 2014年11月10日 17:17, Zubair Lutfullah Kakakhel wrote: Nice work. This patch should move the binding from Documentation/devicetree/bindings/staging/imx-drm/hdmi.txt to the location instead of making a new file and leaving the old one in

Re: [PATCH V5 7/7] dt-bindings: add document for dw-hdmi

2014-11-10 Thread Andy Yan
On 2014年11月10日 17:53, Zubair Lutfullah Kakakhel wrote: On 10/11/14 09:44, Andy Yan wrote: Hi ZubairLK On 2014年11月10日 17:17, Zubair Lutfullah Kakakhel wrote: Nice work. This patch should move the binding from Documentation/devicetree/bindings/staging/imx-drm/hdmi.txt to the location instead

Re: [PATCH V5 7/7] dt-bindings: add document for dw-hdmi

2014-11-10 Thread Andy Yan
Hi ZubairLK On 2014年11月10日 17:17, Zubair Lutfullah Kakakhel wrote: Nice work. This patch should move the binding from Documentation/devicetree/bindings/staging/imx-drm/hdmi.txt to the location instead of making a new file and leaving the old one in place. And use git format-patch -M to

Re: Inconsistent lock state with Hyper-V memory balloon?

2014-11-10 Thread Peter Zijlstra
On Sat, Nov 08, 2014 at 02:36:54PM +, Sitsofe Wheeler wrote: I've been trying to use the Hyper-V balloon driver to allow the host to reclaim unused memory but have been hitting issues. With a Hyper-V 2012 R2 guest with 4GBytes of RAM, dynamic memory on, 1GByte minimum 10GByte maximum, 8

[PATCH v6 2/7] staging: imx-drm: imx-hdmi: return defer if can't get ddc i2c adapter

2014-11-10 Thread Andy Yan
drm driver may probe before the i2c bus, so the driver should defer probing until it is available Signed-off-by: Andy Yan andy@rock-chips.com --- Changes in v6: None Changes in v5: None Changes in v4: - defer probe ddc i2c adapter Changes in v3: None Changes in v2: None

[PATCH v6 3/7] staging: imx-drm: imx-hdmi: split imx soc specific code from imx-hdmi

2014-11-10 Thread Andy Yan
imx6 and rockchip rk3288 and JZ4780 (Ingenic Xburst/MIPS) use the interface compatible Designware HDMI IP, but they also have some lightly difference, such as phy pll configuration, register width, 4K support, clk useage, and the crtc mux configuration is also platform specific. To reuse the imx

[PATCH v6 1/7] staging: imx-drm: imx-hdmi: make checkpatch happy

2014-11-10 Thread Andy Yan
CHECK: Alignment should match open parenthesis + if ((hdmi-vic == 10) || (hdmi-vic == 11) || + (hdmi-vic == 12) || (hdmi-vic == 13) || CHECK: braces {} should be used on all arms of this statement + if (hdmi-hdmi_data.video_mode.mdvi) [...] + else { [...]

[PATCH v6 0/7] dw-hdmi: convert imx hdmi to bridge/dw-hdmi

2014-11-10 Thread Andy Yan
We found freescale imx6 and rockchip rk3288 and Ingenic JZ4780 (Xburst/MIPS) use the interface compatible Designware HDMI IP, but they also have some lightly difference, such as phy pll configuration, register width(imx hdmi register is one byte, but rk3288 is 4 bytes width and can only access by

[PATCH v6 4/7] staging: imx-drm: imx-hdmi: move imx-hdmi to bridge/dw-hdmi

2014-11-10 Thread Andy Yan
the original imx hdmi driver is under staging/imx-drm, which depends on imx-drm, so move the imx hdmi drvier out to drm/bridge and rename imx-hdmi to dw-hdmi Signed-off-by: Andy Yan andy@rock-chips.com --- Changes in v6: None Changes in v5: None Changes in v4: None Changes in v3: None

[PATCH v6 6/7] drm: bridge/dw-hdmi: convert dw-hdmi to drm_bridge mode

2014-11-10 Thread Andy Yan
From: Yakir Yang y...@rock-chips.com handle encoder in dw_hdmi-imx.c, keep the connector birdge in dw_hdmi.c Signed-off-by: Andy Yan andy@rock-chips.com Signed-off-by: Yakir Yang y...@rock-chips.com --- Changes in v6: - move some modification from patch#5 Changes in v5: None Changes in

Re: [PATCH V5 1/7] imx-drm: imx-hdmi: split imx soc specific code from imx-hdmi

2014-11-10 Thread Zubair Lutfullah Kakakhel
Hi Andy, A few comments inline. On 08/11/14 05:28, Andy Yan wrote: imx6 and rockchip rk3288 and JZ4780 (Ingenic Xburst/MIPS) use the interface compatible Designware HDMI IP, but they also have some lightly difference, such as phy pll configuration, register width, 4K support, clk useage,

Re: [PATCH v5 10/10] [media] v4l: Forbid usage of V4L2_MBUS_FMT definitions inside the kernel

2014-11-10 Thread Hans Verkuil
On 11/08/2014 04:47 PM, Boris Brezillon wrote: Place v4l2_mbus_pixelcode in a #ifndef __KERNEL__ section so that kernel users don't have access to these definitions. We have to keep this definition for user-space users even though they're encouraged to move to the new media_bus_format enum.

Re: [PATCH V5 1/7] imx-drm: imx-hdmi: split imx soc specific code from imx-hdmi

2014-11-10 Thread Andy Yan
Hi Zubair: thanks very much for your comments. On 2014年11月10日 18:51, Zubair Lutfullah Kakakhel wrote: Hi Andy, A few comments inline. On 08/11/14 05:28, Andy Yan wrote: imx6 and rockchip rk3288 and JZ4780 (Ingenic Xburst/MIPS) use the interface compatible Designware HDMI IP, but they also

Re: [PATCH 0/3] Tools: hv: vssdaemon: freeze/thaw logic improvement for the failure case

2014-11-10 Thread Vitaly Kuznetsov
Dexuan Cui de...@microsoft.com writes: -Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Saturday, November 8, 2014 1:09 AM To: KY Srinivasan; Haiyang Zhang; Greg Kroah-Hartman Cc: de...@linuxdriverproject.org; linux-ker...@vger.kernel.org; Dexuan Cui

[PATCH] staging: unisys: fix CamelCase methods/variables in virthba.c

2014-11-10 Thread Ken Depro
This patch fixes the CamelCase checks for methods/variables in virthba.c found with the checkpatch script. Signed-off-by: Ken Depro kenneth.de...@unisys.com --- drivers/staging/unisys/virthba/virthba.c | 122 +++--- 1 file changed, 61 insertions(+), 61 deletions(-) diff

[PATCH] staging: unisys: add comment to spinlock in struct charqueue

2014-11-10 Thread Benjamin Romer
Add a comment to the charqueue's spinlock to explain that it is a lock for the structure. Now with spaces! :) Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/visorutil/charqueue.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] staging: unisys: remove unused types from visorchipset.h

2014-11-10 Thread Benjamin Romer
Delete the following unused types, and unused function prototypes: VISORCHIPSET_SWITCH_INFO VISORCHIPSET_EXTERNALPORT_INFO VISORCHIPSET_INTERNALPORT_INFO visorchipset_get_switch_info() visorchipset_get_externalport_info() Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com ---

[PATCH v2 2/2] Tools: hv: vssdaemon: skip all filesystems mounted readonly

2014-11-10 Thread Vitaly Kuznetsov
Instead of making a list of exceptions for readonly filesystems in addition to iso9660 we already have it is better to skip freeze operation for all readonly-mounted filesystems. Signed-off-by: Vitaly Kuznetsov vkuzn...@redhat.com --- tools/hv/hv_vss_daemon.c | 2 +- 1 file changed, 1

[PATCH v6 09/10] gpu: ipu-v3: Make use of media_bus_format enum

2014-11-10 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definition to include/uapi/linux/media-bus-format.h and prefixed enum values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Reference new definitions in the ipu-v3 driver. Signed-off-by: Boris Brezillon

[PATCH v6 04/10] [media] i2c: Make use of media_bus_format enum

2014-11-10 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definitions to include/uapi/linux/media-bus-format.h and prefixed values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Replace all references to the old definitions in i2c drivers. Signed-off-by: Boris Brezillon

[PATCH v6 10/10] [media] v4l: Forbid usage of V4L2_MBUS_FMT definitions inside the kernel

2014-11-10 Thread Boris Brezillon
Place v4l2_mbus_pixelcode in a #ifndef __KERNEL__ section so that kernel users don't have access to these definitions. We have to keep this definition for user-space users even though they're encouraged to move to the new media_bus_format enum. Signed-off-by: Boris Brezillon

[PATCH v6 05/10] [media] pci: Make use of MEDIA_BUS_FMT definitions

2014-11-10 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definition to include/uapi/linux/media-bus-format.h and prefixed values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Replace all references to the old definitions in pci drivers. Signed-off-by: Boris Brezillon

[PATCH v6 02/10] [media] v4l: Update subdev-formats doc with new MEDIA_BUS_FMT values

2014-11-10 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definition to include/uapi/linux/media-bus-format.h and prefixed them with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Update the v4l documentation accordingly. Signed-off-by: Boris Brezillon

[PATCH v6 06/10] [media] platform: Make use of media_bus_format enum

2014-11-10 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definition to include/uapi/linux/media-bus-format.h and prefixed values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Reference new definitions in all platform drivers. Signed-off-by: Boris Brezillon

[PATCH v6 08/10] staging: media: Make use of MEDIA_BUS_FMT_ definitions

2014-11-10 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definition to include/uapi/linux/media-bus-format.h and prefixed values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Reference new definitions in all media drivers residing in staging. Signed-off-by: Boris

[PATCH v6 00/10] [media] Make mediabus format subsystem neutral

2014-11-10 Thread Boris Brezillon
Hello, This patch series prepares the use of media bus formats outside of the V4L2 subsytem (my final goal is to use it in the Atmel HLCDC DRM driver where I have to configure my DPI/RGB bus according to the connected display). The series first defines MEDIA_BUS_FMT_ macros, and then replace all

[PATCH v6 01/10] [media] Move mediabus format definition to a more standard place

2014-11-10 Thread Boris Brezillon
Define MEDIA_BUS_FMT macros (re-using the values defined in the v4l2_mbus_pixelcode enum) into a separate header file so that they can be used from the DRM/KMS subsystem without any reference to the V4L2 subsystem. Then set V4L2_MBUS_FMT definitions to the MEDIA_BUS_FMT values using the

[PATCH v6 03/10] [media] Make use of the new media_bus_format definitions

2014-11-10 Thread Boris Brezillon
Replace references to the v4l2_mbus_pixelcode enum with the new media_bus_format enum in all common headers. Signed-off-by: Boris Brezillon boris.brezil...@free-electrons.com Acked-by: Sakari Ailus sakari.ai...@linux.intel.com Acked-by: Hans Verkuil hans.verk...@cisco.com ---

[PATCH v6 07/10] [media] usb: Make use of media_bus_format enum

2014-11-10 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definition to include/uapi/linux/media-bus-format.h and prefixed enum values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Reference new definitions in all usb drivers. Signed-off-by: Boris Brezillon

[PATCH v6 RESEND 07/10] [media] usb: Make use of media_bus_format enum

2014-11-10 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definition to include/uapi/linux/media-bus-format.h and prefixed enum values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Reference new definitions in all usb drivers. Signed-off-by: Boris Brezillon

[PATCH v6 RESEND 03/10] [media] Make use of the new media_bus_format definitions

2014-11-10 Thread Boris Brezillon
Replace references to the v4l2_mbus_pixelcode enum with the new media_bus_format enum in all common headers. Signed-off-by: Boris Brezillon boris.brezil...@free-electrons.com Acked-by: Sakari Ailus sakari.ai...@linux.intel.com Acked-by: Hans Verkuil hans.verk...@cisco.com ---

Re: [PATCH v6 00/10] [media] Make mediabus format subsystem neutral

2014-11-10 Thread Boris Brezillon
On Mon, 10 Nov 2014 18:21:44 +0100 Boris Brezillon boris.brezil...@free-electrons.com wrote: Hello, This patch series prepares the use of media bus formats outside of the V4L2 subsytem (my final goal is to use it in the Atmel HLCDC DRM driver where I have to configure my DPI/RGB bus

[PATCH v6 RESEND 10/10] [media] v4l: Forbid usage of V4L2_MBUS_FMT definitions inside the kernel

2014-11-10 Thread Boris Brezillon
Place v4l2_mbus_pixelcode in a #ifndef __KERNEL__ section so that kernel users don't have access to these definitions. We have to keep this definition for user-space users even though they're encouraged to move to the new media_bus_format enum. Signed-off-by: Boris Brezillon

[PATCH v6 RESEND 05/10] [media] pci: Make use of MEDIA_BUS_FMT definitions

2014-11-10 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definition to include/uapi/linux/media-bus-format.h and prefixed values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Replace all references to the old definitions in pci drivers. Signed-off-by: Boris Brezillon

[PATCH v6 RESEND 02/10] [media] v4l: Update subdev-formats doc with new MEDIA_BUS_FMT values

2014-11-10 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definition to include/uapi/linux/media-bus-format.h and prefixed them with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Update the v4l documentation accordingly. Signed-off-by: Boris Brezillon

[PATCH v6 RESEND 00/10] [media] Make mediabus format subsystem neutral

2014-11-10 Thread Boris Brezillon
Hello, This patch series prepares the use of media bus formats outside of the V4L2 subsytem (my final goal is to use it in the Atmel HLCDC DRM driver where I have to configure my DPI/RGB bus according to the connected display). The series first defines MEDIA_BUS_FMT_ macros, and then replace all

[PATCH v6 RESEND 04/10] [media] i2c: Make use of media_bus_format enum

2014-11-10 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definitions to include/uapi/linux/media-bus-format.h and prefixed values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Replace all references to the old definitions in i2c drivers. Signed-off-by: Boris Brezillon

[PATCH v6 RESEND 09/10] gpu: ipu-v3: Make use of media_bus_format enum

2014-11-10 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definition to include/uapi/linux/media-bus-format.h and prefixed enum values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Reference new definitions in the ipu-v3 driver. Signed-off-by: Boris Brezillon

[PATCH v6 RESEND 06/10] [media] platform: Make use of media_bus_format enum

2014-11-10 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definition to include/uapi/linux/media-bus-format.h and prefixed values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Reference new definitions in all platform drivers. Signed-off-by: Boris Brezillon

[PATCH v6 RESEND 08/10] staging: media: Make use of MEDIA_BUS_FMT_ definitions

2014-11-10 Thread Boris Brezillon
In order to have subsytem agnostic media bus format definitions we've moved media bus definition to include/uapi/linux/media-bus-format.h and prefixed values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Reference new definitions in all media drivers residing in staging. Signed-off-by: Boris

[PATCH 0/7] staging: unisys: more cleanup in visorchipset

2014-11-10 Thread Benjamin Romer
Another set of checkpatch.pl fixups for visorchipset files. Benjamin Romer (7): staging: unisys: remove testing.h from visorchipset staging: unisys: refactor visorchipset_file_init() staging: unisys: fix CamelCase global variables in file.c staging: unisys: get rid of HAVE_UNLOCKED_IOCTL

[PATCH 1/7] staging: unisys: remove testing.h from visorchipset

2014-11-10 Thread Benjamin Romer
Nothing in this file is used anymore, so remove it and the one last reference to it from visorchipset_main.c. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/visorchipset/testing.h | 43 -- .../unisys/visorchipset/visorchipset_main.c

[PATCH 6/7] staging: unisys: fix CamelCase in visorchipset_mmap()

2014-11-10 Thread Benjamin Romer
Fix CamelCase name: physAddr = physaddr Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/visorchipset/file.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/unisys/visorchipset/file.c

[PATCH 5/7] staging: unisys: get rid of goto in visorchipset_open()

2014-11-10 Thread Benjamin Romer
This goto uses CamelCase and was completely unnecessary. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/visorchipset/file.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/drivers/staging/unisys/visorchipset/file.c

[PATCH 3/7] staging: unisys: fix CamelCase global variables in file.c

2014-11-10 Thread Benjamin Romer
Fix CamelCase names: Cdev = file_cdev PControlVm_channel = file_controlvm_channel MajorDev = majordev Registered = registered Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/visorchipset/file.c | 54 +++--- 1 file changed, 27

[PATCH 7/7] staging: unisys: get rid of goto in visorchipset_ioctl()

2014-11-10 Thread Benjamin Romer
Remove another completely unnecessary goto and just return the values directly. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/visorchipset/file.c | 18 +- 1 file changed, 5 insertions(+), 13 deletions(-) diff --git

[PATCH 2/7] staging: unisys: refactor visorchipset_file_init()

2014-11-10 Thread Benjamin Romer
Fix the declaration so it is a single line. Fix CamelCase parameter names: MajorDev = major_dev pControlVm_channel = controlvm_channel Remove the unnecessary gotos and just return directly in error cases. Fix the last error condition so it returns the result of cdev_add() instead of always zero.

[PATCH v6 RESEND 01/10] [media] Move mediabus format definition to a more standard place

2014-11-10 Thread Boris Brezillon
Define MEDIA_BUS_FMT macros (re-using the values defined in the v4l2_mbus_pixelcode enum) into a separate header file so that they can be used from the DRM/KMS subsystem without any reference to the V4L2 subsystem. Then set V4L2_MBUS_FMT definitions to the MEDIA_BUS_FMT values using the

[PATCH 0/4] staging: unisys: visorchannel: CamelCase and comment cleanup

2014-11-10 Thread a
From: Bryan Thompson bryan.thomp...@unisys.com This series of patches addresses two CamelCase issues and a spinlock commenting issue reported by checkpatch on the visorchannel tree. Bryan Thompson (3): staging: unisys: visorchannel: Rename CamelCase variable channelBytes staging: unisys:

[PATCH 1/3] staging: unisys: visorchannel: Rename CamelCase variable channelBytes

2014-11-10 Thread a
From: Bryan Thompson bryan.thomp...@unisys.com Rename channelBytes to channel_bytes in the series of visorchannel_create functions provided by visorchannel. Signed-off-by: Bryan Thompson bryan.thomp...@unisys.com --- drivers/staging/unisys/visorchannel/visorchannel.h | 10

[PATCH 3/3] staging: unisys: visorchannel: Describe spinlocks in VISORCHANNEL_Tag struct

2014-11-10 Thread a
From: Bryan Thompson bryan.thomp...@unisys.com Add comments indicating the requirements and use of the insert_lock and remove_lock in the VISORCHANNEL_Tag structure. Signed-off-by: Bryan Thompson bryan.thomp...@unisys.com --- .../unisys/visorchannel/visorchannel_funcs.c |7 --- 1

[PATCH 2/3] staging: unisys: visorchannel: Rename CamelCase variable nQueues

2014-11-10 Thread a
From: Bryan Thompson bryan.thomp...@unisys.com Rename the visorchannel_debug parameter nQueues to num_queues. Signed-off-by: Bryan Thompson bryan.thomp...@unisys.com --- drivers/staging/unisys/visorchannel/visorchannel.h |2 +- .../unisys/visorchannel/visorchannel_funcs.c |4 ++--

[PATCH 2/3] staging: unisys: visorchannel: Rename CamelCase variable nQueues

2014-11-10 Thread Bryan Thompson
Rename the visorchannel_debug parameter nQueues to num_queues. Signed-off-by: Bryan Thompson bryan.thomp...@unisys.com --- drivers/staging/unisys/visorchannel/visorchannel.h |2 +- .../unisys/visorchannel/visorchannel_funcs.c |4 ++-- 2 files changed, 3 insertions(+), 3

[PATCH 3/3] staging: unisys: visorchannel: Describe spinlocks in VISORCHANNEL_Tag struct

2014-11-10 Thread Bryan Thompson
Add comments indicating the requirements and use of the insert_lock and remove_lock in the VISORCHANNEL_Tag structure. Signed-off-by: Bryan Thompson bryan.thomp...@unisys.com --- .../unisys/visorchannel/visorchannel_funcs.c |7 --- 1 file changed, 4 insertions(+), 3 deletions(-)

[PATCH 0/4] staging: unisys: visorchannel: CamelCase and comment cleanup

2014-11-10 Thread Bryan Thompson
Resending due to return address error. This series of patches addresses two CamelCase issues and a spinlock commenting issue reported by checkpatch on the visorchannel tree. Bryan Thompson (3): staging: unisys: visorchannel: Rename CamelCase variable channelBytes staging: unisys:

[PATCH 1/3] staging: unisys: visorchannel: Rename CamelCase variable channelBytes

2014-11-10 Thread Bryan Thompson
Rename channelBytes to channel_bytes in the series of visorchannel_create functions provided by visorchannel. Signed-off-by: Bryan Thompson bryan.thomp...@unisys.com --- drivers/staging/unisys/visorchannel/visorchannel.h | 10 .../unisys/visorchannel/visorchannel_funcs.c | 26

Re: XVME 6300 with TSI148 bridge on 64 bit Debian (Linux 3.2.57) vme_user issue

2014-11-10 Thread Maurice Moss
Hi Martyn, Thanks for your reply. I have pasted the code below. It is very much similar to your test code from the forum. Thanks! #define _XOPEN_SOURCE 500 #define u32 unsigned int #include stdio.h #include stdlib.h #include sys/ioctl.h #include sys/types.h #include sys/stat.h #include fcntl.h

[PATCH] rtl8188eu: Simplify rtw_endofpktfile() in xmit_linux.c

2014-11-10 Thread Krzysztof Konopko
scripts/checkpatch.pl reports a coding style problem in xmit_linux.c WARNING:BRACES: braces {} are not necessary for single statement blocks #61852: FILE: rtl8188eu/os_dep/xmit_linux.c:70: This patch removes unnecessary braces and simplifies the function to a single return statement.

[PATCH] staging: rtl8723au: change typecast to match type returned by htons()

2014-11-10 Thread Chris Ruffin
Using a u16 pointer typecast for a result from htons() results in the following warning from sparse: drivers/staging/rtl8723au/core/rtw_xmit.c:1279:36: warning: incorrect type in assignment (different base types) drivers/staging/rtl8723au/core/rtw_xmit.c:1279:36:expected unsigned short

Re: [PATCH] staging: rtl8723au: change typecast to match type returned by htons()

2014-11-10 Thread Jes Sorensen
Chris Ruffin cmruf...@gmail.com writes: Using a u16 pointer typecast for a result from htons() results in the following warning from sparse: drivers/staging/rtl8723au/core/rtw_xmit.c:1279:36: warning: incorrect type in assignment (different base types)

Re: [PATCH] staging: rtl8723au: change typecast to match type returned by htons()

2014-11-10 Thread Jes Sorensen
Arend van Spriel ar...@broadcom.com writes: On 10-11-14 21:21, Jes Sorensen wrote: Chris Ruffin cmruf...@gmail.com writes: Using a u16 pointer typecast for a result from htons() results in the following warning from sparse: drivers/staging/rtl8723au/core/rtw_xmit.c:1279:36: warning:

[PATCH 00/10] staging: rtl8723au: Fixes and remove dead code

2014-11-10 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com Hi, Here's a set of patches removing a pile of unused code from the rtl8723au drivers. It also includes an alignment issue pointed out by Arend van Spriel. Note this set goes on top of Chris Ruffin's patch: staging: rtl8723au: change typecast to match

[PATCH 09/10] staging: rtl8723au: Remove no-op function Hal_InitChannelPlan23a()

2014-11-10 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com Signed-off-by: Jes Sorensen jes.soren...@redhat.com --- drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c | 4 drivers/staging/rtl8723au/hal/usb_halinit.c | 4 drivers/staging/rtl8723au/include/rtl8723a_hal.h | 2 -- 3 files changed, 10

[PATCH 01/10] staging: rtl8723au: Remove unused rtw_calculate_wlan_pkt_size_by_attribue23a()

2014-11-10 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com Signed-off-by: Jes Sorensen jes.soren...@redhat.com --- drivers/staging/rtl8723au/core/rtw_xmit.c| 17 - drivers/staging/rtl8723au/include/rtw_xmit.h | 2 -- 2 files changed, 19 deletions(-) diff --git

[PATCH 05/10] staging: rtl8723au: portctrl(): Remove a pile of unnecessary clutter

2014-11-10 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com Signed-off-by: Jes Sorensen jes.soren...@redhat.com --- drivers/staging/rtl8723au/core/rtw_recv.c | 40 --- 1 file changed, 4 insertions(+), 36 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_recv.c

[PATCH 07/10] staging: rtl8723au: pxmitframe-frame_tag is never set to a value 8

2014-11-10 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com No point of masking out high bits since we never set the value to anything exceeding bits 0-3. Signed-off-by: Jes Sorensen jes.soren...@redhat.com --- drivers/staging/rtl8723au/core/rtw_xmit.c | 2 +-

[PATCH 04/10] staging: rtl8723au: rtw_put_snap23a(): Make it static

2014-11-10 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com Signed-off-by: Jes Sorensen jes.soren...@redhat.com --- drivers/staging/rtl8723au/core/rtw_xmit.c| 40 ++-- drivers/staging/rtl8723au/include/rtw_xmit.h | 1 - 2 files changed, 20 insertions(+), 21 deletions(-) diff --git

[PATCH 06/10] staging: rtl8723au: rtw_dump_xframe(): Use proper ETH_P_* types

2014-11-10 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com Signed-off-by: Jes Sorensen jes.soren...@redhat.com --- drivers/staging/rtl8723au/hal/rtl8723au_xmit.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/rtl8723au_xmit.c

[PATCH 03/10] staging: rtl8723au: rtw_put_snap23a(): Use put_unaligned to set protocol

2014-11-10 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com Reported-by: Arend van Spriel ar...@broadcom.com Signed-off-by: Jes Sorensen jes.soren...@redhat.com --- drivers/staging/rtl8723au/core/rtw_xmit.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 02/10] staging rtl8723au: rtw_put_smap23a(): Use common SNAP header defines

2014-11-10 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com Signed-off-by: Jes Sorensen jes.soren...@redhat.com --- drivers/staging/rtl8723au/core/rtw_xmit.c | 26 +++--- 1 file changed, 7 insertions(+), 19 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_xmit.c

[PATCH 08/10] staging: rtl8723au: Remove unused rtl8723a_update_txdesc() and child functions

2014-11-10 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com Signed-off-by: Jes Sorensen jes.soren...@redhat.com --- drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c | 248 -- drivers/staging/rtl8723au/include/rtl8723a_xmit.h | 1 - 2 files changed, 249 deletions(-) diff --git

[PATCH 10/10] staging: rtl8723au: Remove sw led handling

2014-11-10 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com The rtl8723au relies on hw led support, so no point carrying a large unused sw led infrastructure around. Signed-off-by: Jes Sorensen jes.soren...@redhat.com --- drivers/staging/rtl8723au/Makefile |2 -

[PATCH 08/15] staging: comedi: addi_apci_1564: fix counter code in main driver source

2014-11-10 Thread H Hartley Sweeten
The Rev 1.0 APCI-1564 boards do not have counters. Fix the code in the main driver source so that the I/O accesses to the counters do not happen if the devpriv-counters member is not initialized. This does not fix the code in hwdrv_apci1564.c. That code violates the comedi API and is currently

[PATCH 10/15] staging: comedi: addi_apci_1564: split timer and counter subdevices

2014-11-10 Thread H Hartley Sweeten
The timer subdevice is currently broken in this driver. The Rev 1.0 and 2.x versions of the board both have a 12-bit timer. But only the Rev 2.x boards have the 3 32-bit counters. Split the current timer subdevice into two separate subdevices: 1) A single channel 12-bit timer subdevice 2) A

[PATCH 11/15] staging: comedi: addi_tcw.h: provide generic defines for the ADDI-DATA TCW

2014-11-10 Thread H Hartley Sweeten
The TCW (timer/counter/watchdog) devices in the various ADDI-DATA drivers use a common register map definition. Provide a common generic define for these registers so they don't have to be replicated in each driver. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ian Abbott

[PATCH 04/15] staging: comedi: addi_apci_1564: use dev-iobase for main registers

2014-11-10 Thread H Hartley Sweeten
According to ADDI-DATA, the PLD Revision 2.x versions of the APCI-1564 use PCI BAR 0 for the main registers of the board. Remove the 'amcc_iobase' member of the private data and use the dev-iobase to store the base address of PCI BAR 1. Signed-off-by: H Hartley Sweeten

[PATCH 06/15] staging: comedi: addi_apci_1564: fix dev-iobase for all PLD revisions

2014-11-10 Thread H Hartley Sweeten
The APCI-1564 has different I/O mapping depending on if the PLD revision is Rev 1.0 or Rev 2.x. Fix the main register defines so they will work for all PLD revisions and initialie the dev-iobase appropriately. Move the register defines to the main driver source file. Signed-off-by: H Hartley

[PATCH 15/15] staging: comedi: addi_watchdog: use addi_tcw.h defines for watchdog

2014-11-10 Thread H Hartley Sweeten
Use the generic TCW (timer/counter/watchdog) defines for the 8-bit watchdog. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ian Abbott abbo...@mev.co.uk Cc: Greg Kroah-Hartman gre...@linuxfoundation.org --- drivers/staging/comedi/drivers/addi_watchdog.c | 30

[PATCH 07/15] staging: comedi: addi_apci_1564: fix timer iobase for all PLD revisions

2014-11-10 Thread H Hartley Sweeten
The APCI-1564 has different I/O mapping depending on if the PLD revision is Rev 1.0 or Rev 2.x. Add a member, 'timer', to the private data for the base address of the 12-bit timer. Fix the register defines so they will work for all PLD revisions and initialize the devpriv-timer appropriately.

[PATCH 01/15] staging: comedi: addi_apci_1564: board does not use an AMCC PCI controller

2014-11-10 Thread H Hartley Sweeten
According to ADDI-DATA, the APCI-1564 board has a FPGA with a PCI core, it does not use an AMCC PCI controller chip. Remove the amcc_s5933.h include as well as the read of the AMCC_OP_REG_INTCSR register. That offset would actually read APCI1564_WDOG_STATUS_REG register in the FPGA which has

[PATCH 05/15] staging: comedi: addi_apci_1564: detect PLD revision for I/O mapping

2014-11-10 Thread H Hartley Sweeten
The APCI-1564 has different I/O mapping depending on if the PLD revision is Rev 1.0 or Rev 2.x. The revision can be determined by reading the EEPROM register at offset 0x00 of PCI BAR 0 and checking the value of bits 7 to 4. Add this check to apci1564_auto_attach(). Currently this driver is coded

[PATCH 00/15] staging: comedi: addi_apci_1564: fix board I/O mapping

2014-11-10 Thread H Hartley Sweeten
The APCI-1564 boards do not actually have an AMCC PCI controller. According to ADDI-DATA, the boards have always used an FPGA with a PCI core. Unfortunately, there are two major revisions of the FPGA and they use different I/O mappings. Fix the driver so that both I/O map revisions work

[PATCH 14/15] staging: comedi: addi_apci_1564: enable support for PLD Rev 1.0 I/O mapping

2014-11-10 Thread H Hartley Sweeten
Remove the error return in the (*auto_attach) and allow the driver to attach to APC-1564 boards that use the PLD Rev 1.0 I/O mapping. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ian Abbott abbo...@mev.co.uk Cc: Greg Kroah-Hartman gre...@linuxfoundation.org ---

[PATCH 09/15] staging: comedi: addi_apci_1564: move counter register defines to driver

2014-11-10 Thread H Hartley Sweeten
Move the defines for the counter registers from the included source file to the main driver source file. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ian Abbott abbo...@mev.co.uk Cc: Greg Kroah-Hartman gre...@linuxfoundation.org ---

[PATCH 03/15] staging: comedi: addi_apci_1564: store PCI BAR 1 base address in private data

2014-11-10 Thread H Hartley Sweeten
According to ADDI-DATA, only the PLD Revision 2.x versions of the APCI-1564 have the 3 counters. The base address for these counters is found in PCI BAR 1. For aesthetics, save this base address in the private data. The dev-iobase can then be used for the main registers of the board.

[PATCH 13/15] staging: comedi: addi_apci_1564: use addi_tcw.h defines for counters

2014-11-10 Thread H Hartley Sweeten
Use the generic TCW (timer/counter/watchdog) defines for the 32-bit counters. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ian Abbott abbo...@mev.co.uk Cc: Greg Kroah-Hartman gre...@linuxfoundation.org --- .../comedi/drivers/addi-data/hwdrv_apci1564.c | 35

[PATCH] staging: comedi: adl_pci9111: use comedi_async 'scans_done' to detect EOA

2014-11-10 Thread H Hartley Sweeten
The comedi core now counts the number of samples added to the async buffer and detects the end-of-scan and increments the comedi_async 'scans_done' counter. Remove the private data member 'stop_counter' and use the 'scans_done' member to detect the end-of-acquisition. This fixes a possible

[PATCH 0/4] staging: comedi: adl_pci9118: tidy up AI acquisition

2014-11-10 Thread H Hartley Sweeten
Use the comedi_async 'scans_done' to detect the EOA and tidy up the DMA code a bit. H Hartley Sweeten (4): staging: comedi: adl_pci9118: use comedi_async 'scans_done' to detect EOA staging: comedi: adl_pci9118: absorb move_block_from_dma() staging: comedi: adl_pci9118: use

[PATCH 3/4] staging: comedi: adl_pci9118: use comedi_bytes_to_samples()

2014-11-10 Thread H Hartley Sweeten
Remove the assumption of the sample size by using the comedi_bytes_to_samples() helper function to convert the number of bytes to the number of samples. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ian Abbott abbo...@mev.co.uk Cc: Greg Kroah-Hartman gre...@linuxfoundation.org

[PATCH 4/4] staging: comedi: adl_pci9118: switch DMA buffers after writing samples

2014-11-10 Thread H Hartley Sweeten
Currently the DMA buffers are switched before writing the current samples to the async buffer. This works but when the EOA event happens we end up with an outstanding DMA operation in progress that gets terminated by the (*cancel). Avoid the outstanding DMA operation by switching the DMA buffers

[PATCH 2/4] staging: comedi: adl_pci9118: absorb move_block_from_dma()

2014-11-10 Thread H Hartley Sweeten
Absorb this simple helper function into interrupt_pci9118_ai_dma(). Remove the unnecessary local variables 'sampls' and 'm'. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ian Abbott abbo...@mev.co.uk Cc: Greg Kroah-Hartman gre...@linuxfoundation.org ---

[PATCH 1/4] staging: comedi: adl_pci9118: use comedi_async 'scans_done' to detect EOA

2014-11-10 Thread H Hartley Sweeten
The comedi core now counts the number of samples added to the async buffer and detects the end-of-scan and increments the comedi_async 'scans_done' counter. Remove the private data member 'ai_act_scan' and use the 'scans_done' member to detect the end-of-acquisition. Signed-off-by: H Hartley

[V2 PATCH 01/10] added media agnostic (MA) USB HCD driver

2014-11-10 Thread Stephanie Wallick
This is where we interface with the existing USB stack and implement the functionality of a USB host controller driver. From the host's perspective, we appear as just another USB host controller. However, instead of passing traffic along a wired USB bus, the driver hands USB packets off for

[V2 PATCH 10/10] added kernel build, configuration, and TODO files

2014-11-10 Thread Stephanie Wallick
Signed-off-by: Sean O. Stalley sean.stal...@intel.com Signed-off-by: Stephanie Wallick stephanie.s.wall...@intel.com --- MAINTAINERS| 7 +++ drivers/staging/Kconfig| 2 ++ drivers/staging/Makefile | 1 +

[V2 PATCH 07/10] added media agnostic (MA) USB management packet handling

2014-11-10 Thread Stephanie Wallick
The Media Agnostic USB Specification outlines a number of managment packet types for management and control functions. Each function is initiated with a particular type of managment request packet and completed with the corresponding management response packet. This is where we fill the fields for

[V2 PATCH 02/10] added media agnostic (MA) USB HCD roothubs

2014-11-10 Thread Stephanie Wallick
This is where we implement USB 2.0 and 3.0 roothubs. From the host's perspective, hub state is set and tracked just like any other USB roothub. Likewise, requests to the roothub appear to be handled like any other wired USB request. Signed-off-by: Sean O. Stalley sean.stal...@intel.com

  1   2   >