On 2015년 10월 23일 15:57, Dan Carpenter wrote:
On Fri, Oct 23, 2015 at 02:28:21PM +0900, Glen Lee wrote:
This patch add new argument wilc to wilc_handle_isr and pass wilc to
the function.
It's not important enough to redo the patch but why are we sometimes
using "wl" and sometimes "wilc"?
We
Hi Arnd,
First of all, I would like to say "thank you" for your efforts and
contributions.
We are updating the driver because new revision came up and making new
patches
to make it stable and elegant as Linux driver. In these days, we are
sending big changes
while testing such patches and also
On Fri, Oct 23, 2015 at 04:36:07PM +0900, Tony Cho wrote:
> In addition, the function parameter names will be wilc
> for the variable of struct wilc.
>
> The "wl" is local variable naming as well.
So if it is a parameter it is wilc but if it is a local variable then it
is wl? That seems like an
On 2015년 10월 23일 16:52, Dan Carpenter wrote:
On Fri, Oct 23, 2015 at 04:36:07PM +0900, Tony Cho wrote:
In addition, the function parameter names will be wilc
for the variable of struct wilc.
The "wl" is local variable naming as well.
So if it is a parameter it is wilc but if it is a local va
`comedi_nsamples_left(s, nsamples)` returns the number of samples
remaining to complete an asynchronous command or the passed in
`nsamples`, whichever is lower. However, it goes wrong in the extreme
case of setting the `nsamples` parameter to `UINT_MAX` when the number
of conversions per "scan" (`
Hi,
Support for RTL8723AU devices was added to
wireless-drivers-next.git recently:
https://marc.info/?l=linux-wireless&m=144541406318463
commit:
https://git.kernel.org/cgit/linux/kernel/git/kvalo/wireless-drivers-next.git/commit/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_regs.h?id=26f1fad29a
On 10/23/2015 05:44 AM, Xose Vazquez Perez wrote:
Hi,
Support for RTL8723AU devices was added to
wireless-drivers-next.git recently:
https://marc.info/?l=linux-wireless&m=144541406318463
commit:
https://git.kernel.org/cgit/linux/kernel/git/kvalo/wireless-drivers-next.git/commit/drivers/net/wir
On 2015/10/22, 22:30, "Nilesh Kokane" wrote:
>Fixed- Return of an errno should typically be negative (ie: return
>-EAGAIN)
Nak. Please do not change these function return values. They are
converted as necessary by the callers before returning to userspace, but
allow the code to distinguish betwe
On Fri, Oct 23, 2015 at 5:10 PM, Dilger, Andreas
wrote:
> On 2015/10/22, 22:30, "Nilesh Kokane" wrote:
>>Fixed- Return of an errno should typically be negative (ie: return
>>-EAGAIN)
>
> Nak. Please do not change these function return values. They are
> converted as necessary by the callers befo
Larry Finger writes:
> On 10/23/2015 05:44 AM, Xose Vazquez Perez wrote:
>> Hi,
>>
>> Support for RTL8723AU devices was added to
>> wireless-drivers-next.git recently:
>> https://marc.info/?l=linux-wireless&m=144541406318463
>> commit:
>> https://git.kernel.org/cgit/linux/kernel/git/kvalo/wireless
The suspend and resume methods in lynx_share are not implemented and
never set. Remove them.
Signed-off-by: Mike Rapoport
---
drivers/staging/sm750fb/sm750.c | 6 --
drivers/staging/sm750fb/sm750.h | 3 ---
2 files changed, 9 deletions(-)
diff --git a/drivers/staging/sm750fb/sm750.c b/drive
The methods in lynx_crtc always use the same implementation and there is
no point to use redirection rather than direct function call.
Signed-off-by: Mike Rapoport
---
drivers/staging/sm750fb/sm750.c | 12
drivers/staging/sm750fb/sm750.h | 10 --
2 files changed, 4 insertion
There is no need to redefine offsetof
Signed-off-by: Mike Rapoport
---
drivers/staging/sm750fb/sm750.h | 4
1 file changed, 4 deletions(-)
diff --git a/drivers/staging/sm750fb/sm750.h b/drivers/staging/sm750fb/sm750.h
index 52fe945..e74a3e1 100644
--- a/drivers/staging/sm750fb/sm750.h
+++
Remove the code that should be compiled for ancient kernel versions
together with controlling '#if LINUX_VERSION_CODE' conditionals
Signed-off-by: Mike Rapoport
---
drivers/staging/sm750fb/sm750_hw.c | 36
1 file changed, 36 deletions(-)
diff --git a/drivers
The proc_setMode method in lynx_output always uses the same
implementation and there is no point to use redirection rather than
direct function call.
Signed-off-by: Mike Rapoport
---
drivers/staging/sm750fb/sm750.c | 4 +---
drivers/staging/sm750fb/sm750.h | 4
2 files changed, 1 insertion(
Hi,
These patches remove some dead code and replace several methods in lynx_ctrc
and lynx_output with direct function calls.
Mike Rapoport (5):
staging: sm750fb: remove defintion of offsetof
staging: sm750fb: remove ancient kernel support
staging: sm750fb: remove unused methods from lynx_sh
>Fixed- Return of an errno should typically be negative (ie: return -EAGAIN)
>
>Signed-off-by: Nilesh Kokane
>---
> drivers/staging/lustre/lnet/lnet/lib-move.c | 22 +++---
> 1 file changed, 11 insertions(+), 11 deletions(-)
Bad idea. I also made this mistake and it broke LNet rea
Luis de Bethencourt writes:
> For loop is outside of the else branch of the above conditional statement.
> Fixing misleading indentation.
>
> Fix a smatch warning:
> drivers/staging/rtl8723au/core/rtw_wlan_util.c:528
> WMMOnAssocRsp23a() warn: curly braces intended?
>
> Signed-off-by: Luis de Beth
The kernel communication code used for HSM and changelog is
entangled. Move the user space bits into the liblustreapi. This will
also help for a possible relicensing. The kernel portion is also moved
from libcfs to obdclass.
The original libcfs_kernelcomm.h header is split into three parts:
* l
From: Henri Doreau
Associate copytool registration to a given MDC import so that
multiple mounts of the same filesystem do not lead to having the
copytool registered multiple time.
Signed-off-by: Henri Doreau
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3882
Reviewed-on: http://review.wh
From: frank zago
Move the kernel portion from libcfs to obdclass. This code is
only used by lustre. This is broken out of the original patch
14270. The part covered by this change is as follows:
The original code in kernel_user_comm.c is split into two parts:
* obdclass/kernelcomm.c for the k
From: frank zago
Split the kernel comm header in libcfs into two new headers
to handle both kernel space and user space for the lustre layer.
This is broken out of the original patch 14270. The part covered
by this change is as follows:
The original libcfs_kernelcomm.h header is split into three
During the removal of the cfs wrappers the kg_sem semaphore
was handled incorrectly. We need to take a write lock when
writing data to the kkuc_groups. The libcfs_kkuc_group_foreach
needs to only take a read lock. This makes use match the
OpenSFS development branch.
Signed-off-by: James Simmons
-
From: frank zago
Point to the right place for GNU license. Update Intel copyright.
Update Nathan Rutmans email address. This was broken out of the
original patch 14270.
Signed-off-by: frank zago
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245
Reviewed-on: http://review.whamcloud.com/14
The following changes since commit 25cb62b76430a91cc6195f902e61c2cb84ade622:
Linux 4.3-rc5 (2015-10-11 11:09:45 -0700)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git/
tags/staging-4.3-rc7
for you to fetch changes up to 4301de3b0ac46
Use the timer API function setup_timer instead of init_timer, removing
the structure field assignments.
@timer@
expression e1,e2,e3,fn_ptr;
@@
-init_timer(&e1);
+setup_timer(&e1, fn_ptr, e2);
... when != fn_ptr = e3
-e1.function = fn_ptr;
-e1.data = e2;
Signed-off-by: Muhammad Falak R Wani
-
Use the timer API function setup_timer instead of init_timer, removing
the structure field assignments.
@timer@
expression e1,e2,e3,fn_ptr;
@@
-init_timer(&e1);
+setup_timer(&e1, fn_ptr, e2);
... when != fn_ptr = e3
-e1.function = fn_ptr;
-e1.data = e2;
Signed-off-by: Muhammad Falak R Wani
-
Use the timer API function setup_timer instead of init_timer, removing
the structure field assignments.
@timer@
expression e1,e2,e3,fn_ptr;
@@
-init_timer(&e1);
+setup_timer(&e1, fn_ptr, e2);
... when != fn_ptr = e3
-e1.function = fn_ptr;
-e1.data = e2;
Signed-off-by: Muhammad Falak R Wani
-
Use the timer API function setup_timer instead of init_timer, removing
the structure field assignments.
Signed-off-by: Muhammad Falak R Wani
---
drivers/staging/rdma/ipath/ipath_sdma.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/rdma/ipath/ipath_sdma
Use the timer API function setup_timer instead of init_timer, removing
the structure field assignments.
Signed-off-by: Muhammad Falak R Wani
---
drivers/staging/rdma/ipath/ipath_verbs.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/rdma/ipath/ipath_verb
Use the timer API function setup_timer instead of init_timer, removing
the structure field assignments.
@timer@
expression e1,e2,e3,fn_ptr;
@@
-init_timer(&e1);
+setup_timer(&e1, fn_ptr, e2);
... when != fn_ptr = e3
-e1.function = fn_ptr;
-e1.data = e2;
Signed-off-by: Muhammad Falak R Wani
-
Use the timer API function setup_timer instead of init_timer, removing
the structure field assignments.
@timer@
expression e1,e2,e3,fn_ptr;
@@
-init_timer(&e1);
+setup_timer(&e1, fn_ptr, e2);
... when != fn_ptr = e3
-e1.function = fn_ptr;
-e1.data = e2;
Signed-off-by: Muhammad Falak R Wani
-
Use the timer API function setup_timer instead of init_timer, removing
the structure field assignments.
@timer@
expression e1,e2,e3,fn_ptr;
@@
-init_timer(&e1);
+setup_timer(&e1, fn_ptr, e2);
... when != fn_ptr = e3
-e1.function = fn_ptr;
-e1.data = e2;
Signed-off-by: Muhammad Falak R Wani
-
33 matches
Mail list logo