[U-Boot] [PATCH v4 0/4] add support for new arch stv0991

[vikasm <vikas.manocha@st.com>]

Hi Simon,

On 11/25/2014 11:50 AM, vikasm wrote:
> Hello Simon,
>
> On 11/18/2014 10:17 AM, vikasm wrote:
>> Hello Simon,
>>
>> On 11/17/2014 09:19 PM, Simon Glass wrote:
>>> Hi Vikas,
>>>
>>> On 18 November 2014 01:30, Vikas Manocha <vikas.manocha@st.com> wrote:
>>>> This patchset add support for new arch stv0991.
>>>>
>>>> Changes in v4:
>>>> - rebased to current master
>>>> - added maintainers info
>>>>
>>>> Changes in v3:
>>>> - removed period from commit messages
>>>>
>>>> Changes in v2:
>>>> - corrected files license to GPL-2.0+
>>>> - replaced printf() usage with puts() for string print
>>>> - sorted sourcing of board Kconfig
>>>>
>>>> Vikas Manocha (4):
>>>>    stv0991: Add basic stv0991 architecture support
>>>>    stv0991: enable ethernet support
>>>>    stv0991: default + misc command configs enabled
>>>>    stv0991: add maintainer information
>>> Can you please look at using driver model for this board? It would be
>>> particularly valuable since it looks like you are using the pl011 UART
>>> which has not previously been tested with driver model (and it seems
>>> you found some problems).
>>>
>>> To do this you probably only need to adjust a few defines and add
>>> platform data for serial. A patch for Raspberry Pi is here:
>>>
>>> http://patchwork.ozlabs.org/patch/392180/
>>
>> Sure, Simon. Infact i was also planning to use the driver model.
>> Thanks for sharing the Raspberry patch link.
>
> I tried using driver model but got into issue after relocation, 
> figured out that it was because 0x0 location
> was being overwritten & pl011 base address was being stored here...
> After tracking "Raspberry Pi with driver model" discussion, seems like 
> you are aware of it now, just wanted
> to provide this information on one more platform.

& with ".priv_auto_alloc_size = sizeof(struct pl01x_priv)," addition, 
the issue is fixed.

>
> Rgds,
> Vikas

Rgds,
Vikas