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

2014-11-12 Thread Oliver Neukum
On Mon, 2014-11-10 at 18:09 -0800, Stephanie Wallick wrote: 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.

Re: [PATCH 01/10] staging: unisys: PARSER_CONTEXT_Tag camel case

2014-11-12 Thread Dan Carpenter
Use your full name in the From and Signed-off-by. It's like signing a legal document to show that you haven't violated copyright in sending this patch. regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org

Re: [PATCH 02/10] staging: unisys: parser_init isLocal camel case

2014-11-12 Thread Dan Carpenter
On Tue, Nov 11, 2014 at 09:46:11AM -0500, Jeffrey wrote: Changed the camel case isLocal to islocal in the struct pointer parser_init on line 36 isLocal = islocal islocal is hard to read. Use is_local instead. Also you may as well fix the .c file at the same time. regards, dan carpenter

Re: [PATCH 03/10] staging: unisys: parser_init tryAgain camel case

2014-11-12 Thread Dan Carpenter
On Tue, Nov 11, 2014 at 09:46:12AM -0500, Jeffrey wrote: Changed the camel case tryAgain to tryagain in the struct pointer parser_init on line 36 of parser.h tryAgain = tryagain Gar... No. Don't fix a single word at a time. This is going to take forever. regards, dan carpenter

Re: [PATCH 10/10] staging: unisys: PARSER_WHICH_STRING typedef

2014-11-12 Thread Dan Carpenter
On Tue, Nov 11, 2014 at 09:46:19AM -0500, Jeffrey wrote: Removed the typedef PARSER_WHICH_STRING in parser.h and added enum in front of it at every instance of the variable in parser.c, and parser.h Signed-off-by: Jeffrey jeffrey.br...@unisys.com ---

Re: [PATCH] hv: hv_fcopy: drop the obsolete message on transfer failure

2014-11-12 Thread Vitaly Kuznetsov
Dexuan Cui de...@microsoft.com writes: In the case the user-space daemon crashes, hangs or is killed, we need to down the semaphore, otherwise, after the daemon starts next time, the obsolete data in fcopy_transaction.message or fcopy_transaction.fcopy_msg will be used immediately. Cc: K.

Re: [PATCH 12/30] staging: comedi: dmm32at: use 8255 module for Digital I/O subdevice

2014-11-12 Thread Ian Abbott
On 11/11/14 23:55, H Hartley Sweeten wrote: The Dimond-MM-32-AT board uses an internal 82C55-type digital I/O circuit to provide the 24 digital I/O lines. The only quirk is the need to set the page selection bits in the control register to select page 1 addresses. Instead of duplicating the

Re: [PATCH 00/30] staging: comedi: dmm32at: cleanup driver

2014-11-12 Thread Ian Abbott
On 11/11/14 23:55, H Hartley Sweeten wrote: Here's the big cleanup series for the dmm32at comedi driver. H Hartley Sweeten (30): staging: comedi: dmm32at: make AI (*cancel) actually cancel async command staging: comedi: dmm32at: use comedi_async 'scans_done' to detect EOA staging:

Re: [PATCH with SmPL?] staging: rtl8188eu: Adjustments around jump labels

2014-11-12 Thread SF Markus Elfring
@@ -212,8 +212,7 @@ efuse_phymap_to_logical(u8 *phymap, u16 _offset, u16 _size_byte, u8 *pbuf) exit: kfree(efuseTbl); - if (eFuseWord) - kfree(eFuseWord); + kfree(eFuseWord); I think that this code has been updated already. It would be better to add labels so that kfree

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

2014-11-12 Thread Dan Carpenter
On Mon, Nov 10, 2014 at 06:11:39PM -0500, jes.soren...@redhat.com wrote: diff --git a/drivers/staging/rtl8723au/core/rtw_xmit.c b/drivers/staging/rtl8723au/core/rtw_xmit.c index 18a9f34..f8b1243 100644 --- a/drivers/staging/rtl8723au/core/rtw_xmit.c +++

[PATCH v8 0/9] dw-hdmi: convert imx hdmi to bridge/dw_hdmi

