[Top] [All Lists]

Re: [RFC][PATCH 01/10] bcma: Use array to store cores.

To: Michael Büsch <>
Subject: Re: [RFC][PATCH 01/10] bcma: Use array to store cores.
From: Hauke Mehrtens <>
Date: Sun, 12 Jun 2011 00:33:33 +0200
Cc: Rafał Miłecki <>, Arend van Spriel <>, Arnd Bergmann <>, George Kashperko <>, Greg KH <>, "" <>, "" <>, "" <>, "" <>, "" <>
In-reply-to: <20110608102001.294a4ff2@maggie>
References: <> <> <> <> <> <> <> <20110608102001.294a4ff2@maggie>
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv: Gecko/20110516 Lightning/1.0b2 Thunderbird/3.1.10
On 06/08/2011 10:20 AM, Michael Büsch wrote:
> On Wed, 8 Jun 2011 02:06:11 +0200
> Rafał Miłecki <> wrote:
>> Because full scanning needs one of the following:
>> 1) Working alloc - not possible for SoCs
> Isn't there a bootmem allocator available on MIPS?

The bootmem allocator is working on mips, but it is initialized after
plat_mem_setup() was called. To use it we have to move the start of bcma
to some other place in the bcm47xx code.

We need access to the common and mips core for different functions in
the bcm47xx code and these functions are getting called by the mips
code, so we can not store these struct bcma_devices on the stack.

I would use this struct on the embedded device and use it in the text
segment of the bcm47xx code.

In include/linux/bcma/bcma.h:
struct bcma_soc {
        struct bcma_bus bus;
        struct bcma_device core_cc;
        struct bcma_device core_mips;

In arch/mips/bcm47xx/setup.c
struct bcma_soc bus;

The chipcommon and mips core can be initilized early without the need
use of any alloc. The bcm47xx code will call
bcma_bus_early_register(struct bcma_soc *soc) and this code will find
these two cores, add then to the list of cores in bcma_bus and run the
init code for them. After that we have all we need to boot up the
device. After the kernel page allocator is fully set up we would search
for all the other cores and add them to the list of cores and do the
initialization for them. The two cores in struct bcma_soc will never be
accessed directly but only through struct bcma_bus so that there is no
difference from early boot and normal mode.


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