Hi Huan,
On Mon, 22 Sep 2014 06:22:44 +0000, Huan Wang
<alison.w...@freescale.com> wrote:
> Hi, Albert,
>
> > On Thu, 18 Sep 2014 13:47:14 +0800, Alison Wang <b18...@freescale.com>
> > wrote:
> >
> > > @@ -111,6 +115,14 @@ static void pbl_parser(char *name)
> > > size_t len = 0;
> > >
> > > fname = name;
> > > +
> > > + if (strstr(fname, "ls102xa")) {
> > > + next_pbl_cmd = 0x8101a000;
> > > + pbl_cmd_initaddr = 0x8101a000;
> > > + pbi_crc_cmd1 = 0x61;
> > > + pbi_crc_cmd2 = 0;
> > > + }
> >
> > Basing the tool's behaviour on the file name is not a good approach IMO.
> > Can't we add some command line option to the tool to select the right
> > set of values?
>
> [Alison Wang] Yes, it is really not a good approach. How about this command
> line approach?
>
> [copy-paste of a diff follows]
Please do not paste entire diffs in mails to the U-Boot Mailing list
unless you are actually submitting them as a patch. When Patchwork sees
a diff in a mail, it creates a new patch entry. Here, your diff made
Patchwork create a second entry for patch 2/7 at
<http://patchwork.ozlabs.org/patch/391796/>
I have manually marked it as "Not Applicable".
Amicalement,
--
Albert.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