2014-11-12 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 differences, such as phy pll configuration, register width(imx hdmi register is one byte, but rk3288 is 4 bytes width and can only access

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

2014-11-12 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 v8: None Changes in v7: None Changes in v6: None Changes in v5: None Changes in v4: - defer probe ddc i2c adapter Changes in v3: None

[PATCH v8 1/9] staging: imx-drm: imx-hdmi: make checkpatch happy

2014-11-12 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 v8 3/9] staging: imx-drm: imx-hdmi: split imx soc specific code from imx-hdmi

2014-11-12 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 differences, 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 v8 4/9] staging: imx-drm: imx-hdmi: move imx-hdmi to bridge/dw_hdmi

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

[PATCH v8 5/9] dt-bindings: add document for dw_hdmi

2014-11-12 Thread Andy Yan
Signed-off-by: Andy Yan andy@rock-chips.com --- Changes in v8: - correct some spelling mistake - modify ddc-i2c-bus and interrupt description Changes in v7: None Changes in v6: None Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: None

[PATCH v8 6/9] drm: bridge/dw_hdmi: add support for multi byte register width access

2014-11-12 Thread Andy Yan
On rockchip rk3288, only word(32-bit) accesses are permitted for hdmi registers. Byte width accesses (writeb, readb) generate an imprecise external abort. Signed-off-by: Andy Yan andy@rock-chips.com --- Changes in v8: None Changes in v7: None Changes in v6: - move some modification to

[PATCH v8 7/9] drm: bridge/dw_hdmi: convert dw-hdmi to drm_bridge mode

2014-11-12 Thread Andy Yan
From: Yakir Yang y...@rock-chips.com keep the connector birdge in dw_hdmi.c, handle encoder in dw_hdmi-imx.c, as most of the encoder operation are platform specific such as crtc select and panel format set Signed-off-by: Andy Yan andy@rock-chips.com Signed-off-by: Yakir Yang

[PATCH v8 8/9] dt-bindings: Add documentation for rockchip dw hdmi

2014-11-12 Thread Andy Yan
Signed-off-by: Andy Yan andy@rock-chips.com --- Changes in v8: - Add documentation for rockchip dw hdmi Changes in v7: None Changes in v6: None Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: None .../devicetree/bindings/video/dw_hdmi-rockchip.txt | 43

[PATCH v8 9/9] drm: bridge/dw_hdmi: add rockchip rk3288 support

2014-11-12 Thread Andy Yan
rk3288 hdmi is compatible with Designware hdmi this patch is depend on patch by Mark Yao Add drm driver for Rockchip Socs see https://lkml.org/lkml/2014/10/8/201 Signed-off-by: Andy Yan andy@rock-chips.com Signed-off-by: Yakir Yang y...@rock-chips.com --- Changes in v8: - add support for

(( CONTACT DR. JIM OVIA FOR YOUR COMPENSATION PAYMENT ))

2014-11-12 Thread U/N
United Nations Compensation Unit, In Affiliation with World Bank Our Ref: U.N.O/W.B.O/11/2014/1982/05/9. Congratulations Beneficiary, You may not understand why this mail came to you. We have been having a meeting for quit sometime now and we just came to a logical conclusion few

Re: [V2 PATCH 04/10] added media agnostic (MA) USB packet handling

2014-11-12 Thread Oliver Neukum
On Mon, 2014-11-10 at 18:09 -0800, Stephanie Wallick wrote: +/** + * Compares 2 request IDs. Returns true if a is less than b. Handles request id + * wraparound. + */ +bool mausb_req_id_lt(u8 a, u8 b) Unify such functions. It's just silly to have so many of them. +/** + * Calculates the

[PATCH 11/15] staging: unisys: Controlvm_payload_bytes_buffered camel case

2014-11-12 Thread Jeffrey
Changed the static ulong the file parser.c Controlvm_payload_bytes_buffered =controlvm_payload_bytes_buffered Signed-off-by: Jeffrey jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH 15/15] staging: unisys: parser_param_start camel cases

2014-11-12 Thread Jeffrey
Fixed camel cases in the void parser_param_start in parser.c Away = cleanups Signed-off-by: Jeffrey jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/unisys/visorchipset/parser.c

[PATCH 04/15] staging: unisys: parser_simpleString_get camel case

2014-11-12 Thread Jeffrey
changed the char pointer parser_simpleString_get to parser_simple- string_get in the file parser.h and this affected the file parser.h parser_simpleString_get = parser_simplestring_get Signed-off-by: Jeffrey jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c |2 +-

