On Mon, Sep 28, 2015 at 11:25:04PM -0700, Anish Bhatt wrote:
> On Mon, Sep 28, 2015 at 5:42 PM, Greg KH wrote:
> > On Fri, Sep 25, 2015 at 12:45:16AM -0700, Anish Bhatt wrote:
> >> Remove leftover comment delimiters that were only partially removed
> >> in a previous cleanup.
> >
> > You also chan
On Mon, Sep 28, 2015 at 5:42 PM, Greg KH wrote:
> On Fri, Sep 25, 2015 at 12:45:16AM -0700, Anish Bhatt wrote:
>> Remove leftover comment delimiters that were only partially removed
>> in a previous cleanup.
>
> You also changed code formatting, which isn't ok to do in the same
> patch, please bre
From: Oleg Drokin
This patch is a first stab at trying to remove structure fields
from obd_export and obd structures that are only used on the
server or make sense on the server. These include tracking
requests in recovery, various recovery stages, lists of
recovered and not yet recovered clients
From: Oleg Drokin
These seem to be unused.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lclient.h| 4
drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 15 ---
2 files changed, 19 deletions(-)
diff --git a/drivers/staging/lustre/lustre/include/
From: Oleg Drokin
Does not appear to be used anywhere.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lclient.h| 1 -
drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 7 ---
2 files changed, 8 deletions(-)
diff --git a/drivers/staging/lustre/lustre/include/lc
From: Oleg Drokin
All of them but the ccc_transient_page_prep are unused, so remove
the unused ones.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lclient.h| 19 -
drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 48 --
2 files changed
From: Arnd Bergmann
This is a large patch to remove all dead code from obdclass and
ptlrpc, purely removing functions and declarations now, so
despite the size should be easy enough to verify.
Signed-off-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
.../lustre/lustre/include/lustre/lustre_
From: Oleg Drokin
These client request/import functions are not used anywhere,
so drop them.
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_net.h | 13 ---
drivers/staging/lustre/lustre/ptlrpc/client.c | 95 --
d
From: Oleg Drokin
These functions are not even referenced in any header files anymore.
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/obdclass/obdo.c | 170 --
1 file changed, 170 deletions(-)
diff --git a/drivers/staging/lustr
From: Oleg Drokin
These functions are mostly used on the server.
class_uuid2obd, get_devices_count, class_obd_list, class_conn2obd,
class_conn2cliimp, class_connected_export, obd_exports_barrier,
kuc_* (kernel-userspace communications).
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
--
From: Oleg Drokin
Since we know lcs_srv is always NULL, just get rid of it completely
and fix up all the code to assumee it was never there.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/fid/fid_request.c| 48 --
drivers/staging/lustre/lustre/fid/lproc_fi
From: Oleg Drokin
nid_hash is used on export to faster find clients based on
their NID. There's no use for that on the client.
Signed-off-by: Oleg Drokin
---
.../staging/lustre/lustre/include/lustre_export.h | 1 -
drivers/staging/lustre/lustre/include/obd.h| 2 -
drivers/staging/lu
From: Oleg Drokin
No callers left.
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_handles.h | 1 -
drivers/staging/lustre/lustre/obdclass/lustre_handles.c | 13 -
2 files changed, 14 deletions(-)
diff --git a/drivers/stagi
From: Oleg Drokin
All users are only in the server code
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
.../staging/lustre/lustre/include/lustre_disk.h| 2 --
drivers/staging/lustre/lustre/obdclass/obd_mount.c | 27 --
2 files changed, 29 deletions(-)
diff
From: Oleg Drokin
Those functions are not present anywhere in the client code.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_fid.h | 24 --
1 file changed, 24 deletions(-)
diff --git a/drivers/staging/lustre/lustre/include/lustre_fid.h
b/driv
From: Oleg Drokin
In fact they could be static as they are only used inside
this file, so remove EXPORT_SYMBOL and declarations.
Also seq_client_init is always called with srv = NULL, so just
drop this argument.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/fid/fid_request.c
From: Oleg Drokin
Drop unused functions.
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_sec.h | 20 --
drivers/staging/lustre/lustre/ptlrpc/sec.c | 116
drivers/staging/lustre/lustre/ptlrpc/sec_bulk.c| 329 -
From: Oleg Drokin
Also while we are at it, remove seq_site_fini forward declaration
as there's no such function anymore.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/fid/fid_request.c| 51 --
drivers/staging/lustre/lustre/include/lustre_fid.h | 3 --
2
From: Oleg Drokin
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_fid.h | 51 --
1 file changed, 51 deletions(-)
diff --git a/drivers/staging/lustre/lustre/include/lustre_fid.h
b/drivers/staging/lustre/lustre/include/lustre_fid.h
index bbe3ff8..
From: Oleg Drokin
It's not referenced anywhere anymore.
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/ldlm/ldlm_lockd.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_lockd.c
b/drivers/staging/lustre/l
From: Oleg Drokin
They are no longer used anywhere.
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
.../lustre/lustre/obdclass/linux/linux-obdo.c | 137 -
1 file changed, 137 deletions(-)
diff --git a/drivers/staging/lustre/lustre/obdclass/linux/linux-obdo.
From: Oleg Drokin
cl_object_header_fini, cl_object_has_locks, cl_attr2lvb,
cl_page_list_own, cl_page_list_unmap, cl_2queue_assume,
cl_io_print, cl_enqueue,
also cleanup extern declarations in cl_object.h
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/i
From: Oleg Drokin
Quite a bunch of them are only used on the server.
lu_object_put_nocache, lu_object_invariant, lu_dev_del_linkage,
lu_context_tags_update, lu_context_tags_clear, lu_session_tags_update,
lu_session_tags_clear, lu_env_refill_by_tags, lu_printk_printer,
lu_object_assign_fid, lu_obj
From: Oleg Drokin
rs_batch is used on the server only.
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_net.h | 6 +-
drivers/staging/lustre/lustre/ptlrpc/service.c | 145 -
2 files changed, 1 insertion(+), 150 de
From: Oleg Drokin
It's only used on the server
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
.../lustre/lustre/include/lustre_req_layout.h | 3 --
drivers/staging/lustre/lustre/ptlrpc/layout.c | 62 --
2 files changed, 65 deletions(-)
diff --git a/d
From: Oleg Drokin
This is a server-side request printing function, so we don't
really need it on the client.
Signed-off-by: Oleg Drokin
Reported-by: Arnd Bergmann
---
.../staging/lustre/lustre/include/lprocfs_status.h | 4 ---
.../staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c| 30 -
From: Oleg Drokin
No point in retaining it if it's unused.
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_net.h | 1 -
drivers/staging/lustre/lustre/ptlrpc/ptlrpcd.c | 42 --
2 files changed, 43 deletions(-)
dif
From: Oleg Drokin
This function is only used on the server where real high-priority
requests actually exist.
This deletes ptlrpc_hpreq_handler() and ptlrpc_request_change_export()
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_net.h | 3
From: Oleg Drokin
Apparently this is only used only on the metadata server.
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
.../staging/lustre/lustre/include/lustre_eacl.h| 4 -
drivers/staging/lustre/lustre/obdclass/acl.c | 119 -
2 files changed, 12
From: Oleg Drokin
It's only used on the server.
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_lib.h | 1 -
drivers/staging/lustre/lustre/obdclass/statfs_pack.c | 14 --
2 files changed, 15 deletions(-)
diff --git a/drive
From: Oleg Drokin
ptlrpc_service_health_check is only used on a service, so
it makes no point to retain it in the client code.
Also removing it's helpers: ptlrpc_svcpt_health_check and
ptlrpc_nrs_req_peek_nolock
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/
From: Oleg Drokin
This set removes a bunch of lustre code that is no lonver relevant,
is server specific or is otherwise unused.
Please consider.
More to come later.
Arnd Bergmann (1):
staging/lustre: remove lots of dead code
Oleg Drokin (28):
staging/lustre: Remove unused target_print_re
On Fri, Sep 25, 2015 at 10:23:44PM +0530, Shraddha Barke wrote:
> Stop using DBG_88E which is a custom macro for printing debugging
> messages. Instead start using pr_debug and in the process define
> pr_fmt.
>
> Signed-off-by: Shraddha Barke
> ---
> drivers/staging/rtl8188eu/os_dep/os_intfs.c |
On Sep 28, 2015, at 10:08 PM, Greg Kroah-Hartman wrote:
> On Mon, Sep 28, 2015 at 11:36:01AM +0200, Arnd Bergmann wrote:
>> On Sunday 27 September 2015 16:46:06 gr...@linuxhacker.ru wrote:
>>> From: Oleg Drokin
>>>
>>> This is a large patch to remove all dead code from obdclass and
>>> ptlrpc,
On Mon, Sep 28, 2015 at 11:36:01AM +0200, Arnd Bergmann wrote:
> On Sunday 27 September 2015 16:46:06 gr...@linuxhacker.ru wrote:
> > From: Oleg Drokin
> >
> > This is a large patch to remove all dead code from obdclass and
> > ptlrpc, purely removing functions and declarations now, so
> > despit
On Mon, Sep 28, 2015 at 11:43:55PM +0530, Chandra S Gorentla wrote:
> - The message queue is replaced with standard Linux linked list
> - kmem_cache is used for list members
> - A check for return value of receive method is added
> - GFP_ATOMIC is changed to GFP_KERNEL
> - A few other related
On Mon, Sep 28, 2015 at 10:10:42PM +0100, Luis de Bethencourt wrote:
> Remove the disabled code, for now, with #if 0.
>
> Signed-off-by: Luis de Bethencourt
> ---
>
> Hi,
>
> The code after the return is dead code. There is a comment saying it is
> disabled for now, it would be good if the code
On Mon, Sep 28, 2015 at 09:54:45PM +0100, Luis de Bethencourt wrote:
> Remove the unused code, which isn't implemented yet, using #if 0.
>
> Signed-off-by: Luis de Bethencourt
> ---
>
> Hi,
>
> The code after the return is dead code. My understanding is that it is
> there for when the output co
On Mon, Sep 28, 2015 at 11:43:55PM +0530, Chandra S Gorentla wrote:
> - The message queue is replaced with standard Linux linked list
> - kmem_cache is used for list members
Why?
> - A check for return value of receive method is added
> - GFP_ATOMIC is changed to GFP_KERNEL
Why? Are you sur
On Mon, Sep 28, 2015 at 05:18:29PM +0200, Christian Gromm wrote:
> This patchset is needed to pull Microchip's latest driver changes into
> the staging tree. The patches include bug fixes, feature updates and
> coding style changes.
I'll take these, but I really don't want to see new "features" ad
On Fri, Sep 25, 2015 at 12:45:16AM -0700, Anish Bhatt wrote:
> Remove leftover comment delimiters that were only partially removed
> in a previous cleanup.
You also changed code formatting, which isn't ok to do in the same
patch, please break it up into different patches.
thanks,
greg k-h
__
On Tue, Sep 29, 2015 at 12:07:43AM +0530, Chandra Gorentla wrote:
> Please do not review this. This is duplicate to -
> [PATCH 2/2] drivers: staging: wilc1000: wilc_msgqueue.c: Remove ineffective
> code
I don't understand, you sent 2 2/2 patches, which one do I look at?
Please just resend the w
On Wed, Sep 23, 2015 at 06:03:49PM +0900, Chaehyun Lim wrote:
> This patch removes wilc_platform.h include file in wilc_oswrapper.h
> Because wilc_platform.h have several standard header files so that some
> header file should be included .c file to avoid compilation errors.
>
> Signed-off-by: Cha
On Wed, Sep 23, 2015 at 04:10:54PM +0900, Glen Lee wrote:
> The define WILC_PARSE_SCAN_IN_HOST is always used in the driver, so just
> delete ifdef WILC_PARSE_SCAN_IN_HOST line, ifndef WILC_PARSE_SCAN_IN_HOST
> line and it's related codes. Finally, remove define in Makefile.
>
> Signed-off-by: Gle
On Mon, Sep 21, 2015 at 11:29:50PM +0530, Punit Vara wrote:
> This patch is to rtsx_chip.h that fixes up following warning reported
> by checkpatch.pl :
>
> -Block comments use * on subsequent lines
> -Block comments use a trailing */ on a separate line
>
> Signed-off-by: Punit Vara
> ---
> dri
Hi Enric,
On Fri, Sep 25, 2015 at 09:29:28PM +0200, Enric Balletbo i Serra wrote:
> At the moment it only supports ANX7814.
>
> The ANX7814 is an ultra-low power Full-HD (1080p60) SlimPort transmitter
> designed for portable devices.
>
> This driver adds initial support and supports HDMI to DP p
fixed style issue comparisons should place the constant on the right side of
the test
Signed-off-by: BryanSPaul
---
drivers/staging/rtl8188eu/core/rtw_cmd.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/rtl8188eu/core/rtw_cmd.c
b/drivers/staging/rt
On Mon, Sep 28, 2015 at 06:02:34PM -0600, BryanSPaul wrote:
> fixed style issue comparisons should place the constant on the right side of
> the test
>
> Signed-off-by: BryanSPaul
I doubt you sign legal papers without a ' ' in them :(
Please fix this up, and properly wrap your changelog at 72
On 2015/09/28, 2:19 AM, "lustre-devel on behalf of Arnd Bergmann"
wrote:
>On Monday 28 September 2015 01:09:18 Dilger, Andreas wrote:
>> On 2015/09/27, 10:45 PM, "gr...@linuxhacker.ru"
>> wrote:
>> >diff --git a/drivers/staging/lustre/lustre/ptlrpc/events.c
>> >b/drivers/staging/lustre/lustre/pt
The while() loop will only exit in a return or a goto ask_for_joinbss,
which means it will never break and execute the return after it.
Removing return _FAIL since it is dead code.
Signed-off-by: Luis de Bethencourt
---
drivers/staging/rtl8712/rtl871x_mlme.c | 2 +-
1 file changed, 1 insertion(+
Remove the disabled code, for now, with #if 0.
Signed-off-by: Luis de Bethencourt
---
Hi,
The code after the return is dead code. There is a comment saying it is
disabled for now, it would be good if the code is removed with #if 0 as
well.
Thanks,
Luis
drivers/staging/comedi/drivers/cb_pcida
Remove the unused code, which isn't implemented yet, using #if 0.
Signed-off-by: Luis de Bethencourt
---
Hi,
The code after the return is dead code. My understanding is that it is
there for when the output commands are implemented in the future.
Meanwhile it would be clearer if the code is remo
Please do not review this. This is duplicate to -
[PATCH 2/2] drivers: staging: wilc1000: wilc_msgqueue.c: Remove ineffective code
On Mon, Sep 28, 2015 at 11:43:56PM +0530, Chandra S Gorentla wrote:
> Signed-off-by: Chandra S Gorentla
> ---
> drivers/staging/wilc1000/wilc_msgqueue.c | 5 -
>
Mateusz Kulikowski writes:
> Firmware was not released properly if kmemdup fails.
>
> Addresses-Coverity-Id: 1269118
> Signed-off-by: Mateusz Kulikowski
> ---
> drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Acked-by: Jes Sorensen
Signed-off-by: Chandra S Gorentla
---
drivers/staging/wilc1000/wilc_msgqueue.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/staging/wilc1000/wilc_msgqueue.c
b/drivers/staging/wilc1000/wilc_msgqueue.c
index a01ada4..1a411d3 100644
--- a/drivers/staging/wilc1000/wilc_msgqueue.c
Signed-off-by: Chandra S Gorentla
---
drivers/staging/wilc1000/wilc_msgqueue.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/staging/wilc1000/wilc_msgqueue.c
b/drivers/staging/wilc1000/wilc_msgqueue.c
index a01ada4..1a411d3 100644
--- a/drivers/staging/wilc1000/wilc_msgqueue.c
- The message queue is replaced with standard Linux linked list
- kmem_cache is used for list members
- A check for return value of receive method is added
- GFP_ATOMIC is changed to GFP_KERNEL
- A few other related minor changes
Signed-off-by: Chandra S Gorentla
---
- Comments of Dan Carpe
From: Ira Weiny
STAGING_RDMA was failing to build when INFINIBAND was set to 'm' and
STAGING_RDMA was set to 'y'.
Making this a tristate properly inherits the 'm' from the INFINIBAND setting.
Reviewed-by: Dalessandro, Dennis
Reviewed-by: John, Jubin
Reviewed-by: Mike Marciniszyn
Signed-off-b
On Mon, Sep 28, 2015 at 09:56:19AM -0500, Larry Finger wrote:
> On 09/28/2015 06:18 AM, Sudip Mukherjee wrote:
> >On Sat, Sep 26, 2015 at 12:45:46PM -0500, Larry Finger wrote:
> >>On 09/26/2015 11:49 AM, Punit Vara wrote:
> >>>This patch is to the rtl871x_ioctl_linux.c that fixes up following
> >>>
By applying this patch the initialization of the most_aim structure is
performed at compile time.
Signed-off-by: Andrey Shvetsov
Signed-off-by: Christian Gromm
---
drivers/staging/most/aim-network/networking.c | 14 --
drivers/staging/most/aim-v4l2/video.c | 13 +++--
This patch reduces the DBR buffer size to prevent an overflow in the DIM2
module. It is needed, because the MediaLB hardware has problems with DBR
buffers that exceed the size of 255 messages.
Signed-off-by: Andrey Shvetsov
Signed-off-by: Christian Gromm
---
drivers/staging/most/hdm-dim2/dim2_h
This patch ensures a fair distribution of buffers, when two AIMs share a
single channel. The AIMs then won't be able to use more than half of all
pre-allocated buffers of the linked channel. However, in case the channel
is not shared, the AIM can exclusively use all available buffers.
Signed-off-b
This patch partly reverts a modification of function most_stop_channel()
that is trying to reset an established link between an AIM and an HDM in
all suitable places. But since the function most_stop_channel() is
stopping the data transfer it is the wrong place to do so.
Signed-off-by: Andrey Shve
This patch simply corrects style violations.
Signed-off-by: Christian Gromm
---
drivers/staging/most/aim-sound/sound.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/most/aim-sound/sound.c
b/drivers/staging/most/aim-sound/sound.c
index dff8f74..18c3
This patch adds 5.1 surround configuration with subbuffer cross-check,
when establishing a link to the core. For the sake of simplicity, only
one specific channel configuration is allowed.
Signed-off-by: Christian Gromm
---
drivers/staging/most/aim-sound/sound.c | 27 +-
This pathch moves the initialization of the PCM middle layer hardware
parameters to function audio_set_hw_params().
Signed-off-by: Christian Gromm
---
drivers/staging/most/aim-sound/sound.c | 28 ++--
1 file changed, 10 insertions(+), 18 deletions(-)
diff --git a/drive
This patch simply rearranges code for better readability.
Signed-off-by: Andrey Shvetsov
Signed-off-by: Christian Gromm
---
drivers/staging/most/hdm-usb/hdm_usb.c | 17 -
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/most/hdm-usb/hdm_usb.c
b/dr
This patch purges a temp. variable to store the functions return value.
Since the content is never being evaluated, it can safely be removed.
Signed-off-by: Christian Gromm
---
drivers/staging/most/aim-sound/sound.c |5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drive
This patch adds Microchip as vendor to the audio card's shortname to
be displayed, when playback and capture devices are queried.
Signed-off-by: Christian Gromm
---
drivers/staging/most/aim-sound/sound.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/most/
Since the PCM interface's hardware parameters are channel/substream
exclusive, the struct snd_pcm_hardware needs to be embedded in the
channel structure.
Signed-off-by: Christian Gromm
---
drivers/staging/most/aim-sound/sound.c | 68 +---
1 file changed, 36 insertio
This patch adds the implementation of the poll syscall to the AIM cdev.
To have the full functionality, a helper function is needed in the
core module to retrieve the instantaneous availability of tx buffers.
Signed-off-by: Christian Gromm
---
drivers/staging/most/aim-cdev/cdev.c | 26
This patch adds missing initialization of channel count for 8-bit mono
audio resolution.
Signed-off-by: Christian Gromm
---
drivers/staging/most/aim-sound/sound.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/staging/most/aim-sound/sound.c
b/drivers/staging/most/aim-sound/sou
This patch renames the function audio_set_pcm_format(). Since the
function doesn't only set the PCM format anymore and to guard against
misunderstandings, its name needs to be changed.
Signed-off-by: Christian Gromm
---
drivers/staging/most/aim-sound/sound.c |4 ++--
1 file changed, 2 insert
This patch removes debug messages and prevents the sound AIM from being
noisy in kernel log.
Signed-off-by: Christian Gromm
---
drivers/staging/most/aim-sound/sound.c | 14 --
1 file changed, 14 deletions(-)
diff --git a/drivers/staging/most/aim-sound/sound.c
b/drivers/staging/mo
This patch keeps the process from sleeping after the PCM middle layer has
stopped playback by calling the pcm trigger callback. The patch is needed
to prevent aplay from causing a pcm_write Input/Output error.
Signed-off-by: Christian Gromm
---
drivers/staging/most/aim-sound/sound.c | 17 +
This patch removes the audio format cross-check, because the definitions
are not compatible.
Signed-off-by: Christian Gromm
---
drivers/staging/most/aim-sound/sound.c |3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/most/aim-sound/sound.c
b/drivers/stagin
This patch prevents the HDM USB from submitting an URB with a buffer size
unaligned to 512 bytes to the USB subsystem.
Signed-off-by: Andrey Shvetsov
Signed-off-by: Christian Gromm
---
drivers/staging/most/hdm-usb/hdm_usb.c |3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git
The struct most_c_obj has the same set of attributes for each of two AIMs.
This patch cleans up the code by introducing the new struct most_c_aim_obj
hat contains those fields.
Signed-off-by: Christian Gromm
---
drivers/staging/most/mostcore/core.c | 108 +-
1 fi
This patch is needed to avoid having DMA on the stack.
Signed-off-by: Andrey Shvetsov
Signed-off-by: Christian Gromm
---
drivers/staging/most/hdm-usb/hdm_usb.c | 26 --
1 file changed, 16 insertions(+), 10 deletions(-)
diff --git a/drivers/staging/most/hdm-usb/hdm_usb
The function drci_rd_reg() always delivers little endian representation of
the 16-bit DCI register. The value returned by this function must always be
carefully converted from __le16 to u16 type. This patch moves all those
conversions to the function itself.
Signed-off-by: Andrey Shvetsov
Signed-
This patch fixes the opening and closing process of a physical channel
when used by different AIMs.
Signed-off-by: Andrey Shvetsov
Signed-off-by: Christian Gromm
---
drivers/staging/most/aim-cdev/cdev.c |5 +-
drivers/staging/most/aim-network/networking.c | 11 ++--
drivers/stagi
This patch fixes the representation of the MAC address within the HDM USB
module.
Signed-off-by: Andrey Shvetsov
Signed-off-by: Christian Gromm
---
drivers/staging/most/hdm-usb/hdm_usb.c | 24
1 file changed, 16 insertions(+), 8 deletions(-)
diff --git a/drivers/stag
The case where the channel type is neither synchronous nor isochronous is
already covered by a previous condition.
Signed-off-by: Christian Gromm
---
drivers/staging/most/hdm-usb/hdm_usb.c |6 --
1 file changed, 6 deletions(-)
diff --git a/drivers/staging/most/hdm-usb/hdm_usb.c
b/drive
If the network device is being opened right after it has been registered
via function register_netdev(), the device state is not yet consistent in
the context of function ndo_open(). This patch cares about having the
initialization done right, before the networking device is registered.
Signed-off
Since conf->extra_len has already been reset in most_start_channel() when
function hdm_configure_channel() gets called, it can safely be removed
here.
Signed-off-by: Christian Gromm
---
drivers/staging/most/hdm-usb/hdm_usb.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/m
Revision D of OS81118 network interface controller have the internal
buffer synchronization mechanism changed. This patch adapts the driver
to this.
Signed-off-by: Christian Gromm
---
drivers/staging/most/hdm-usb/hdm_usb.c | 41
1 file changed, 10 insertions(+)
This patch removes the wrongly used macros cpu_to_le16
Signed-off-by: Christian Gromm
---
drivers/staging/most/hdm-usb/hdm_usb.c |8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/most/hdm-usb/hdm_usb.c
b/drivers/staging/most/hdm-usb/hdm_usb.c
index 3
This patchset is needed to pull Microchip's latest driver changes into
the staging tree. The patches include bug fixes, feature updates and
coding style changes.
Christian Gromm (29):
staging: most: change structure initialization
staging: most: remove aim reset
staging: most: fix race condi
Since there is no way find out whether the INIC has generated an
interrupt, the I2C interrupt must not be registered as a shared
interrupt.
Reported-by: PrasannaKumar Muralidharan
Signed-off-by: Christian Gromm
---
drivers/staging/most/hdm-i2c/hdm_i2c.c |2 +-
1 file changed, 1 insertion(+
On 09/28/2015 06:18 AM, Sudip Mukherjee wrote:
On Sat, Sep 26, 2015 at 12:45:46PM -0500, Larry Finger wrote:
On 09/26/2015 11:49 AM, Punit Vara wrote:
This patch is to the rtl871x_ioctl_linux.c that fixes up following
warning reported by checkpatch.pl :
- Comparisons should place the constant
On Mon, Sep 28, 2015 at 4:48 PM, Sudip Mukherjee
wrote:
> On Sat, Sep 26, 2015 at 12:45:46PM -0500, Larry Finger wrote:
>> On 09/26/2015 11:49 AM, Punit Vara wrote:
>> >This patch is to the rtl871x_ioctl_linux.c that fixes up following
>> >warning reported by checkpatch.pl :
>> >
>> >- Comparisons
On Sun, Sep 27, 2015 at 09:48:19PM +0200, Mateusz Kulikowski wrote:
> Firmware was not released properly if kmemdup fails.
>
> Addresses-Coverity-Id: 1269118
> Signed-off-by: Mateusz Kulikowski
> ---
Not seen your v2 while replying to your v1. Sorry for the noise.
regards
sudip
_
On Sun, Sep 27, 2015 at 09:30:54PM +0200, Mateusz Kulikowski wrote:
> Firmware was not released properly if kmemdup fails.
>
> Addresses-Coverity-Id: 1269118
> Signed-off-by: Mateusz Kulikowski
> ---
> drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c | 2 +-
> 1 file changed, 1 insertion(+), 1
On Sat, Sep 26, 2015 at 12:45:46PM -0500, Larry Finger wrote:
> On 09/26/2015 11:49 AM, Punit Vara wrote:
> >This patch is to the rtl871x_ioctl_linux.c that fixes up following
> >warning reported by checkpatch.pl :
> >
> >- Comparisons should place the constant on the right side of the test
> >
> >
On Sunday 27 September 2015 16:46:06 gr...@linuxhacker.ru wrote:
> From: Oleg Drokin
>
> This is a large patch to remove all dead code from obdclass and
> ptlrpc, purely removing functions and declarations now, so
> despite the size should be easy enough to verify.
>
> Signed-off-by: Arnd Bergma
On Sun, Sep 27, 2015 at 07:43:18PM +0530, Chandra S Gorentla wrote:
> The message queue is replaced with standard Linux linked list. A check for
> return value of receive method is added.
>
> Signed-off-by: Chandra S Gorentla
> ---
> drivers/staging/wilc1000/host_interface.c | 7 +++-
> driver
Don't worry too much. I think your patches are basically fine. I'm a
fairly experience kernel dev but I don't know what an N/M patch is...
Just don't work on staging/rtl* any more. The rest of staging is fine.
regards,
dan carpenter
___
devel mailin
On Monday 28 September 2015 01:09:18 Dilger, Andreas wrote:
> On 2015/09/27, 10:45 PM, "gr...@linuxhacker.ru"
> wrote:
> >diff --git a/drivers/staging/lustre/lustre/ptlrpc/events.c
> >b/drivers/staging/lustre/lustre/ptlrpc/events.c
> >index 53f6b62..afd869b 100644
> >--- a/drivers/staging/lustre/l
On Monday 28 September 2015 00:38:37 Dilger, Andreas wrote:
> >Changing this would unfortunately change the format in an incompatible
> >way, breaking all existing user space tools that access the data.
>
> It seems that it would be possible to fix this declaration to be
> compatible with current
On Sunday 27 September 2015 23:57:30 Dilger, Andreas wrote:
> >diff --git a/drivers/staging/lustre/lustre/libcfs/debug.c
> >b/drivers/staging/lustre/lustre/libcfs/debug.c
> >index e93f556..ae325f7 100644
> >--- a/drivers/staging/lustre/lustre/libcfs/debug.c
> >+++ b/drivers/staging/lustre/lustre/li
100 matches
Mail list logo
