Re: [U-Boot] [PATCH 1/5] vybrid: add vybrid CPU support

2013-05-07 Thread Wang Huan-B18965
Hi, Liu Hui, >-Original Message- >From: Wang Huan-B18965 >Sent: Sunday, April 28, 2013 12:48 PM >To: Liu Hui-R64343; u-boot@lists.denx.de >Cc: sba...@denx.de; Estevam Fabio-R49496; TsiChung Liew; Jin Zhengxiong- >R64188 >Subject: RE: [PATCH 1/5] vybrid: add vybrid CPU support > >Hi, Liu H

Re: [U-Boot] [PATCH 1/5] vybrid: add vybrid CPU support

2013-04-28 Thread Stefano Babic
On 28/04/2013 12:36, Wang Huan-B18965 wrote: >> >> I would like to understand there common code should be put. As you >> can see, for i.MX there is a imx_common directory that is valid >> across ARM-Core. > [Alison Wang] I'm trying to align the soc with i.mx platform and will > use i.MX's imx_com

Re: [U-Boot] [PATCH 1/5] vybrid: add vybrid CPU support

2013-04-13 Thread wang alison
Hi, Fabio, On Fri, Apr 12, 2013 at 3:53 AM, Alison Wang wrote: > +static char *get_reset_cause(void) > +{ > + char *resetcause[32] = {"POR", > + "Cortex A5 WDOG Timer Reset", > + 0, You mix strings and an integer zero here. Shou

Re: [U-Boot] [PATCH 1/5] vybrid: add vybrid CPU support

2013-04-12 Thread Fabio Estevam
On Fri, Apr 12, 2013 at 3:53 AM, Alison Wang wrote: > +static char *get_reset_cause(void) > +{ > + char *resetcause[32] = {"POR", > + "Cortex A5 WDOG Timer Reset", > + 0, You mix strings and an integer zero here. Shouldn't you rep