Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1220774 > unrolled thread

[PATCH 0/4] xtensa: fix build failure

Back to article view | Back to linux.kernel

Contents

  [PATCH 0/4] xtensa: fix build failure Sudip Mukherjee <sudipm.mukherjee@gmail.com> - 2015-09-08 15:20 +0200
    [PATCH 3/4] xtensa: fix errors with dma_supported Sudip Mukherjee <sudipm.mukherjee@gmail.com> - 2015-09-08 15:20 +0200
    [PATCH 2/4] xtensa: fix error with dma_supported Sudip Mukherjee <sudipm.mukherjee@gmail.com> - 2015-09-08 15:20 +0200
    Re: [PATCH 0/4] xtensa: fix build failure Max Filippov <jcmvbkbc@gmail.com> - 2015-09-08 19:40 +0200
      Re: [PATCH 0/4] xtensa: fix build failure Sudip Mukherjee <sudipm.mukherjee@gmail.com> - 2015-09-09 09:20 +0200
        Re: [PATCH 0/4] xtensa: fix build failure Max Filippov <jcmvbkbc@gmail.com> - 2015-09-09 15:00 +0200
          Re: [PATCH 0/4] xtensa: fix build failure Sudip Mukherjee <sudipm.mukherjee@gmail.com> - 2015-09-09 15:50 +0200

#1220774 — [PATCH 0/4] xtensa: fix build failure

While building with allmodconfig the build failed with different dma
related errors. It turned out that the dma functions were moved to
asm-generic/dma-mapping-common.h() and as a result we were getting the
errors of "redefinition".
This patch series will move the functions as required and at the end of
the series, allmodconfig builds.
Only build tested with allmodconfig, common_defconfig, iss_defconfig on
next-20150908.

Sudip Mukherjee (4):
  xtensa: fix error with dma_{alloc,free}_{attrs,coherent}
  xtensa: fix error with dma_supported
  xtensa: fix errors with dma_supported
  xtensa: fix error with dma_set_mask

 arch/xtensa/include/asm/dma-mapping.h | 57 +++--------------------------------
 1 file changed, 4 insertions(+), 53 deletions(-)

-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1220775 — [PATCH 3/4] xtensa: fix errors with dma_supported

commit eec17ba9f049 ("dma-mapping: consolidate dma_supported") has
removed dma_supported() from individual arch files to
asm-generic/dma-mapping-common.h or if arch has its own implementation
then we need to set the flag HAVE_ARCH_DMA_SUPPORTED. Otherwise we were
getting error like:
error: redefinition of 'dma_supported'
while building with allmodconfig.

Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
---
 arch/xtensa/include/asm/dma-mapping.h | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/arch/xtensa/include/asm/dma-mapping.h b/arch/xtensa/include/asm/dma-mapping.h
index 5762d8d..019a94a 100644
--- a/arch/xtensa/include/asm/dma-mapping.h
+++ b/arch/xtensa/include/asm/dma-mapping.h
@@ -30,10 +30,7 @@ static inline struct dma_map_ops *get_dma_ops(struct device *dev)
 		return &xtensa_dma_map_ops;
 }
 
-#include <asm-generic/dma-mapping-common.h>
-
-#define dma_alloc_noncoherent(d, s, h, f) dma_alloc_attrs(d, s, h, f, NULL)
-#define dma_free_noncoherent(d, s, v, h) dma_free_attrs(d, s, v, h, NULL)
+#define HAVE_ARCH_DMA_SUPPORTED 1
 
 static inline int
 dma_supported(struct device *dev, u64 mask)
@@ -41,6 +38,11 @@ dma_supported(struct device *dev, u64 mask)
 	return 1;
 }
 
+#include <asm-generic/dma-mapping-common.h>
+
+#define dma_alloc_noncoherent(d, s, h, f) dma_alloc_attrs(d, s, h, f, NULL)
+#define dma_free_noncoherent(d, s, v, h) dma_free_attrs(d, s, v, h, NULL)
+
 static inline int
 dma_set_mask(struct device *dev, u64 mask)
 {
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1220778 — [PATCH 2/4] xtensa: fix error with dma_supported

commit 11bd9421d0ad ("dma-mapping: cosolidate dma_mapping_error") has
removed dma_mapping_error() from arch specific file to
asm-generic/dma-mapping-common.h but it was not removed from xtensa
architecture and as a result we were getting error like:
error: redefinition of 'dma_mapping_error'
while building with allmodconfig.

Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
---
 arch/xtensa/include/asm/dma-mapping.h | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/arch/xtensa/include/asm/dma-mapping.h b/arch/xtensa/include/asm/dma-mapping.h
index bf24c90..5762d8d 100644
--- a/arch/xtensa/include/asm/dma-mapping.h
+++ b/arch/xtensa/include/asm/dma-mapping.h
@@ -36,15 +36,6 @@ static inline struct dma_map_ops *get_dma_ops(struct device *dev)
 #define dma_free_noncoherent(d, s, v, h) dma_free_attrs(d, s, v, h, NULL)
 
 static inline int
-dma_mapping_error(struct device *dev, dma_addr_t dma_addr)
-{
-	struct dma_map_ops *ops = get_dma_ops(dev);
-
-	debug_dma_mapping_error(dev, dma_addr);
-	return ops->mapping_error(dev, dma_addr);
-}
-
-static inline int
 dma_supported(struct device *dev, u64 mask)
 {
 	return 1;
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1221014

Sudip,

On Tue, Sep 8, 2015 at 4:18 PM, Sudip Mukherjee
<sudipm.mukherjee@gmail.com> wrote:
> While building with allmodconfig the build failed with different dma
> related errors. It turned out that the dma functions were moved to
> asm-generic/dma-mapping-common.h() and as a result we were getting the
> errors of "redefinition".
> This patch series will move the functions as required and at the end of
> the series, allmodconfig builds.

I believe it's fixed in the -mm tree and will be merged soon,
see the thread at https://lkml.org/lkml/2015/8/26/118

-- 
Thanks.
-- Max
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1221281

On Tue, Sep 08, 2015 at 08:34:56PM +0300, Max Filippov wrote:
> Sudip,
> 
> On Tue, Sep 8, 2015 at 4:18 PM, Sudip Mukherjee
> <sudipm.mukherjee@gmail.com> wrote:
> > While building with allmodconfig the build failed with different dma
> > related errors. It turned out that the dma functions were moved to
> > asm-generic/dma-mapping-common.h() and as a result we were getting the
> > errors of "redefinition".
> > This patch series will move the functions as required and at the end of
> > the series, allmodconfig builds.
> 
> I believe it's fixed in the -mm tree and will be merged soon,
> see the thread at https://lkml.org/lkml/2015/8/26/118
I guess its already merged in Linus tree now. I tested in the morning
with Linus tree and allmodconfig worked. But next-20150909 still failed.
BTW, yesterday when I was testing few of the defconfigs were not
working. Are they also being fixed?

regards
sudip
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1221403

On Wed, Sep 9, 2015 at 10:11 AM, Sudip Mukherjee
<sudipm.mukherjee@gmail.com> wrote:
> On Tue, Sep 08, 2015 at 08:34:56PM +0300, Max Filippov wrote:
>> Sudip,
>>
>> On Tue, Sep 8, 2015 at 4:18 PM, Sudip Mukherjee
>> <sudipm.mukherjee@gmail.com> wrote:
>> > While building with allmodconfig the build failed with different dma
>> > related errors. It turned out that the dma functions were moved to
>> > asm-generic/dma-mapping-common.h() and as a result we were getting the
>> > errors of "redefinition".
>> > This patch series will move the functions as required and at the end of
>> > the series, allmodconfig builds.
>>
>> I believe it's fixed in the -mm tree and will be merged soon,
>> see the thread at https://lkml.org/lkml/2015/8/26/118
>
> I guess its already merged in Linus tree now. I tested in the morning
> with Linus tree and allmodconfig worked. But next-20150909 still failed.

AFAICS it's not merged yet: arch/xtensa/include/dma-mapping.h still
has all functions deleted by patch in the mentioned thread.

> BTW, yesterday when I was testing few of the defconfigs were not
> working. Are they also being fixed?

There are many ways they can possibly fail, but mostly build errors are
caused by using wrong compiler. For more details please refer to
http://wiki.linux-xtensa.org/index.php/Linux_Kernel#Configuring_and_building_the_kernel

-- 
Thanks.
-- Max
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1221472

On Wed, Sep 09, 2015 at 03:52:11PM +0300, Max Filippov wrote:
> On Wed, Sep 9, 2015 at 10:11 AM, Sudip Mukherjee
> <sudipm.mukherjee@gmail.com> wrote:
> > On Tue, Sep 08, 2015 at 08:34:56PM +0300, Max Filippov wrote:
> >> Sudip,
> >>
> >> On Tue, Sep 8, 2015 at 4:18 PM, Sudip Mukherjee
> >> <sudipm.mukherjee@gmail.com> wrote:
> >> > While building with allmodconfig the build failed with different dma
> >> > related errors. It turned out that the dma functions were moved to
> >> > asm-generic/dma-mapping-common.h() and as a result we were getting the
> >> > errors of "redefinition".
> >> > This patch series will move the functions as required and at the end of
> >> > the series, allmodconfig builds.
> >>
> >> I believe it's fixed in the -mm tree and will be merged soon,
> >> see the thread at https://lkml.org/lkml/2015/8/26/118
> >
> > I guess its already merged in Linus tree now. I tested in the morning
> > with Linus tree and allmodconfig worked. But next-20150909 still failed.
> 
> AFAICS it's not merged yet: arch/xtensa/include/dma-mapping.h still
> has all functions deleted by patch in the mentioned thread.
Ohhh.. I didn't notice it in the morning. I saw Linus tree is building
and I thought it may have been merged. Now on checking I see that the
series by Christoph Hellwig which removes the arch specific functions is
still not in Linus tree and thats why I didnot get any error.


regards
sudip
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web