There is a macro RLINIT_TO_NICE in linux/sched/prio.h to convert priority in
rlimit
to nice value.
This patch replace the opened implementation with RLINIT_TO_NICE.
Signed-off-by: Dongsheng Yang
cc: de...@driverdev.osuosl.org
cc: Masanari Iida
cc: Bojan Prtvar
cc: Serban Constantinescu
---
Signed-off-by: Dongsheng Yang
cc: de...@driverdev.osuosl.org
cc: Xiong Zhou
cc: Andreas Dilger
cc: Cyril Roelandt
cc: Kent Overstreet
cc: Peng Tao
---
drivers/staging/lustre/lustre/llite/lloop.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/lustre/lustre
On Wed, Mar 05, 2014 at 10:20:51AM +0100, Philipp Zabel wrote:
> Hi,
>
> this latest version of the imx-drm DT binding patches applies
> on top of staging-next and also depends on the OF graph binding
> patchset that moves the v4l2_of helpers to drivers/of.
> Currently, the two patchsets are also
On 03/03/2014 04:48 PM, Roger wrote:
On 02/24/2014 11:00 AM, Chris Ball wrote:
Hi,
On Mon, Feb 24 2014, Roger wrote:
Would you please help comment PATCH 2/3 and 3/3 (for mmc and memstick,
respectively) or give Acks? Thus we can proceed with applying or
revision if necessary.
For 2/3:
Acked-
unsplit strings from two lines to one. Changes resulted
in line over 80 characters warning. Leaving the latter
of the two warnings for possible string grepping reasons.
Signed-off-by: Gary Alan Rookard
---
On branch staging-next
drivers/staging/lustre/lustre/fld/fld_request.c | 15 ++
It doesn't need to use while loop for getting newrate,
because it always breaks out the end of while loop with
"break". So just replace while with if.
And the type of newrate is "unsigned int", this type
is never less than zero. If it can be set to negative value by
user application with ioctl(),
fixed up a spacing issue.
Signed-off-by: Gary Alan Rookard
---
On branch staging-next
drivers/staging/lustre/lustre/fld/fld_cache.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/lustre/lustre/fld/fld_cache.c
b/drivers/staging/lustre/lustre/fld/fld_cache.c
unsplit string from two lines to one.
Signed-off-by: Gary Alan Rookard
---
On branch staging-next
drivers/staging/lustre/lustre/fld/fld_cache.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/lustre/lustre/fld/fld_cache.c
b/drivers/staging/lustre/lustre/f
unsplit strings from two lines to one.
Signed-off-by: Gary Alan Rookard
---
On branch staging-next
drivers/staging/lustre/lustre/fid/fid_request.c | 24
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/drivers/staging/lustre/lustre/fid/fid_request.c
b/dri
> -Original Message-
> From: David Miller [mailto:da...@davemloft.net]
> Sent: Tuesday, March 11, 2014 1:26 AM
> To: KY Srinivasan
> Cc: net...@vger.kernel.org; linux-ker...@vger.kernel.org;
> de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com;
> jasow...@redhat.com
> Subjec
Good day,
We are Christian organization, we give out loan to those that have given
there lives to Christ, contact us via email marieloanlend...@gmail.com
Regard
Mrs Marie
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdri
This CamelCase function is just a wrapper around inb(). Remove it.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/c6xdigio.c | 102 ++
1 file changed, 48 insertions(+), 54 deletions(-)
diff --git a/drivers/
Factor out the common code that busywaits for the status to change.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/c6xdigio.c | 187 --
1 file changed, 48 insertions(+), 139 deletions(-)
diff --git a/driver
Rename the CamelCase function.
For aesthetics, pass the comedi_device pointer instead of the dev->iobase.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/c6xdigio.c | 20 ++--
1 file changed, 10 insertions(+), 10 deleti
The 24-bit encoder value is read using 3-bits in the status register. The
data register is banged between each read of the status register to advance
the bits.
Introduce a helper function to handle this and remove the union encvaluetype
and struct encbitsbyte.
Signed-off-by: H Hartley Sweeten
Cc
Rename the CamelCase function.
For aesthetics, pass the comedi_device pointer instead of the dev->iobase
as the first parameter.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/c6xdigio.c | 38 +++
1 file ch
The value read from the encoders is in two's complement format. Use
the comedi_offset_munge() helper to convert the value to offset binary.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/c6xdigio.c | 15 ++-
1 file changed,
The register map was already defined but it was not being used. Rename
to defines and use them in the code.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/c6xdigio.c | 29 +++--
1 file changed, 15 insertions(+),
For aesthetics, rename this function so it has namespace associated
with the driver.
Absorb the c6xdigio_pwm_init() and c6xdigio_encoder_reset() helpers.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/c6xdigio.c | 33 -
Remove the commented out init of the 'trig'.
For aesthetics, add some whitespace to the init and rename the 'insn_read'
function.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/c6xdigio.c | 20 ++--
1 file changed, 10
Add and (*insn_read) for the PWM subdevice to allow reading back the
last value written to the channels.
There are only 2 PWM channels and they have a maxdata of 500. Pack
the last values in the subdevice 'state' instead of adding a private
data struct to this driver.
Signed-off-by: H Hartley Swe
For aesthetics, pass the comedi_device pointer instead of the dev->iobase
as the first parameter.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/c6xdigio.c | 52 +++
1 file changed, 26 insertions(+), 26 dele
Reformat the multi-line comments at the beginning of the code to
follow the CodingStyle.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/c6xdigio.c | 55 +++
1 file changed, 27 insertions(+), 28 deletions(-)
Rename the CamelCase function.
For aesthetics, pass the comedi_device pointer instead of the dev->iobase
as the first parameter.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/c6xdigio.c | 65 ---
1 file ch
To clarify the code, define the bits in the data register and remove the
magic numbers.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/c6xdigio.c | 45 +--
1 file changed, 19 insertions(+), 26 deletions(-)
From: H Hartley Sweeten
Change the generic MODULE_DESCRIPTION text to something more
specific for this driver.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/c6xdigio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --gi
The 10-bit pwm value (2-498) is written by banging 2-bits in the data register.
Refactor this function to remove the need for the union pcmcmdtype and struct
pwmbitstype.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/c6xdigio.c | 41
All writes to the hardware involve writing the the data register then
checking the status register. Introduce a helper function to handle this.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/c6xdigio.c | 90 +++-
This subdevice is a PWM output not and analog output, fix the 'type'.
Fix the subdevice 'range', range_bipolar does not make sense for a PWM
output. The range_unknown is a better choice.
Remove the commented out init of the 'trig'.
For aesthetics, add some whitespace to the init and rename the '
Cleanup the driver.
H Hartley Sweeten (21):
staging: comedi: c6xdigio: remove ReadByteFromHwPort()
staging: comedi: c6xdigio: remove WriteByteToHwPort()
staging: comedi: c6xdigio: remove C6XDIGIO_SIZE
staging: comedi: c6xdigio: factor out status check busywait
staging: comedi: c6xdigio:
This CamelCase function is just a wrapper around outb_p(). Remove it.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/c6xdigio.c | 53 ++-
1 file changed, 24 insertions(+), 29 deletions(-)
diff --git a/drive
Rename the CamelCase function.
For aesthetics, pass the comedi_device pointer instead of the dev->iobase.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/c6xdigio.c | 20 ++--
1 file changed, 10 insertions(+), 10 deleti
This define is only used in the comedi_request_region() call and does
not add any additional clarity to the code. Remove it.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/c6xdigio.c | 4 +---
1 file changed, 1 insertion(+), 3 deletion
On Mon, 2014-03-10 at 12:33 -0700, Greg KH wrote:
> On Mon, Mar 10, 2014 at 11:48:50AM -0700, Joe Perches wrote:
[]
> > I've no strong opinion of the need to write code
> > like:
> >
> > {
> > u32 iterations = 0;
> >
> > while (arch_spin_is_locked(lock))
> > delay_
From: Haiyang Zhang
Date: Sun, 9 Mar 2014 16:10:59 -0700
> Due to a bug in the Hyper-V host verion 2008R2, we need to use a slightly
> smaller
> receive buffer size, otherwise the buffer will not be accepted by the legacy
> hosts.
>
> Signed-off-by: Haiyang Zhang
Applied.
__
From: "K. Y. Srinivasan"
Date: Sat, 8 Mar 2014 19:22:42 -0800
> This patch set enables both checksum as well as segmentation offload.
> As part of this effort I have enabled scatter gather I/O a well.
>
> In version 2 of these patches, I addressed comments from David Miller and
> Dan Carpenter.
This patch fixes a few more 80+ char lines as reported by checkpatch
Signed-off-by: Mark Hounschell
Cc: Greg Kroah-Hartman
---
drivers/staging/dgap/dgap.c | 162 ++--
1 file changed, 113 insertions(+), 49 deletions(-)
diff --git a/drivers/staging/dgap/dg
This patch fixes a few more 80+ char lines as reported by checkpatch
Signed-off-by: Mark Hounschell
Cc: Greg Kroah-Hartman
---
drivers/staging/dgap/dgap.c | 26 +-
1 file changed, 17 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/dgap/dgap.c b/drivers/stagin
sparse reported dgap_do_fep_load is redeclared with different type. while
fixing, I noticed __user attribute is used incorrectly in declaration.
There is no need to define __user for firware->data.
Replaced the __user with 'const uchar *' from function dgap_do_fep_load and
did the same for functio
On Mon, Mar 10, 2014 at 12:34:22PM -0700, Greg KH wrote:
> On Sun, Mar 09, 2014 at 12:33:32AM -0800, Masood Mehmood wrote:
> > On Mon, Mar 10, 2014 at 11:37:12AM -0700, Greg KH wrote:
> > > On Sat, Mar 08, 2014 at 11:18:42PM -0800, Masood Mehmood wrote:
> > > > 1 file changed, 6 insertions(+), 7 d
On Sun, Mar 09, 2014 at 12:33:32AM -0800, Masood Mehmood wrote:
> On Mon, Mar 10, 2014 at 11:37:12AM -0700, Greg KH wrote:
> > On Sat, Mar 08, 2014 at 11:18:42PM -0800, Masood Mehmood wrote:
> > > 1 file changed, 6 insertions(+), 7 deletions(-)
> >
> > I don't see a patch 1/2 here, did you forget
On Mon, Mar 10, 2014 at 11:48:50AM -0700, Joe Perches wrote:
> On Mon, 2014-03-10 at 09:50 -0700, Greg KH wrote:
> > On Mon, Mar 10, 2014 at 09:02:26AM -0700, Joe Perches wrote:
> > > On Fri, 2014-03-07 at 01:30 -0800, Joe Perches wrote:
> > > > On Fri, 2014-03-07 at 10:54 +0300, Dan Carpenter wrot
On Mon, 2014-03-10 at 09:50 -0700, Greg KH wrote:
> On Mon, Mar 10, 2014 at 09:02:26AM -0700, Joe Perches wrote:
> > On Fri, 2014-03-07 at 01:30 -0800, Joe Perches wrote:
> > > On Fri, 2014-03-07 at 10:54 +0300, Dan Carpenter wrote:
> > (a question about a new message warning of a missing
> > bl
On Mon, Mar 10, 2014 at 11:37:12AM -0700, Greg KH wrote:
> On Sat, Mar 08, 2014 at 11:18:42PM -0800, Masood Mehmood wrote:
> > 1 file changed, 6 insertions(+), 7 deletions(-)
>
> I don't see a patch 1/2 here, did you forget to send it to me?
Patch 1/2 has some unrelated changes. My bad, I forgot
On Sat, Mar 08, 2014 at 11:18:42PM -0800, Masood Mehmood wrote:
> sparse reported dgap_do_fep_load is redeclared with different type. while
> fixing, I noticed __user attribute is used incorrectly in declaration.
> There is no need to define __user for firware->data.
>
> Replaced the __user with '
This patch fixes a few 80+ char lines
as reported by checkpatch
Signed-off-by: Mark Hounschell
Cc: Greg Kroah-Hartman
---
drivers/staging/dgap/dgap.c | 55 -
1 file changed, 30 insertions(+), 25 deletions(-)
diff --git a/drivers/staging/dgap/dgap.c b
sparse reported dgap_do_fep_load is redeclared with different type. while
fixing, I noticed __user attribute is used incorrectly in declaration.
There is no need to define __user for firware->data.
Replaced the __user with 'const uchar *' from function dgap_do_fep_load and
did the same for functio
On Mon, Mar 10, 2014 at 09:02:26AM -0700, Joe Perches wrote:
> On Fri, 2014-03-07 at 01:30 -0800, Joe Perches wrote:
> > On Fri, 2014-03-07 at 10:54 +0300, Dan Carpenter wrote:
> (a question about a new message warning of a missing
> blank line between variable declaration blocks and
> code in
On Fri, 2014-03-07 at 01:30 -0800, Joe Perches wrote:
> On Fri, 2014-03-07 at 10:54 +0300, Dan Carpenter wrote:
(a question about a new message warning of a missing
blank line between variable declaration blocks and
code in a function)
> > How many warnings does this generate does this generat
On 03/08/2014 04:59 AM, Mark Hounschell wrote:
On 03/07/2014 05:46 PM, Dan Carpenter wrote:
I still prefer my patch. Unless I'm missing something? Could you
review it and I will submit it as a patch tomorrow.
I've reviewed it and see no obvious problems. If you like it better, I'm
OK with
50 matches
Mail list logo