[PATCH 14/15] staging: unisys: parser_init_bytestream camel cases parser.c

2014-11-12 Thread Jeffrey
Fixed the camel cases in parser_init_bytestream in parser.c isLocal = is_local tryAgain = tryagain Signed-off-by: Jeffrey jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 01/15] staging: unisys: PARSER_CONTEXT_Tag camel case

2014-11-12 Thread Jeffrey
Changed the camel case PARSER_CONTEXT_Tag to parser_context_tag in the files parser.h and parser.c PARSER_CONTEXT_Tag = parser_context_tag Signed-off-by: Jeffrey jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c |2 +- drivers/staging/unisys/visorchipset/parser.h |

[PATCH 10/15] staging: unisys: braces parser.c camel case

2014-11-12 Thread Jeffrey
Added pair of braces to an if statement on line 146 Signed-off-by: Jeffrey jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/unisys/visorchipset/parser.c

[PATCH 00/15] parser.h and parser.c patches

2014-11-12 Thread Jeffrey
Sorry for the amount of patches given I was told to be a little conservative with my patches. I grouped the camel cases together based on which struct or void they come from to make it easier. The first seven are patches for parser.h and the last eight are for parser.c, but they will still affect

[PATCH 12/15] staging: unisys: parser_init_guts camel cases

2014-11-12 Thread Jeffrey
Fixed the camel cases for the arguments inside the struct parser_init_struct isLocal = is_local tryAgain = tryagain Away = cleanups hasStandardPayloadHeader = has_standard_payload_header Signed-off-by: Jeffrey jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c | 42

[PATCH 09/15] staging: unisys: Logical continuation parser.c

2014-11-12 Thread Jeffrey
Put || operator on line 390 where it belongs, to have a proper logical continuation Signed-off-by: Jeffrey jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 07/15] staging: unisys: PARSER_WHICH_STRING typedef

2014-11-12 Thread Jeffrey
Removed the typedef PARSER_WHICH_STRING in parser.h and added enum in front of it at every instance of the variable in parser.c, and parser.h Signed-off-by: Jeffrey jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c |2 +- drivers/staging/unisys/visorchipset/parser.h |

[PATCH 05/15] staging: unisys: parser_byteStream_get camel case

2014-11-12 Thread Jeffrey
Changed the camel case parser_byteStream_get to parser_bytestream_ get in parser.h and this also changed in the file parser.c parser_byteStream = parser_bytestream Signed-off-by: Jeffrey jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c |2 +-

[PATCH 02/15] staging: unisys: parser_init parser_init_byteStream camel case

2014-11-12 Thread Jeffrey
Fixed all of the camel cases in the two structs parser_init and parser_init_byteStream in parser.h isLocal = is_local tryAgain = tryagain Signed-off-by: Jeffrey jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.h |6 +++--- 1 files changed, 3 insertions(+), 3

[PATCH 06/15] staging: unisys: PARSER_CONTEXT typedef

2014-11-12 Thread Jeffrey
Removed typedef declaration for PARSER_CONTEXT on line 34 of parser.h and replaced every instance of PARSER_CONTEXT with struct parser_context_tag because PARSER_CONTEXT is actually parser_context_tag Signed-off-by: Jeffrey jeffrey.br...@unisys.com ---

[PATCH 13/15] staging: unisys: parser_init camel cases parser.c

2014-11-12 Thread Jeffrey
Fixed the camel cases inside the definition of the struct parser_init in parser.c isLocal = is_local tryAgain = tryagain Signed-off-by: Jeffrey jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 03/15] staging: unisys: parser_init_byteStream camel case

2014-11-12 Thread Jeffrey
Changed the struct pointer parser_init_byteStream to parser_init_bytestream on line 37 in parser.h, parser.c and visor- chipset_main.c parser_init_byteStream = parser_init_bytestream Signed-off-by: Jeffrey jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c |2 +-

[PATCH 08/15] staging: unisys: spaces after casts parser.c

2014-11-12 Thread Jeffrey
Removed all spaces before casts in parser.c Signed-off-by: Jeffrey jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c | 18 +- 1 files changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/unisys/visorchipset/parser.c

Re: [PATCH 00/15] parser.h and parser.c patches

2014-11-12 Thread Dan Carpenter
Your from header and sign offs are still wrong. Should be your full name. regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: comedi: fix scan_end_arg == chanlist_len assumption

