On Wed, 18 Dec 2013, micky_ch...@realsil.com.cn wrote:
> From: Micky Ching
>
> we add a macro to simplify setting pull control, and use a common init
> function to init the common params for 8411-like chips. at last we add
> support for rtl8402 chip.
>
> Micky Ching (2):
> mfd: rtsx: add set
Gary emailed me privately that fixing my comments would be a problem.
1) Greg complained about your earlier patch where you moved the
comments from this:
/* comment */
code();
To this:
code(); /* comment */
Greg's complaint was not about that you were doing more than one
On Wed, 18 Dec 2013, Dan Carpenter wrote:
Date: Wed, 18 Dec 2013 17:00:16 +0300
From: Dan Carpenter
To: Gary Rookard
Cc: de...@driverdev.osuosl.org, gre...@linuxfoundation.org,
l...@xenapiadmin.com, j...@joshtriplett.org, nandu.hgo...@gmail.com
Subject: Re: [PATCH] Staging: bcm: DDRInit: f
- Original Message -
> From: "Greg KH"
> To: "Aaron Sierra"
> Cc: "martyn welch" , de...@driverdev.osuosl.org,
> "manohar vanga" , "dan
> carpenter"
> Sent: Tuesday, December 17, 2013 7:06:06 PM
> Subject: Re: [PATCH v2] vme_user: Update API to work in mixed environments
>
> On Mon, De
This patch fixes "ERROR: trailing whitespace" found by
checkpatch.pl, within dgap_sysfs.c
Signed-off-by: Masanari Iida
---
drivers/staging/dgap/dgap_sysfs.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/dgap/dgap_sysfs.c
b/drivers/staging/dg
Now that the VME userspace API structures compile to a consistent size
in mixed environments (32-bit userspace and 64-bit kernel), enable the
.compat_ioctl to allow ioctls to execute in this environment.
Signed-off-by: Aaron Sierra
---
drivers/staging/vme/devices/vme_user.c |1 +
1 file chan
Signed-off-by: Aaron Sierra
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/vme/devices/vme_user.h | 22 +++---
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/drivers/staging/vme/devices/vme_user.h
b/drivers/staging/vme/devices/vme_user.h
index c3b4964.
This patch fixes "ERROR: trailing whitespace" found by
checkpatch.pl, within dgap_driver.h
Signed-off-by: Masanari Iida
---
drivers/staging/dgap/dgap_driver.h | 22 +++---
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/drivers/staging/dgap/dgap_driver.h
b/driver
This patch fixes "ERROR: trailing whitespace" found by
checkpatch.pl, within dgap_downld.h and dgap_sysfs.h.
Signed-off-by: Masanari Iida
---
drivers/staging/dgap/dgap_downld.h | 4 ++--
drivers/staging/dgap/dgap_sysfs.h | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/dri
This patch fixes "ERROR: trailing whitespace" found by
checkpatch.pl, within dgap_fep5.h
Signed-off-by: Masanari Iida
---
drivers/staging/dgap/dgap_fep5.h | 45
1 file changed, 22 insertions(+), 23 deletions(-)
diff --git a/drivers/staging/dgap/dgap_fep5
This patch fixed "ERROR:trailing whitespace
found by checkpatch.pl within dgap_tty.c
Signed-off-by: Masanari Iida
---
drivers/staging/dgap/dgap_tty.c | 174
1 file changed, 87 insertions(+), 87 deletions(-)
diff --git a/drivers/staging/dgap/dgap_tty.c b/
This patch fix "ERROR: trailing whitespace found by
checkpatch.pl, whithin downld.c
Signed-off-by: Masanari Iida
---
drivers/staging/dgap/downld.c | 168 +-
1 file changed, 84 insertions(+), 84 deletions(-)
diff --git a/drivers/staging/dgap/downld.c b/dri
I have deleated some unintelligible comments, and made a
few minor white space corrections.
Signed-off-by: Gary Alan Rookard
---
On branch staging-next
drivers/staging/bcm/DDRInit.c | 149 +-
1 file changed, 74 insertions(+), 75 deletions(-)
diff --git a/d
The following changes since commit 319e2e3f63c348a9b66db4667efa73178e18b17d:
Linux 3.13-rc4 (2013-12-15 12:31:33 -0800)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git/
tags/staging-3.13-rc5
for you to fetch changes up to fd6040ed57d
Removed the developer debug prints BCM_DEBUG_PRINT()
as per the TODO list, also removed braces for the
if-statement to match coding style
Signed-off-by: Gokulnath A
---
drivers/staging/bcm/InterfaceDld.c | 70 --
1 file changed, 14 insertions(+), 56 deletions(
As per comments from Greg KH and Dan Carpenter updated and
split the patch series.
Gokulnath A (2):
staging:bcm: Removed developer debug prints in InterfaceDld.c
staging: bcm: line over 80 characters in InterfaceDld.c
drivers/staging/bcm/InterfaceDld.c | 140 +
Fixed all the line over 80 characters warning found by
checkpatch.pl script.
Signed-off-by: Gokulnath A
---
drivers/staging/bcm/InterfaceDld.c | 70 ++
1 file changed, 49 insertions(+), 21 deletions(-)
diff --git a/drivers/staging/bcm/InterfaceDld.c
b/driver
This feature allows multiple channels to be used by each virtual NIC.
It is available on Hyper-V host 2012 R2.
Signed-off-by: Haiyang Zhang
Reviewed-by: K. Y. Srinivasan
---
drivers/net/hyperv/hyperv_net.h | 159 ++
drivers/net/hyperv/netvsc.c | 120 +++
On Thu, Dec 19, 2013 at 01:05:27AM +0900, Masanari Iida wrote:
> This patch fixes "ERROR: trailing whitespace" found by
> checkpatch.pl, within dgap_sysfs.c
>
> Signed-off-by: Masanari Iida
I see a bunch of patches missing in this series, can you please resend
the whole series, as a new one, so
This patch cleans up bypass.c of the silicom driver in the staging area. All
errors and most warnings according to checkpatch.pl should be fixed.
There are still two warnings left related to too many leading tabs at nested
blocks. I did not touch this issue because the code needs really some
refac
Hi Dan,
Thank you for the patch.
On Wednesday 18 December 2013 10:29:24 Dan Carpenter wrote:
> The spin_lock_irqsave() macro is not nestable. The second call will
> overwrite the first record of "flags" so the IRQs will not be enabled
> correctly at the end of the function.
>
> In the current c
On Thu, 2013-12-19 at 00:45 +0100, Michael Hoefler wrote:
> This patch cleans up bypass.c of the silicom driver in the staging area. All
> errors and most warnings according to checkpatch.pl should be fixed.
When you make patch against staging, please use the
latest staging-next branch of the stag
On Thu, Dec 19, 2013 at 12:45:09AM +0100, Michael Hoefler wrote:
> This patch cleans up bypass.c of the silicom driver in the staging area. All
> errors and most warnings according to checkpatch.pl should be fixed.
>
> There are still two warnings left related to too many leading tabs at nested
>
23 matches
Mail list logo