From mboxrd@z Thu Jan 1 00:00:00 1970 From: vikasm Date: Tue, 25 Nov 2014 12:13:11 -0800 Subject: [U-Boot] [PATCH v4 0/4] add support for new arch stv0991 In-Reply-To: <5474DD69.4070801@st.com> References: <1416274228-19333-1-git-send-email-vikas.manocha@st.com> <546B8D24.20902@st.com> <5474DD69.4070801@st.com> Message-ID: <5474E2D7.8000202@st.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 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