2014-11-12 Thread Ian Abbott
Some comedi drivers allow the `scan_end_arg` value of an asynchronous command to be a multiple ( 1) of the `chanlist_len` although most require them to be the same value. `comedi_bytes_per_scan()` is incorrectly using `chanlist_len` as the length of the scan. Change it to use `scan_end_arg`.

Re: [PATCH 02/15] staging: unisys: parser_init parser_init_byteStream camel case

2014-11-12 Thread Dan Carpenter
On Wed, Nov 12, 2014 at 09:10:01AM -0500, Jeffrey wrote: Fixed all of the camel cases in the two structs parser_init and parser_init_byteStream in parser.h isLocal = is_local tryAgain = tryagain Use try_again instead of tryagain. The one thing per patch rule is a bit vague. Really it's one

[PATCH 12/30] staging: comedi: dmm32at: use 8255 module for Digital I/O subdevice

2014-11-12 Thread Hartley Sweeten
On Wednesday, November 12, 2014 3:12 AM, Ian Abbott wrote: On 11/11/14 23:55, H Hartley Sweeten wrote: The Dimond-MM-32-AT board uses an internal 82C55-type digital I/O circuit to provide the 24 digital I/O lines. The only quirk is the need to set the page selection bits in the control

[PATCH for 3.18] staging: r8188eu: Fix scheduling while atomic error introduced in commit fadbe0cd

2014-11-12 Thread Larry Finger
In commit fadbe0cd5292851608e2e01b91d9295fa287b9fe entitled staging: rtl8188eu:Remove rtw_zmalloc(), wrapper for kzalloc(), the author failed to note that the original code in the wrapper tested whether the caller could sleep, and set the flags argument to kzalloc() appropriately. After the patch,

Re: [PATCH 12/30] staging: comedi: dmm32at: use 8255 module for Digital I/O subdevice

2014-11-12 Thread Ian Abbott
On 12/11/14 16:07, Hartley Sweeten wrote: On Wednesday, November 12, 2014 3:12 AM, Ian Abbott wrote: On 11/11/14 23:55, H Hartley Sweeten wrote: The Dimond-MM-32-AT board uses an internal 82C55-type digital I/O circuit to provide the 24 digital I/O lines. The only quirk is the need to set the

RE: [PATCH] staging: comedi: fix scan_end_arg == chanlist_len assumption

2014-11-12 Thread Hartley Sweeten
On Wednesday, November 12, 2014 9:01 AM, Ian Abbott wrote: Some comedi drivers allow the `scan_end_arg` value of an asynchronous command to be a multiple ( 1) of the `chanlist_len` although most require them to be the same value. `comedi_bytes_per_scan()` is incorrectly using `chanlist_len`

RE: [PATCH 12/30] staging: comedi: dmm32at: use 8255 module for Digital I/O subdevice

2014-11-12 Thread Hartley Sweeten
On Wednesday, November 12, 2014 9:20 AM, Ian Abbott wrote: On 12/11/14 16:07, Hartley Sweeten wrote: The write to the Miscellaneous Control register (DMM32AT_CTRL_REG) in the ISR routine is actually safe. According to the user manual: INTRST Writing a 1 to this location resets the

[PATCH 08/10] staging: unisys: parser.c space after casts

2014-11-12 Thread Jeffrey Brown
Removed all spaces after casts in parser.c Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/unisys/visorchipset/parser.c

[PATCH 06/10] staging: unisys: Controlvm_Payload_Bytes_Buffered camel case

2014-11-12 Thread Jeffrey Brown
Changed the camel case of the static ulong Controlvm_Payload_Bytes_ Buffered Controlvm_Payload_Bytes_Buffered =controlvm_payload_bytes_buffered Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c | 8 1 file changed, 4 insertions(+), 4

[PATCH 04/10] staging: unisys: PARSER_CONTEXT typedef

2014-11-12 Thread Jeffrey Brown
Completely removed all trace of the typedef of PARSER_CONTEXT and replaced all instance of that typedef with parser_context_tag also changed PARSER_CONTEXT_Tag PARSER_CONTEXT_Tag = parser_context_tag Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com ---

[PATCH 00/10] parser.h and parser.c patches

