Hello,
On Sat, Jan 17, 2015 at 12:51:50PM -0800, Ray Jui wrote:
> On 1/17/2015 12:18 PM, Uwe Kleine-König wrote:
> > Hello,
> >
> > On Sat, Jan 17, 2015 at 11:58:33AM -0800, Ray Jui wrote:
> >> On 1/17/2015 8:01 AM, Uwe Kleine-König wrote:
> >>> On Fri, Jan 16, 2015 at 02:09:28PM -0800, Ray Jui w
Hi,
On Sat, Jan 17, 2015 at 12:12 PM, Jon Masters wrote:
> Hi Catalin, all,
>
> I would like to ensure that the SMBIOS data provided by firmware is
> always readable from userspace on AArch64, through /dev/mem.
Seems like this would be a good opportunity for cleanup and fixing
userspace to use /
Hi!
> I'm looking into moving the sysfs interface to debugfs. Doesn't look too
> hard and you and Alan are making lots of sense about this.
Debugfs: please don't. That's meant for debugging, not for production
use.
Pavel
--
T: Bus=01 Lev=02 Prnt=03 Port=00 Cnt=01 Dev#= 5 Spd=12 MxCh= 0
D: Ver= 1.10 Cls=e0(wlcon) Sub=01 Prot=01 MxPS=64 #Cfgs= 1
P: Vendor=13d3 ProdID=3423 Rev= 0.01
C:* #Ifs= 2 Cfg#= 1 Atr=e0 MxPwr=100mA
A: FirstIf#= 0 IfCount= 2 Cls=e0(wlcon) Sub=01 Prot=01
I:* If#= 0 Alt= 0 #EPs= 3 Cls=e0(wlcon) Sub=01
Add support of 13d3:3423 device.
BugLink: https://bugs.launchpad.net/bugs/1411193
T: Bus=01 Lev=02 Prnt=03 Port=00 Cnt=01 Dev#= 5 Spd=12 MxCh= 0
D: Ver= 1.10 Cls=e0(wlcon) Sub=01 Prot=01 MxPS=64 #Cfgs= 1
P: Vendor=13d3 ProdID=3423 Rev= 0.01
C:* #Ifs= 2 Cfg#= 1 Atr=e0 MxPwr=100mA
A: FirstIf#= 0 I
On 1/17/2015 1:10 PM, Uwe Kleine-König wrote:
> Hello,
>
> On Sat, Jan 17, 2015 at 12:51:50PM -0800, Ray Jui wrote:
>> On 1/17/2015 12:18 PM, Uwe Kleine-König wrote:
>>> Hello,
>>>
>>> On Sat, Jan 17, 2015 at 11:58:33AM -0800, Ray Jui wrote:
On 1/17/2015 8:01 AM, Uwe Kleine-König wrote:
>>>
On Tue, Jan 13, 2015 at 02:24:04PM +0900, Masahiro Yamada wrote:
> Hi Sam,
>
> IIRC, you mentioned scripts/headers.sh is pointless
> and should be removed from the code base.
Took a quik look at it.
It seems that there is use for it still.
um require some special handling.
And the HDR_ARCH_LIST f
Quoting Stephen Boyd (2015-01-05 11:34:32)
> On 01/05/2015 08:04 AM, Stanimir Varbanov wrote:
> > The commit 646cafc6 (clk: Change clk_ops->determine_rate to
> > return a clk_hw as the best parent) opens a possibility for
> > null pointer dereference, fix this.
> >
> > Signed-off-by: Stanimir Varba
Quoting Sergei Shtylyov (2014-12-24 06:43:27)
> In case CLK_GATE_HIWORD_MASK flag is passed to clk_register_gate(), the bit #
> should be no higher than 15, however the corresponding check is obviously off-
> by-one.
>
> Fixes: 045779942c04 ("clk: gate: add CLK_GATE_HIWORD_MASK")
> Signed-off-by:
> T: Bus=01 Lev=02 Prnt=03 Port=00 Cnt=01 Dev#= 5 Spd=12 MxCh= 0
> D: Ver= 1.10 Cls=e0(wlcon) Sub=01 Prot=01 MxPS=64 #Cfgs= 1
> P: Vendor=13d3 ProdID=3423 Rev= 0.01
> C:* #Ifs= 2 Cfg#= 1 Atr=e0 MxPwr=100mA
> A: FirstIf#= 0 IfCount= 2 Cls=e0(wlcon) Sub=01 Prot=01
> I:* If#= 0 Alt= 0 #EPs= 3 Cls=e0(w
Quoting Stephen Boyd (2014-12-22 11:26:42)
> On 12/22/2014 10:38 AM, Stephen Boyd wrote:
> > On 12/22/2014 03:45 AM, Krzysztof Kozlowski wrote:
> >
> >> diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c
> >> index f4963b7d4e17..35079302a650 100644
> >> --- a/drivers/clk/clk.c
> >> +++ b/drivers/cl
Hello.
On 01/18/2015 01:01 AM, Mike Turquette wrote:
In case CLK_GATE_HIWORD_MASK flag is passed to clk_register_gate(), the bit #
should be no higher than 15, however the corresponding check is obviously off-
by-one.
Fixes: 045779942c04 ("clk: gate: add CLK_GATE_HIWORD_MASK")
Signed-off-by:
Hi Guenter, Greg,
> >>> This commit uses all the UGO bits returned by is_visible instead
> >>> of OR'ing them with the default attribute mode.
> >>>
> >>> Concretely, this allows a driver to use macros like DEVICE_ATTR_RW
> >>> to set the attribute show and store functions and remove the
> >>> S_I
A Donation Has Been Made To You Email:gloriamack...@outlook.com For More
Details
.
We believe 1 family torn apart by war is too many.
Join UNHCR and share their stories:
http://stories.unhcr.org?link=email
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body
On Sun, Dec 07, 2014 at 03:37:20PM +0100, Emil Renner Berthing wrote:
> This also fixes a sparse warning.
What sparse warning? What's wrong with the original code? Unless Jes
resends this to me, I don't see the need to apply it, sorry.
greg k-h
--
To unsubscribe from this list: send the line "u
On Sat, Jan 17, 2015 at 04:13:16PM +0100, Antonio Ospite wrote:
> On Sat, 17 Jan 2015 17:23:45 +0530
> Shailendra Verma wrote:
>
> >
> > Signed-off-by: Shailendra Verma
> > ---
> > drivers/input/evdev.c| 30 --
> > drivers/input/mousedev.c | 18 -
On Wed, Dec 17, 2014 at 09:36:54PM +0100, Rickard Strandqvist wrote:
> Removes some functions that are not used anywhere:
> rtw_hw_resume23a() rtw_hw_suspend23a()
>
> This was partially found by using a static code analysis program called
> cppcheck.
>
> Signed-off-by: Rickard Strandqvist
> Sig
On Sun, Dec 21, 2014 at 01:59:39PM +0100, Rickard Strandqvist wrote:
> Removes some functions that are not used anywhere:
> rtl8723au_DeInitSwLeds() rtl8723au_InitSwLeds()
>
> This was partially found by using a static code analysis program called
> cppcheck.
>
> Signed-off-by: Rickard Strandqvi
On Sat, Dec 20, 2014 at 06:10:13PM +0100, Rickard Strandqvist wrote:
> Removes some functions that are not used anywhere:
> rtw_set_ch_cmd23a() rtw_cmd_clr_isr23a()
>
> This was partially found by using a static code analysis program called
> cppcheck.
>
> Signed-off-by: Rickard Strandqvist
> -
Hi Vishnu,
On Fri, Jan 16, 2015 at 07:33:38PM +0530, Vishnu Patekar wrote:
> Signed-off-by: VishnuPatekar
> ---
> drivers/input/serio/Kconfig | 11 ++
> drivers/input/serio/Makefile|1 +
> drivers/input/serio/sun4i-ps2.c | 330
> +++
> 3 files
On Sat, Dec 20, 2014 at 03:58:32PM +0100, Rickard Strandqvist wrote:
> Remove the function r8712_os_read_port() that is not used anywhere.
>
> This was partially found by using a static code analysis program called
> cppcheck.
>
> Signed-off-by: Rickard Strandqvist
> ---
> drivers/staging/rtl8
On Sat, Dec 20, 2014 at 05:24:11PM +0100, Rickard Strandqvist wrote:
> Removes some functions that are not used anywhere:
> CAM_empty_entry() get_bsstype() rtw_get_oper_choffset()
> rtw_get_oper_bw() is_ap_in_wep() should_forbid_n_rate()
>
> This was partially found by using a static code analysis
On Sat, Jan 10, 2015 at 02:19:11PM +, Lorenzo Stoakes wrote:
> This patch fixes errors raised by checkpatch.pl relating to use of C99
> comments
> in r8192U_dm.c, and cleans up existing ANSI C comments.
>
> Signed-off-by: Lorenzo Stoakes
> ---
> drivers/staging/rtl8192u/r8192U_dm.c | 811
>
On Sat, Jan 17, 2015 at 01:26:41PM -0800, Ray Jui wrote:
> time_left = wait_for_completion_timeout(&iproc_i2c->done, time_left);
>
> /* disable all interrupts */
> writel(0, iproc_i2c->base + IE_OFFSET);
>
> if (!time_left && !atomic_read(&iproc_i2c->transfer_is_successful
Dave,
On Sat, Jan 17, 2015 at 06:00:29PM +, David Solda wrote:
> Can we just reverse the prior change that caused the bug, or are you saying
> that the owner of the prior patch has to fix it?
>
> Dave
>
I tried to revert the patch but it didn't do so cleanly.
git revert ea2c67bb4aff
So
resp_rsup_opcodes() may get called from atomic context and would need to
use GFP_ATOMIC for allocations:
[ 1237.913419] BUG: sleeping function called from invalid context at
mm/slub.c:1262
[ 1237.914865] in_atomic(): 1, irqs_disabled(): 0, pid: 7556, name: trinity-c311
[ 1237.916142] 3 locks held
Hi Olof,
On 01/17/2015 04:10 PM, Olof Johansson wrote:
> Hi,
>
> On Sat, Jan 17, 2015 at 12:12 PM, Jon Masters wrote:
>> Hi Catalin, all,
>>
>> I would like to ensure that the SMBIOS data provided by firmware is
>> always readable from userspace on AArch64, through /dev/mem.
>
> Seems like this
On 01/17/2015 05:52 PM, Jon Masters wrote:
> Hi Olof,
>
> On 01/17/2015 04:10 PM, Olof Johansson wrote:
>> Hi,
>>
>> On Sat, Jan 17, 2015 at 12:12 PM, Jon Masters wrote:
>>> Hi Catalin, all,
>>>
>>> I would like to ensure that the SMBIOS data provided by firmware is
>>> always readable from users
Wensong, this is something we discussed 10 years ago and you liked it, but
it didn't actually get into the kernel. I've updated it, tested it, and
would like to work toward inclusion.
Thanks,
Chris
---
From: Chris Caputo
IPVS wlib (Weighted Least Incoming Byterate) and wlip (Weighted Least
On Sat, Jan 17, 2015 at 03:12:13PM -0500, Jon Masters wrote:
> Hi Catalin, all,
>
> I would like to ensure that the SMBIOS data provided by firmware is
> always readable from userspace on AArch64, through /dev/mem.
No, we need to ensure /dev/mem can be completely disabled on any
system that ever
From: Chris Caputo
IPVS wlib (Weighted Least Incoming Byterate) and wlip (Weighted Least Incoming
Packetrate) scheduler docs for ipvsadm-1.27.
Signed-off-by: Chris Caputo
---
diff -upr ipvsadm-1.27-stock/SCHEDULERS ipvsadm-1.27/SCHEDULERS
--- ipvsadm-1.27-stock/SCHEDULERS 2013-09-06 08:
On Sat, Jan 10, 2015 at 10:20:03AM +0530, jitendra kumar khasdev wrote:
> This is patch to pcl812.c that fix line over 80 characters warning
> which is found by checkpatch.pl tool. Reduce the extra space around
> equal sign that fix warning.
>
> Signed-off-by: Jitendra kumar khasdev
> ---
> driv
chown() and write() should clear all privilege attributes on
a file - setuid, setgid, setcap and any other extended
privilege attributes.
However, any attributes beyond setuid and setgid are managed by the
LSM and not directly by the filesystem, so they cannot be set along
with the other attribute
On Wed, Jan 14, 2015 at 04:12:58AM +1100, Emrys Bayliss wrote:
> This patch fixes the following checkpatch.pl error:
> rxtx.c:558: WARNING: else is not generally useful after a break or return
>
> Signed-off-by: Emrys Bayliss
> ---
> The else statement was replaced with a block to avoid the follo
On Sun, Dec 21, 2014 at 08:18:19AM +0530, Mohammad Jamal wrote:
> This patch solves space prohibited before , warning in gdm_mux.c
>
> Signed-off-by: Mohammad Jamal
> ---
> drivers/staging/gdm724x/gdm_mux.c |2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/stag
On Sun, Dec 21, 2014 at 11:44:43AM +0100, Robert Nachlinger wrote:
> This is a patch to the gdm_mux.c file that fixes a space before comma
> coding style issue found by the checkpatch.pl tool.
>
> Signed-off-by: Robert Nachlinger
> ---
> drivers/staging/gdm724x/gdm_mux.c | 2 +-
> 1 file changed
Removes some functions that are not used anywhere:
sst_hsw_stream_unmute() sst_hsw_stream_mute() msg_set_stage_type()
sst_hsw_dx_get_state() sst_hsw_stream_set_write_position()
sst_hsw_stream_get_vol_reg() sst_hsw_stream_get_peak_reg()
sst_hsw_stream_get_pointer_reg() sst_hsw_stream_get_read_reg()
On Mon, Jan 12, 2015 at 06:27:38PM +0200, Tero Marttila wrote:
> Fix missing blank lines after declarations in octeon/ethernet-rx.c
>
> Signed-off-by: Tero Marttila
> ---
> drivers/staging/octeon/ethernet-rx.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/staging/octeon/eth
On Sun, Jan 04, 2015 at 09:23:44AM +0100, Ning Zhou wrote:
> This is a patch to fix a space error found by the checkpatch.pl in
> file gdm_mux.c.
>
> Signed-off-by: Ning Zhou
> ---
> drivers/staging/gdm724x/gdm_mux.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drive
The following changes since commit 97bf6af1f928216fd6c5a66e8a57bfa95a659672:
Linux 3.19-rc1 (2014-12-20 17:08:50 -0800)
are available in the git repository at:
https://git.linaro.org/people/mike.turquette/linux.git
tags/clk-fixes-for-linus
for you to fetch changes up to c7662fc59ca38517e0e
On Mon, Dec 22, 2014 at 02:17:31AM +0100, Simone Weiss wrote:
>
> Signed-off-by: Simone Weiss
> Signed-off-by: Helene Gsaenger
> ---
> drivers/staging/dgnc/dpacompat.h | 6 --
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/dgnc/dpacompat.h
> b/drivers/s
On Sat, Jan 10, 2015 at 04:00:47PM +0100, Arno Tiemersma wrote:
> Remove do {} while (0) loops around single statements in
> skein/skein_block.c
>
> Signed-off-by: Arno Tiemersma
> ---
> drivers/staging/skein/skein_block.c | 16
> 1 file changed, 4 insertions(+), 12 deletions(-)
On Sun, Jan 11, 2015 at 03:41:05PM -0800, Shirish Gajera wrote:
> This patch fixes the checkpatch.pl warning:
>
> WARNING: Single statement macros should not use a do {} while (0) loop
>
> I remove do while from the single statement macro.
>
> Signed-off-by: Shirish Gajera
> ---
> drivers/stag
On Wed, Jan 07, 2015 at 12:57:05PM +0530, Arjun AK wrote:
> Remove unnecessary braces from single statement blocks.
>
> Signed-off-by: Arjun AK
> ---
> drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 101
> ---
> 1 file changed, 35 insertions(+), 66 deletions(-)
Doesn't
From: Michael Karcher
---
drivers/platform/x86/fujitsu-laptop.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/drivers/platform/x86/fujitsu-laptop.c
b/drivers/platform/x86/fujitsu-laptop.c
index 7c21c1c..2a9afa2 100644
--- a/drivers/platform/x86/fujitsu-laptop.c
+++
Signed-Off-By: Michael Karcher
---
scripts/recordmcount.pl | 1 -
1 file changed, 1 deletion(-)
diff --git a/scripts/recordmcount.pl b/scripts/recordmcount.pl
index 56ea99a..537c38c 100755
--- a/scripts/recordmcount.pl
+++ b/scripts/recordmcount.pl
@@ -255,7 +255,6 @@ if ($arch eq "x86_64") {
On Sat, Dec 06, 2014 at 04:03:22PM -0800, Tristan Lelong wrote:
> This patch fix a sparse warning in lustre sources
>
> warning: incorrect type in argument 1 (different address spaces)
> expected void [noderef] *to
> got char *
>
> This is done by adding the missing __user attribute on us
On Tue, Jan 13, 2015 at 10:34:00PM +0100, Bilel DRIRA wrote:
> This patch fix the checkpatch.pl error:
>
> ERROR: else should follow close brace '}'
>
> Signed-off-by: Bilel DRIRA
> ---
> v2: remove braces instead of just putting else after close brace. (suggested
> by Joe Perches).
>
>
Quoting Ulf Hansson (2015-01-15 02:04:04)
> On 15 January 2015 at 10:20, Krzysztof Kozlowski
> wrote:
> > On czw, 2015-01-15 at 09:20 +0100, Ulf Hansson wrote:
> >> + Mike, Stephen (Clock maintainers)
> >>
> >> On 12 January 2015 at 10:23, Krzysztof Kozlowski
> >> wrote:
> >> > Hi,
> >> >
> >> >
On Sat, Dec 27, 2014 at 02:26:53AM +0300, Alexey Khoroshilov wrote:
> obd_class_open() ignores error code of try_module_get(),
> while it can lead to race with module unload.
>
> Found by Linux Driver Verification project (linuxtesting.org).
>
> Signed-off-by: Alexey Khoroshilov
> ---
> drivers
On Mon, Dec 29, 2014 at 08:49:40PM +0800, Jia He wrote:
> This patch is to fix the below warnings generated by sparse:
> warning: incorrect type in argument 1 (different address spaces)
> expected void const volatile [noderef] *
> got int *
> Adding the macro "__user" can suppress the warnings,
On Wed, Jan 07, 2015 at 02:28:12PM +0200, Andrey Utkin wrote:
> 2015-01-07 13:32 GMT+02:00 Dilger, Andreas :
> > On 2015/01/07, 1:36 AM, "Dan Carpenter" wrote:
> >> Didn't Al change these to kvec instead of iovec? You have to look at
> >> the callers to figure out if it's actually a user space po
From: Michael Karcher
Signed-off-by: Michael Karcher
---
drivers/platform/x86/fujitsu-laptop.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/drivers/platform/x86/fujitsu-laptop.c
b/drivers/platform/x86/fujitsu-laptop.c
index 7c21c1c..2a9afa2 100644
--- a/drivers/pl
On Sun, Jan 11, 2015 at 01:44:03PM -0500, Nickolaus Woodruff wrote:
> This patch fixes the following checkpatch.pl error in libcfs_string.c:
>
> ERROR: spaces required around that '='
>
> Signed-off-by: Nickolaus Woodruff
> ---
> drivers/staging/lustre/lustre/libcfs/libcfs_string.c | 2 +-
> 1
On Tue, Jan 13, 2015 at 11:33:03PM +0100, Luca Ceresoli wrote:
> Fixes dozens of sparse warnings like:
> warning: symbol 'cfs_tracefile_sem' was not declared. Should it be static?
>
> Signed-off-by: Luca Ceresoli
> Cc: Peng Tao
> Cc: Jinshan Xiong
> Cc: Srikrishan Malik
> Cc: hpdd-disc...@li
On Mon, Jan 05, 2015 at 03:51:54PM +, Tang, Feng wrote:
> Hi Alan,
>
> Thanks for the review.
>
> > -Original Message-
> > From: One Thousand Gnomes [mailto:gno...@lxorguk.ukuu.org.uk]
> > Sent: Monday, January 05, 2015 10:25 PM
> > To: Tang, Feng
> > Cc: Greg KH; John Stultz; Colin C
On Sat, Dec 20, 2014 at 10:11:56PM -0800, Sören Brinkmann wrote:
> Hi Mohammad,
>
> On Sun, 2014-12-21 at 08:03AM +0530, Mohammad Jamal wrote:
> > Fix the coding style issue by adding blank line after declaration
> >
> > Signed-off-by: Mohammad Jamal
> > ---
> > .../clocking-wizard/clk-xlnx-clo
On Tue, Jan 06, 2015 at 01:29:01AM +1100, Emrys Bayliss wrote:
> Fix the checkpatch.pl errors:
> ERROR: space prohibited before that ',' (ctx:WxW)
>
> Signed-off-by: Emrys Bayliss
> ---
> drivers/staging/rts5208/ms.c | 10 +-
> 1 file changed, 5 insertions(+), 5 deletions(-)
Someone els
On Thu, Jan 01, 2015 at 05:44:39PM +0900, Akira Hayakawa wrote:
> This patch adds dm-writeboost to staging tree.
>
> dm-writeboost is a log-structured SSD-caching driver.
> It caches data in log-structured way on the cache device
> so that the performance is maximized.
>
> The merit of putting th
On Wed, Dec 31, 2014 at 05:56:43PM +0530, Sudip Mukherjee wrote:
> This reverts commit "dc93c85235efa5201e9a3c116bc3fbd1afc1a182"
>
> Signed-off-by: Sudip Mukherjee
> ---
>
> Hi Greg K-H
> With reference to our discussion at https://lkml.org/lkml/2014/11/22/54 ,
> now I have the hardware and I w
On Thu, Jan 08, 2015 at 03:01:17PM +0100, Sven Dziadek wrote:
> This fixes the following checkpatch.pl warning:
> WARNING: else is not generally useful after a break or return
>
> Signed-off-by: Sven Dziadek
> ---
> drivers/staging/wlan-ng/hfa384x_usb.c | 9 -
> 1 file changed, 4 inserti
On Wed, Dec 03, 2014 at 01:53:18PM +, 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
>
> ---
> drivers/staging/slicoss/slicoss.c | 39
> ++-
> 1 fi
On 1/17/2015 2:40 PM, Russell King - ARM Linux wrote:
> On Sat, Jan 17, 2015 at 01:26:41PM -0800, Ray Jui wrote:
>> time_left = wait_for_completion_timeout(&iproc_i2c->done, time_left);
>>
>> /* disable all interrupts */
>> writel(0, iproc_i2c->base + IE_OFFSET);
>>
>> if (!ti
On Mon, Jan 05, 2015 at 03:25:23PM +0100, Guillaume Vercoutere wrote:
> Replace explicit NULL comparison with !
Not a good idea, you just lost typechecking :(
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majo
On Mon, Jan 05, 2015 at 03:25:37PM +0100, Guillaume Vercoutere wrote:
> Replace sizeof struct
>
> Signed-off-by: Guillaume Vercoutere
Doesn't apply as I didn't take patch 2, please fix up and resend.
thanks,
greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
On Mon, Jan 12, 2015 at 05:31:21PM +0100, Moritz Fischer wrote:
> The file triggered the following warning:
>
> WARNING: line over 80 characters
>
> Signed-off-by: Moritz Fischer
> ---
> drivers/staging/android/sync_debug.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
Someone else
On Fri, Jan 16, 2015 at 08:58:10PM -0800, Andrew Milkovich wrote:
> Fixed coding style errors per checkpatch.pl. Inline comments removed; chanptr
> comment is implied by the first argument being the channel. No parent bus
> comment is already understood by the argument being NULL.
>
> Signed-off-b
On Sat, Jan 17, 2015 at 12:04:21PM +0530, Shailendra Verma wrote:
>
> Signed-off-by: Shailendra Verma
> ---
> drivers/staging/android/ion/ion.c |1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/staging/android/ion/ion.c
> b/drivers/staging/android/ion/ion.c
> index 9e56d32..0d
From: Rafael J. Wysocki
Commit f25c0ae2b4c4 (ACPI / PM: Avoid resuming devices in ACPI PM
domain during system suspend) modified the ACPI PM domain's system
suspend callbacks to allow devices attached to it to be left in the
runtime-suspended state during system suspend so as to optimize
the susp
Removes some functions that are not used anywhere:
rtw_hw_resume23a() rtw_hw_suspend23a()
This was partially found by using a static code analysis program called
cppcheck.
Signed-off-by: Rickard Strandqvist
---
drivers/staging/rtl8723au/include/osdep_intf.h |3 -
drivers/staging/rtl8723au/
On 01/17/2015 06:21 PM, Leif Lindholm wrote:
> On Sat, Jan 17, 2015 at 03:12:13PM -0500, Jon Masters wrote:
>> Hi Catalin, all,
>>
>> I would like to ensure that the SMBIOS data provided by firmware is
>> always readable from userspace on AArch64, through /dev/mem.
>
> No, we need to ensure /dev/m
Removes some functions that are not used anywhere:
rtw_set_ch_cmd23a() rtw_cmd_clr_isr23a()
This was partially found by using a static code analysis program called
cppcheck.
Signed-off-by: Rickard Strandqvist
---
drivers/staging/rtl8723au/core/rtw_cmd.c| 61 ---
d
On Sat, 2015-01-03 at 09:51 -0800, Joe Perches wrote:
> There is a copy/paste typo in the dev_dbg_once macro.
>
> It uses dev_info instead of dev_dbg, so use the correct
> function instead.
>
> Signed-off-by: Joe Perches
> Noticed-by: Marc Finet
Ping?
> ---
> include/linux/device.h | 2 +-
>
On Thu, 2014-12-25 at 15:07 -0800, Joe Perches wrote:
> No caller or macro uses the return value so make all
> the functions return void.
Ping?
> Compiled x86 allyesconfig and defconfig w/o CONFIG_PRINTK
>
> Signed-off-by: Joe Perches
> ---
> This change is associated to a desire to eventually
Remove the function r8712_os_read_port() that is not used anywhere.
This was partially found by using a static code analysis program called
cppcheck.
Signed-off-by: Rickard Strandqvist
---
drivers/staging/rtl8712/recv_linux.c | 14 --
drivers/staging/rtl8712/recv_osdep.h |1 -
The syscall and sysenter code use KERNEL_STACK_OFFSET to set the
initial stack pointer a bit below the top of the kernel stack page.
Stop doing that.
Signed-off-by: Alexander van Heukelum
---
arch/x86/ia32/ia32entry.S | 4 ++--
arch/x86/include/asm/thread_info.h | 6 ++
arch/x86/ker
Hi Andy,
Seems you are the person to talk now :). Good to see some activity in
the "entry" area of the kernel. I found some (very) old patches in this
area lying around and some time to dust them off.
Any chance you can review them?
They were redone on top of 22f2aa4a0361707a5cfb1de9d45260b39965
In the macro THREAD_INFO(reg,offset), the offset is relative to
KERNEL_STACK_OFFSET. Change the macro and all its uses so that
offset is given as the current top of stack in the pt_regs frame.
The generated code is identical.
Signed-off-by: Alexander van Heukelum
---
arch/x86/ia32/ia32entry.S
Removes some functions that are not used anywhere:
CAM_empty_entry() get_bsstype() rtw_get_oper_choffset()
rtw_get_oper_bw() is_ap_in_wep() should_forbid_n_rate()
This was partially found by using a static code analysis program called
cppcheck.
Signed-off-by: Rickard Strandqvist
---
drivers/st
Create an IRET-compatible top of stack at syscall entry and use this
information to return to user mode in the sysret path. This removes
the need for the FIXUP_TOP_OF_STACK and RESTORE_TOP_OF_STACK macros.
Signed-off-by: Alexander van Heukelum
---
arch/x86/kernel/entry_64.S | 77 +---
On Sat, Jan 17, 2015 at 04:56:20PM -0800, Joe Perches wrote:
> On Thu, 2014-12-25 at 15:07 -0800, Joe Perches wrote:
> > No caller or macro uses the return value so make all
> > the functions return void.
>
> Ping?
It's in my queue, just starting to dig out from under it, give me a few
more days
Hi Stephan,
On 01/17/2015 10:23 AM, Stephan Mueller wrote:
> during testing of my algif_aead patch with the different GCM implementations
> I
> am able to trigger a kernel crash from user space using __driver-gcm-aes-
> aesni.
>
> As I hope that algif_aead is going to be included, unprivileged u
Signed-off-by: Nicholas Mc Guire
---
return type of wait_for_completion_timeout is unsigned long as ret is
used only for checking wait_for_completion_timeout here the type was
changed to unsigned long, wait_for_completion_timeout return >= 0 only
so the checks for negative return canbe dropped.
On 01/17/2015 02:09 PM, Vivien Didelot wrote:
Hi Guenter, Greg,
[ .. ]
BTW Guenter, does this patch make sense to you?
It does make sense to me to only use the return value from is_visible
for the mode.
As for which bits to use, I am not entirely sure. I think it would be
more important
From: Richard Weinberger
Subject: Re: [PATCH] hostfs: Use noop_fsync for directories
Date: Wed, 14 Jan 2015 09:43:53 +0100
> Daniel, are you interested in a small kernel project?
Sure, I'm just pretty busy with university right now, I'll probably
pick this back up after exams are finished in a m
Signed-off-by: Nicholas Mc Guire
---
The return value of wait_for_completion_timeout is unsigned long,
as it is used here for wait_for_completion_timeout only the type
of commretval was changed to unsigned long.
As wait_for_completion_timeout does not return negative values
the commretval < 0
From: Mathieu Poirier
mpoirier@t430:~/work/kernel1$ make C=1 M=drivers/staging/vt6655/
LD drivers/staging/vt6655/built-in.o
CHECK drivers/staging/vt6655/device_main.c
drivers/staging/vt6655/device_main.c:1672:5: warning: symbol 'vnt_init' was not
declared. Should it be static?
CC [M
From: Mathieu Poirier
>From a v3.19-rc4 baseline sparse finds the following errors:
LD drivers/staging/vt6655/built-in.o
CHECK drivers/staging/vt6655/device_main.c
drivers/staging/vt6655/device_main.c:1503:25: warning: incorrect type in
argument 1 (different address spaces)
drivers/s
From: Mathieu Poirier
mpoirier@t430:~/work/kernel1$ make C=1 M=drivers/staging/vt6655/
LD drivers/staging/vt6655/built-in.o
CHECK drivers/staging/vt6655/device_main.c
drivers/staging/vt6655/device_main.c:1503:25: warning: incorrect type in
argument 1 (different address spaces)
drivers
From: Mathieu Poirier
mpoirier@t430:~/work/kernel1$ make C=1 M=drivers/staging/vt6655/
LD drivers/staging/vt6655/built-in.o
CHECK drivers/staging/vt6655/device_main.c
CC [M] drivers/staging/vt6655/device_main.o
CHECK drivers/staging/vt6655/card.c
CC [M] drivers/staging/vt6655
From: Mathieu Poirier
mpoirier@t430:~/work/kernel1$ make C=1 M=drivers/staging/vt6655/
CHECK drivers/staging/vt6655/mac.c
drivers/staging/vt6655/mac.c:162:6: warning: symbol 'MACvGetShortRetryLimit'
was not declared. Should it be static?
CC [M] drivers/staging/vt6655/mac.o
Commit:
87299d
Hi,
On Sat, Jan 17, 2015 at 04:22:35PM -0800, Greg KH wrote:
> On Fri, Jan 16, 2015 at 08:58:10PM -0800, Andrew Milkovich wrote:
> > Fixed coding style errors per checkpatch.pl. Inline comments removed;
> > chanptr
> > comment is implied by the first argument being the channel. No parent bus
> >
On 01/16/2015 09:48 AM, Stefan Wahren wrote:
> This patch adds root compatible properties for the following boards:
> - Raspberry Pi Model B
> - Raspberry Pi Model B+
We may as well add all the models while at it. I think the following is
a complete list so far:
"model-b-i2c0": early model B with
On 01/16/2015 09:49 AM, Stefan Wahren wrote:
> This patch converts all bcm2835 dts files to use the pinctrl
> header file.
Aside from the one comment I made, the series,
Acked-by: Stephen Warren
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
Hi again,
On Sat, Jan 17, 2015 at 04:22:35PM -0800, Greg KH wrote:
> On Fri, Jan 16, 2015 at 08:58:10PM -0800, Andrew Milkovich wrote:
> > Fixed coding style errors per checkpatch.pl. Inline comments removed;
> > chanptr
> > comment is implied by the first argument being the channel. No parent bu
On Sat, Jan 17, 2015 at 07:01:41PM -0800, Andrew Milkovich wrote:
> Hi,
>
> On Sat, Jan 17, 2015 at 04:22:35PM -0800, Greg KH wrote:
> > On Fri, Jan 16, 2015 at 08:58:10PM -0800, Andrew Milkovich wrote:
> > > Fixed coding style errors per checkpatch.pl. Inline comments removed;
> > > chanptr
> >
Fixed coding style errors per checkpatch.pl. Inline comments removed; chanptr
comment is implied by the first argument being the channel. No parent bus
comment is already understood by the argument being NULL.
Signed-off-by: Andrew Milkovich
---
drivers/staging/unisys/virtpci/virtpci.c | 24
On Sat, Jan 17, 2015 at 05:45:15PM -0800, Guenter Roeck wrote:
> On 01/17/2015 02:09 PM, Vivien Didelot wrote:
> >Hi Guenter, Greg,
> >
> [ .. ]
>
> >
> >BTW Guenter, does this patch make sense to you?
> >
>
> It does make sense to me to only use the return value from is_visible
> for the mode.
>
Fixed coding style errors per checkpatch.pl. Inline comments removed; chanptr
comment is implied by the first argument being the channel. No parent bus
comment is already understood by the argument being NULL.
Signed-off-by: Andrew Milkovich
---
drivers/staging/unisys/virtpci/virtpci.c | 24
On Sat, 2015-01-17 at 19:30 -0800, Andrew Milkovich wrote:
> Fixed coding style errors per checkpatch.pl.
which ones?
As far as I can tell, none of the checkpatch
warnings on this file are actually addressed
by this patch.
--
To unsubscribe from this list: send the line "unsubscribe linux-kerne
101 - 200 of 223 matches
Mail list logo
