public inbox for ecos-discuss@sourceware.org
 help / color / mirror / Atom feed
From: "Trenton D. Adams" <tadams@theone.dnsalias.com>
To: <bartv@redhat.com>
Cc: <ecos-discuss@sourceware.cygnus.com>
Subject: RE: [ECOS] ethernet driver package idea
Date: Wed, 29 Aug 2001 10:18:00 -0000	[thread overview]
Message-ID: <000f01c130ae$747a0100$090110ac@TRENT> (raw)
In-Reply-To: <002101c12cb4$0568e360$090110ac@TRENT>

One more thing.  You might want to do what I've done below.  It's a
little nicer using a drop down box than a text box for selection of the
ethX device name!

	cdl_option CYGDAT_ETHDRV_NAME {
	    display       "Name to use for this network device"
	    flavor        data
          legal_values {"\"eth0\"" "\"eth1\""}
	    default_value { (1 == CYGHWR_NET_DRIVERS) ? "\"eth0\"" :
"\"eth1\"" }
	    description "
	        The name of this network device for control purposes.
	    "
	}


-----Original Message-----
From: ecos-discuss-owner@sources.redhat.com
[ mailto:ecos-discuss-owner@sources.redhat.com ] On Behalf Of Trenton D.
Adams
Sent: Friday, August 24, 2001 9:47 AM
To: bartv@redhat.com
Cc: ecos-discuss@sourceware.cygnus.com
Subject: RE: [ECOS] ethernet driver package idea


We are going to have two interfaces on our board.  The cs8900 and the
Wavelan PC card.  So, this option is helpful because I can change them
to any device I want without going and editing the CDL and source.

The new CDL is attached if you want to incorporate it.

ETH_DRV_SC(edb7xxx_sc,
           &_cs8900_priv_data, // Driver specific data
           CYGDAT_ETHDRV_NAME,             // Name for this interface
           cs8900_start,
           cs8900_stop,
           cs8900_control,
           cs8900_can_send,
           cs8900_send,
           cs8900_recv,
           cs8900_deliver,     // "pseudoDSR" called from fast net
thread
           cs8900_int,         // poll function, encapsulates ISR and
DSR
           cs8900_int_vector);

-----Original Message-----
From: ecos-discuss-owner@sources.redhat.com
[ mailto:ecos-discuss-owner@sources.redhat.com ] On Behalf Of Bart Veer
Sent: Friday, August 24, 2001 9:42 AM
To: tadams@theone.dnsalias.com
Cc: ecos-discuss@sourceware.cygnus.com
Subject: Re: [ECOS] ethernet driver package idea


>>>>> "Trenton" == Trenton D Adams <tadams@theone.dnsalias.com> writes:

    Trenton> But this is only done on the USB interface then?

When an ethernet driver is likely to be the only one present on a
given target then it can just default to eth0 - in most circumstances
that will work just fine. For example, if the processor has a single
on-chip ethernet device then usually that will be the only ethernet
device in the target system.

The USB ethernet device is different, in that it has no way of
guessing whether or not another ethernet device is present. Hence it
has more complicated CDL which makes sure that the right thing
happens: if it provides the only ethernet interface in the system then
it will use eth0; if there is a real ethernet device then that can
stay as eth0 and the USB-ethernet will become eth1.

For something like a driver for a PCMCIA plugin card, much the same
applies. That driver has no way of knowing whether or not another
ethernet device is present on the target hardware, so it should adapt. 

    Trenton> Doing it the way I did it wouldn't require an update of
    Trenton> the config tool though! Also, do you have the new tool
    Trenton> ready for download? I personally don't have the time at
    Trenton> the moment to install wxWindows, and then recompile the
    Trenton> tool. If anything goes wrong, it's wasted my employer's
    Trenton> money, and I don't want to do that.

    Trenton> Besides, yours seems to work with the Windows GUI config
    Trenton> tool 2.04. I can't be sure because I don't have two
    Trenton> ethernet interfaces to check it with. Is the conflict
    Trenton> resolution thing the only problem with it?

I do not know offhand whether 2.04 includes the required libcdl
changes, that is why I included the libcdl ChangeLog entry. If 2.04
was built before 2001-07-10 then it cannot have the required changes.
If it was built after 2001-07-10 then it almost certainly does have
the required changes. Take a look at Help->About to find out when the
configtool was built. The libcdl changes were specifically to cope
with targets that already have an ethernet interface and you are now
adding a second one such as USB-ethernet.

Bart

  reply	other threads:[~2001-08-29 10:18 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-08-23 11:41 Trenton D. Adams
2001-08-23 12:24 ` Trenton D. Adams
2001-08-23 13:30   ` [ECOS] CS8900 ethernet driver PATCH Trenton D. Adams
2001-08-24  4:22   ` [ECOS] ethernet driver package idea Bart Veer
2001-08-24  7:59     ` Trenton D. Adams
2001-08-24  8:42       ` Bart Veer
2001-08-24  8:47         ` Trenton D. Adams
2001-08-29 10:18           ` Trenton D. Adams [this message]
2001-09-03 11:01             ` Bart Veer
2001-09-06 12:20               ` Jonathan Larmour
2001-09-11 10:58                 ` Hugo Tyson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='000f01c130ae$747a0100$090110ac@TRENT' \
    --to=tadams@theone.dnsalias.com \
    --cc=bartv@redhat.com \
    --cc=ecos-discuss@sourceware.cygnus.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).