[Top] [All Lists]

Re: [PATCH v2] Revert "MIPS: make CAC_ADDR and UNCAC_ADDR account for PH

To: Ralf Baechle <>
Subject: Re: [PATCH v2] Revert "MIPS: make CAC_ADDR and UNCAC_ADDR account for PHYS_OFFSET"
From: Markos Chandras <>
Date: Wed, 3 Jul 2013 15:01:05 +0100
Cc: "Steven J. Hill" <>,, Leonid Yegoshin <>, Florian Fainelli <>
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=20120113; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc:content-type; bh=p6tzu1pwq9Qyveu7YGkwgh2MqdEwEajk5JGXWgSUUL8=; b=y+Hi5iuD5kYMxDyidNFcicayKXhT6G9/dG0Jd5VkD9KsmQcLCfYnPhaJ+SIrIR8lvJ J62eEVVMyEMEMjQDQdjjz3IgzFtwJw5DwuPhvTyPr8KTBK6lxvAJbqksekNUmef6mamA Cdi5bP5R1XHaztQsfFwlI4EkcA9emns2AvxLbFkWGTS87KXswZ1evqUAolSdLuW+Zdsi 8dIIbhlp/nI7OElj9FGcxQAuxJga00N2dIuCJIUtdM2DWGlJezy8XuBo0h1bcHfVaR1S p4hJbtqXeiuHOfrCcZK4x/1YtDM2tdxBJfuYJhvjISoJ60OyAsOO8WgtqCYMCMZ/1Fnu g/qA==
In-reply-to: <>
List-archive: <>
List-help: <>
List-id: linux-mips <>
List-owner: <>
List-post: <>
List-software: Ecartis version 1.0.0
List-subscribe: <>
List-unsubscribe: <>
Original-recipient: rfc822;
References: <> <>
On 26 June 2013 15:52, Ralf Baechle <> wrote:
> On Thu, Jun 20, 2013 at 10:36:30AM -0500, Steven J. Hill wrote:
>> From: Leonid Yegoshin <>
>> This reverts commit 3f4579252aa166641861a64f1c2883365ca126c2. It is
>> invalid because the macros CAC_ADDR and UNCAC_ADDR have a kernel
>> virtual address as an argument and also returns a kernel virtual
>> address. Using and physical address PHYS_OFFSET is blatantly wrong
>> for a macro common to multiple platforms.
> While the patch itself is looking sane at a glance, I'm wondering if this
> is fixing any actual bug or is just the result of a code review?
>   Ralf

I am afraid this commit[1] broke the build in
upstream-sfr/mips-for-linux-next with errors like this

arch/mips/include/asm/mach-generic/spaces.h:29:0: warning:
"UNCAC_BASE" redefined [enabled by default]
In file included from arch/mips/include/asm/addrspace.h:13:0,
                 from arch/mips/include/asm/barrier.h:11,
                 from arch/mips/include/asm/bitops.h:18,
                 from include/linux/bitops.h:22,
                 from include/linux/kernel.h:10,
                 from include/asm-generic/bug.h:13,
                 from arch/mips/include/asm/bug.h:41,
                 from include/linux/bug.h:4,
                 from include/linux/page-flags.h:9,
                 from kernel/bounds.c:9:
arch/mips/include/asm/mach-ar7/spaces.h:20:0: note: this is the
location of the previous definition


Markos Chandras

<Prev in Thread] Current Thread [Next in Thread>