[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <20121109.224909.1450759513114986795.davem@davemloft.net>
Date: Fri, 09 Nov 2012 22:49:09 -0500 (EST)
From: David Miller <davem@...emloft.net>
To: andreas@...sler.com
Cc: sam@...nborg.org, sparclinux@...r.kernel.org, software@...sler.com,
grant.likely@...retlab.ca, rob.herring@...xeda.com,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] of/address: sparc: Declare of_address_to_resource()
as an extern function for sparc again
From: Andreas Larsson <andreas@...sler.com>
Date: Mon, 5 Nov 2012 11:53:18 +0100
> This bug-fix makes sure that of_address_to_resource is defined extern for sparc
> so that the sparc-specific implementation of of_address_to_resource() is once
> again used when including include/linux/of_address.h in a sparc context. A
> number of drivers in mainline relies on this function working for sparc.
>
> The bug was introduced in a850a7554442f08d3e910c6eeb4ee216868dda1e, "of/address:
> add empty static inlines for !CONFIG_OF". Contrary to that commit title, the
> static inlines are added for !CONFIG_OF_ADRESS, and CONFIG_OF_ADRESS is never
> defined for sparc. This is good behavior for the other functions in
> include/linux/of_address.h, as the extern functions defined in
> drivers/of/address.c only gets linked when OF_ADDRESS is configured. However,
> for of_address_to_resource is that there exists a sparc-specific implementation
> in arch/sparc/arch/sparc/kernel/of_device_common.c
>
> Signed-off-by: Andreas Larsson <andreas@...sler.com>
Applied.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists