Re: [avrdude-dev] [bug #30498] Add support for new avr devices (atmega324pa)

[jo]

I think it's a good idea, smaller conf file and fewer things to do when 
adding a new family of devices.
But I have to ask if someone can do the change in the code below as I'm 
not good enough to do it myself. (I'm helping in the part definition as 
it is (almost) the only things I can be usefull at)

I'll post another version of my conf using the alias field.


Le 19/08/2011 12:16, Joerg Wunsch a écrit :
> Right now, the search for a particular device match is done in
> avrpart.c based on their "id" or "desc" field:
>
>    for (ln1=lfirst(parts); ln1&&  !found; ln1=lnext(ln1)) {
>      p = ldata(ln1);
>      if ((strcasecmp(partdesc, p->id) == 0) ||
>          (strcasecmp(partdesc, p->desc) == 0))
>        found = 1;
>    }
>
> My suggestion is to add an "alias" field to the config syntax, which
> by itself is threated as a list of aliases, and is also searched.
> That way, devices that are completely identical to other devices
> (including their signature) can be mentioned that way, like:
>
> part
>      id               = "m88p";
>      desc             = "ATMEGA88P";
>      alias            = "ATMEGA88PA", "ATMEGA88PV";
>
> (In case someone would want to be diligent enough, we could add all
> the different suffix variants like "V" or "L" that way.)
>
> I don't think there's a point in adding more abbreviations (the "id"
> field) for the "A" devices: the abbreviation is only useful for humans
> to save some typing work, so it makes sense to keep them at the
> shortest string possible that will select a particular entry.
>
> Opinions?