lkml.org 
[lkml]   [2015]   [Jun]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [bp:tip-mm 11/13] drivers/video/fbdev/aty/atyfb_base.c:3461:2: error: implicit declaration of function 'ioremap_uc'
From
On Fri, Jun 26, 2015 at 12:29 PM, Borislav Petkov <bp@suse.de> wrote:
> On Fri, Jun 26, 2015 at 04:28:51PM +0800, kbuild test robot wrote:
>> tree: git://git.kernel.org/pub/scm/linux/kernel/git/bp/bp tip-mm
>> head: 7b0982330a1b4d4bdb99bb8382e2363165231a90
>> commit: fd50bea9cedd8fdbd091dfccb4b526be0ef9839f [11/13] drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC
>> config: xtensa-allyesconfig (attached as .config)
...
>> All error/warnings (new ones prefixed by >>):
>>
>> drivers/video/fbdev/aty/atyfb_base.c: In function 'atyfb_setup_generic':
>> >> drivers/video/fbdev/aty/atyfb_base.c:3461:2: error: implicit declaration of function 'ioremap_uc' [-Werror=implicit-function-declaration]
>> par->ati_regbase = ioremap_uc(info->fix.mmio_start, 0x1000);
>> ^
>> >> drivers/video/fbdev/aty/atyfb_base.c:3461:19: warning: assignment makes pointer from integer without a cast
>> par->ati_regbase = ioremap_uc(info->fix.mmio_start, 0x1000);
>> ^
...
> Looks to me, xtensa needs something like that:
>
> ---
> From d8ac67e37a1a99e305c5ec57cff084f9f3513e61 Mon Sep 17 00:00:00 2001
> From: Borislav Petkov <bp@suse.de>
> Date: Fri, 26 Jun 2015 10:54:40 +0200
> Subject: [PATCH] xtensa/asm: Add ioremap_uc
>
> ... and default to the uncacheable version.
>
> Reported-by: kbuild test robot <fengguang.wu@intel.com>
> Signed-off-by: Borislav Petkov <bp@suse.de>
> Cc: "Luis R. Rodriguez" <mcgrof@suse.com>
> Cc: Chris Zankel <chris@zankel.net>
> Cc: Max Filippov <jcmvbkbc@gmail.com>
> Cc: Will Deacon <will.deacon@arm.com>
> Cc: Ingo Molnar <mingo@kernel.org>
> Cc: Toshi Kani <toshi.kani@hp.com>
> Cc: linux-xtensa@linux-xtensa.org
> ---
> arch/xtensa/include/asm/io.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/xtensa/include/asm/io.h b/arch/xtensa/include/asm/io.h
> index c39bb6e61911..34f707ea619b 100644
> --- a/arch/xtensa/include/asm/io.h
> +++ b/arch/xtensa/include/asm/io.h
> @@ -60,6 +60,7 @@ static inline void __iomem *ioremap_cache(unsigned long offset,
>
> #define ioremap_wc ioremap_nocache
> #define ioremap_wt ioremap_nocache
> +#define ioremap_uc ioremap_nocache
>
> static inline void __iomem *ioremap(unsigned long offset, unsigned long size)
> {
> --

Acked-by: Max Filippov <jcmvbkbc@gmail.com>

--
Thanks.
-- Max


\
 
 \ /
  Last update: 2015-06-27 03:41    [W:0.254 / U:0.028 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site