Re: [PATCH]drivers:staging:visorbus:Fix checkpatch warnings

On Sun, Jun 28, 2015 at 10:24:49PM +0530, Ravi Teja wrote: > > This patch fixes checkpatch warnings What warnings? Be specific please. ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverde

[RESEND] mfd: rtsx: add support for rts522A

From: Micky Ching rts522a(rts5227s) is derived from rts5227, and mainly same with rts5227. Add it to file mfd/rts5227.c to support this chip. Signed-off-by: Micky Ching --- drivers/mfd/Kconfig | 7 ++-- drivers/mfd/rts5227.c| 77 ++-- d

[PATCH 11/11] staging: sm750fb: add required spaces around C operators

Add spaces according to checkpatch.pl messages: "ERROR: spaces required around that '=' (ctx:VxV)" "ERROR: need consistent spacing around '-' (ctx:WxV)" "ERROR: spaces required around that '?' (ctx:VxE)" "ERROR: need consistent spacing around '&' (ctx:VxW)" Signed-off-by: Anatoly Stepanov ---

[PATCH 09/11] staging: sm750fb: remove assignment from if condition

Remove assignment from if condition according to checkpatch.pl message: "ERROR: do not use assignment in if condition" Signed-off-by: Anatoly Stepanov --- drivers/staging/sm750fb/sm750_hw.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/staging/sm750fb/sm750

[PATCH 06/11] staging: sm750fb: put open brace on the previous line

Fix open braces according to checkpatch.pl message: "ERROR: that open brace { should be on the previous line" Signed-off-by: Anatoly Stepanov --- drivers/staging/sm750fb/sm750_hw.c | 24 1 file changed, 8 insertions(+), 16 deletions(-) diff --git a/drivers/staging/sm7

[PATCH 08/11] staging: sm750fb: fix 'switch and case' indentation

Fix according to checkpatch.pl message: "ERROR: switch and case should be at the same indent" Signed-off-by: Anatoly Stepanov --- drivers/staging/sm750fb/sm750_hw.c | 172 ++--- 1 file changed, 86 insertions(+), 86 deletions(-) diff --git a/drivers/staging/sm75

[PATCH 04/11] staging: sm750fb: insert space before open parenthesis

Insert spaces before '(' according to checkpatch.pl message: "ERROR: space required before the open parenthesis" Signed-off-by: Anatoly Stepanov --- drivers/staging/sm750fb/sm750_hw.c | 62 +++--- 1 file changed, 31 insertions(+), 31 deletions(-) diff --git a/d

[PATCH 10/11] staging: sm750fb: fix open brace in function declaration

Fix according to checkpatch.pl message: "ERROR: open brace '{' following function declarations go on the next line" Signed-off-by: Anatoly Stepanov --- drivers/staging/sm750fb/sm750_hw.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/sm750fb/sm750_hw.c b

[PATCH 07/11] staging: sm750fb: fix indentation in 'else' statements

Fix indentation according to checkpatch.pl message: "ERROR: space required after that close brace '}'" Signed-off-by: Anatoly Stepanov --- drivers/staging/sm750fb/sm750_hw.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/sm750fb/sm750_hw.c

[PATCH 03/11] staging: sm750fb: insert space between ')' and '{'

Insert spaces according to checkpatch.pl message: "ERROR: space required before the open brace '{'" Signed-off-by: Anatoly Stepanov --- drivers/staging/sm750fb/sm750_hw.c | 40 +++--- 1 file changed, 20 insertions(+), 20 deletions(-) diff --git a/drivers/stagin

[PATCH 05/11] staging: sm750fb: fix indentation of pointer operator

Fix indentation of pointer operator '*' according to checkpatch.pl message: "ERROR: 'foo* bar' should be 'foo *bar' " Signed-off-by: Anatoly Stepanov --- drivers/staging/sm750fb/sm750_hw.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/drivers/

[PATCH 02/11] staging: sm750fb: remove trailing whitespaces

Remove trailing whitespaces according to checkpatch.pl error message: "ERROR: trailing whitespace" Signed-off-by: Anatoly Stepanov --- drivers/staging/sm750fb/sm750_hw.c | 25 - 1 file changed, 12 insertions(+), 13 deletions(-) diff --git a/drivers/staging/sm750fb/sm75

[PATCH 01/11] staging: sm750fb: replace spaces with tabs

Replace spaces with tabs according to checkpatch.pl error message: "ERROR: code indent should use tabs where possible" Signed-off-by: Anatoly Stepanov --- drivers/staging/sm750fb/sm750_hw.c | 70 +++--- 1 file changed, 35 insertions(+), 35 deletions(-) diff --git

[PATCH 00/11] staging: sm750fb: fix checkpatch issues

This patch series fixes chekpatch.pl errors in sm750fb. Anatoly Stepanov (11): staging: sm750fb: replace spaces with tabs staging: sm750fb: remove trailing whitespaces staging: sm750fb: insert space between ')' and '{' staging: sm750fb: insert space before open parenthesis staging: sm750

Re: LIBCFS_ALLOC

Yeah. You're right. Doing a vmalloc() when kmalloc() doesn't have even a tiny sliver of RAM isn't going to work. It's easier to use libcfs_kvzalloc() everywhere, but it's probably the wrong thing. regards, dan carpenter ___ devel mailing list de...@l