2014-11-12 Thread Jeffrey Brown
Sorry for all of the messy patches from before but here are the improved patches for parser.h and parser.c. The camel cases are grouped on functions and structs that occurs in both files. Jeffrey Brown (10): staging: unisys: parser_init camel cases staging: unisys: parser_init_byteStream

[PATCH 01/10] staging: unisys: parser_init camel cases

2014-11-12 Thread Jeffrey Brown
Fixed camel cases for parser_init isLocal = is_local tryAgain = try_again Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c | 4 ++-- drivers/staging/unisys/visorchipset/parser.h | 3 ++- 2 files changed, 4 insertions(+), 3 deletions(-) diff

[PATCH 02/10] staging: unisys: parser_init_byteStream camel case

2014-11-12 Thread Jeffrey Brown
Changed the camel cases for the struct parser_init_bytestream parser_init_byteStream = parser_init_bytestream isLocal = is_local tryAgain = try_again Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c| 4 ++--

[PATCH 07/10] staging: unisys: parser_init_guts and parser_param_start camel case

2014-11-12 Thread Jeffrey Brown
Fixed camel cases in the struct parser_init_guts and an Away camel case in parser_param_start isLocal = is_local tryAgain = try_again HasStandardPayloadHeader = has_standard_payload_header Away = cleanups Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com ---

[PATCH 05/10] staging: unisys: PARSER_WHICH_STRING typedef

2014-11-12 Thread Jeffrey Brown
Removed the typedef of PARSER_WHICH_STRING and replaced all instance of the typedef with enum parser_which_string. Also changed the name of it to parser_which_string PARSER_WHICH_STRING = parser_which_string Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com ---

[PATCH 09/10] staging: unisys: parser.c braces

2014-11-12 Thread Jeffrey Brown
Inserted a necessary brace for an if statement on line 146 of parser.c Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/unisys/visorchipset/parser.c

[PATCH 03/10] staging: unisys: simpleString_get byteStream_get camel case

2014-11-12 Thread Jeffrey Brown
Fixed the camel case for parser_byteStream_get and parser_simpleString_get parser_simpleString_get = parser_simplestring_get parser_byteStream_get = parser_bytestream_get Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c | 4 ++--

[PATCH 10/10] staging: unisys: parser.c logical continuation

2014-11-12 Thread Jeffrey Brown
Fixed a logical continuation on line 391 by placing the '||' operator on line 390 Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 04/71] staging: unisys: kzalloc visorchipset_main.c

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Inserted preferred sizeof statements for kzalloc in visorchipset_ main.c line 1127: kzalloc(sizeof(*pBusInfo)...) line 1277: kzalloc(sizeof(*pDevInfo)...) Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com ---

[PATCH 00/71] visorchipset_main.c patches

2014-11-12 Thread Jeffrey Brown
I am really sorry about the exterme number of patches I gave you, it was a very large file to take care of I could make the number of patches a little less if you want. The patches start off simple by taking care of trivial stuff like alignment and space after casts. The patches go into the camel

[PATCH 05/71] staging: unisys: visorchipset_main.c braces and spaces

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Inserted and formatted braces properly, also removed unnecessary spaces that were usually surrounding the braces Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- .../unisys/visorchipset/visorchipset_main.c| 24 +- 1

[PATCH 01/71] staging: unisys: space after a cast visorchipset_main.c

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Removed all spaces after casts in visorchipset_main.c Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- .../unisys/visorchipset/visorchipset_main.c| 62 +++--- 1 file changed, 31 insertions(+), 31 deletions(-) diff --git

[PATCH 41/71] staging: unisys: textId camel cases

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the u32 textId to textid in the static ssize_ts textid_show and textid_store textId = textid Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 12 ++-- 1 file changed, 6

[PATCH 10/71] staging: unisys: NotifierLock camel case

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Replaced the static DEFINE_SEMAPHORE NotifierLock with notifierlock NotifierLock = notifierlock Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- .../staging/unisys/visorchipset/visorchipset_main.c| 18 +- 1 file changed, 9

[PATCH 13/71] staging: unisys: invalid bus/device number camel cases

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed two static ulongs that are supposed to represent invalid bus/device numbers g_diagpoolBusNo = g_diagpoolbusno g_diagpoolDevNo = g_diagpooldevno Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com ---

[PATCH 34/71] staging: unisys: Controlvm_Pending_Msg_Valid camel case

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the static BOOL to controlvm_pending_msg_valid Controlvm_Pending_Msg_Valid = controlvm_pending_msg_valid Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 8 1 file changed,

