On Sun 2015-01-11 10:29:00, atull wrote:
> On Sat, 10 Jan 2015, Pavel Machek wrote:
>
> > On Sat 2015-01-10 10:10:51, Pantelis Antoniou wrote:
> > > Hi Pavel,
> > >
> > > > On Jan 9, 2015, at 22:56 , Pavel Machek wrote:
> > > >
> > > > On Fri 2015-01-09 13:14:24, atull wrote:
> > > >> On Wed, 7
These patches look really nice. Thanks for doing this.
regards,
dan carpenter
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
On 01/12/2015 09:45 AM, Pavel Machek wrote:
> On Sun 2015-01-11 10:29:00, atull wrote:
>> On Sat, 10 Jan 2015, Pavel Machek wrote:
>>
>>> On Sat 2015-01-10 10:10:51, Pantelis Antoniou wrote:
Hi Pavel,
> On Jan 9, 2015, at 22:56 , Pavel Machek wrote:
>
> On Fri 2015-01-09 13:1
On 01/11/2015 10:31 PM, Pavel Machek wrote:
> On Sun 2015-01-11 21:58:00, Konrad Zapalowicz wrote:
>> On 01/11, Pavel Machek wrote:
>>> On Sun 2015-01-11 17:24:26, Konrad Zapalowicz wrote:
On 01/11, atull wrote:
> On Sat, 10 Jan 2015, Konrad Zapalowicz wrote:
>
>> On 01/06, at...@o
On Sun, Jan 11, 2015 at 08:52:19PM +0100, Pavel Machek wrote:
>
> Actually, why not? Tools should know to look at From: in the
> commit. AFAICT, Alan has it set up correctly.
> Pavel
Normally, the Extra From header should only be used
Arno,
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 delet
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 Mon, Jan 12, 2015 at 09:14:18AM -0500, Jason Cooper wrote:
> Arno,
>
> 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/
On Mon, Jan 12, 2015 at 05:33:47PM +0300, Dan Carpenter wrote:
> On Mon, Jan 12, 2015 at 09:14:18AM -0500, Jason Cooper wrote:
> > Arno,
> >
> > 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
On Sun, Jan 11, 2015 at 10:29 AM, atull wrote:
> On Sat, 10 Jan 2015, Pavel Machek wrote:
>
>> On Sat 2015-01-10 10:10:51, Pantelis Antoniou wrote:
>> > Hi Pavel,
>> >
>> > > On Jan 9, 2015, at 22:56 , Pavel Machek wrote:
>> > >
>> > > On Fri 2015-01-09 13:14:24, atull wrote:
>> > >> On Wed, 7 Ja
The global shrinker will invoke lowmem_shrink in a loop.
The loop will be run (total_scan_pages/batch_size) times.
The default batch_size will be 128 which will make
shrinker invoking 100s of times. LMK does meaningful
work only during first 2-3 times and then rest of the
invocations are just CPU c
Please ignore this patch. My extreme bad that I merged commit messages
applicable to some very old kernel into this patch. Updating shortly.
On 01/12/2015 09:38 PM, Chintan Pandya wrote:
The global shrinker will invoke lowmem_shrink in a loop.
The loop will be run (total_scan_pages/batch_size)
The global shrinker will invoke lowmem_shrink in a loop.
The loop will be run (total_scan_pages/batch_size) times.
The default batch_size will be 128 which will make
shrinker invoking 100s of times. LMK does meaningful
work only during first 2-3 times and then rest of the
invocations are just CPU c
Fixed a coding style issue
Signed-off-by: Igor Ishchenko
---
drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c
b/drivers/staging/lustre/lnet/klnds/o2iblnd/o2i
At Sun, 11 Jan 2015 15:04:55 -0600,
Chris Rorvick wrote:
>
> > At Fri, 9 Jan 2015 23:35:46 -0600,
> > Chris Rorvick wrote:
> >>
> >> I have a TonePort UX2 that I've used for testing, meaning that some of
> >> this is really only compile-tested.
> >
> > If anyone is responsible for testing with re
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/ethernet-rx.c
b/drivers/staging/octeon/ethernet-rx.c
index fcbe836..22667db 10064
When an SMP Hyper-V guest is running on top of 2012R2 Server and secondary
cpus are sent offline (with echo 0 > /sys/devices/system/cpu/cpu$cpu/online)
the system freeze is observed. This happens due to the fact that on newer
hypervisors (Win8, WS2012R2, ...) vmbus channel handlers are distributed
This patch fixes the following checkpatch.pl error:
rxtx.c:588: WARNING: else is not generally useful after a break or return
Signed-off-by: Emrys Bayliss
---
drivers/staging/vt6656/rxtx.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/vt6656/rxtx.c b/drivers/staging/vt6656
On Mon, Jan 12, 2015 at 10:05:49AM -0600, Rob Herring wrote:
> On Sun, Jan 11, 2015 at 10:29 AM, atull wrote:
> > Previous uses of the firmware layer has been to use it to load once after
> > bootup; this is different since some use cases will want to switch out
> > the FPGA image. If someone wa
> Here is a proposal to include in the staging tree the drivers from the
> fbtft project at https://github.com/notro/fbtft. This project contains
> a number of drivers small TFT LCD display modules, which are not
> otherwise supported by the Linux kernel. This set of drivers appears
> to be quite
For aesthetics, introduce a struct to hold the DMA descriptor data.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/ni_at_a2150.c | 76 +++-
1 file changed, 41 insertions(+), 35 deletions(-)
diff --git a/drivers
For aesthetics, introduce a struct to hold the DMA descriptor data.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/pcl818.c | 44 +++--
1 file changed, 25 insertions(+), 19 deletions(-)
diff --git a/drivers
The IRA and DMA are optional with this driver but both are required to
support async commands. Introduce a helper function to request the IRQ
and DMA channel and allocate the buffers. Don't fail the driver attach
if the user passed an invalid IRQ or DMA channel or they cannot be
requested.
Signed-
As indicated by checkpatch.pl, "WARNING: Use of volatile is usually
wrong: ...", the 'count' member of the private data does not need to
be volatile.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/ni_at_a2150.c | 3 +--
1 file changed,
This driver no longer uses virt_to_bus(). Remove the dependancy.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/Kconfig | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/comedi/Kconfig b/drivers/staging/comedi/Kconfig
index a8
DMA is optional with this driver. Introduce a helper function to request
the DMA channel and allocate the buffers. Don't fail the driver attach
if the user passed an invalid DMA channel or the channel cannot be
requested.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
--
This series removes all the VIRT_TO_BUS dependancies from the legacy comedi
drivers. It also tidies up the allocation and freeing of the DMA and the
buffers used for DMA.
H Hartley Sweeten (37):
staging: comedi: adl_pci9118: remove VIRT_TO_BUS dependancy
staging: comedi: das1800: reduce indent
Use dma_{alloc,free}_coherent() to allocate and free the DMA buffers.
This removes the dependancy on VIRT_TO_BUS.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/Kconfig | 2 +-
drivers/staging/comedi/drivers/pcl812.c | 20 +--
From: H Hartley Sweeten
For aesthetics, use the helper function to get the byte size of each sample
instead of using sizeof() the first buffer element.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/ni_at_a2150.c | 8
1 file
For aesthetics, move the freeing of the DMA channel and the buffers to
a helper function.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/pcl816.c | 25 +
1 file changed, 17 insertions(+), 8 deletions(-)
diff --
Use dma_{alloc,free}_coherent() to allocate and free the DMA buffers.
This removes the dependancy on VIRT_TO_BUS.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/Kconfig | 2 +-
drivers/staging/comedi/drivers/dt282x.c | 13 ---
Use dma_{alloc,free}_coherent() to allocate and free the DMA buffers.
This removes the dependancy on VIRT_TO_BUS.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/Kconfig | 2 +-
drivers/staging/comedi/drivers/pcl816.c | 20 +--
From: H Hartley Sweeten
For aesthetics, introduce a helper function to free the DMA channel and
buffer.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/ni_at_a2150.c | 20 +---
1 file changed, 13 insertions(+), 7 delet
This driver can use DMA or the hardware FIFO to read analog input samples.
When DMA is enabled it's also possible for some async commands to cause
DMA to be disabled and the FIFO is used instead.
Currently the first DMA channels buffer is used for the bounce buffer
when using the FIFO. For aesthet
Refactor this function so that a for loop can be used to request and
set up the two DMA descriptors.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/das1800.c | 52 +++-
1 file changed, 25 insertions(+), 27 d
DMA support is optional for the labpc driver. The return value from
labpc_init_dma_chan() is not even checked by the caller.
Change the return type to void and tidy up the function a bit.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers
For aesthetics, move the freeing of the DMA channel and the buffers to
a helper function.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/das16.c | 33 +++--
1 file changed, 19 insertions(+), 14 deletions(-)
For aesthetics, introduce a struct to hold the DMA descriptor data.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/dt282x.c | 111 +++-
1 file changed, 53 insertions(+), 58 deletions(-)
diff --git a/drivers
For aesthetics, introduce a struct to hold the DMA descriptor data.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/pcl816.c | 51 ++---
1 file changed, 28 insertions(+), 23 deletions(-)
diff --git a/drivers
DMA is optional with this driver. Introduce a helper function to request
the DMA channel and allocate the buffers. Don't fail the driver attach
if the user passed an invalid DMA channel or the channel cannot be
requested.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
--
An IRQ and at least one DMA channel are required to use DMA with this driver.
Move the check to the caller to reduce the indent level of this function.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/das1800.c | 129
Use dma_{alloc,free}_coherent() to allocate and free the DMA buffers.
This removes the dependancy on VIRT_TO_BUS.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/Kconfig | 2 +-
drivers/staging/comedi/drivers/ni_at_a2150.c | 17 +
The IRA and DMA are optional with this driver but both are required to
support async commands. Introduce a helper function to request the IRQ
and DMA channel and allocate the buffers. Don't fail the driver attach
if the user passed an invalid IRQ or DMA channel or they cannot be
requested.
Signed-
For aesthetics, introduce a struct to hold the DMA descriptor data.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/das1800.c | 145 +++
1 file changed, 70 insertions(+), 75 deletions(-)
diff --git a/drivers
DMA is optional with this driver. Introduce a helper function to request
the DMA channel and allocate the buffers. Don't fail the driver attach
if the user passed an invalid DMA channel or the channel cannot be
requested.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
--
For aesthetics, introduce a struct to hold the DMA descriptor data.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/das16.c | 48 ++
1 file changed, 26 insertions(+), 22 deletions(-)
diff --git a/drivers
Currently this function always transfers the full DMA buffer to the comedi
async buffer. When the cmd->stop_src == TRIG_COUNT the last DMA transfer
might be smaller than the buffer size. This results in invalid data being
added to the asunc buffer.
Add a 'size' member to the DMA descriptor and ini
Use dma_{alloc,free}_coherent() to allocate and free the DMA buffers.
This removes the dependancy on VIRT_TO_BUS.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/Kconfig | 2 +-
drivers/staging/comedi/drivers/pcl818.c | 25 +++
The DAS16 board is an ISA device not a PCI device. For aesthetics, use
dma_{alloc,free}_coherent() to allocate and free the DMA buffers instead
of the PCI versions.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/das16.c | 9 -
For aesthetics, introduce a struct to hold the DMA descriptor data.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/pcl812.c | 100
1 file changed, 50 insertions(+), 50 deletions(-)
diff --git a/drivers
Use dma_{alloc,free}_coherent() to allocate and free the DMA buffers.
This removes the dependancy on VIRT_TO_BUS.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/Kconfig | 2 +-
drivers/staging/comedi/drivers/das1800.c | 16 +
For aesthetics, move the freeing of the DMA channel and the buffers to
a helper function.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/pcl812.c | 27 +--
1 file changed, 17 insertions(+), 10 deletions(-)
diff
For aesthetics, move the freeing of the DMA channel and the buffers to
a helper function.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/pcl818.c | 23 +--
1 file changed, 17 insertions(+), 6 deletions(-)
diff --gi
For aesthetics, introduce a helper function to free the DMA channels and
allocated buffers.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/das1800.c | 30 --
1 file changed, 20 insertions(+), 10 deletions(-)
Use dma_{alloc,free}_coherent() to allocate and free the DMA buffers.
This removes the dependancy on VIRT_TO_BUS.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/Kconfig | 2 +-
drivers/staging/comedi/drivers/ni_labpc.h
For aesthetics, introduce a struct to hold the DMA descriptor data.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/ni_labpc.h| 15 --
drivers/staging/comedi/drivers/ni_labpc_isadma.c | 58
drivers/s
From: H Hartley Sweeten
This driver requires an IRQ and DMA in order to support async commands.
For aesthetics, introduce a helper function to request the IRQ and DMA
channels and allocate the DMA buffer. Since the async command support
is optional, make the helper function handle any request/al
When the cmd->stop_src == TRIG_COUNT the last DMA transfer might be smaller
than the buffer size. This results in invalid data being added to the async
buffer.
Add a 'size' member to the DMA descriptor and initialize it with the
actual size of the DMA transfer. Use that in interrupt and ai subdevi
On Sun, Jan 11, 2015 at 10:29:00AM -0600, atull wrote:
> the FPGA image. If someone wants there to be only one FPGA image on
> the FGPA forever, they will probably not be using this framework; their
> FPGA will probably be loaded before Linux boots up.
Nonsense, loading the FPGA through Linux is
> -Original Message-
> From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com]
> Sent: Monday, January 12, 2015 8:50 AM
> To: KY Srinivasan; de...@linuxdriverproject.org
> Cc: Haiyang Zhang; Greg Kroah-Hartman; linux-ker...@vger.kernel.org;
> Dexuan Cui
> Subject: [PATCH v3] Drivers: hv: vmbu
On Wed, Dec 03, 2014 at 07:34:42PM +0100, Sam van Kampen wrote:
> This patch fixes the error "Macros with complex values should be enclosed in
> parentheses", as reported by checkpatch.pl.
>
> Signed-off-by: Sam van Kampen
> ---
> drivers/staging/line6/usbdefs.h | 2 +-
> 1 file changed, 1 inser
On Mon, Jan 12, 2015 at 05:35:01PM +0100, Takashi Iwai wrote:
> At Sun, 11 Jan 2015 15:04:55 -0600,
> Chris Rorvick wrote:
> >
> > > At Fri, 9 Jan 2015 23:35:46 -0600,
> > > Chris Rorvick wrote:
> > >>
> > >> I have a TonePort UX2 that I've used for testing, meaning that some of
> > >> this is re
At Mon, 12 Jan 2015 11:52:27 -0800,
Greg Kroah-Hartman wrote:
>
> On Mon, Jan 12, 2015 at 05:35:01PM +0100, Takashi Iwai wrote:
> > At Sun, 11 Jan 2015 15:04:55 -0600,
> > Chris Rorvick wrote:
> > >
> > > > At Fri, 9 Jan 2015 23:35:46 -0600,
> > > > Chris Rorvick wrote:
> > > >>
> > > >> I have
On Sat, Dec 27, 2014 at 05:00:10PM +1100, Damon Swayn wrote:
> Fixed a coding style issue reported by checkpatch surrounding the
> spacing of comma's
>
> Signed-off-by: Damon Swayn
> ---
> drivers/staging/line6/pcm.h | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
Sorry, someone e
On Mon, Dec 22, 2014 at 10:45:01PM +0200, Dmitriy Polyanskiy wrote:
> This is a patch to pcm.h file that fixes up some space warnings
> found by checkpatch.pl
>
> Signed-off-by: Dmitriy Polyanskiy
> ---
> drivers/staging/line6/pcm.h |6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-
From: Chris Rorvick
Define an enum containing the supported devices and associate each entry
in the device table to the respective value.
Signed-off-by: Chris Rorvick
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/line6/driver.c | 54 ++
From: Chris Rorvick
Wrap USB_DEVICE to avoid repeating the Line 6 vendor ID.
Signed-off-by: Chris Rorvick
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/line6/driver.c | 40 +---
1 file changed, 21 insertions(+), 19 dele
From: Chris Rorvick
The `device_bit' member was no longer used as of commit 2807904441d4
(staging: line6: drop MIDI parameter sysfs attrs).
Signed-off-by: Chris Rorvick
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/line6/driver.c | 44 +++
From: Rickard Strandqvist
The NULL check was done to late, and there it was a risk
of a possible null pointer dereference.
This was partially found by using a static code analysis program called
cppcheck.
Signed-off-by: Rickard Strandqvist
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Greg Kro
Hi Takashi,
Here are all of the line6 patches that have been sent to me that were in
my queue, including the series from Chris that caused you to want to
move the driver out of staging. I included it here as it had to be
tweaked due to some earlier patches in my queue.
thanks,
greg k-h
__
From: Chris Rorvick
Support for these devices appears to have never been completed. Remove
them from the device table along with a minimal amount of supporting
code.
Signed-off-by: Chris Rorvick
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/line6/driver.
From: Chris Rorvick
The device type can now be used to determine the altsetting for the
interface. Drop the conditional logic and make this value a property.
Signed-off-by: Chris Rorvick
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/line6/driver.c | 74 +
From: Jonas Lundqvist
Fix three space prohibited errors in pcm.h found by checkpatch.pl.
Signed-off-by: Jonas Lundqvist
Reviewed-by: Jeremiah Mahler
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/line6/pcm.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/dri
From: Chris Rorvick
Commit 1027f476f507 (staging: line6: sync with upstream) removed PCM
from the Pocket POD capabilities but left the endpoint configuration.
Signed-off-by: Chris Rorvick
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/line6/pcm.c | 5 -
From: Chris Rorvick
This function was no longer used as of commit 2807904441d4 (staging:
line6: drop MIDI parameter sysfs attrs).
Signed-off-by: Chris Rorvick
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/line6/pod.c | 9 -
drivers/staging/line6/p
From: Chris Rorvick
The driver only supports interface 0 of the TonePort UX2 and POD Studio
UX2 devices. Use the device table to filter on this.
Signed-off-by: Chris Rorvick
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/line6/driver.c | 20 --
From: Chris Rorvick
The driver currently uses the device's idProduct as input to several
switch statements. In some cases this is not sufficiently granular and
the interface number must be taken into account.
Store the device type in `usb_line6' and key off of it instead. New
types can then be
From: Chris Rorvick
Remove declarations from the header and move the definitions up in the
source so they need not be forward declared.
Signed-off-by: Chris Rorvick
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/line6/pod.c | 58 ++
From: Chris Rorvick
Casting the `struct usb_line6' pointer at the call point makes the code
difficult to read. This is substantially cleaned up by moving the cast
into the callees.
Signed-off-by: Chris Rorvick
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Greg Kroah-Hartman
---
drivers/stagin
From: Chris Rorvick
Specify the index of the properties entry explicitly to define this
structure more robustly. Also, drop the `L6PROP' macro in favor of
initializing each member explicitly on its own line since horizontal
space is limited and more attributes will be added later.
Signed-off-by
From: Chris Rorvick
Which *_disconnect() to call on disconnect is known at initialization.
Add a function pointer to the `usb_line6' struct and use to call into
the appropriate logic instead of evaluating the conditional logic.
Signed-off-by: Chris Rorvick
Reviewed-by: Stefan Hajnoczi
Signed-o
From: Chris Rorvick
The PODxt Live device has both a POD and a Variax interface. Add device
type entries for each of these.
Signed-off-by: Chris Rorvick
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/line6/driver.c | 112 --
From: Chris Rorvick
The `LINE6_CAP_CTRL_PCM_HW' macro combines three capabilities to save
horizontal space when defining the properties entries. Now that these
are no longer limited to single lines this is not such a concern.
Specify capabilities individually when defining each property for
bett
From: Chris Rorvick
Which *_process_message() function (if any) to call when data is
received is known at initialization. Add a function pointer to the
`usb_line6' struct and use to call into the appropriate logic instead
of evaluating the conditional logic for each message.
Signed-off-by: Chri
From: Chris Rorvick
Casting the `struct usb_line6' pointer at the call point makes the code
difficult to read. This is substantially cleaned up by moving the cast
into the callees.
Signed-off-by: Chris Rorvick
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Greg Kroah-Hartman
---
drivers/stagin
From: Chris Rorvick
Signed-off-by: Chris Rorvick
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/line6/driver.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/line6/driver.c b/drivers/staging/line6/driver.c
index 6dc8a0d4c4b6..acde205a6
From: Chris Rorvick
Including "BIT" in the macro name is pointless. Replace with "CAP" to
provide some context for what its value represents.
Signed-off-by: Chris Rorvick
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/line6/driver.c | 8
drive
From: Chris Rorvick
The device type can now be used to determine the addresses of the
control endpoints for the interface. Drop the conditional logic and
make these values properties.
Signed-off-by: Chris Rorvick
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Greg Kroah-Hartman
---
drivers/sta
From: Chris Rorvick
The driver uses a different altsetting depending on the interface. Add
device type entries for each of these.
Signed-off-by: Chris Rorvick
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/line6/driver.c | 41 -
From: Chris Rorvick
The current logic uses the index of the matched entry from the device
table as an offset to the corresponding properties entry. The values
of the new `line6_device_type' enum are ordered such that they can be
used as an index into either of these arrays. Drop the device entr
From: Chris Rorvick
The device type can now be used to determine the addresses of the
audio endpoints for the interface. Drop the conditional logic and make
these values properties.
Signed-off-by: Chris Rorvick
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Greg Kroah-Hartman
---
drivers/stagi
From: Chris Rorvick
These are now only used to build the device table. Each entry in this
table is already clearly documented as to what device it maps to so the
macros become unnecessary indirection.
Signed-off-by: Chris Rorvick
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Greg Kroah-Hartman
From: Chris Rorvick
The driver only supports interface 1 of the Pocket POD. Use the device
table to filter on this.
Signed-off-by: Chris Rorvick
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/line6/driver.c | 12 ++--
1 file changed, 2 insertions(
> Then configure udev to load right firmware for you, or ln -s
> image-i-want-now socfpga-fpga-image to select the one to read...?
Your conceptual model is wrong. FPGA firmware is dynamic. There are
already people who lazy reload FPGA firmware on taskswitches. This
proposed fpga manager is broken
On Mon, Jan 12, 2015 at 09:01:34PM +, One Thousand Gnomes wrote:
> There are plenty of people today who treat the FPGA as an entirely
> dynamic resource. It's not like flashing a controller, its near
> immediate.
But this is a completely different use case. Remember, there are
*megabytes* of i
At Mon, 12 Jan 2015 12:42:33 -0800,
Greg Kroah-Hartman wrote:
>
> Hi Takashi,
>
> Here are all of the line6 patches that have been sent to me that were in
> my queue, including the series from Chris that caused you to want to
> move the driver out of staging. I included it here as it had to be
>
At Mon, 12 Jan 2015 17:35:01 +0100,
Takashi Iwai wrote:
>
> At Sun, 11 Jan 2015 15:04:55 -0600,
> Chris Rorvick wrote:
> >
> > > At Fri, 9 Jan 2015 23:35:46 -0600,
> > > Chris Rorvick wrote:
> > >>
> > >> I have a TonePort UX2 that I've used for testing, meaning that some of
> > >> this is reall
On Mon, Jan 12, 2015 at 11:01:03PM +0100, Takashi Iwai wrote:
> At Mon, 12 Jan 2015 12:42:33 -0800,
> Greg Kroah-Hartman wrote:
> >
> > Hi Takashi,
> >
> > Here are all of the line6 patches that have been sent to me that were in
> > my queue, including the series from Chris that caused you to wan
The Hyper-V clocksource is continuous; mark it accordingly.
Signed-off-by: K. Y. Srinivasan
Cc: stable
---
arch/x86/kernel/cpu/mshyperv.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/arch/x86/kernel/cpu/mshyperv.c b/arch/x86/kernel/cpu/mshyperv.c
index a450373..93915
On Tue, Jan 13, 2015 at 8:26 AM, K. Y. Srinivasan
wrote:
The Hyper-V clocksource is continuous; mark it accordingly.
Signed-off-by: K. Y. Srinivasan
Cc: stable
---
arch/x86/kernel/cpu/mshyperv.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
Acked-by: Jason Wang
diff --gi
1 - 100 of 118 matches
Mail list logo
