* [PATCH RFT] ARM64: dma-mapping: support debug_dma_mapping_error
@ 2012-10-25 22:47 Shuah Khan
2012-10-26 8:38 ` Santosh Shilimkar
2012-10-26 14:48 ` Catalin Marinas
0 siblings, 2 replies; 7+ messages in thread
From: Shuah Khan @ 2012-10-25 22:47 UTC (permalink / raw)
To: linux-arm-kernel
Add support for debug_dma_mapping_error() call to avoid warning from
debug_dma_unmap() interface when it checks for mapping error checked
status. Without this patch, device driver failed to check map error
warning is generated.
Signed-off-by: Shuah Khan <shuah.khan@hp.com>
---
arch/arm64/include/asm/dma-mapping.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/include/asm/dma-mapping.h b/arch/arm64/include/asm/dma-mapping.h
index 538f4b4..9947768 100644
--- a/arch/arm64/include/asm/dma-mapping.h
+++ b/arch/arm64/include/asm/dma-mapping.h
@@ -50,6 +50,7 @@ static inline phys_addr_t dma_to_phys(struct device *dev, dma_addr_t dev_addr)
static inline int dma_mapping_error(struct device *dev, dma_addr_t dev_addr)
{
struct dma_map_ops *ops = get_dma_ops(dev);
+ debug_dma_mapping_error(dev, dev_addr);
return ops->mapping_error(dev, dev_addr);
}
--
1.7.9.5
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH RFT] ARM64: dma-mapping: support debug_dma_mapping_error
2012-10-25 22:47 [PATCH RFT] ARM64: dma-mapping: support debug_dma_mapping_error Shuah Khan
@ 2012-10-26 8:38 ` Santosh Shilimkar
2012-10-26 14:48 ` Catalin Marinas
1 sibling, 0 replies; 7+ messages in thread
From: Santosh Shilimkar @ 2012-10-26 8:38 UTC (permalink / raw)
To: linux-arm-kernel
On Friday 26 October 2012 04:17 AM, Shuah Khan wrote:
> Add support for debug_dma_mapping_error() call to avoid warning from
> debug_dma_unmap() interface when it checks for mapping error checked
> status. Without this patch, device driver failed to check map error
> warning is generated.
>
> Signed-off-by: Shuah Khan <shuah.khan@hp.com>
> ---
Looks good. Similar fix for 32bit ARM port is done in below commit.
From 871ae57adc5ed092c1341f411514d0e8482e2611 Mon Sep 17 00:00:00 2001
From: Ming Lei <ming.lei@canonical.com>
Date: Mon, 22 Oct 2012 20:44:03 +0800
Subject: [PATCH] ARM: dma-mapping: support debug_dma_mapping_error
Regards
Santosh
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH RFT] ARM64: dma-mapping: support debug_dma_mapping_error
2012-10-25 22:47 [PATCH RFT] ARM64: dma-mapping: support debug_dma_mapping_error Shuah Khan
2012-10-26 8:38 ` Santosh Shilimkar
@ 2012-10-26 14:48 ` Catalin Marinas
2012-10-26 15:23 ` [PATCH linux-next] " Shuah Khan
1 sibling, 1 reply; 7+ messages in thread
From: Catalin Marinas @ 2012-10-26 14:48 UTC (permalink / raw)
To: linux-arm-kernel
On 25 October 2012 23:47, Shuah Khan <shuah.khan@hp.com> wrote:
> Add support for debug_dma_mapping_error() call to avoid warning from
> debug_dma_unmap() interface when it checks for mapping error checked
> status. Without this patch, device driver failed to check map error
> warning is generated.
>
> Signed-off-by: Shuah Khan <shuah.khan@hp.com>
> ---
> arch/arm64/include/asm/dma-mapping.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm64/include/asm/dma-mapping.h b/arch/arm64/include/asm/dma-mapping.h
> index 538f4b4..9947768 100644
> --- a/arch/arm64/include/asm/dma-mapping.h
> +++ b/arch/arm64/include/asm/dma-mapping.h
> @@ -50,6 +50,7 @@ static inline phys_addr_t dma_to_phys(struct device *dev, dma_addr_t dev_addr)
> static inline int dma_mapping_error(struct device *dev, dma_addr_t dev_addr)
> {
> struct dma_map_ops *ops = get_dma_ops(dev);
> + debug_dma_mapping_error(dev, dev_addr);
> return ops->mapping_error(dev, dev_addr);
> }
The patch looks fine but debug_dma_mapping_error() is not in mainline
yet, so it will have to wait. If you want to add it to your series
(which I assume adds debug_dma_mapping_error() support), you can add
my Acked-by on the patch.
--
Catalin
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH linux-next] ARM64: dma-mapping: support debug_dma_mapping_error
2012-10-26 14:48 ` Catalin Marinas
@ 2012-10-26 15:23 ` Shuah Khan
2012-11-02 15:58 ` Shuah Khan
0 siblings, 1 reply; 7+ messages in thread
From: Shuah Khan @ 2012-10-26 15:23 UTC (permalink / raw)
To: linux-arm-kernel
Add support for debug_dma_mapping_error() call to avoid warning from
debug_dma_unmap() interface when it checks for mapping error checked
status. Without this patch, device driver failed to check map error
warning is generated.
Signed-off-by: Shuah Khan <shuah.khan@hp.com>
Acked-by: Catalin Marinas <catalin.marinas@arm.com>
---
arch/arm64/include/asm/dma-mapping.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/include/asm/dma-mapping.h b/arch/arm64/include/asm/dma-mapping.h
index 538f4b4..9947768 100644
--- a/arch/arm64/include/asm/dma-mapping.h
+++ b/arch/arm64/include/asm/dma-mapping.h
@@ -50,6 +50,7 @@ static inline phys_addr_t dma_to_phys(struct device *dev, dma_addr_t dev_addr)
static inline int dma_mapping_error(struct device *dev, dma_addr_t dev_addr)
{
struct dma_map_ops *ops = get_dma_ops(dev);
+ debug_dma_mapping_error(dev, dev_addr);
return ops->mapping_error(dev, dev_addr);
}
--
1.7.9.5
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH linux-next] ARM64: dma-mapping: support debug_dma_mapping_error
2012-10-26 15:23 ` [PATCH linux-next] " Shuah Khan
@ 2012-11-02 15:58 ` Shuah Khan
2012-11-05 16:46 ` Catalin Marinas
0 siblings, 1 reply; 7+ messages in thread
From: Shuah Khan @ 2012-11-02 15:58 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, 2012-10-26 at 09:23 -0600, Shuah Khan wrote:
> Add support for debug_dma_mapping_error() call to avoid warning from
> debug_dma_unmap() interface when it checks for mapping error checked
> status. Without this patch, device driver failed to check map error
> warning is generated.
>
> Signed-off-by: Shuah Khan <shuah.khan@hp.com>
> Acked-by: Catalin Marinas <catalin.marinas@arm.com>
Do you want this patch going through linux-next or the ARM64 tree? Let
me know your preference.
-- Shuah
> ---
> arch/arm64/include/asm/dma-mapping.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm64/include/asm/dma-mapping.h b/arch/arm64/include/asm/dma-mapping.h
> index 538f4b4..9947768 100644
> --- a/arch/arm64/include/asm/dma-mapping.h
> +++ b/arch/arm64/include/asm/dma-mapping.h
> @@ -50,6 +50,7 @@ static inline phys_addr_t dma_to_phys(struct device *dev, dma_addr_t dev_addr)
> static inline int dma_mapping_error(struct device *dev, dma_addr_t dev_addr)
> {
> struct dma_map_ops *ops = get_dma_ops(dev);
> + debug_dma_mapping_error(dev, dev_addr);
> return ops->mapping_error(dev, dev_addr);
> }
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH linux-next] ARM64: dma-mapping: support debug_dma_mapping_error
2012-11-02 15:58 ` Shuah Khan
@ 2012-11-05 16:46 ` Catalin Marinas
2012-11-15 17:11 ` Shuah Khan
0 siblings, 1 reply; 7+ messages in thread
From: Catalin Marinas @ 2012-11-05 16:46 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Nov 02, 2012 at 03:58:53PM +0000, Shuah Khan wrote:
> On Fri, 2012-10-26 at 09:23 -0600, Shuah Khan wrote:
> > Add support for debug_dma_mapping_error() call to avoid warning from
> > debug_dma_unmap() interface when it checks for mapping error checked
> > status. Without this patch, device driver failed to check map error
> > warning is generated.
> >
> > Signed-off-by: Shuah Khan <shuah.khan@hp.com>
> > Acked-by: Catalin Marinas <catalin.marinas@arm.com>
>
> Do you want this patch going through linux-next or the ARM64 tree? Let
> me know your preference.
I think it can go together with your other patches since I can't tell
when you'll push the debug_dma_mapping_error() implementation.
--
Catalin
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH linux-next] ARM64: dma-mapping: support debug_dma_mapping_error
2012-11-05 16:46 ` Catalin Marinas
@ 2012-11-15 17:11 ` Shuah Khan
0 siblings, 0 replies; 7+ messages in thread
From: Shuah Khan @ 2012-11-15 17:11 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, 2012-11-05 at 16:46 +0000, Catalin Marinas wrote:
> On Fri, Nov 02, 2012 at 03:58:53PM +0000, Shuah Khan wrote:
> > On Fri, 2012-10-26 at 09:23 -0600, Shuah Khan wrote:
> > > Add support for debug_dma_mapping_error() call to avoid warning from
> > > debug_dma_unmap() interface when it checks for mapping error checked
> > > status. Without this patch, device driver failed to check map error
> > > warning is generated.
> > >
> > > Signed-off-by: Shuah Khan <shuah.khan@hp.com>
> > > Acked-by: Catalin Marinas <catalin.marinas@arm.com>
> >
> > Do you want this patch going through linux-next or the ARM64 tree? Let
> > me know your preference.
>
> I think it can go together with your other patches since I can't tell
> when you'll push the debug_dma_mapping_error() implementation.
>
Marek,
This one is for ARM64 and ARM tree has already been fixed.
-- Shuah
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2012-11-15 17:11 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-10-25 22:47 [PATCH RFT] ARM64: dma-mapping: support debug_dma_mapping_error Shuah Khan
2012-10-26 8:38 ` Santosh Shilimkar
2012-10-26 14:48 ` Catalin Marinas
2012-10-26 15:23 ` [PATCH linux-next] " Shuah Khan
2012-11-02 15:58 ` Shuah Khan
2012-11-05 16:46 ` Catalin Marinas
2012-11-15 17:11 ` Shuah Khan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).