[PATCH 39/71] staging: unisys: toolAction camel cases

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the camel cases of toolAction to toolaction in the static ssize_ts toolaction_store, and toolaction_show toolAction = toolaction Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 12

[PATCH 36/71] staging: unisys: MajorDev camel case

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the static dev_t at line 252 to majordev Major = majordev Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH 03/71] staging: unisys: visorchipset_main.c logical continuations

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Fixed all logical continuations by moving the operators to their correct lines, (which is the line previous to the operator) Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- .../unisys/visorchipset/visorchipset_main.c| 23

[PATCH 18/71] staging: unisys: LIVEDUMP_INFO camel cases

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the camel cases for the variables for the struct LIVEDUMP_ INFO Dumpcapture_header = dumpcapture_header Gettextdump_header = gettextdump_header Dumpcomplete_header = dumpcomplete_header Gettextdump_outstanding = gettextdump_outstanding

[PATCH 02/71] staging: unisys: Alignment fixes visorchipset_main.c

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Fixed all alignment issues in visorchipset_main.c, kept the 80 character warnings in mind when doing so, and if there are any warnings coming from this patch it is most likely that there was an 80 character warning at the line before I fixed the alignment

[PATCH 26/71] staging: unisys: Parahotplug_request_list camel case

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the static LIST_HEAD to parahotplug_request_list Parahotplug_request = parahotplug_request Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 8 1 file changed, 4

[PATCH 42/71] staging: unisys: remainingSteps camel cases

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the camel case of remainingSteps in the static ssize_ts of remaining_steps_show and remaining_steps_store remainingSteps = remainingsteps Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com ---

[PATCH 38/71] staging: unisys: controlvm prototype camel cases

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the camel case msgHdr to msghdr in the static voids controlvm_respond, controlvm_respond_chipset_init, controlvm_- respond_physdev_changestate msgHdr = msghdr Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com ---

[PATCH 47/71] staging: unisys: visorchipset_save_message camel cases

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the camel cases for the void visorchipset_save_message localSavedCrashMsgOffset - localsavedcrashmsgoffset localSavedCrashMsgCount - localsavedcrashmsgcount Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com ---

[PATCH 31/71] staging: unisys: device_destroy_response camel cases

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the camel cases for the static void device_destroy_response busNo - busno devNo - devno Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 6 +++--- 1 file changed, 3 insertions(+), 3

[PATCH 09/71] stating: unisys: Periodic_controlvm_workqueue camel case

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed this static struct pointer to periodic_controlvm_workqueue Periodic_controlvm_workqueue = periodic_controlvm_workqueue Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- .../staging/unisys/visorchipset/visorchipset_main.c| 18

[PATCH 44/71] staging: unisys: devInfo_clear camel case

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the static void devInfo_clear to devinfo_clear devInfo_clear = devinfo_clear Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 6 +++--- 1 file changed, 3 insertions(+), 3

[PATCH 07/71] staging: unisys: Most_recent_message_jiffies

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the static ulong to most_recent_message_jiffies Most_recent_message_jiffies = most_recent_message_jiffies Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 8 1 file

[PATCH 25/71] staging: unisys: atomic_t camel case

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the varaible Visorchipset_cache_buffers_in_use to visorchipset_cache_buffers_in_use Visorchipset_cache_buffer_in_use =visorchipset_cache_buffer_in_use Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com ---

[PATCH 08/71] staging: unisys: Periodic_controlvm_work camel case

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed this camel case to periodic_controlvm_work Periodic_controlvm_work = periodic_controlvm_work Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 14 +++--- 1 file changed, 7

[PATCH 06/71] staging: unisys: Poll_jiffies camel case

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the static ulong Poll_jiffies to poll_jiffies using refactor and rename Poll_jiffies = poll_jiffies Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- .../staging/unisys/visorchipset/visorchipset_main.c | 20 ++-- 1 file

[PATCH 19/71] staging: unisys: Controlvm_payload_info camel case

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed this variable to controlvm_payload_info Controlvm_payload_info = controlvm_payload_info Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 8 1 file changed, 4 insertions(+),

[PATCH 14/71] staging: unisys: controlvm_message_packet camel case

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the camel case g_DeviceChangeStatePacket to g_deicechange- statepacket g_DeviceChangeStatePacket = g_devicechangestatepacket Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 10

