On Sun, Oct 16, 2016 at 01:16:50PM -0400, Oleg Drokin wrote:
> It's multiple values per file, so it has no business being in sysfs,
> besides it was assuming seqfile anyway.
>
> Introduced by
> commit d806f30e639b ("staging: lustre: osc: revise unstable pages accounting")
In the future, the "corr
Hello there,
linux-4.9-rc1/drivers/staging/lustre/lustre/osc/osc_request.c:973]: (style)
Checking if unsigned variable 'cli.cl_avail_grant' is less than zero.
Source code is
if (cli->cl_avail_grant < 0) {
Suggest code rework.
Regards
David Binderman
On Mon, Oct 17, 2016 at 01:13:55AM +0530, Pankaj Bharadiya wrote:
> Replace CamelCase names with underscores to comply with the standard
> kernel coding style.
>
> Signed-off-by: Pankaj Bharadiya
> ---
> drivers/staging/dgnc/dgnc_tty.c | 20 ++--
> 1 file changed, 10 insertions(+
On Mon, Oct 17, 2016 at 12:40:01AM +0100, Cathal Mullaney wrote:
> Fixed sparse warning; function not declared static.
>
> Signed-off-by: Cathal Mullaney
> ---
> drivers/staging/dgnc/dgnc_tty.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Someone else already made this change before
On Sun, Oct 16, 2016 at 10:16:00PM +0200, Carlos Maiolino wrote:
> A small patchset fixing some coding style errors identified by checkpatch.pl
>
> Greg. I splitted the single patch into 3 different patches now, hope I set it
> right now :)
>
> Also added a blank line before the signed-off.
None
On Fri, Oct 14, 2016 at 07:22:57PM +0200, Fernando Apesteguia wrote:
> The patch replaces the macro with a function (dgnc_get_board) and
> substitutes the macro statement with a call to that function and a
> comparison on the returned value.
>
> This removes a checkpatch warning.
>
> Signed-off-b
On Mon, Oct 17, 2016 at 07:33:33AM +, David Binderman wrote:
> Hello there,
>
>
>
> linux-4.9-rc1/drivers/staging/lustre/lustre/osc/osc_request.c:973]: (style)
> Checking if unsigned variable 'cli.cl_avail_grant' is less than zero.
>
>
>
> Source code is
>
>
>
> if (cli->cl_avail_g
gb_audio_manager_module_descriptor's intf_id field maintains the
information about the interface on which module is connected hence
having an extra slot field is redundant.
Thus remove the slot field and its associated code.
Signed-off-by: Pankaj Bharadiya
---
drivers/staging/greybus/audio_code
On Mon, Oct 17, 2016 at 2:42 PM, Pankaj Bharadiya
wrote:
> gb_audio_manager_module_descriptor's intf_id field maintains the
> information about the interface on which module is connected hence
> having an extra slot field is redundant.
>
> Thus remove the slot field and its associated code.
>
> Si
On Mon, Oct 17, 2016 at 2:42 PM, Pankaj Bharadiya
wrote:
> gb_audio_manager_module_descriptor's intf_id field maintains the
> information about the interface on which module is connected hence
> having an extra slot field is redundant.
>
> Thus remove the slot field and its associated code.
>
> Si
Replace CamelCase names with underscores to comply with the standard
kernel coding style.
Signed-off-by: Pankaj Bharadiya
---
drivers/staging/dgnc/dgnc_tty.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/dgnc/dgnc_tty.c b/drivers/staging/dgnc/dgnc_tty.
Hi,
This patchset does minor cleanup for dgnc driver.
v1->v2:
- Split into multiple patches
- remove redundant dgnc_tmp_write_buf variable and function calls
- rename dgnc_default_termios to default_termios
Pankaj Bharadiya (2):
staging: dgnc: Remove some redundant functions
staging: dgnc:
dgnc_tty_preinit() and dgnc_tty_post_uninit() functions are used to
allocate and free "dgnc_TmpWriteBuf" and this "dgnc_TmpWriteBuf" is
not really getting used. Hence remove these redundant functions.
Also remove dgnc_TmpWriteBuf variable as it is not used anymore.
Signed-off-by: Pankaj Bharadiya
Hi Greg,
I've based these patches on top of linux-next. Thanks anyway.
Which tree are you applying these patches on? I'll make sure to submit
next patches based on your tree.
Cheers
On Mon, Oct 17, 2016 at 10:27 AM, Greg KH wrote:
> On Sun, Oct 16, 2016 at 10:16:00PM +0200, Carlos Maiolino wro
On Mon, Oct 17, 2016 at 01:26:51PM +0200, Carlos Maiolino wrote:
> Hi Greg,
>
> I've based these patches on top of linux-next. Thanks anyway.
>
> Which tree are you applying these patches on? I'll make sure to submit
> next patches based on your tree.
While the -rc1 merge window was open, I coul
Fixed following checkpatch warning
WARNING: Symbolic permissions 'S_IRUGO' are not preferred. Consider
using octal permissions '0444'.
Signed-off-by: Shyam Saini
---
drivers/staging/gs_fpgaboot/gs_fpgaboot.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/gs_f
Hi Paolo,
Thanks for reviews. I will incorporate your feedbacks in v2.
On 10/13/2016 06:19 AM, Paolo Bonzini wrote:
On 23/08/2016 01:23, Brijesh Singh wrote:
TODO:
- send qemu/seabios RFC's on respective mailing list
- integrate the psp driver with CCP driver (they share the PCI id's)
- add
From: Wei Yongjun
In case of error, the function kthread_create() returns ERR_PTR() and
never returns NULL. The NULL test in the return value check should be
replaced with IS_ERR().
Signed-off-by: Wei Yongjun
---
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c | 6 +++---
1 file
From: Wei Yongjun
Remove .owner field if calls are used which set it automatically.
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
Signed-off-by: Wei Yongjun
---
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/
From: Wei Yongjun
In case of error, the function kthread_create() returns ERR_PTR() and
never returns NULL. The NULL test in the return value check should be
replaced with IS_ERR().
Signed-off-by: Wei Yongjun
---
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 2 +-
1 file chan
[+CC: staging list ]
On Sun, Oct 16, 2016 at 02:19:48PM +0530, sayli karnik wrote:
> This patch replaces kmalloc and memcpy with kmemdup for duplication of
> memory.
>
> Signed-off-by: sayli karnik
Acked-by: Johan Hovold
> ---
> drivers/staging/greybus/es2.c | 3 +--
> 1 file changed, 1 inse
On Sun, Oct 16, 2016 at 03:29:14PM +0530, Pankaj Bharadiya wrote:
> gb_audio_manager_module_descriptor's cport field is actually used to
> manage and pass interface id to user space.
>
> Thus rename gb_audio_manager_module_descriptor's 'cport' field and
> few other things to avoid confusion.
Plea
Fix coding style errors reported by checkpatch.pl, specifically:
ERROR: space prohibited before that ',' (ctx:WxW)
This patch get rid of all "ERROR" messages from checkpatch.pl for this file
Signed-off-by: Carlos Maiolino
---
drivers/staging/rtl8188eu/core/rtw_mlme_ext.c | 28 +
Hello Mauro,
I noticed that module autoload won't be working in a bunch of media
platform drivers because the module alias information is not filled
in the modules. This patch series contains the fixes for them.
Best regards,
Javier
Javier Martinez Canillas (5):
[media] v4l: vsp1: Fix module
If the driver is built as a module, autoload won't work because the module
alias information is not filled. So user-space can't match the registered
device with the corresponding module.
Export the module alias information using the MODULE_DEVICE_TABLE() macro.
Before this patch:
$ modinfo drive
If the driver is built as a module, autoload won't work because the module
alias information is not filled. So user-space can't match the registered
device with the corresponding module.
Export the module alias information using the MODULE_DEVICE_TABLE() macro.
Before this patch:
$ modinfo drive
Create an option CONFIG_LED_TRIGGER_PHY (default n), which will create a
set of led triggers for each instantiated PHY device. There is one LED
trigger per link-speed, per-phy.
The triggers are registered during phy_attach and unregistered during
phy_detach.
This allows for a user to configure the
phy_supported_speeds provides a means to get a list of all the speeds a
phy device currently supports.
Signed-off-by: Zach Brown
---
drivers/net/phy/phy.c | 35 +++
include/linux/phy.h | 15 +++
2 files changed, 50 insertions(+)
diff --git a/drivers
Adding led support for phy causes namespace conflicts for some
phy drivers.
The marvel skge driver declared an enum for representing the states of
Link LED Register. The enum contained constant LED_OFF which conflicted
with declartation found in linux/leds.h.
LED_OFF changed to LED_REG_OFF
Also ch
Fix skge driver that declared enum contants that conflicted with enum
constants in linux/leds.h
Create function that encapsulates actions taken during the adjust phy link step
of phy state changes.
Create function that provides list of speeds currently supported by the phy.
Add support for led t
During phy state machine state transitions some set of actions should
occur whenever the link state changes. These actions should be
encapsulated into a single function
This patch adds the phy_adjust_link function, which is called whenever
phydev->adjust_link would have been called before. Actions
Fixed following checkpatch.pl warning
WARNING: Symbolic permissions 'S_IRUGO | S_IWUSR' are not preferred.
Consider using octal permissions '0644'.
Signed-off-by: Shyam Saini
---
drivers/staging/media/bcm2048/radio-bcm2048.c | 40 +--
1 file changed, 20 insertions(+), 20
Fixed following checkpatch warning
Symbolic permissions 'S_IRUGO' are not preferred. Consider using octal
permissions '0444'
Signed-off-by: Shyam Saini
---
drivers/staging/media/bcm2048/radio-bcm2048.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers
On Mon, Oct 17, 2016 at 05:39:22PM +0200, Carlos Maiolino wrote:
> Fix coding style errors reported by checkpatch.pl, specifically:
>
> ERROR: space prohibited before that ',' (ctx:WxW)
>
> This patch get rid of all "ERROR" messages from checkpatch.pl for this file
>
> Signed-off-by: Carlos Maio
On Mon, Oct 17, 2016 at 09:24:19PM +0530, Shyam Saini wrote:
> Fixed following checkpatch warning
> Symbolic permissions 'S_IRUGO' are not preferred. Consider using octal
> permissions '0444'
>
> Signed-off-by: Shyam Saini
> ---
> drivers/staging/media/bcm2048/radio-bcm2048.c | 18 +-
On Mon, Oct 17, 2016 at 09:23:07PM +0530, Shyam Saini wrote:
> Fixed following checkpatch.pl warning
> WARNING: Symbolic permissions 'S_IRUGO | S_IWUSR' are not preferred.
> Consider using octal permissions '0644'.
>
> Signed-off-by: Shyam Saini
> ---
> drivers/staging/media/bcm2048/radio-bcm204
>> ---
>> drivers/staging/rtl8188eu/core/rtw_mlme_ext.c | 28
>> +--
>> 1 file changed, 14 insertions(+), 14 deletions(-)
>
> Any reason you didn't use scripts/get_maintainer.pl to cc: the relevant
> maintainers and developers of this code?
>
> thanks,
>
> greg k-h
Not at
On Mon, Oct 17, 2016 at 06:17:21PM +0200, Carlos Maiolino wrote:
> >> ---
> >> drivers/staging/rtl8188eu/core/rtw_mlme_ext.c | 28
> >> +--
> >> 1 file changed, 14 insertions(+), 14 deletions(-)
> >
> > Any reason you didn't use scripts/get_maintainer.pl to cc: the relevan
Yes, thanks for the heads up. I'll try to avoid such mistakes in the
next patches
On Mon, Oct 17, 2016 at 6:29 PM, Greg KH wrote:
> On Mon, Oct 17, 2016 at 06:17:21PM +0200, Carlos Maiolino wrote:
>> >> ---
>> >> drivers/staging/rtl8188eu/core/rtw_mlme_ext.c | 28
>> >> +
From: Wei Yongjun
This node pointer is returned by of_find_compatible_node() with
refcount incremented in this function. of_node_put() on it before
exitting this function.
This is detected by Coccinelle semantic patch.
Signed-off-by: Wei Yongjun
---
drivers/staging/greybus/arche-platform.c |
On Mon, 2016-10-17 at 18:07 +0200, Greg KH wrote:
> On Mon, Oct 17, 2016 at 09:24:19PM +0530, Shyam Saini wrote:
> >
> > Fixed following checkpatch warning
> > Symbolic permissions 'S_IRUGO' are not preferred. Consider using
> > octal
> > permissions '0444'
> >
> > Signed-off-by: Shyam Saini
> >
On Mon, Oct 17, 2016 at 04:37:20PM +, Wei Yongjun wrote:
> From: Wei Yongjun
>
> This node pointer is returned by of_find_compatible_node() with
> refcount incremented in this function. of_node_put() on it before
> exitting this function.
>
> This is detected by Coccinelle semantic patch.
>
OBD_INIT_CHECK was previously used by a conditional group. This is
no longer the case so it can be removed.
Signed-off-by: Aya Mahfouz
---
drivers/staging/lustre/lustre/obdclass/class_obd.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/lustre/lustre/obdclass/class_obd.c
b/d
Hi Paolo,
On 10/13/2016 05:45 AM, Paolo Bonzini wrote:
On 23/08/2016 01:28, Brijesh Singh wrote:
The ioctl will be used by qemu to issue the Secure Encrypted
Virtualization (SEV) guest commands to transition a guest into
into SEV-enabled mode.
a typical usage:
struct kvm_sev_launch_start s
The Kconfig dependency previously included ARM64, which is not
strictly correct. Change it to ARCH_LAYERSCAPE which is the
SoC platform that includes the DPAA2 hardware.
Signed-off-by: Stuart Yoder
---
drivers/staging/fsl-mc/bus/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
dif
fsl-mc-private.h references several definitions that it does not explicitly
have includes for. Up until now we've gotten lucky due to include ordering
that things compile. Add the missing includes.
Signed-off-by: Stuart Yoder
---
drivers/staging/fsl-mc/bus/fsl-mc-private.h | 3 +++
1 file chang
improve readibility of the DPRC interrupt name in sysfs by
shortening this and just using the device name. There is only
one interrupt DPRC used, so no further differentiation is needed.
Signed-off-by: Stuart Yoder
---
drivers/staging/fsl-mc/bus/dprc-driver.c | 2 +-
1 file changed, 1 insertion
Be consistent and use kernel-preferred multi-line comment style
everywhere.
Signed-off-by: Stuart Yoder
---
drivers/staging/fsl-mc/bus/dpbp-cmd.h | 3 ++-
drivers/staging/fsl-mc/bus/dpbp.c | 3 ++-
drivers/staging/fsl-mc/bus/dpcon-cmd.h | 3 ++-
drivers/staging/fsl-mc/bus/dpmcp-cmd.h
Cleanup and clarify messages
-use "fsl-mc" prefix for all messages that don't come from dev_*
functions
-remove unnecessary verbosity to make things clearer
-use explicit __func__ arguments to clarify some error messages
-improve misc wording for clarity
Signed-off-by: Stuart Yoder
---
From: Ioana Ciornei
Make whitespace consistent with other fsl-mc source files.
-delete extraneous tabs
-align start of function arguments properly
-for structs and function definitions, separate type and
variable name by a single space
-remove multiple blank lines in comments
Signed-o
The dpbp and dpcon cmd header files should not be in the public
include directory but should be private to the bus driver.
Signed-off-by: Stuart Yoder
---
drivers/staging/fsl-mc/{include => bus}/dpbp-cmd.h | 0
drivers/staging/fsl-mc/bus/dpbp.c | 3 ++-
drivers/staging/fsl-mc/
This patch does miscellaneous minor cleanup and uprevs the supported
MC firmware version to 10.x.
I believe with this series the TODO items on our list are complete,
with the exception of "Add at least one device driver for a DPAA2
object", which I expect to send a patch for very soon. I plan to
Except for copyrights we are avoiding all references
to Freescale, which no longer exists as a brand. Cleanup
Freescale references and simplify the Kconfig description
of the fsl-mc bus.
Signed-off-by: Stuart Yoder
---
drivers/staging/fsl-mc/bus/Kconfig | 22 +++---
1 file chang
improve and clarify miscellaneous comments
-when referring to the bus call it "fsl-mc"
-reduce the verbosity of some comments for clarity
-improve wording of the slightly confusing term "MC object device"
Signed-off-by: Stuart Yoder
---
drivers/staging/fsl-mc/bus/fsl-mc-allocator.c | 68 ++
For /proc/interrupts readability, platform bus MSIs are named
"ITS-pMSI" in the GIC ITS implementation for that bus. Follow
a similar naming convention and call fsl-mc bus MSIs
"ITS-fMSI".
Signed-off-by: Stuart Yoder
---
drivers/staging/fsl-mc/bus/irq-gic-v3-its-fsl-mc-msi.c | 2 +-
1 file chan
From: Ioana Ciornei
Replace all uses of uintX_t types with uX types in order to comply with
kernel coding style and resolve checkpatch warnings.
Signed-off-by: Ioana Ciornei
Signed-off-by: Stuart Yoder
---
drivers/staging/fsl-mc/bus/dpmcp.h | 84 +++---
1 file
Many source files have evolved without copyright date
updates. Update the dates to reflect work through
2016.
Signed-off-by: Stuart Yoder
---
drivers/staging/fsl-mc/bus/dpcon-cmd.h | 2 +-
drivers/staging/fsl-mc/bus/dpmcp.h | 2 +-
drivers/staging/fsl-mc/bus/
From: Ioana Ciornei
DPAA2 will not support MC firmware versions prior to MC v10.x.
Update the MC interface code and drivers to reflect this.
-update the object .h files and code that builds commands to include
the new command version in the command header
-object versions are no longer
If the driver is built as a module, autoload won't work because the module
alias information is not filled. So user-space can't match the registered
device with the corresponding module.
Export the module alias information using the MODULE_DEVICE_TABLE() macro.
Before this patch:
$ modinfo drive
-German has moved on to other things and wished to be
removed as a maintainer
-cleanup the driver description to use the proper name
of the driver (i.e. the fsl-mc bus driver) and remove incorrect
references to Freescale
Signed-off-by: Stuart Yoder
Acked-by: J. German Rivera
---
MAINTAINERS
Fixed: sparse WARNING: symbol 'lov_read_and_clear_async_rc' was not declared.
Should it be static?
Signed-off-by: Ryan Swan
---
drivers/staging/lustre/lustre/lov/lov_object.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/lustre/lustre/lov/lov_object.c
b/dr
From: Stephen Hemminger
This is a new driver to enable userspace networking on VMBus.
It is based largely on the similar driver that already exists
for PCI, and earlier work done by Brocade to support DPDK.
Signed-off-by: Stephen Hemminger
---
v2 - put in UIO not staging
rename to uio-hv-g
From: Stephen Hemminger
This patch adds sysfs interface to dynamically bind new UUID values
to existing VMBus device. This is useful for generic UIO driver to
act similar to uio_pci_generic.
Signed-off-by: Stephen Hemminger
---
v2 - allow device driver to have empty id table, and fix bugs
dri
From: Stephen Hemminger
Update UIO documentation to include basic information about
uio_hv_generic.
Signed-off-by: Stephen Hemminger
---
Documentation/DocBook/uio-howto.tmpl | 62
1 file changed, 62 insertions(+)
diff --git a/Documentation/DocBook/uio-howt
On Mon, Oct 17, 2016 at 10:10:08PM +0530, Shyam Saini wrote:
> On Mon, 2016-10-17 at 18:07 +0200, Greg KH wrote:
> > On Mon, Oct 17, 2016 at 09:24:19PM +0530, Shyam Saini wrote:
> > >
> > > Fixed following checkpatch warning
> > > Symbolic permissions 'S_IRUGO' are not preferred. Consider using
>
On Mon, Oct 17, 2016 at 03:21:52PM -0400, Ryan Swan wrote:
> Fixed: sparse WARNING: symbol 'lov_read_and_clear_async_rc' was not declared.
> Should it be static?
I don't believe you actually test-built this change :(
>
> Signed-off-by: Ryan Swan
> ---
> drivers/staging/lustre/lustre/lov/lov_o
> I am not sure if I fully understand this feedback. Let me summaries what
> we have right now.
>
> At highest level SEV key management commands are divided into two sections:
>
> - platform management : commands used during platform provisioning. PSP
> drv provides ioctl's for these commands. Q
class_devno_max is an inline function that is written with
the sole purpose of returning the value of MAX_OBD_DEVICES.
Since no computations are made by the class_devno_max,
replace it with MAX_OBD_DEVICES.
Aya Mahfouz (2):
staging: lustre: replace uses of class_devno_max by MAX_OBD_DEVICES
s
class_devno_max is an inline function that returns
MAX_OBD_DEVICES. Replace all calls to the function
by MAX_OBD_DEVICES.
Signed-off-by: Aya Mahfouz
---
drivers/staging/lustre/lustre/obdclass/class_obd.c | 6 +++---
drivers/staging/lustre/lustre/obdclass/genops.c| 22 +++---
On Mon, Oct 17, 2016 at 10:29 AM, Greg KH wrote:
> On Fri, Oct 14, 2016 at 07:22:57PM +0200, Fernando Apesteguia wrote:
>> The patch replaces the macro with a function (dgnc_get_board) and
>> substitutes the macro statement with a call to that function and a
>> comparison on the returned value.
>>
class_devno_max is an inline function that is not used
anymore. Hence, it is removed.
Signed-off-by: Aya Mahfouz
---
drivers/staging/lustre/lustre/include/obd_class.h | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/staging/lustre/lustre/include/obd_class.h
b/drivers/staging/lust
A recent rework removed the initialization of the successful return
code from lpfc_write_firmware:
drivers/scsi/lpfc/lpfc_init.c: In function 'lpfc_write_firmware':
drivers/scsi/lpfc/lpfc_init.c:10333:214: error: format '%ld' expects argument
of type 'long int', but argument 9 has type 'size_t {a
After a recent bugfix, we get a warning about the use of an uninitialized
variable:
drivers/staging/lustre/lnet/libcfs/linux/linux-cpu.c: In function
'cfs_cpt_table_create_pattern':
drivers/staging/lustre/lnet/libcfs/linux/linux-cpu.c:833:7: error: 'str' may be
used uninitialized in this functio
The newly added __sca3000_get_base_freq function handles all valid
modes of the SCA3000_REG_ADDR_MODE register, but gcc notices
that any other value (i.e. 0x00) causes the base_freq variable to
not get initialized:
drivers/staging/iio/accel/sca3000_core.c: In function 'sca3000_write_raw':
drivers/
The hdr_offset variable is only if we deal with a TCP or UDP packet,
but as the check surrounding its usage tests for skb_is_gso()
instead, the compiler has no idea if the variable is initialized
or not at that point:
drivers/net/hyperv/netvsc_drv.c: In function ‘netvsc_start_xmit’:
drivers/net/hy
On Monday, October 17, 2016 10:23:00 PM CEST Patrick Farrell wrote:
> Arnd,
>
>
> The description and the actual patch don't seem to match up. Am I missing
> something?
Sorry, I pasted the wrong error message when writing the changelog.
> From: lustre-devel on behalf of Arnd
> Bergmann
> S
On Mon, Oct 17, 2016 at 3:29 PM, Arnd Bergmann wrote:
>
> Sorry, I pasted the wrong error message when writing the changelog.
Not just the warning, the summary above it talks about the wrong
function too. And the commit it references doesn't actually exist
either. So apparently this is against so
On Oct 17, 2016, at 15:46, Aya Mahfouz wrote:
>
> class_devno_max is an inline function that returns
> MAX_OBD_DEVICES. Replace all calls to the function
> by MAX_OBD_DEVICES.
Thanks for your patch, but unfortunately it can't be accepted.
This function was added in preparation of being able to
A recent rework dropped the initialization of the initialization of the
successful return code in lov_getstripe:
drivers/staging/lustre/lustre/lov/lov_pack.c: In function 'lov_getstripe':
drivers/staging/lustre/lustre/lov/lov_pack.c:426:9: error: 'rc' may be used
uninitialized in this function [-
On Monday, October 17, 2016 3:37:11 PM CEST Linus Torvalds wrote:
> On Mon, Oct 17, 2016 at 3:29 PM, Arnd Bergmann wrote:
> >
> > Sorry, I pasted the wrong error message when writing the changelog.
>
> Not just the warning, the summary above it talks about the wrong
> function too. And the commit
From: Wei Yongjun
sizeof() when applied to a pointer typed expression gives the
size of the pointer, not that of the pointed data.
Fixes: 2eb9d8cbb3c3 ("staging: rts5208: rtsx.c: Alloc sizeof struct")
Signed-off-by: Wei Yongjun
---
drivers/staging/rts5208/rtsx.c | 2 +-
1 file changed, 1 inser
On Mon, Oct 17, 2016 at 1:16 PM, Greg KH wrote:
>
> On Mon, Oct 17, 2016 at 01:13:55AM +0530, Pankaj Bharadiya wrote:
> > Replace CamelCase names with underscores to comply with the standard
> > kernel coding style.
> >
> > Signed-off-by: Pankaj Bharadiya
> > ---
> > drivers/staging/dgnc/dgnc_tt
On Mon, Oct 17, 2016 at 11:46:36PM +0200, Fernando Apesteguía wrote:
> On Mon, Oct 17, 2016 at 10:29 AM, Greg KH wrote:
> > On Fri, Oct 14, 2016 at 07:22:57PM +0200, Fernando Apesteguia wrote:
> >> The patch replaces the macro with a function (dgnc_get_board) and
> >> substitutes the macro stateme
83 matches
Mail list logo