Subject: [PATCH 43/74] ST SPEAr : EMI (Extrenal Memory Interface) controller driver

From: [email-masked] (Vipin Kumar)
Date: Fri, 10 Sep 2010 13:41:22 +0530

On 9/7/2010 7:02 PM, Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 17:24 Tue 07 Sep     , viresh kumar wrote:
>> On 9/7/2010 5:08 PM, Jean-Christophe PLAGNIOL-VILLARD wrote:
>>>>>> +/* emi nor flash device registeration */
>>>>>>>>>>> +static struct physmap_flash_data emi_norflash_data;
>>>>>>>>>>> +
>>>>>>>>>>> +static struct resource emi_nor_resources[] = {
>>>>>>>>>>> +     {
>>>>>>>>>>> +             .start  = SPEAR310_EMI_MEM_0_BASE,
>>>>>>>>>>> +             .end    = SPEAR310_EMI_MEM_0_BASE + SPEAR310_EMI_MEM_SIZE - 1,
>>>>>>>>>>> +             .flags  = IORESOURCE_MEM,
>>>>>>>>>>> +     },
>>>>>>>>>>> +};
>>>>>>> it's board specfic not mach
>>>>>>>
>>>>>>> NACK
>>>>>
>>>>> No. This is machine specific. Same for all boards.
>>> NACK as you can have a flash at other place and more than one flash
>>> and the size depend on the flash
>>
>> Correct!! Size has to be board specific. Will modify it.
> and the start also as you may have 2 nor flash on different bank
> 

The start addresses in our case are fixed for a particular bank.

Regards
Vipin

> Best Regards,
> J.
> .