[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From: jo
Subject: Re: [avrdude-dev] [bug #30498] Add support for new avr devices (atmega324pa)
Date: Fri, 19 Aug 2011 14:04:04 +0200
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv: Gecko/20110617 Thunderbird/3.1.11

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:

     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.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]