* K. Y. Srinivasan wrote:
> If the hypervisor supports MSR based access to the APIC registers
> (EOI, TPR and ICR), implement the MSR based access.
>
> Signed-off-by: K. Y. Srinivasan
> ---
> arch/x86/include/uapi/asm/hyperv.h |5 +++
> arch/x86/kernel/cpu/mshyperv.c | 69
> +++
On Fri, Mar 13, 2015 at 12:53:16AM +0100, Mateusz Kulikowski wrote:
> - 'WARNING: break quoted strings at a space character'
> @@ -1361,9 +1365,10 @@ static u8 HwRateToMRate90(bool bIsHT, u8 rate)
> break;
>
> default:
> - RT_TRACE(COMP_REC
On Fri, Mar 13, 2015 at 05:42:03PM +0100, Isaac Lleida wrote:
> From: isaky
Not your name. Leave this out unless you are passing on someone else's
patch.
>
> This path implements a bit array representing the LCD signal states instead
> of the old "struct bits", which used char to represent a
On Sun, Mar 15, 2015 at 10:03:56AM +0100, Isaac Lleida wrote:
> This path implements a bit array representing the LCD signal states instead
> of the old "struct bits", which used char to represent a single bit. This
> will reduce the memory usage.
Sudip told you to line wrap the changelog but it
What Joe said. Also the subject is too long, there is no changelog and
no signed-off-by.
regards,
dan carpenter
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
We have made helping people our business. We only do data recovery, we do
it all day, every day - we have become the very best in this field. We have
seen all types of problems and solved nearly all of them!
Why we are here?
Data – Important assets of enterprises and individuals, is the key
eleme
This patch fixes this sparse warning
CHECK drivers/staging/sm750fb/ddk750_swi2c.c
drivers/staging/sm750fb/ddk750_swi2c.c:223:6: warning: symbol
'swI2CStart' was not declared. Should it be static?
drivers/staging/sm750fb/ddk750_swi2c.c:234:6: warning: symbol
'swI2CStop' was not declared. Shou
On Sun, Mar 15, 2015 at 05:21:09PM -0700, Ragavendra Nagraj wrote:
the convention to write the subject is :
staging: sm750fb: your subject line
this is for all drivers, if you see the mails in lkml you will see
that almost everyone follows the same convention.
>
> Signed-off-by: Ragavendra Nagraj
> -Original Message-
> From: Ingo Molnar [mailto:mingo.kernel@gmail.com] On Behalf Of Ingo
> Molnar
> Sent: Monday, March 16, 2015 12:37 AM
> To: KY Srinivasan
> Cc: x...@kernel.org; gre...@linuxfoundation.org; linux-
> ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle
On Sun, Mar 15, 2015 at 09:39:43PM +0100, Mateusz Kulikowski wrote:
> Fix checkpatch.pl warnings:
> - 'WARNING: quoted string split across lines'
> - 'WARNING: break quoted strings at a space character'
>
> Signed-off-by: Mateusz Kulikowski
> ---
This patch fails to apply to my staging-testing b
On Sat, Mar 07, 2015 at 10:56:53AM +0530, Sudip Mukherjee wrote:
> modified the code to keep the logic same but removed some indention.
>
> Signed-off-by: Sudip Mukherjee
> ---
>
> this patch will generate checkpatch warning about line more than 80char,
> and too many use of tab. but unless the
On Sat, Mar 07, 2015 at 10:56:52AM +0530, Sudip Mukherjee wrote:
> these variables were assigned some values but they were never being
> reused again.
But some of them should have been checked, right? Or, if no one cares,
fix up the function to not return anything, like for all of the
read_regist
On Thu, Mar 12, 2015 at 07:44:20PM +0530, Sudip Mukherjee wrote:
> these variable were being assigned some values but never reused.
>
> Signed-off-by: Sudip Mukherjee
> ---
> drivers/staging/unisys/virthba/virthba.c | 18 ++
> drivers/staging/unisys/virtpci/virtpci.c | 22 +++
On Thu, Mar 12, 2015 at 11:00:09PM +0530, Sudip Mukherjee wrote:
> these variable were being assigned some values but never reused.
>
> Signed-off-by: Sudip Mukherjee
> ---
>
> v2: no change
Then why resend?
confused,
greg k-h
___
devel mailing list
On Thu, Mar 12, 2015 at 11:00:10PM +0530, Sudip Mukherjee wrote:
> dev_t is defined to be of unsigned int type, no use initializing
> it to -1.
>
> Signed-off-by: Sudip Mukherjee
> ---
> v2: it was not in v1
>
> drivers/staging/unisys/visorchipset/file.c | 2 +-
> 1 file changed, 1 insertion(+)
This error is found by smatch tool.
Signed-off-by: Madhusudhanan Ravindran
---
drivers/staging/fbtft/fb_agm1264k-fl.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/staging/fbtft/fb_agm1264k-fl.c
b/drivers/staging/fbtft/fb_agm1264k-fl.c
index 578fdbe..939b7b9 100644
--- a/driver
On 03/15/2015 08:07 AM, Mark Hounschell wrote:
On 03/14/2015 04:44 AM, Greg KH wrote:
On Fri, Mar 13, 2015 at 04:55:55PM -0400, Mark Hounschell wrote:
On 03/12/2015 12:14 PM, Giedrius Statkevičius wrote:
On 2015.03.12 12:08, Greg KH wrote:
On Mon, Mar 09, 2015 at 06:29:38PM +0200, Giedrius St
On Thu, Mar 12, 2015 at 08:08:24PM +0300, Dan Carpenter wrote:
> "brd->nasync" amd "brd->maxports" are the same. They hold the number of
> filled out channels in the brd->channels[] array. These tests should
> be ">=" instead of ">" so that we don't read one element past the end.
>
> Signed-off-
Fix all comparisons to NULL so that they don't use NULL anymore.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/virthba/virthba.c | 30 +++---
1 file changed, 15 insertions(+), 15 deletions(-)
diff --git a/drivers/staging/unisys/virthba/virthba.c
b/drivers/sta
Fix the CamelCase global variable in visorchipset_main.c:
Most_recent_message_jiffies => most_recent_message_jiffies
Update all references to use the corrected name.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 8
1 file changed, 4 inserti
Fix the name and references:
Poll_jiffies => poll_jiffies
Signed-off-by: Benjamin Romer
---
.../unisys/visorchipset/visorchipset_main.c| 22 ++
1 file changed, 10 insertions(+), 12 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset/visorchipset_main.c
b/d
Fix this CamelCase function name:
parser_byteStream_get => parser_byte_stream_get
Update all references to use the fixed name. Fix the spacing in the typecast.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/parser.c | 5 ++---
drivers/staging/unisys/visorchipset/parser.h
Fix camelcase name:
Controlvm_Payload_Bytes_Buffered => controlvm_payload_bytes_buffered
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/parser.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset/parser.c
b/driv
Fix CamelCase names:
isLocal => local
tryAgain => retry
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/parser.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset/parser.c
b/drivers/staging/unisys/visorchipset/parse
Rename the channel:
ControlVm_channel => controlvm_channel
Update all references to use the new name. Fix the odd bracketing in the
while() statement with the reference to controlvm_channel, too, so the
patch doesn't generate a warning.
Signed-off-by: Benjamin Romer
---
.../unisys/visorchipset
Rename the semaphore
NotifierLock => notifier_lock
Update all references to use the new name.
Signed-off-by: Benjamin Romer
---
.../staging/unisys/visorchipset/visorchipset_main.c| 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/unisys/vi
Fix the CamelCase variable names:
Periodic_controlvm_work => periodic_controlvm_work
Periodic_controlvm_workqueue => periodic_controlvm_workqueue
Update all references to use the corrected names.
Signed-off-by: Benjamin Romer
---
.../unisys/visorchipset/visorchipset_main.c| 32 +
Nobody is using this, so remove it.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset/visorchipset_main.c
b/drivers/staging/unisys/visorchipset/visorchipset_main.c
i
Remove unnecessary NULLs by just using the pointer value directly.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/virtpci/virtpci.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/unisys/virtpci/virtpci.c
b/drivers/staging/unisys/virtpci/vi
Rename CamelCase local variable:
Features_addr => features_addr
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/virthba/virthba.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/unisys/virthba/virthba.c
b/drivers/staging/unisys/virthba/virthba.
Just remove a couple of extra spaces after typecasts.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset/visorchipset_main.c
b/drivers/staging/unisy
Fix the CamelCase variable names:
g_DiagMsgHdr => g_diag_msg_hdr
g_ChipSetMsgHdr => g_chipset_msg_hdr
g_DelDumpMsgHdr => g_del_dump_msg_hdr
Update all references to use the corrected names.
Signed-off-by: Benjamin Romer
---
.../unisys/visorchipset/visorchipset_main.c| 28 +++---
Take out the variable used to compute a comparison against NULL, and
just use the parameter directly.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/virtpci/virtpci.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/virtpci/virtpci.c
b/drive
There is an unused variable in the visorchipset_bus_info structure that
used to be for a proc entry, so remove it, and the code that referenced
it. We don't need it anymore.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset.h | 1 -
drivers/staging/unisys/vi
Correct all comparisons with NULL so that NULL isn't needed.
Signed-off-by: Benjamin Romer
---
.../unisys/visorchannel/visorchannel_funcs.c | 23 +++---
1 file changed, 11 insertions(+), 12 deletions(-)
diff --git a/drivers/staging/unisys/visorchannel/visorchannel_funcs.c
Rename the CamelCased list names:
BusInfoList => bus_info_list
DevInfoList => dev_info_list
Update all references to use the fixed names.
Signed-off-by: Benjamin Romer
---
.../unisys/visorchipset/visorchipset_main.c| 48 +++---
1 file changed, 24 insertions(+), 24 delet
Fix CamelCase global variable:
UltraDiagPoolChannelProtocolGuid => spar_diag_pool_channel_protocol_uuid
Update all references to use the fixed name.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions
Fix CamelCase names:
isLocal => local
hasStandardPayloadHeader => standard_payload_header
tryAgain => retry
Away => cleanup
Fix spacing after typecasts, add missing braces to the if statement, and
eliminate the NULL comparison by just using the pointer value directly.
Signed-off-by: Benjamin Rom
Simplify the code a little and shorten the indentation levels by
reversing the check for time_after_eq().
Signed-off-by: Benjamin Romer
---
.../unisys/visorchipset/visorchipset_main.c| 22 --
1 file changed, 12 insertions(+), 10 deletions(-)
diff --git a/drivers/stag
Fix the CamelCase label:
Away => cleanup
Update all references to use the fixed name.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset
Convert all references to PARSER_CONTEXT into struct parser_context.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/parser.c | 28 +++---
drivers/staging/unisys/visorchipset/parser.h | 22 -
.../unisys/visorchipset/visorchipset_m
The block of code is old and #if 0'd out, so just remove it.
Signed-off-by: Benjamin Romer
---
.../unisys/visorchipset/visorchipset_main.c| 29 --
1 file changed, 29 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset/visorchipset_main.c
b/drivers/staging/
Rename CamelCased local variable names:
localSavedCrashMsgOffset => crash_msg_offset
localSavedCrashMsgCount => crash_msg_count
Update references to use the new names.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 16
1 file change
Just fix the spacing around the logical or operator in this function.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/unisys/visorchipset/visorchipset_main.c
b/drivers/s
Fix CamelCase names:
busNo => bus_no
pBusInfo => bus_info
Update all references to use the corrected names, and remove the goto
statements and labels entirely.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 17 ++---
1 file changed, 6 in
Rename the CamelCase global structure
ControlVm_payload_info => controlvm_payload_info
Move the declaration to the struct definition, and update all references
to use the new name.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 14 ++
1
Fix the CamelCase global names:
ControlVm_Pending_Msg => controlvm_pending_msg
ControlVm_Pending_Msg_Valid => controlvm_pending_msg_valid
Update all references to use the corrected names.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 14 +++
Rename the CamelCase global:
MajorDev => major_dev
Update references to use the corrected name.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/unisys/visorchip
Fix these CamelCased names:
BusDev_Server_Notifiers => busdev_server_notifiers
BusDev_Client_Notifiers => busdev_client_notifiers
Update references to use the fixed names.
Signed-off-by: Benjamin Romer
---
.../unisys/visorchipset/visorchipset_main.c| 40 +++---
1 file c
Fix the CamelCase-named structure clearing functions in
visorchipset_main.c:
busInfo_clear => bus_info_clear
devInfo_clear => dev_info_clear
Update references to use the fixed names.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 12 ++--
1
Rename this variable:
Visorchipset_cache_buffers_in_use => visorchipset_cache_buffers_in_use
Update all references to use the fixed name.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff
Fix the variable's CamelCase name:
g_DeviceChangeStatePacket => g_devicechangestate_packet
Update all references to use the fixed name.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
di
Fix the CamelCase variable names:
gotACommand => got_command
Poll_Count => poll_count
Away => cleanup
Fix all references to use the new names. Remove the empty braces in the
while() and put in a trailing semicolon, for clarity.
Signed-off-by: Benjamin Romer
---
.../unisys/visorchipset/visorchi
Fix all of the NULL comparison checks generated by checkpatch.pl.
Signed-off-by: Benjamin Romer
---
.../staging/unisys/visorchipset/visorchipset_main.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset/visorchipset_mai
Fix local CamelCase variable names:
busNo => bus_no
devNo => dev_no
pDevinfo => dev_info
pBusInfo => bus_info
Away => cleanup
Update references to use the corrected names, and change the kzalloc()
to use the variable itself rather than the type.
Signed-off-by: Benjamin Romer
---
.../unisys/vis
Fix the CamelCase parameter
msgHdr => msg_hdr
In all of the chipset functions. Update all references to use the corrected
name.
Signed-off-by: Benjamin Romer
---
.../unisys/visorchipset/visorchipset_main.c| 22 +++---
1 file changed, 11 insertions(+), 11 deletions(-)
d
Fix the CamelCase parameters:
busNo => bus_no
devNo => dev_no
In the functions bus_create_response(), bus_destroy_response(),
device_create_response(), device_destroy_response(), and
device_resume_response().
Fix the CamelCase global structure name
BusDev_Responders => busdev_responders
And up
All of the proc code using easyproc and procobjecttree support is gone,
so we can remove it from the source entirely.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorutil/Makefile | 3 +-
drivers/staging/unisys/visorutil/easyproc.c | 356 --
driv
Rename the CamelCase variable:
Visorchipset_platform_device => visorchipset_platform_device
Update all references to use the corrected name.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 16
1 file changed, 8 insertions(+), 8 delet
Rename the CamelCase local
toolAction => tool_action
In both of the sysfs functions for exposing this value, and update all
references to use the corrected name.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 12 ++--
1 file changed, 6 inser
Fix the CamelCase member names and that global variable:
Dumpcapture_header => dumpcapture_header
Gettextdump_header => gettextdump_header
Dumpcomplete_header => dumpcomplete_header
Gettextdump_outstanding => gettextdump_outstanding
LiveDump_info => livedump_info
Update all references and merge t
Fix the CamelCase variable names:
busNo => bus_no
devNo => dev_no
pDevInfo => dev_info
Update all the references to the updated names, then remove the goto and
its label.
Signed-off-by: Benjamin Romer
---
.../staging/unisys/visorchipset/visorchipset_main.c | 21 +
1 file ch
Rename the CamelCase parameters:
busNo => bus_no
devNo => dev_no
needResponse => need_response
And the local variables
pBusInfo => bus_info
pDevInfo => dev_info
Update all references to use the corrected names.
Signed-off-by: Benjamin Romer
---
.../unisys/visorchipset/visorchipset_main.c
Fix the CamelCased local variable
textId => text_id
In both of the textid sysfs functions, and update all references to use the
corrected name.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 12 ++--
1 file changed, 6 insertions(+), 6 deleti
First, fix the CamelCase names:
busNo => bus_no
pBusInfo => bus_info
Update all references to use the corrected names, and remove the messy
goto and label.
Signed-off-by: Benjamin Romer
---
.../unisys/visorchipset/visorchipset_main.c| 49 ++
1 file changed, 22 inser
Fix CamelCase names:
parser_init_byteStream => parser_init_byte_stream
isLocal => local
tryAgain => retry
Update references to the changed function name.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/parser.c| 4 ++--
drivers/staging/unisys/visorchipset/parse
This set of patches cleans up most of the remaining checkpatch.pl issues in the
unisys tree.
Benjamin Romer (69):
staging: unisys: clean up NULL comparisons in visorchannel_funcs.c
staging: unisys: cleanup NULL comparisons in virthba.c
staging: unisys: fix CamelCase in virthba_probe()
stag
Fix the CamelCase names:
Parahotplug_request_list => parahotplug_request_list
Parahotplug_request_list_lock => parahotplug_request_list_lock
Update references to use the corrected names.
Signed-off-by: Benjamin Romer
---
.../unisys/visorchipset/visorchipset_main.c| 24 +++--
Take out extra braces from if statements with only a single line of code
inside.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/uislib/uislib.c | 18 ++
1 file changed, 6 insertions(+), 12 deletions(-)
diff --git a/drivers/staging/unisys/uislib/uislib.c
b/drivers/stag
Fix the CamelCase parameter names in the set of responder functions:
cmdId => cmd_id
busNo => bus_no
devNo => dev_no
responseState => response_state
Update all references to use the fixed names.
Signed-off-by: Benjamin Romer
---
.../unisys/visorchipset/visorchipset_main.c| 42 +
Fix up the line spacing around braces in visorchipset_main.c.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 4
1 file changed, 4 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset/visorchipset_main.c
b/drivers/staging/unisys/visorch
Rename the local CamelCase variable
efiSparIndication => efi_spar_indication
Update all references to use the corrected name.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git
First, fix all CamelCase names:
busNo => bus_no
pBusInfo => bus_info
Away => cleanup
Fix the kzalloc() call so it uses the variable name rather than the
type, and update all of the references to the renamed variables and
labels.
Signed-off-by: Benjamin Romer
---
.../unisys/visorchipset/visorch
Fix CamelCase names:
localCrashCreateBusMsg => local_crash_bus_msg
localCrashCreateDevMsg => local_crash_dev_msg
localSavedCrashMsgOffset => local_crash_msg_offset
localSavedCrashMsgCount => local_crash_msg_count
Away => cleanup
Update all references to use the fixed names.
Signed-off-by: Benjam
First, fix the CamelCase local variable names:
parametersAddr => parm_addr
parametersBytes => parm_bytes
isLocalAddr => local_addr
Change the type of local_addr to bool instead of using the #defined
type. Update all references to use the fixed names. Fix the indentation of that
line that's over t
Just remove the NULL from the check and test the pointer directly.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/unisys/visorchipset/visorchipset_main.c
b/drivers/stag
Change pointer comparisons to NULL to just use the pointer directly.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/uislib/uislib.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/unisys/uislib/uislib.c
b/drivers/staging/unisys/uislib
Fix the CamelCase names:
busNo => bus_no
devNo => dev_no
pDevInfo => dev_info
Update all references to use the new names, and remove the goto and
label entirely.
Signed-off-by: Benjamin Romer
---
.../unisys/visorchipset/visorchipset_main.c| 27 ++
1 file changed, 12
Fix the CamelCase parameter name
msgHdr => msg_hdr
In all of the controlvm functions in visorchipset_main.c. Update all references
to use the corrected name.
Signed-off-by: Benjamin Romer
---
.../unisys/visorchipset/visorchipset_main.c| 24 +++---
1 file changed, 12 ins
Fix as many parenthesis alignment problems in this file as possible.
In cases where there was no good way to align to the leading parenthesis, the
content of the parenthesis were aligned as if there were no limit, and then the
leading indentation was moved back to one tab in from the previous line
Fix CamelCase names:
g_diagpoolBusNo => g_diagpool_bus_no
g_diagpoolDevNo => g_diagpool_dev_no
Update all references to the changed names.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 16
1 file changed, 8 insertions(+), 8 deletio
Fix the CamelCase local variables:
payloadOffset => payload_offset
payloadBytes => payload_bytes
Update all references to use the new names.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-
Fix the CamelCased goto label:
Away => cleanup
and get rid of the NULL comparison.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset/visorc
Several if statement lines had the logical operators in the wrong place,
so fix these by moving the operator to the end of the previous line.
Signed-off-by: Benjamin Romer
---
.../staging/unisys/visorchipset/visorchipset_main.c | 20 ++--
1 file changed, 10 insertions(+), 10 del
Rename the CamelCase local variable
remainingSteps => remaining_steps
Update all references to use the corrected name.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/driv
Fix the CamelCase global names:
Putfile_buffer_list_pool => putfile_buffer_list_pool
Putfile_buffer_list_pool_name => putfile_buffer_list_pool_name
Putfile_request_list => putfile_request_list
Update all references to use the corrected names.
Signed-off-by: Benjamin Romer
---
.../staging/unisy
Rename the label in both places:
Away => cleanup
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset/visorchipset_main.c
b/drivers/staging/unisys/vi
Fix all of the bracing mistakes by adding missing braces, and removing
unnecessary braces.
Signed-off-by: Benjamin Romer
---
.../staging/unisys/visorchipset/visorchipset_main.c | 20 +++-
1 file changed, 11 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/unisys/visor
> -Original Message-
> From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com]
> Sent: Wednesday, March 11, 2015 6:29 AM
> To: KY Srinivasan; de...@linuxdriverproject.org
> Cc: Haiyang Zhang; linux-ker...@vger.kernel.org; Dexuan Cui; Radim Krcmar;
> Greg Kroah-Hartman; linux-...@vger.kernel.o
On Mon, Mar 16, 2015 at 02:47:43PM -0300, casca...@linux.vnet.ibm.com wrote:
> On Thu, Mar 12, 2015 at 08:08:24PM +0300, Dan Carpenter wrote:
> > "brd->nasync" amd "brd->maxports" are the same. They hold the number of
> > filled out channels in the brd->channels[] array. These tests should
> > be
On 16 March 2015 at 10:51, Ricardo Ribalda Delgado
wrote:
> This patch fixes this sparse warning
Hi Ricardo,
I currently have a patch series awaiting the final ok which includes
fixes for these warnings
(http://lkml.iu.edu/hypermail/linux/kernel/1503.1/02535.html),
apologies but assuming there i
This small patchset adds const to of_device_id arrays in
drivers branch.
Fabian Frederick (35):
ata: constify of_device_id array
regulator: constify of_device_id array
thermal: constify of_device_id array
tty/hvc_opal: constify of_device_id array
tty: constify of_device_id array
power:
of_device_id is always used as const.
(See driver.of_match_table and open firmware functions)
Signed-off-by: Fabian Frederick
---
drivers/staging/octeon-usb/octeon-hcd.c | 2 +-
drivers/staging/octeon/ethernet.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers
On Thu, Mar 12, 2015 at 02:16:23PM +, KY Srinivasan wrote:
>
>
> > -Original Message-
> > From: Greg KH [mailto:gre...@linuxfoundation.org]
> > Sent: Thursday, March 12, 2015 6:29 AM
> > To: KY Srinivasan
> > Cc: a...@canonical.com; de...@linuxdriverproject.org; o...@aepfle.de;
> > li
Hello Lorenzo
On Mon, Mar 16, 2015 at 8:03 PM, Lorenzo Stoakes wrote:
>
> I currently have a patch series awaiting the final ok which includes
> fixes for these warnings
> (http://lkml.iu.edu/hypermail/linux/kernel/1503.1/02535.html),
> apologies but assuming there isn't something hideously wron
> -Original Message-
> From: Greg KH [mailto:gre...@linuxfoundation.org]
> Sent: Monday, March 16, 2015 1:22 PM
> To: KY Srinivasan
> Cc: a...@canonical.com; de...@linuxdriverproject.org; o...@aepfle.de;
> linux-ker...@vger.kernel.org
> Subject: Re: [PATCH 1/6] Drivers: hv: vmbus: Perform
On 16.03.2015 09:44, Dan Carpenter wrote:
> On Fri, Mar 13, 2015 at 12:53:16AM +0100, Mateusz Kulikowski wrote:
>> -RT_TRACE(COMP_RECV, "HwRateToMRate90(): Non supported"
>> - "Rate [%x], bIsHT = %d!!!\n", rate, bIsHT);
>> -
On 13.03.2015 01:27, Joe Perches wrote:
>
> Another thing you might consider is to change these
> embedded function names in another patch to use
> "%s: ", __func__
>
> RT_TRACE(COMP_ERR,
>"%s: "unknown hardware version\n
dmam_alloc_coherent does not return a __iomem pointer.
here is its prototype:
void * dmam_alloc_coherent(struct device *dev, size_t size,
dma_addr_t *dma_handle, gfp_t gfp)
This fixes these sparse warnings:
drivers/staging/goldfish/goldfish_audio.c:134:43: warning: inc
Add new function ft1000_read_dsp_timer() replacing recurring code block for
reading DSP timer. Such code refactoring solves all remaining "line over 80
characters" warnings reported by checkpatch.pl.
Signed-off-by: Daniele Alessandrelli
---
drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 199
1 - 100 of 120 matches
Mail list logo
