[Top] [All Lists]

Re: [PATCH] MIPS: Fix build failure for IP22

To: Ralf Baechle <>
Subject: Re: [PATCH] MIPS: Fix build failure for IP22
From: Dmitri Vorobiev <>
Date: Thu, 9 Dec 2010 22:34:32 +0200
Cc: Dmitri Vorobiev <>,
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=gamma; h=domainkey-signature:mime-version:received:received:in-reply-to :references:date:message-id:subject:from:to:cc:content-type :content-transfer-encoding; bh=nGrAPdr9zO5+lpeXfFXukK8zXPM99ZRbVxk3flXSxss=; b=rD1szn7ksItnZKIrYcZOJGCoL+UMpHxXtKxFsshPZ87aoCIJDqn8+p28E6yloSaobo JDw7fSJ4h5TUlCplHGbmqIKyCYMMeeQR4m4J4nP7uGkJgrQi9KEixHVG5CzsUQmxZUp4 pXhZJa8EbuvR6h3b2MjFh/WN6VX1nZBIVSnHo=
Domainkey-signature: a=rsa-sha1; c=nofws;; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; b=NzurbfGhJlCWBwmvjbcowXw8tSJ4V2Dm/uSR76pDA02/vpGNZeX79y0SeSoA1uswrj ltqMl8ima0HGA97Tv8WgDmm3cNZkWX3yvo3APYDSAP+luJJTT8GqRx17IqZL8iIO4R4+ j+K7NT6oadcWcwXTyUwGTfFhNaJU+9v9tFDQA=
In-reply-to: <>
Original-recipient: rfc822;
References: <> <>
On Thu, Dec 9, 2010 at 8:46 PM, Ralf Baechle <> wrote:
> On Tue, Nov 23, 2010 at 01:32:28PM +0200, Dmitri Vorobiev wrote:
>> Commit 48e1fd5a81416a037f5a48120bf281102f2584e2 changed the name
>> of the MIPS-specific dma_cache_sync() routine by prefixing it with
>> `mips_', and removed the export for its symbol. Two drivers, which
>> did use dma_cache_sync(), namely, sgiseeq and sgiwd93, were not
>> converted to use the new function, which led to build failure for
>> the IP22 platform.
>> This patch fixes the build failure by fixing the call sites of
>> mips_dma_cache_sync() and exporting the symbol for this routine as
>> a GPL symbol. While at it, some minor changes to improve Kconfig
>> help entries were done.
> dma_cache_sync isn't MIPS specific - mips_dma_cache_sync is but the
> function wasn't renamed everywhere.  Looking into what went wrong now
> but your patch surely is not correct.

OK, the main thing would be that the build failure be fixed. Thanks
for looking into that.


>  Ralf

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