On Thu, Nov 29, 2012 at 1:56 AM, Andrew Morton
wrote:
> On Wed, 21 Nov 2012 16:31:11 +0200
> Andy Shevchenko wrote:
>
>> When build a kernel with "make W=1" we will get a warning about missing
>> initializer. It comes from kfifo usage style. The DEFINE_KFIFO macr
On Thu, Nov 29, 2012 at 11:21 AM, Andy Shevchenko
wrote:
> On Thu, Nov 29, 2012 at 1:56 AM, Andrew Morton
> wrote:
>> On Wed, 21 Nov 2012 16:31:11 +0200
>> Andy Shevchenko wrote:
>>
>>> When build a kernel with "make W=1" we will get a warning about m
On Thu, Nov 29, 2012 at 3:51 AM, Andrew Morton
wrote:
> On Wed, 21 Nov 2012 16:31:11 +0200
> Andy Shevchenko wrote:
>
>> When build a kernel with "make W=1" we will get a warning about missing
>> initializer. It comes from kfifo usage style. The DEFINE_KFIFO macr
leted.
Signed-off-by: Andy Shevchenko
---
drivers/dma/dw_dmac.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/dma/dw_dmac.c b/drivers/dma/dw_dmac.c
index 43a5329..43e2e89 100644
--- a/drivers/dma/dw_dmac.c
+++ b/drivers/dma/dw_dmac.c
@@ -1030,10 +10
The commit c30f46bc (drivers/Kconfig: add several missing GENERIC_HARDIRQS
dependencies) introduced a warning on make defconfig stage. This patch fixes a
typo.
Signed-off-by: Andy Shevchenko
---
drivers/dma/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma
From: Mika Westerberg
Intel Lynxpoint PCH Low Power Subsystem has DMA controller to support general
purpose serial buses like SPI, I2C, and HSUART. This controller is enumerated
from ACPI namespace with ACPI ID INTL9C60.
Signed-off-by: Andy Shevchenko
---
drivers/dma/dw_dmac.c |6
On Thu, Feb 7, 2013 at 5:39 PM, Viresh Kumar wrote:
> On 7 February 2013 21:06, Andy Shevchenko
> wrote:
>> From: Mika Westerberg
>>
>> Intel Lynxpoint PCH Low Power Subsystem has DMA controller to support general
>> purpose serial buses like SPI, I2C, and HSUART.
On Fri, Feb 8, 2013 at 6:28 AM, Viresh Kumar wrote:
> On Thu, Feb 7, 2013 at 10:35 PM, Andy Shevchenko
> wrote:
>> On Thu, Feb 7, 2013 at 5:39 PM, Viresh Kumar wrote:
>>> On 7 February 2013 21:06, Andy Shevchenko
>>> wrote:
>>>> From: Mika Westerber
to match by the
driver name. This will make it similar to the DT or ACPI cases.
Signed-off-by: Andy Shevchenko
Reported-by: Mika Westerberg
Cc: Eric Miao
Cc: Greg Kroah-Hartman
---
drivers/base/platform.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/base
to match by the
driver name. This will make it similar to the DT or ACPI cases.
Signed-off-by: Andy Shevchenko
Reported-by: Mika Westerberg
Cc: Eric Miao
Cc: Greg Kroah-Hartman
---
drivers/base/platform.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/base
When status is DMA_SUCCESS the residue should be zero. Otherwise it's a bug.
Signed-off-by: Andy Shevchenko
Cc: Linus Walleij
Cc: linux-arm-ker...@lists.infradead.org
---
drivers/dma/coh901318.c |4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/dma/coh90131
It's possible to have an inconsistency in the list due to unprotected operation
on it. The patch adds a proper locking on the list operation.
Signed-off-by: Andy Shevchenko
Acked-by: Rob Herring
---
drivers/of/dma.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/of/dm
This is a patcset of independent fixes of the DMAC drivers. They are simple and
being understandable in their context.
Andy Shevchenko (6):
of: dma: protect list write operation by spin_lock
dmaengine.h: remove redundant else keyword
dma: coh901318_lli: avoid unbalanced locking
dma
I have not checked the code but there is a possibility that custom pool->lock
is not needed at all due to dma_pool contains it's own lock.
Signed-off-by: Andy Shevchenko
Cc: Linus Walleij
Cc: linux-arm-ker...@lists.infradead.org
---
drivers/dma/coh901318_lli.c |2 +-
1 file ch
Accordingly to commentary in the platform_device_register_full the memory
allocated for dma_mask will not going to be freed. That's why is better to
assign dma_mask afterwards.
Signed-off-by: Andy Shevchenko
---
drivers/dma/edma.c |6 --
1 file changed, 4 insertions(+), 2 dele
dmaengine_device_control returns -ENOSYS in case the dma driver doesn't have
such functionality.
Signed-off-by: Andy Shevchenko
---
include/linux/dmaengine.h |3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/include/linux/dmaengine.h b/include/linux/dmaengine.h
There is no need to assign 0 to residue, because dma_cookie_status() does this
for us.
Signed-off-by: Andy Shevchenko
---
drivers/dma/tegra20-apb-dma.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/dma/tegra20-apb-dma.c b/drivers/dma/tegra20-apb-dma.c
index 46e4630..fcee27e
On Tue, Feb 12, 2013 at 5:56 PM, Vinod Koul wrote:
> On Wed, Jan 30, 2013 at 09:32:56AM +0200, Andy Shevchenko wrote:
>> On Tue, 2013-01-29 at 20:57 +0530, Viresh Kumar wrote:
>> > On Tue, Jan 29, 2013 at 10:52 AM, Mika Westerberg
>> > wrote:
>> > > We ha
On Tue, Feb 12, 2013 at 6:16 PM, Vinod Koul wrote:
> On Tue, Jan 29, 2013 at 05:06:24PM +0200, Andy Shevchenko wrote:
>> In some cases we got the device without dma_mask configured. We have to apply
>> the default value to avoid crashes during memory mapping.
>>
> what wa
rent address spaces)
drivers/dma/dw_dmac.c:1691:32:expected void const *ptr
drivers/dma/dw_dmac.c:1691:32:got void [noderef] *[assigned] regs
--
With Best Regards,
Andy Shevchenko
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a messa
In some cases we got the device without dma_mask configured. We have to apply
the default value to avoid crashes during memory mapping.
Signed-off-by: Andy Shevchenko
---
drivers/dma/dw_dmac.c |6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/dma/dw_dmac.c b/drivers/dma
T is a platform device (see
drivers/acpi/csrt.c for the details).
That's why we have to pass this info somehow to the DMAC driver. And
like Mika already mentioned we have no other generic way except
IORESOURCE_DMA. If you have something better in mind, please share. We
could adopt our code the
On Wed, Jan 30, 2013 at 8:41 AM, Matt Porter wrote:
> On Mon, Jan 28, 2013 at 09:27:24PM +0200, Andy Shevchenko wrote:
>> On Tue, Jan 15, 2013 at 10:32 PM, Matt Porter wrote:
>> > Adds support for parsing the TI EDMA DT data into the required
>> > EDMA private API pla
On Wed, 2013-01-30 at 13:31 +0530, Viresh Kumar wrote:
> On Wed, Jan 30, 2013 at 1:02 PM, Andy Shevchenko
> wrote:
> > We get a device from the ACPI CSRT table. During enumeration we don't
> > know what kind of the device it is. From the enumeration point of view
> >
On Thu, 2013-01-24 at 10:37 +0530, Viresh Kumar wrote:
> On Wed, Jan 23, 2013 at 9:07 PM, Andy Shevchenko
> wrote:
> > diff --git a/drivers/dma/dw_dmac.c b/drivers/dma/dw_dmac.c
>
> > static void dwc_scan_descriptors(struct dw_dma *dw, struct
by this patch due to len field in the descriptor
structure is left untouched by the original code.
Signed-off-by: Andy Shevchenko
---
drivers/dma/dw_dmac.c | 44 +---
drivers/dma/dw_dmac_regs.h |1 +
2 files changed, 42 insertions(+), 3 deletions
It will be useful to have the length of the transfer in the descriptor. The
cyclic transfer functions remained untouched.
Signed-off-by: Andy Shevchenko
Acked-by: Viresh Kumar
---
drivers/dma/dw_dmac.c |3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/dma/dw_dmac.c b/drivers
By this new field we distinguish a total length of the chain and the individual
length of each descriptor in the chain.
Signed-off-by: Andy Shevchenko
Acked-by: Viresh Kumar
---
drivers/dma/dw_dmac.c | 12 ++--
drivers/dma/dw_dmac_regs.h |1 +
2 files changed, 7 insertions
The soft LLP mode is working for active descriptor only. So, we do not need to
have a copy of its pointer.
Signed-off-by: Andy Shevchenko
Acked-by: Viresh Kumar
---
drivers/dma/dw_dmac.c | 20 +++-
drivers/dma/dw_dmac_regs.h |1 -
2 files changed, 15 insertions(+), 6
Since v1:
- everything is rewritten to address Viresh's and Vinod's comments.
*** BLURB HERE ***
Andy Shevchenko (4):
dw_dmac: remove unnecessary tx_list field in dw_dma_chan
dw_dmac: introduce total_len field in struct dw_desc
dw_dmac: fill individual length of descriptor
dw_dm
On Thu, Jan 24, 2013 at 12:45 PM, Viresh Kumar wrote:
> On Thu, Jan 24, 2013 at 4:04 PM, Andy Shevchenko
> wrote:
>
> I don't see residue - len from interrupt handler. It isn't required?
Where exactly?
>> @@ -1062,6 +1090,7 @@ dwc_tx_status(struct dma_chan *c
by this patch due to len field in the descriptor
structure is left untouched by the original code.
Signed-off-by: Andy Shevchenko
---
drivers/dma/dw_dmac.c | 47 ++--
drivers/dma/dw_dmac_regs.h |1 +
2 files changed, 46 insertions(+), 2 deletions
On Fri, 2013-01-25 at 09:43 +0530, Viresh Kumar wrote:
> On Thu, Jan 24, 2013 at 6:37 PM, Andy Shevchenko
> wrote:
> > Currently the driver returns full length of the active descriptor which is
> > wrong. We have to go throught the active descriptor and substract the
> >
On Fri, Jan 25, 2013 at 10:56 AM, Andy Shevchenko
wrote:
> On Fri, 2013-01-25 at 09:43 +0530, Viresh Kumar wrote:
>> On Thu, Jan 24, 2013 at 6:37 PM, Andy Shevchenko
>> wrote:
>> > diff --git a/drivers/dma/dw_dmac.c b/drivers/dma/dw_dmac.c
>> > +static
On Fri, Jan 25, 2013 at 11:07 AM, Viresh Kumar wrote:
> On 25 January 2013 14:34, Andy Shevchenko wrote:
>> Okay, we have to have a protection here because get_sent reads two
>> registers consequentially. This means we could end up with scenario
>> with threads 1 and 2
>
On Fri, Jan 25, 2013 at 11:39 AM, Viresh Kumar wrote:
> On 25 January 2013 15:07, Andy Shevchenko wrote:
>> Yeah, the keyword is "look". 1 per million cases it could be true. I
>> think this discussion is going to the dead end.
>> Anyone else would like to arg
On Fri, Jan 25, 2013 at 11:30 AM, Viresh Kumar wrote:
> On Fri, Jan 25, 2013 at 2:49 PM, Andy Shevchenko
> wrote:
>> On Fri, Jan 25, 2013 at 11:07 AM, Viresh Kumar
>> wrote:
>>> Who is going to right on ctlhi/lo?
>
> Ahh, my English :( (/s/right/write)
>
It will be useful to have the length of the transfer in the descriptor. The
cyclic transfer functions remained untouched.
Signed-off-by: Andy Shevchenko
Acked-by: Viresh Kumar
---
drivers/dma/dw_dmac.c |3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/dma/dw_dmac.c b/drivers
The soft LLP mode is working for active descriptor only. So, we do not need to
have a copy of its pointer.
Signed-off-by: Andy Shevchenko
Acked-by: Viresh Kumar
---
drivers/dma/dw_dmac.c | 20 +++-
drivers/dma/dw_dmac_regs.h |1 -
2 files changed, 15 insertions(+), 6
get rid of dwc_update_residue(): for soft LLP mode we assign residue in
dwc_do_start() and decrease it at each interrupt. In the middle of transfer
we will return that result substracted by amount of sent bytes
Since v1:
- everything is rewritten to address Viresh's and Vinod's comments.
Andy Shevc
By this new field we distinguish a total length of the chain and the individual
length of each descriptor in the chain.
Signed-off-by: Andy Shevchenko
Acked-by: Viresh Kumar
---
drivers/dma/dw_dmac.c | 12 ++--
drivers/dma/dw_dmac_regs.h |1 +
2 files changed, 7 insertions
by this patch due to len field in the descriptor
structure is left untouched by the original code.
Signed-off-by: Andy Shevchenko
---
drivers/dma/dw_dmac.c | 47 ++--
drivers/dma/dw_dmac_regs.h |1 +
2 files changed, 46 insertions(+), 2 deletions
On some hardware configurations we have got the request line with the offset.
The patch introduces convert_slave_id() helper for that cases. The request line
base is got from the platform device resources provided by the IORESOURCE_DMA
type.
Signed-off-by: Mika Westerberg
Signed-off-by: Andy
On Mon, 2013-01-28 at 04:10 -0800, Vinod Koul wrote:
> > On Fri, Jan 18, 2013 at 5:44 PM, Andy Shevchenko
> > > The proper place for the main logic of the soft LLP mode is
> > > dwc_scan_descriptors. It prevents to get the transfer unexpectedly
> > > abo
rm_get_irq_byname(pdev, irq_name);
> + if (node)
> + err_irq[j] = irq_of_parse_and_map(node, 2);
> + else {
> + sprintf(irq_name, "edma%d_err", j);
> + err_irq[j] = platform_get_irq_byname
gt; --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
--
With Best
On Mon, 2013-01-28 at 13:04 +0200, Andy Shevchenko wrote:
> On some hardware configurations we have got the request line with the offset.
> The patch introduces convert_slave_id() helper for that cases. The request
> line
> base is got from the platform device resources pro
On some hardware configurations we have got the request line with the offset.
The patch introduces convert_slave_id() helper for that cases. The request line
base is came from the driver data provided by the platform_device_id table.
Signed-off-by: Mika Westerberg
Signed-off-by: Andy Shevchenko
ase the idr_alloc is never
called, but idr_find is. The hint field is never initialized and could not be
dereferenced.
The proposed fix uses the idr_slowpath call which reflects old behaviour.
Signed-off-by: Andy Shevchenko
Reported-by: Kirill A. Shutemov
---
kernel/events/core.c | 2 +-
1 file
On Wed, Feb 20, 2013 at 7:16 PM, Tejun Heo wrote:
> On Wed, Feb 20, 2013 at 07:09:36PM +0200, Andy Shevchenko wrote:
>> The guilty commit is cc5b5f6 "events: convert to idr_alloc()" together with
>> f49318a "idr: implement lookup hint". In our case the idr_allo
On Wed, Feb 20, 2013 at 1:52 PM, Andy Shevchenko
wrote:
> On some hardware configurations we have got the request line with the offset.
> The patch introduces convert_slave_id() helper for that cases. The request
> line
> base is came from the driver data provided by the platform_dev
You right. I have a patch to support few options like that already.
However, I can't share
it yet by some legal reasons.
--
With Best Regards,
Andy Shevchenko
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel
0
> +#defineDW_MEM_WIDTH_32 1 /* e.g. for avr32 */
> + u8 max_mem_width;
Might be I missed something, but why is it slave configuration?
I think the controller (actually channel) structure is more suitable
to keep that field inside.
--
With Best Reg
up dst_master as well?
> + /* Limit maximum transfer width to 32-bit */
> + slave->sdata.max_mem_width = DW_MEM_WIDTH_32;
It's under question regarding to my comments on previous patch.
--
With Best Regards,
Andy Shevchenko
--
To unsubscribe from this list: send the lin
On Mon, Sep 3, 2012 at 11:30 AM, Viresh Kumar wrote:
> On 3 September 2012 13:55, Andy Shevchenko wrote:
>>> #define DW_MEM_WIDTH_64 0 /* default */
>>> #define DW_MEM_WIDTH_32 1 /* e.g. for avr32 */
>> There are 4 options: 32, 64, 128, and
usage of the
custom slave config structure.
Signed-off-by: Andy Shevchenko
---
drivers/dma/dw_dmac.c |5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/dma/dw_dmac.c b/drivers/dma/dw_dmac.c
index f0c9403..7a67673 100644
--- a/drivers/dma/dw_dmac.c
+++ b/drivers/dma/dw_dmac.c
down to 2. The first value that works correctly will be
> used as the maximum.
Oh, it might be good idea to get this value in case neither IP nor
platform data provides it.
I'm pretty sure the platform device driver has to know this beforehand.
--
With Best Regards,
Andy Shevchenko
me one
> could
> (who has used dw_dmac.c) could share their code to use dw_dmac.c.
I published all my patches already here in the mailing list. Part of
them is under review, but another part has been pushed to the
linux-next.
I highly encourage you to try most recent version from the li
t?
Please, follow the discussions in the mailing list. Most of the related
patches are tagged as "dw_dmac".
--
Andy Shevchenko
Intel Finland Oy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More
From: Heikki Krogerus
The driver will be used as a core part for various implementations of the
DesignWare DMA device. The patch adjusts description on the top and corrects
paragraph indentation in few places across the code.
Signed-off-by: Heikki Krogerus
Signed-off-by: Andy Shevchenko
From: Heikki Krogerus
It's good to have a quasistatic name for the platform driver.
Signed-off-by: Heikki Krogerus
Signed-off-by: Andy Shevchenko
Reviewed-by: Felipe Balbi
Acked-by: Viresh Kumar
---
drivers/dma/dw_dmac.c |1 +
1 file changed, 1 insertion(+)
diff --git a/driver
From: Heikki Krogerus
This driver could be used on different platforms. Thus, the HAVE_CLK dependency
is dropped away.
Signed-off-by: Heikki Krogerus
Signed-off-by: Andy Shevchenko
Reviewed-by: Felipe Balbi
Acked-by: Viresh Kumar
---
drivers/dma/Kconfig |1 -
1 file changed, 1 deletion
There are few cleanups to the driver which already acked and reviewed.
I decide to split last series to two parts. This is first part.
Heikki Krogerus (4):
dmaengine: dw_dmac: use helper macro module_platform_driver()
dmaengine: dw_dmac: add module alias
dmaengine: dw_dmac: remove CLK depend
From: Heikki Krogerus
Since v3.2 we have nice macro to define the platform driver's init and exit
calls. This patch simplifies the dw_dmac driver by using that macro.
Signed-off-by: Heikki Krogerus
Signed-off-by: Andy Shevchenko
Reviewed-by: Felipe Balbi
Acked-by: Viresh Kumar
---
dr
/') + 1;
Moreover there is already basename() in the lib/dynamic_debug.c you
could rename, export and reuse.
--
Andy Shevchenko
Intel Finland Oy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
ne TRACE_GOTO(chip, label) goto label
> +#define TRACE_RET(chip, ret) return(ret)
> +#define TRACE_GOTO(chip, label) goto(label)
> #endif
In this specific case I think checkpatch.pl wants more than needed.
--
Andy Shevchenko
Intel Finland Oy
--
To unsubscribe from this li
The custom filename function mostly repeats the kernel's kbasename. This patch
simplifies it. The updated filename() will not check for the '\' in the
filenames. It seems redundant in Linux.
Signed-off-by: Andy Shevchenko
Cc: YAMANE Toshiaki
Cc: Greg Kroah-Hartman
---
Signed-off-by: Andy Shevchenko
Cc: Greg Kroah-Hartman
Cc: linux-...@vger.kernel.org
---
drivers/usb/core/file.c |9 ++---
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/drivers/usb/core/file.c b/drivers/usb/core/file.c
index e5387a4..eff9af9 100644
--- a/drivers/usb/core
Signed-off-by: Andy Shevchenko
Cc: Steven Rostedt (maintainer:TRACING)
Cc: Frederic Weisbecker (maintainer:TRACING)
---
kernel/trace/trace_uprobe.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/kernel/trace/trace_uprobe.c b/kernel/trace/trace_uprobe.c
index
Remove the custom implementation of the functionality similar to kbasename().
Signed-off-by: Andy Shevchenko
Cc: Jason Baron
---
lib/dynamic_debug.c |9 +
1 file changed, 1 insertion(+), 8 deletions(-)
diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c
index e7f7d99..1db1fc6
Signed-off-by: Andy Shevchenko
---
fs/proc/proc_devtree.c |7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/fs/proc/proc_devtree.c b/fs/proc/proc_devtree.c
index df7dd08..3d9fd66 100644
--- a/fs/proc/proc_devtree.c
+++ b/fs/proc/proc_devtree.c
@@ -13,6 +13,7
Signed-off-by: Andy Shevchenko
Cc: linux...@kvack.org
---
mm/memory.c |8 +++-
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/mm/memory.c b/mm/memory.c
index 0e3a516..6b101a2 100644
--- a/mm/memory.c
+++ b/mm/memory.c
@@ -58,6 +58,7 @@
#include
#include
#include
There are several places in kernel that use functionality like shell's basename
function. Let's do it common helper for them.
Signed-off-by: Andy Shevchenko
Cc: YAMANE Toshiaki
---
include/linux/string.h | 11 +++
1 file changed, 11 insertions(+)
diff --git a/include/linu
On Tue, Oct 2, 2012 at 8:34 PM, Greg KH wrote:
> On Tue, Oct 02, 2012 at 06:00:54PM +0300, Andy Shevchenko wrote:
>> There are several places in kernel that use functionality like shell's
>> basename
>> function. Let's do it common helper for them.
>>
>
On Tue, Oct 2, 2012 at 6:00 PM, Andy Shevchenko
wrote:
> --- a/drivers/usb/core/file.c
> +++ b/drivers/usb/core/file.c
> @@ -200,14 +200,9 @@ int usb_register_dev(struct usb_interface *intf,
>
> /* create a usb class device for this usb interface */
> snprintf
There are several places in the kernel that use functionality like basename(3)
with the exception: in case of '/foo/bar/' we expect to get an empty string.
Let's do it common helper for them.
Signed-off-by: Andy Shevchenko
---
include/linux/string.h | 11 +++
1 f
The custom filename function mostly repeats the kernel's kbasename. This patch
simplifies it. The updated filename() will not check for the '\' in the
filenames. It seems redundant in Linux.
Signed-off-by: Andy Shevchenko
Cc: YAMANE Toshiaki
Cc: Greg Kroah-Hartman
---
Signed-off-by: Andy Shevchenko
Cc: linux...@kvack.org
---
mm/memory.c |8 +++-
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/mm/memory.c b/mm/memory.c
index 0e3a516..6b101a2 100644
--- a/mm/memory.c
+++ b/mm/memory.c
@@ -58,6 +58,7 @@
#include
#include
#include
Signed-off-by: Andy Shevchenko
Cc: Steven Rostedt (maintainer:TRACING)
Cc: Frederic Weisbecker (maintainer:TRACING)
---
kernel/trace/trace_uprobe.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/kernel/trace/trace_uprobe.c b/kernel/trace/trace_uprobe.c
index
Signed-off-by: Andy Shevchenko
---
fs/proc/proc_devtree.c |7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/fs/proc/proc_devtree.c b/fs/proc/proc_devtree.c
index df7dd08..3d9fd66 100644
--- a/fs/proc/proc_devtree.c
+++ b/fs/proc/proc_devtree.c
@@ -13,6 +13,7
Remove the custom implementation of the functionality similar to kbasename().
Signed-off-by: Andy Shevchenko
Cc: Jason Baron
---
lib/dynamic_debug.c |9 +
1 file changed, 1 insertion(+), 8 deletions(-)
diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c
index e7f7d99..1db1fc6
There are several places in kernel that duplicate code to get last part of the
pathname.
This patchset introduces a helper.
Since v1:
- fix changelog of the patch 1 - we are doing basname(3) alike helper
- usb related patch temporary excluded from series (under discussion with
Greg)
Andy
Signed-off-by: Andy Shevchenko
Cc: Steven Rostedt
Cc: Frederic Weisbecker
---
kernel/trace/trace_uprobe.c |8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/kernel/trace/trace_uprobe.c b/kernel/trace/trace_uprobe.c
index 03003cd..c7ba4f6 100644
--- a/kernel/trace
st possible size, and keep as much of
>> common part in dmac.c...
>>
>> Similar is done in drivers/mmc/host/sdhci*...
> Yes that IMHO would be simpler approach :)
Oh, thanks, I got the idea. Will implement soon.
--
With Best Regards,
Andy Shevchenko
--
To unsubscribe from this
On Wed, 2012-10-03 at 14:39 -0400, Nick Bowler wrote:
> On 2012-10-02 11:12 -0700, Greg KH wrote:
> > On Tue, Oct 02, 2012 at 08:52:05PM +0300, Andy Shevchenko wrote:
> > > On Tue, Oct 2, 2012 at 8:34 PM, Greg KH
> > > wrote:
[...]
> > Well, if you want your
On Fri, Oct 5, 2012 at 7:49 PM, Steven Rostedt wrote:
> On Wed, 2012-10-03 at 11:53 +0300, Andy Shevchenko wrote:
>> Signed-off-by: Andy Shevchenko
>> Cc: Steven Rostedt
>> Cc: Frederic Weisbecker
>> ---
>> kernel/trace/trace_uprobe.c |8
>&
On Fri, Oct 5, 2012 at 8:12 PM, Steven Rostedt wrote:
> On Fri, 2012-10-05 at 20:02 +0300, Andy Shevchenko wrote:
>
>> > I don't see kbasename() anywhere. Is this based off of other patches?
>> It's introduced by first patch in the series.
> Usually a series
> --
> To unsubscribe from this list: send the line "unsubscribe linux-usb" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
With Best Regards,
Andy Shevchenko
--
To unsubscribe from this list:
Signed-off-by: Andy Shevchenko
Acked-by: Grant Likely
Cc: Rob Herring
---
drivers/of/fdt.c | 10 ++
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/drivers/of/fdt.c b/drivers/of/fdt.c
index 91a375f..0ca0f9e 100644
--- a/drivers/of/fdt.c
+++ b/drivers/of/fdt.c
@@ -486,14
Not all of the controllers support the 64 bit data width. Make it configurable
via platform data. The driver will try to get a value from the component
parameters, otherwise it will use the platform data.
Signed-off-by: Andy Shevchenko
---
arch/arm/mach-spear13xx/spear13xx.c |2 ++
arch
The maximum block size is a configurable parameter for the chip. So, driver
will try to get it from the encoded component parameters. Otherwise it will
come from the platform data.
Signed-off-by: Andy Shevchenko
---
arch/arm/mach-spear13xx/spear13xx.c |1 +
arch/avr32/mach-at32ap
transfers. It means that
normal linked list transfer is substituted by set of single block transfers
transparently to the user.
Since v1:
- addressed all Viresh's comments.
Andy Shevchenko (6):
dw_dmac: mark dwc_dump_chan_regs as inline
dw_dmac: fill optional encoded parameters in register stru
In case the controller has the encoded parameters feature enabled the driver
will use it to get the number of channels. In the future it will be used for
the other important parameters as well.
Signed-off-by: Andy Shevchenko
---
drivers/dma/dw_dmac.c | 33 +++--
1
There is a block of the registers that are optional. However, if enabled they
contain useful information about the controller hardware configuration. We will
use this piece of data to autoconfigure the driver.
Signed-off-by: Andy Shevchenko
---
drivers/dma/dw_dmac_regs.h | 30
On Fri, 2012-09-21 at 19:30 +0530, viresh kumar wrote:
> On Fri, Sep 21, 2012 at 5:35 PM, Andy Shevchenko
> wrote:
> > diff --git a/drivers/dma/dw_dmac.c b/drivers/dma/dw_dmac.c
> > @@ -1385,6 +1375,7 @@ static int __devinit dw_probe(struct platform_device
> > *pde
Not all of the controllers support the 64 bit data width. Make it configurable
via platform data. The driver will try to get a value from the component
parameters, otherwise it will use the platform data.
Signed-off-by: Andy Shevchenko
---
Since v2:
- sometimes memory-to-memory test is failed
duplicating a
core driver functionality.
Andy Shevchenko (2):
MAINTAINERS: fix indentation for Viresh Kumar
MAINTAINERS: add recently created files to dw_dmac section
Heikki Krogerus (4):
dmaengine: dw_dmac: Remove clk API dependency
dmaengine: dw_dmac: add driver for Atmel AT32
dmaengine
From: Heikki Krogerus
This is the PCI part of the DesignWare DMAC driver. The controller is usually
used in the Intel hardware such as Medfield.
Signed-off-by: Heikki Krogerus
Signed-off-by: Andy Shevchenko
---
drivers/dma/Kconfig|9 +++
drivers/dma/Makefile |1
From: Heikki Krogerus
This driver should be usable on all platforms that depend on clk API.
Signed-off-by: Heikki Krogerus
Signed-off-by: Andy Shevchenko
---
drivers/dma/Kconfig|9 +++
drivers/dma/Makefile |1 +
drivers/dma/dw_dmac.c | 23 +--
drivers/dma
Signed-off-by: Andy Shevchenko
Cc: Viresh Kumar
---
MAINTAINERS | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 8fa0255..479a655 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -572,7 +572,7 @@ F: drivers/net/appletalk/
F
1 - 100 of 10954 matches
Mail list logo