New Order

Dear Sir/Madam, Kindly find the attached reversed order quantity, technical details and full list of our demand which we intend to supply on urgent bases to our end buyer. After study please give me feedback promptly so we can finalize discussion on that, because we need the products u

[PATCH REESEND 06/10] Drivers: hv: vmbus: prefer 'die' notification chain to 'panic'

From: Vitaly Kuznetsov current_pt_regs() sometimes returns regs of the userspace process and in case of a kernel crash this is not what we need to report. E.g. when we trigger crash with sysrq we see the following: ... RIP: 0010:[] [] sysrq_handle_crash+0x16/0x20 RSP: 0018:8800db0a7d88 E

[PATCH RESEND 05/10] Drivers: hv: vmbus: add special crash handler

From: Vitaly Kuznetsov Full kernel hang is observed when kdump kernel starts after a crash. This hang happens in vmbus_negotiate_version() function on wait_for_completion() as Hyper-V host (Win2012R2 in my testing) never responds to CHANNELMSG_INITIATE_CONTACT as it thinks the connection is alrea

[PATCH RESEND 01/10] Drivers: hv: vmbus: remove hv_synic_free_cpu() call from hv_synic_cleanup()

From: Vitaly Kuznetsov We already have hv_synic_free() which frees all per-cpu pages for all CPUs, let's remove the hv_synic_free_cpu() call from hv_synic_cleanup() so it will be possible to do separate cleanup (writing to MSRs) and final freeing. This is going to be used to assist kexec. Signed

[PATCH RESEND 10/10] Drivers: hv: vmbus: Permit sending of packets without payload

The guest may have to send a completion packet back to the host. To support this usage, permit sending a packet without a payload - we would be only sending the descriptor in this case. Signed-off-by: K. Y. Srinivasan --- drivers/hv/channel.c |4 +++- 1 files changed, 3 insertions(+), 1 dele

[PATCH RESEND 02/10] kexec: define kexec_in_progress in !CONFIG_KEXEC case

From: Vitaly Kuznetsov If some piece of code wants to check kexec_in_progress it has to be put in #ifdef CONFIG_KEXEC block to not break the build in !CONFIG_KEXEC case. Overcome this limitation by defining kexec_in_progress to false. Signed-off-by: Vitaly Kuznetsov Signed-off-by: K. Y. Sriniva

[PATCH RESEND 04/10] Drivers: hv: don't do hypercalls when hypercall_page is NULL

From: Vitaly Kuznetsov At the very late stage of kexec a driver (which are not being unloaded) can try to post a message or signal an event. This will crash the kernel as we already did hv_cleanup() and the hypercall page is NULL. Move all common (between 32 and 64 bit code) declarations to the

[PATCH RESEND 07/10] Drivers: hv: kvp: check kzalloc return value

From: Vitaly Kuznetsov kzalloc() return value check was accidentally lost in 11bc3a5fa91f: "Drivers: hv: kvp: convert to hv_utils_transport" commit. We don't need to reset kvp_transaction.state here as we have the kvp_timeout_func() timeout function and in case we're in OOM situation it is prefe

[PATCH RESEND 09/10] Drivers: hv: balloon: Enable dynamic memory protocol negotiation with Windows 10 hosts

From: Alex Ng Support Win10 protocol for Dynamic Memory. Thia patch allows guests on Win10 hosts to hot-add memory even when dynamic memory is not enabled on the guest. Signed-off-by: Alex Ng Signed-off-by: K. Y. Srinivasan --- drivers/hv/hv_balloon.c | 26 -- 1 fil

[PATCH RESEND 08/10] Drivers: hv: fcopy: dynamically allocate smsg_out in fcopy_send_data()

From: Vitaly Kuznetsov struct hv_start_fcopy is too big to be on stack on i386, the following warning is reported: >> drivers/hv/hv_fcopy.c:159:1: warning: the frame size of 1088 bytes is larger >> than 1024 bytes [-Wframe-larger-than=] Reported-by: kbuild test robot Signed-off-by: Vitaly Kuz

[PATCH RESEND 03/10] Drivers: hv: vmbus: add special kexec handler

From: Vitaly Kuznetsov When general-purpose kexec (not kdump) is being performed in Hyper-V guest the newly booted kernel fails with an MCE error coming from the host. It is the same error which was fixed in the "Drivers: hv: vmbus: Implement the protocol for tearing down vmbus state" commit - mo

[PATCH RESEND 00/10] Drivers: hv: vmbus: Enable kexec and other misc cleanup

In addition to enabling kexec, this patch-set has a bunch of miscellaneous fixes. I have been having mail issues and hence resending the series. Alex Ng (1): Drivers: hv: balloon: Enable dynamic memory protocol negotiation with Windows 10 hosts K. Y. Srinivasan (1): Drivers: hv: vmbus: P

[PATCH]drivers:staging:visorbus:Fix checkpatch warnings

This patch fixes checkpatch warnings Signed-off-by: Ravi Teja Darbha --- drivers/staging/unisys/visorbus/visorchipset.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/unisys/visorbus/visorchipset.c b/drivers/staging/unisys/visorbus/visorchipset.c