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

[Simon Glass]

Hi Vikas,

On 25 November 2014 at 12:50, vikasm <vikas.manocha at st.com> 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 at 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.

Yes thanks, the patch is here:

http://patchwork.ozlabs.org/patch/414224/

Unfortunate that my test platform had RAM at 0!

Regards,
Simon