[PATCH 23/71] staging: unisys: Putfile_buffer_list_pool camel case

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the static struct Putfile_buffer_list_pool to putfile_buffer_list_pool Putfile_buffer_list_pool = putfile_buffer_list_pool Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 12

[PATCH 60/71] staging: unisys: initialize_controlvm_payload camel case

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Fixed the static void initialize_controlvm_payload camel cases payloadOffset - payloadoffset payloadBytes - payloadbytes Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 10 +- 1

[PATCH 32/71] staging: unisys: device_resume_response camel cases

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the camel cases fore the static void device_resume_response busNo - busno devNo - devno Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 6 +++--- 1 file changed, 3 insertions(+), 3

[PATCH 12/71] staging: unisys: static const uuid_le camel case

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the uuid_le UltraDiagpoolChannelProtocolGuid to ultradiagpoolchannelprotocolguid in visorchipset_main.c UltraDiagpoolChannelProtocolGuid = ultradiagpoolchannelprotocolguid Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com ---

[PATCH 43/71] staging: unisys: busInfo_clear camel cases

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Chagned the static void's name to businfo_clear busInfo_clear = businfo_clear Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)

[PATCH 24/71] staging: unisys: Putfile_request_list camel case

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the static LIST_HEAD to putfile_request_list Putfile_request_list = putfile_request_list Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH 30/71] staging: unisys: device_create_response camel cases

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the camel cases for ethe static void device_create_response busNo = busno devNo = devno Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 6 +++--- 1 file changed, 3 insertions(+), 3

[PATCH 52/71] staging: unisys: device_epilog camel case

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the camel cases in the static void device_epilog busNo - busno devNo - devNo msgHdr - msghdr needResponse - needresponse pDevInfo - pdevinfo Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- .../unisys/visorchipset/visorchipset_main.c

[PATCH 27/71] staging: unisys: visorchipset_busdev_notifiers camel cases

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the camel cases that manages the info for a CONTROLVM_DUMP_ CAPTURESTATE and CONTROLVM/REPORTEVENT BusDev_Server_Notifiers = busdev_server_notifiers BusDev_Client_Notifiers = busdev_client_notifiers Signed-off-by: Jeffrey Brown

[PATCH 45/71] staging: unisys: chipset_init camel cases

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Cleaned up all of the camel cases in the static void chipset_init Away - cleanups Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

[PATCH 20/71] staging: unisys: Livedump_info camel case

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed this static struct LIVEDUMP_INFO into livedump_info Livedump_info = livedump_info Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 6 +++--- 1 file changed, 3 insertions(+), 3

[PATCH 69/71] staging: unisys: CONTROLVM_PAYLOAD_INFO typedef

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the typedef struct CONTROLVM_PAYLOAD_INFO on line 107 to a normal struct and replaced every instnace of CONTROLVM_PAYLOAD_INFO with struct CONTROLVM_PAYLOAD_INFO Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com ---

[PATCH 70/71] staging: unisys: LIVEDUMP_INFO typedef

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the typedef LIVEDUMP_INFO to a normal struct and replaced the only other instance of LIVEDUMP_INFO to struct LIVEDUMP_INFO on line 133 Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c

[PATCH 46/71] staging: unisys: controlvm camel cases

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the camel case of msgHdr for the static voids of controlvm_respond, controlvm_init_response, cnotrolvm_respond_- chipset_init, controlvm_respond_physdev_changestate msgHdr = msghdr Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com ---

[PATCH 21/71] staging: unisys: Controlvm_pending_msg camel case

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed this variable to controlvm_pending_msg Controlvm_pending_msg = controlvm_pending_msg Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 6 +++--- 1 file changed, 3 insertions(+), 3

[PATCH 11/71] staging: unisys: controlvm_message_header camel case

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Changed the camel cases of three controlvm_message_headers that are related to each other g_DiagMsgHdr = g_diagmsghdr g_ChipsetMsgHdr = g_chipsetmsghdr g_DelDumpMsgHdr = g_deldumpmsghdr Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com ---

[PATCH 65/71] staging: unisys: visorchipset_init camel cases

2014-11-12 Thread Jeffrey Brown
From: Jeffrey jeffrey.br...@unisys.com Fixed camel cases in the static int _init at line 2365 Away - cleanups Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-)

  1   2   >