On Tue, Dec 02, 2014 at 04:30:51PM -0800, Greg KH wrote:
> On Sat, Nov 29, 2014 at 11:48:07AM +0100, Abel Moyo wrote:
> > Removed inline function get_my_bssid and open coded all its references
> >
> > Signed-off-by: Abel Moyo
> > ---
> > Changes in v2:
> > - Remove inline function entirely an
From: Fred Chou
v2: alignment to match open parenthesis
Fixed the following warnings in sparse:
drivers/staging/rtl8192u/r8192U_core.c:670:6: warning:
symbol 'dump_eprom' was not declared.
Should it be static?
drivers/staging/rtl8192u/r8192U_core.c:1489:5: warning:
symbol
On Tue, Dec 02, 2014 at 11:45:00PM +, Sean Cleator wrote:
>
> A patch to the slicoss.c file to fix some of the long line issues found by
> the
> checkpath.pl tool
> Signed-off-by: Sean Cleator
do not use whitespace before Signed-off-by
thanks
sudip
>
> ---
> drivers/staging/slicoss/
On Tue, Dec 02, 2014 at 07:30:40PM -0600, Paul Hedman wrote:
> Fixes spacing issues around operators, e.g. i=0 to i = 0, as
> found by the checkpatch.pl tool.
no Signed-off-by ???
please run your patch through checkpatch before sending.
thanks
sudip
> ---
> drivers/staging/ft1000/ft1000-usb/ft
On Wed, Dec 03, 2014 at 10:38:27AM +0800, Fred Chou wrote:
> From: Fred Chou
>
> Fixed the following warnings in sparse:
> drivers/staging/rtl8192u/r8192U_core.c:670:6: warning:
> symbol 'dump_eprom' was not declared.
> Should it be static?
> drivers/staging/rtl8192u/r8192U_core.c:148
On Tue, Dec 02, 2014 at 11:25:22PM -0500, Kevin Pietsch wrote:
> Removed braces from single line if, else, for statements.
> Replaced space indents with tabs. Fixed whitespace errors around operators.
That's a whole lot of different things to do in one patch.
Please, one patch per "thing" you do,
Removed braces from single line if, else, for statements.
Replaced space indents with tabs. Fixed whitespace errors around operators.
Signed-off-by: Kevin Pietsch
---
drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 225 +--
1 file changed, 90 insertions(+), 135 deletions(-
On Wed, Dec 03, 2014 at 10:31:55AM +0800, Fred Chou wrote:
> From: Fred
One word name, and then:
> Signed-off-by: Fred Chou
Two word name.
Unacceptable, you know this...
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdrive
From: Fred Chou
Fixed the following warnings in sparse:
drivers/staging/rtl8192u/r8192U_core.c:670:6: warning:
symbol 'dump_eprom' was not declared.
Should it be static?
drivers/staging/rtl8192u/r8192U_core.c:1489:5: warning:
symbol 'ComputeTxTime' was not declared.
This patch fixes a brace warning in rtl871x_sta_mgt.c found
by the checkpatch.pl tool.
Signed-off-by: Paul Hedman
---
drivers/staging/rtl8712/rtl871x_sta_mgt.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8712/rtl871x_sta_mgt.c
b/drivers/staging/rtl87
From: Fred
Fixed the following warnings in sparse:
drivers/staging/rtl8192u/r8192U_core.c:670:6: warning:
symbol 'dump_eprom' was not declared.
Should it be static?
drivers/staging/rtl8192u/r8192U_core.c:1489:5: warning:
symbol 'ComputeTxTime' was not declared.
Sho
Well shoot, please ignore what I just sent in then, I thought I had it.
-Paul
On Tue, Dec 2, 2014 at 7:31 PM, Greg KH wrote:
> On Tue, Dec 02, 2014 at 07:10:41PM -0600, Paul H wrote:
>> A tutorial I read suggested using a different branch for applying the
>> patches; I named it "fix-spaces". It
On Tue, Dec 02, 2014 at 07:10:41PM -0600, Paul H wrote:
> A tutorial I read suggested using a different branch for applying the
> patches; I named it "fix-spaces". It would appear it is mistaken
> then?
You have to make your patch against the linux-next tree, not Linus's
tree, so as to not duplic
Fixes spacing issues around operators, e.g. i=0 to i = 0, as
found by the checkpatch.pl tool.
---
drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 28
1 file changed, 14 insertions(+), 14 deletions(-)
diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_debug.c
b/driv
A tutorial I read suggested using a different branch for applying the
patches; I named it "fix-spaces". It would appear it is mistaken
then?
Thanks again, hope I'm not bothering at all, first time trying to give back.
-Paul
On Tue, Dec 2, 2014 at 6:48 PM, Greg KH wrote:
> On Tue, Dec 02, 2014
Hi Philipp:
On 2014年12月03日 02:23, Philipp Zabel wrote:
Hi Andy,
Am Dienstag, den 02.12.2014, 15:42 +0800 schrieb Andy Yan:
diff --git a/Documentation/devicetree/bindings/drm/bridge/dw_hdmi.txt
b/Documentation/devicetree/bindings/drm/bridge/dw_hdmi.txt
new file mode 100644
index 000..107c1c
On Mon, Dec 01, 2014 at 09:54:45AM +0800, Fred wrote:
> Fixed the following warnings in sparse:
> drivers/staging/rtl8192u/r8192U_core.c:670:6: warning:
> symbol 'dump_eprom' was not declared.
> Should it be static?
> drivers/staging/rtl8192u/r8192U_core.c:1489:5: warning:
> symbo
On Tue, Dec 02, 2014 at 09:11:15AM -0600, Paul Hedman wrote:
> This fix fixes spacing issues around operators, e.g. i=0 to i = 0, found by
> the checkpatch.pl tool.
Please wrap your changelog at the correct column.
>
> Signed-off-by: Paul Hedman
> ---
> drivers/staging/ft1000/ft1000-usb/ft100
On Sat, Nov 29, 2014 at 06:25:30PM +0530, Anjana Sasindran wrote:
> This patch fix the checkpatch.pl warning
>
> WARNING:Missing a blank line after declaration
>
> Signed-off-by: Anjana Sasindran
> ---
> drivers/staging/clocking-wizard/clk-xlnx-clock-wizard.c | 1 +
> 1 file changed, 1 insertio
On Sat, Nov 29, 2014 at 11:04:00AM +0530, Athira Lekshmi wrote:
> Fixed the checkpatch warning:
> Missing a blank line after declarations
>
> Signed-off-by: Athira Lekshmi
> ---
> drivers/staging/octeon/ethernet-rx.c |1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/staging/oc
On Sat, Nov 29, 2014 at 11:48:07AM +0100, Abel Moyo wrote:
> Removed inline function get_my_bssid and open coded all its references
>
> Signed-off-by: Abel Moyo
> ---
> Changes in v2:
> - Remove inline function entirely and open code the
> implementation instead of replacing __inlin
> We have had the Hyper-V clocksource for sometime now and this patch
just marks this
> clocksource as being continuous. Nothing has changed with regards to
timesynch.
Alright, that cleared up that question. Gleaning around from the source
tree, I don't seem to
comprehend what changes are being m
> -Original Message-
> From: Jeff Leung [mailto:jle...@v10networks.ca]
> Sent: Tuesday, December 2, 2014 3:58 PM
> To: KY Srinivasan; x...@kernel.org; gre...@linuxfoundation.org; linux-
> ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de;
> a...@canonical.com; jasow...@
Does this mean that host-to guest timesync now works properly with Linux
guests?
-- Jeff
> -Original Message-
> From: linux-kernel-ow...@vger.kernel.org
[mailto:linux-kernel-ow...@vger.kernel.org]
> On Behalf Of K. Y. Srinivasan
> Sent: Tuesday, December 2, 2014 17:04
> To: x...@kernel.or
The clocksource based on Hyper-V per-partition reference count MSR is
continuous. Mark it accordingly.
Signed-off-by: K. Y. Srinivasan
cc: sta...@vger.kernel.org
---
arch/x86/kernel/cpu/mshyperv.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/arch/x86/kernel/cpu/mshype
A patch to the slicoss.c file to fix some of the long line issues found by the
checkpath.pl tool
Signed-off-by: Sean Cleator
---
drivers/staging/slicoss/slicoss.c | 39 ++-
1 file changed, 26 insertions(+), 13 deletions(-)
diff --git a/drivers/staging/sl
Removes some functions that are not used anywhere.
rtw_hal_interrupt_handler() rtw_hal_sreset_get_wifi_status()
This was partially found by using a static code analysis program called
cppcheck.
Signed-off-by: Rickard Strandqvist
---
drivers/staging/rtl8188eu/hal/hal_intf.c | 16 -
"asm/unaligned.h" is more generic than "access_ok.h", and it may include
"access_ok.h", so need use it instead of "access_ok.h".
During building, "rtllib.h" has already include "asm/unaligned.h", so
will cause building issue. The related error (with allmodconfig under
parisc):
CC [M] drivers
> -Original Message-
> From: Dan Carpenter [mailto:dan.carpen...@oracle.com]
> Sent: Tuesday, December 2, 2014 1:51 PM
> To: KY Srinivasan
> Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org;
> de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com;
> jasow...@redhat.
On Tue, Dec 02, 2014 at 02:29:13PM -0800, K. Y. Srinivasan wrote:
> Make adjustments in computing the balloon floor.
>
The changelog should say what the user visible effects of the bug are.
regards,
dan carpenter
___
devel mailing list
de...@linuxdriv
On Nov 28, 2014, at 11:50 AM, Greg KH wrote:
> On Thu, Nov 27, 2014 at 07:34:10PM +0100, Loïc Pefferkorn wrote:
>> Hello Greg,
>>
>> After some investigation, I think that removing these wrappers is not going
>> to improve the code readability:
>>
>> On Wed, Nov 26, 2014 at 12:54:43PM -0800, Gr
We currently release memory (balloon down) in the interrupt context and we also
post memory status while releasing memory. Rather than posting the status
in the interrupt context, wakeup the status posting thread to post the status.
This will address the inconsistent lock state that Sitsofe Wheeler
We support memory hot-add in the Hyper-V balloon driver by hot adding an
appropriately
sized and aligned region and controlling the on-lining of pages within that
region
based on the pages that the host wants us to online. We do this because the
granularity and alignment requirements in Linux are
Make adjustments in computing the balloon floor.
Signed-off-by: K. Y. Srinivasan
---
drivers/hv/hv_balloon.c |9 +
1 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/drivers/hv/hv_balloon.c b/drivers/hv/hv_balloon.c
index b958ded..9cbbb83 100644
--- a/drivers/hv/hv_balloo
Some bug fixes for the balloon driver.
K. Y. Srinivasan (3):
Drivers: hv: hv_balloon: Make adjustments in computing the floor
Drivers: hv: hv_balloon: Fix a locking bug in the balloon driver
Drivers: hv: hv_balloon: Don't post pressure status from interrupt
context
drivers/hv/hv_balloo
Hi Andy,
Am Dienstag, den 02.12.2014, 15:42 +0800 schrieb Andy Yan:
> diff --git a/Documentation/devicetree/bindings/drm/bridge/dw_hdmi.txt
> b/Documentation/devicetree/bindings/drm/bridge/dw_hdmi.txt
> new file mode 100644
> index 000..107c1ca
> --- /dev/null
> +++ b/Documentation/devicetree
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Hi Matthias,
Thanks for your feedback!
That's weird. I can still see the driver in the 'master' branch on
https://github.com/torvalds/linux/tree/master/drivers/staging/bcm
I've cloned from
git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux
Hi Pieter,
On 02-12-2014 19:04:31, 1Ago - Pieter Waegeman wrote:
> That's weird. I can still see the driver in the 'master' branch on
> https://github.com/torvalds/linux/tree/master/drivers/staging/bcm
> I've cloned from
> git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
>
> Or am
Hi Pieter,
as far as I know, this driver was removed from the source tree. Please
update your local copy of the kernel.
--
Mit freundlichen Grüßen,
Kind regards,
Matthias Beyer
Proudly sent with mutt.
Happily signed with gnupg.
On 02-12-2014 18:25:49, Pieter Waegeman wrote:
> ---
> drivers/st
---
drivers/staging/bcm/InterfaceMacros.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/bcm/InterfaceMacros.h
b/drivers/staging/bcm/InterfaceMacros.h
index fedb794..67e7b62 100644
--- a/drivers/staging/bcm/InterfaceMacros.h
+++ b/drivers/staging/bcm/Interface
This define is not used by the driver. Remove it.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/addi-data/hwdrv_apci1500.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/staging/comedi/drivers/addi-data/hwdrv_apci1500.c
The Z8536 CIO registers are indirectly read by writing the register value
to the control register then reading the control register. Introduce a helper
function to read the Z8536 CIO registers.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
.../comedi/drivers/addi-d
Introduce a helper function to reset the Z8536 CIO device.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
.../comedi/drivers/addi-data/hwdrv_apci1500.c | 185 -
1 file changed, 69 insertions(+), 116 deletions(-)
diff --git a/drivers/staging
The interrupt used by this driver is shared. If the board did not cause
the interrupt the driver should return IRQ_NONE so that another driver
can handle it. Fix the interrupt handler so this happens.
Tidy up the interrupt handler a bit.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg
PCI Bar 0 accesses the AMCC S5933 PCI controller used on this board.
Use the defines from amcc_35933.h and remove the "magic" numbers.
Rename the CamelCase private data member 'i_IobaseAmcc' used to hold
the address for PCI Bar 0.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-H
This member of the private data is the same as the comedi_device 'iobase'.
Remove the private data member and use that instead.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
.../comedi/drivers/addi-data/hwdrv_apci1500.c | 28 +-
drivers/sta
For aesthetics, convert the enum for the PCI Bar 1 register map to
defines and move them to the main driver source file. Rename the
registers.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
.../comedi/drivers/addi-data/hwdrv_apci1500.c | 35
The Z8536 CIO registers are indirectly written by writing the register value
to the control register then writing the value to the control register.
Introduce a helper function to write the Z8536 CIO registers.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
.../come
Move the PCI Bar 1 register map defines to the main driver source file.
For aesthetics, rename the defines.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
.../comedi/drivers/addi-data/hwdrv_apci1500.c | 21 -
drivers/staging/comedi/drivers/
This driver is pretty broken but in the current state its difficult to
figure out where to fix it.
Introduce a couple helper functions to handle the read/write and reset
of the Z8536 CIO chip. This removes quite a bit of lines of code and
makes the driver a _bit_ easier to follow.
Tidy up the reg
This member of the private data is set but not used by the driver.
Remove it.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/addi_apci_1500.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/comedi/drivers/addi_apci
Rename this CamelCase member of the private data.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
.../comedi/drivers/addi-data/hwdrv_apci1500.c| 20
drivers/staging/comedi/drivers/addi_apci_1500.c | 4 ++--
2 files changed, 10 inser
On Mon, Dec 01, 2014 at 02:53:52PM +0800, sunwxg wrote:
> From: Sun Wang
>
> Incorrect type when assign value to varible iov_base.
Oh, for pity sake... Folks, these struct iovec in there are bloody wrong.
It's not iovec at all. There are exactly two declarations in the whole
drivers/staging/lu
On Tue, 2014-12-02 at 16:20 +0530, Sudip Mukherjee wrote:
> these header files were included multiple times
>
> Signed-off-by: Sudip Mukherjee
Looks good to me, thanks for the help. :)
Signed-off-by: Benjamin Romer
___
devel mailing list
de...@linuxd
This fix fixes spacing issues around operators, e.g. i=0 to i = 0, found by the
checkpatch.pl tool.
Signed-off-by: Paul Hedman
---
drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 28
1 file changed, 14 insertions(+), 14 deletions(-)
diff --git a/drivers/staging/ft10
From: Paul
This patch fixes various coding standards violations found by the checkpatch.pl
tool
Signed-off-by: Paul Hedman
---
drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 948 ---
1 file changed, 478 insertions(+), 470 deletions(-)
diff --git a/drivers/staging/ft10
On Tue, Dec 02, 2014 at 08:27:46AM -0600, Paul Hedman wrote:
> From: Paul
>
> This patch fixes various coding standards violations found by the
> checkpatch.pl tool
>
> Signed-off-by: Paul Hedman
> ---
1) you are doing multiple things in this patch. please do only one type of fix
in a patch.
Hi Andy,
Am Dienstag, den 02.12.2014, 20:34 +0800 schrieb Andy Yan:
> Hi Philipp:
> On 2014年12月02日 18:24, Philipp Zabel wrote:
> > Hi Andy,
> >
> > Am Dienstag, den 02.12.2014, 15:45 +0800 schrieb Andy Yan:
> > [...]
> >> +static int dw_hdmi_rockchip_bind(struct device *dev, struct device
> >> *m
Hi Philipp:
On 2014年12月02日 18:24, Philipp Zabel wrote:
Hi Andy,
Am Dienstag, den 02.12.2014, 15:45 +0800 schrieb Andy Yan:
[...]
+static int dw_hdmi_rockchip_bind(struct device *dev, struct device *master,
+void *data)
+{
+ struct platform_device *pdev = to
Fixes checkpatch warning:
WARNING: else is not generally useful after a break or return
Signed-off-by: Mike Krinkin
---
drivers/staging/rtl8712/rtl8712_recv.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/rtl8712/rtl8712_recv.c
b/drivers/staging/rt
Fixes checkpatch warning:
WARNING: else is not generally useful after a break or return
Signed-off-by: Mike Krinkin
---
drivers/staging/rtl8712/rtl8712_recv.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/rtl8712/rtl8712_recv.c
b/drivers/staging/rt
From: Markus Elfring
Date: Tue, 2 Dec 2014 11:40:33 +0100
The functions free_ll_remote_perm(), free_rmtperm_hash() and iput() test
whether their argument is NULL and then return immediately.
Thus the test around their calls is not needed.
This issue was detected by using the Coccinelle software.
these header files were included multiple times
Signed-off-by: Sudip Mukherjee
---
change in v2: missed seeing that linux/uuid.h is included 3 times
drivers/staging/unisys/include/timskmod.h | 1 -
drivers/staging/unisys/uislib/uisutils.c | 2 --
2 files changed, 3 deletions(-)
diff --git a/
On Tue, Dec 02, 2014 at 01:46:54AM -0800, Jeremiah Mahler wrote:
> Sudip,
>
> On Tue, Dec 02, 2014 at 02:34:58PM +0530, Sudip Mukherjee wrote:
> > these header files were included two times
> >
> > Signed-off-by: Sudip Mukherjee
> > ---
> > drivers/staging/unisys/include/timskmod.h | 1 -
> > d
Hi Andy,
Am Dienstag, den 02.12.2014, 15:45 +0800 schrieb Andy Yan:
[...]
> +static int dw_hdmi_rockchip_bind(struct device *dev, struct device *master,
> + void *data)
> +{
> + struct platform_device *pdev = to_platform_device(dev);
> + const struct dw_hdmi_pl
On Sun, Nov 23, 2014 at 07:39:23PM +0100, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sun, 23 Nov 2014 18:48:15 +0100
>
> The functions ion_heap_destroy() and vfree() perform also input
> parameter validation. Thus the test around the call is not needed.
>
> This issue was detected b
Sudip,
On Tue, Dec 02, 2014 at 02:34:58PM +0530, Sudip Mukherjee wrote:
> these header files were included two times
>
> Signed-off-by: Sudip Mukherjee
> ---
> drivers/staging/unisys/include/timskmod.h | 1 -
> drivers/staging/unisys/uislib/uisutils.c | 1 -
> 2 files changed, 2 deletions(-)
>
these header files were included two times
Signed-off-by: Sudip Mukherjee
---
drivers/staging/unisys/include/timskmod.h | 1 -
drivers/staging/unisys/uislib/uisutils.c | 1 -
2 files changed, 2 deletions(-)
diff --git a/drivers/staging/unisys/include/timskmod.h
b/drivers/staging/unisys/includ
68 matches
Mail list logo
