From: Roger Tseng
Some platform have both UEFI driver and MFD/mmc driver, if entering
linux while card in the slot, the card power is already on, and rtsx-mmc
driver have no chance to make card power off. This will lead UHSI card
failed to enter UHSI mode.
It is hard to control the UEFI driver l
From: Micky Ching
Fix rts5249 failed send buffer cmd after suspend,
PM_CTRL3 should reset before send any buffer cmd after suspend.
Otherwise, buffer cmd will failed, this will lead resume fail.
Signed-off-by: Micky Ching
---
drivers/mfd/rts5249.c | 17 +
1 file changed, 17 i
From: Micky Ching
v2:
using (err < 0) to check if a function failed, not using
"if (err)" and "if (err < 0)" in mixing way.
This patch fix rts5227 and rts5249 suspend issue, when card reader
resumed from suspend state, the power state should reset before send
buffer command. The original not res
From: Micky Ching
Fix rts5227 failed send buffer cmd after suspend,
PM_CTRL3 should reset before send any buffer cmd after suspend.
Otherwise, buffer cmd will failed, this will lead resume fail.
Signed-off-by: Micky Ching
---
drivers/mfd/rts5227.c| 19 +++
include/lin
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/ni_6527.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/pcl726.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/addi_apci_1032.c | 2 +-
1 file changed, 1 insertion(+), 1 d
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
Also, cfc_handle_events() does nothing if no events are set so the local
variable 'oldevents' can be removed.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
All the pci230_{ao,ai}_stop() calls to cancel the command can then be removed.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/sta
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/ni_65xx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/amplc_pc236_common.c | 2 +-
1 file changed, 1 insertion(+),
Currently cfc_handle_events() is done by both the DMA and non-DMA helper
functions that are called by the interrupt handler. For aesthetics, move
the cfc_handle_events() to the end of the interrupt handler do it in one
place.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/comedi_parport.c | 2 +-
1 file changed, 1 insertion(+), 1 d
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
Also, cfc_handle_events() does nothing if no events are set so the local
variable 'do_event' can be removed.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg K
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 13 -
1 file change
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
Also, cfc_handle_events() does nothing if no events are set so the check
can be removed.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
d
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
The code to cancel the async command can then be removed from rtd_interrupt().
Since the (*cancel) will also clear the FIFO, the ai_read_dregs() function
can also be rem
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/addi_apci_1564.c | 2 +-
1 file changed, 1 insertion(+), 1 d
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/dt2814.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/addi-data/hwdrv_apci3200.c | 3 +--
1 file changed, 1 insert
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/dmm32at.c | 3 +--
1 file changed, 1 insertion(+), 2 deletio
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
Also, cfc_handle_events() does nothing if no events are set so the local
variable 'oldevents' can be removed.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg
Clear the A/D FIFO as part of the analog input (*cancel) to help with
cleaning up the async command.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/rtd520.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/comedi/dri
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/ni_atmio16d.c | 2 +-
1 file changed, 1 insertion(+), 1 dele
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
The pcl711_ai_set_mode() call when the end-of-acquisition is detected can
then be removed. The (*cancel) does the same thing.
Signed-off-by: H Hartley Sweeten
Cc: Ian A
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
The s626_mc_disable() call when the end-of-acquisition is detected can then
be removed. The (*cancel) does the same thing.
For aesthetics, also remove the 'finished' loc
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/addi_apci_3xxx.c | 2 +-
1 file changed, 1 insertion(+), 1 d
The comedi async commands should stop if an error/overflow happens or when
the end-of-acquisition for the command is detected. Some of the drivers
do this correctly but many don't.
The cfc_handle_events() function automatically detects the events that would
terminate a command. If the events are s
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
Also, cfc_handle_events() does nothing if no events are set so the local
variable 'oldevents' can be removed.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg
Use cfc_handler_events() to automatically (*cancel) the async command for
an end-of-acquisition or if an error/overflow occurs.
For aesthetics, add a local variable for the comedi_subdevice pointer.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/com
On Wed, Sep 10, 2014 at 10:10:49PM +0100, Purnendu Kapadia wrote:
>- add line after declaration
>- allignment should match open parenthesis
>- remove unnecessary new line
>
> Signed-off-by: Purnendu Kapadia
> ---
> drivers/staging/android/sw_sync.c | 9 +
> drivers/staging/an
On Thursday, September 11, 2014 2:38 PM, Greg Kroah-Hartman wrote:
> On Thu, Sep 11, 2014 at 05:57:34PM +, Hartley Sweeten wrote:
>> Just wanted to give you a heads up on this series and the ones I have
>> posted. The series are:
>>
>>> From Ian, [PATCH 00/48] staging: comedi: avoid using come
The number of spinlocks has been halved, from 8 to 4 since this
comment was made, let's see if this is enough.
Signed-off-by: Mark Einon
---
drivers/staging/et131x/README | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/et131x/README b/drivers/staging/et131x/README
index a19840
In nic_rx_pkts(), we check that a multicast packet received (when using
a multicast list) is one that was requested - despite setting the list
up with the hardware. We shouldn't expect to get a mc packet we didn't
ask for, so remove these extra checks.
This also means that the surrounding code can
Remove et131x_send_packets() and replace the only use in et131x_tx
with the removed function's body.
Signed-off-by: Mark Einon
---
drivers/staging/et131x/et131x.c | 76 -
1 file changed, 29 insertions(+), 47 deletions(-)
diff --git a/drivers/staging/et131
Tackle the last few major TODO items for this driver.
Mark Einon (8):
staging: et131x: Remove et131x_send_packets() function
staging: et131x: In et131x_tx(), don't return NETDEV_TX_BUSY, just
drop the packet
staging: et131x: Tidy up rx/tx dropped & bytes stats
staging: et131x: Remove u
'struct tcb' member 'flags' was only used to collect tx stats, now
we are no longer collecting those particular stats, we no longer
need tcb->flags or the code used to peek into the skb to set it's
value.
Signed-off-by: Mark Einon
---
drivers/staging/et131x/et131x.c | 18 ++
1 fi
Both these if statements have the same effect when true, so combine
them and save a few lines.
Signed-off-by: Mark Einon
---
drivers/staging/et131x/et131x.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c
Memory allocation errors do not denote NETDEV_TX_BUSY, simply drop
the packet silently with kfree_skb() and return NETDEV_TX_OK.
Also remove this item from the TODO list.
Signed-off-by: Mark Einon
---
drivers/staging/et131x/README | 1 -
drivers/staging/et131x/et131x.c | 51 -
>From struct ce_stats; unicast_pkts_rcvd, unicast_pkts_xmtd,
multicast_pkts_xmtd, broadcast_pkts_rcvd and broadcast_pkts_xmtd
are not returned or used for anything meaningful - remove the code
that collects them, and the struct members too.
Signed-off-by: Mark Einon
---
drivers/staging/et131x/et
Remove some fairly useless comments regarding rx/tx _bytes and
_dropped, and use rcvd_pkts_dropped stat value to provide rx_dropped.
Signed-off-by: Mark Einon
---
drivers/staging/et131x/et131x.c | 12 ++--
1 file changed, 2 insertions(+), 10 deletions(-)
diff --git a/drivers/staging/et1
On Mon, Sep 08, 2014 at 03:27:35PM +0100, Luke Hart wrote:
> Copy the channel type into a temporary buffer so that code will work
> for architectures that don't support MMIO. This now works in same way
> as other tests in same function.
>
> Signed-off-by: Luke Hart
> ---
> drivers/staging/unisys
On Thu, Sep 11, 2014 at 05:57:34PM +, Hartley Sweeten wrote:
> On Tuesday, September 09, 2014 3:26 AM, Ian Abbott wrote:
> > The `comedi_board` inline function takes a single parameter of type
> > `struct comedi_device *` and merely returns the value of the `board_ptr`
> > member therein. This
On Thu, Sep 11, 2014 at 10:07:53AM +0200, Philipp Zabel wrote:
> Am Mittwoch, den 10.09.2014, 11:32 -0700 schrieb Greg Kroah-Hartman:
> > On Wed, Sep 10, 2014 at 11:35:29AM +0200, Philipp Zabel wrote:
> > > Hi Greg,
> > >
> > > I have asked Dave to pull Steve Longerbeam's preparation series for
>
Hi Philipp,
Thank you for the patch.
On Thursday 11 September 2014 17:33:04 Philipp Zabel wrote:
> This patch adds a function to get a port device tree node by port id,
> or reg property value.
>
> Signed-off-by: Philipp Zabel
Acked-by: Laurent Pinchart
> ---
> Changes since v2:
> - Fixed a
On Tuesday, September 09, 2014 3:26 AM, Ian Abbott wrote:
> The `comedi_board` inline function takes a single parameter of type
> `struct comedi_device *` and merely returns the value of the `board_ptr`
> member therein. This is somewhat superfluous as the member can be
> accessed directly.
>
> Re
Hi,
this series converts all existing users of of_graph_get_next_endpoint that pass
a non-NULL prev argument to the function and decrement its refcount themselves
to stop doing that. The of_node_put is moved into of_graph_get_next_endpoint
instead.
This allows to add a for_each_endpoint_of_node he
In preparation for a following patch, stop decrementing the endpoint node
refcount in the loop. This temporarily leaks a reference to the endpoint node,
which will be fixed by having of_graph_get_next_endpoint decrement the refcount
of its prev argument instead.
Signed-off-by: Philipp Zabel
---
Decrementing the reference count of the previous endpoint node allows to
use the of_graph_get_next_endpoint function in a for_each_... style macro.
Prior to this patch, all current users of this function that actually pass
a non-NULL prev parameter should be changed to not decrement the passed
prev
Using the for_each_... macro should make the code bit shorter and
easier to read.
Signed-off-by: Philipp Zabel
---
drivers/staging/imx-drm/imx-drm-core.c | 17 +++--
1 file changed, 7 insertions(+), 10 deletions(-)
diff --git a/drivers/staging/imx-drm/imx-drm-core.c
b/drivers/stagi
Using the for_each_... macro should make the code bit shorter and
easier to read. This patch also properly decrements the endpoint node
reference count before returning out of the loop.
Signed-off-by: Philipp Zabel
---
drivers/staging/imx-drm/imx-drm-core.c | 9 +++--
1 file changed, 3 inser
Using the for_each_... macro should make the code a bit shorter and
easier to read.
Signed-off-by: Philipp Zabel
Acked-by: Laurent Pinchart
---
drivers/gpu/drm/drm_of.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/drm_of.c b/drivers/gpu/drm/drm_of.
In preparation for the following patch, stop decrementing the endpoint node
refcount in the loop. This temporarily leaks a reference to the endpoint node,
which will be fixed by having of_graph_get_next_endpoint decrement the refcount
of its prev argument instead.
Signed-off-by: Philipp Zabel
---
This patch adds a function to get a port device tree node by port id,
or reg property value.
Signed-off-by: Philipp Zabel
---
Changes since v2:
- Fixed and simplified of_graph_get_port_by_id function
---
drivers/of/base.c| 26 ++
include/linux/of_graph.h | 7 +++
Note that while of_graph_get_next_endpoint decrements the reference count
of the child node passed to it, of_node_put(child) still has to be called
manually when breaking out of the loop.
Signed-off-by: Philipp Zabel
Acked-by: Laurent Pinchart
---
include/linux/of_graph.h | 11 +++
1 fi
On 2014-09-10 19:39, Hartley Sweeten wrote:
On Wednesday, September 10, 2014 3:56 AM, Ian Abbott wrote:
On 2014-09-10 00:15, H Hartley Sweeten wrote:
The validation of the cmd->stop_arg when the cmd->stop_src == TRIG_EXT
is a bit over thought. The comments state that the stop_arg is validated
t
Hi Philipp,
Thank you for the patch.
On Wednesday 10 September 2014 12:58:25 Philipp Zabel wrote:
> This patch adds a function to get a port device tree node by port id,
> or reg property value.
>
> Signed-off-by: Philipp Zabel
> ---
> Changes since v1:
> - Fixed whitespace in comment
> - Cha
Hi Philipp,
Thank you for the patch.
On Wednesday 10 September 2014 12:58:24 Philipp Zabel wrote:
> Note that while of_graph_get_next_endpoint decrements the reference count
> of the child node passed to it, of_node_put(child) still has to be called
> manually when breaking out of the loop.
>
>
Hi Philipp,
Thank you for the patch.
On Wednesday 10 September 2014 12:58:23 Philipp Zabel wrote:
> Decrementing the reference count of the previous endpoint node allows to
> use the of_graph_get_next_endpoint function in a for_each_... style macro.
> Prior to this patch, all current users of thi
Am Mittwoch, den 10.09.2014, 11:32 -0700 schrieb Greg Kroah-Hartman:
> On Wed, Sep 10, 2014 at 11:35:29AM +0200, Philipp Zabel wrote:
> > Hi Greg,
> >
> > I have asked Dave to pull Steve Longerbeam's preparation series for
> > capture support. It starts off with the following two commits that also
On Thu, Sep 11, 2014 at 03:17:52PM +0800, micky_ch...@realsil.com.cn wrote:
> + /* reset PM_CTRL3 before send buffer cmd */
> + err = rtsx_pci_write_register(pcr, PM_CTRL3, 0x10, 0x00);
> + if (err < 0)
> + return err;
> +
> + return err;
Just do:
return rtsx_pc
From: Micky Ching
Fix rts5249 failed send buffer cmd after suspend,
PM_CTRL3 should reset before send any buffer cmd after suspend.
Otherwise, buffer cmd will failed, this will lead resume fail.
Signed-off-by: Micky Ching
---
drivers/mfd/rts5249.c | 21 +
1 file changed,
From: Micky Ching
This patch fix rts5227 and rts5249 suspend issue, when card reader
resumed from suspend state, the power state should reset before send
buffer command. The original not reset PM state first, so this will
lead resume failed, and can not do anything more.
Micky Ching (2):
mfd:
From: Micky Ching
Fix rts5227 failed send buffer cmd after suspend,
PM_CTRL3 should reset before send any buffer cmd after suspend.
Otherwise, buffer cmd will failed, this will lead resume fail.
Signed-off-by: Micky Ching
---
drivers/mfd/rts5227.c| 23 +++
include
64 matches
Mail list logo
