This is the mail archive of the
ecos-discuss@sources.redhat.com
mailing list for the eCos project.
RE: Confirmed error in eth_drv.c
- To: "Trenton D. Adams" <tadams at theone dot dnsalias dot com>
- Subject: RE: [ECOS] Confirmed error in eth_drv.c
- From: Gary Thomas <gthomas at redhat dot com>
- Date: Mon, 20 Aug 2001 11:22:53 -0600 (MDT)
- Cc: eCos Discussion <ecos-discuss at sourceware dot cygnus dot com>
- Organization: Red Hat, Inc.
On 20-Aug-2001 Trenton D. Adams wrote:
> I added things, but never removed anything. So if anything was disabled
> it was due to a conflict. If you want, I can do it again to see what
> happens!
>
No, but I'd like to see your 'ecos.ecc' file. You can send it to me
privately.
> -----Original Message-----
> From: gary@chez-thomas.org [mailto:gary@chez-thomas.org] On Behalf Of
> Gary Thomas
> Sent: Monday, August 20, 2001 11:16 AM
> To: Trenton D. Adams
> Cc: eCos Discussion
> Subject: RE: [ECOS] Confirmed error in eth_drv.c
>
>
>
> On 20-Aug-2001 Trenton D. Adams wrote:
>> I just used default settings when creating a project using Config Tool
>> 2.04.
>>
>
> You changed nothing? E.g. turning on GDB stubs in your code, or taking
> over the virtual vector communications channel.
>
>>
>>
>> -----Original Message-----
>> From: ecos-discuss-owner@sources.redhat.com
>> [mailto:ecos-discuss-owner@sources.redhat.com] On Behalf Of Gary
> Thomas
>> Sent: Monday, August 20, 2001 11:00 AM
>> To: Trenton D. Adams
>> Cc: eCos Discussion
>> Subject: RE: [ECOS] Confirmed error in eth_drv.c
>>
>>
>> Note: this error would only occur if you turn _LOCK_WITH_ROM_MONITOR
>> off,
>> which obviously we don't think is a good idea.
>>
>> I'll change the source to build in this case anyway.
>>
>> On 20-Aug-2001 Trenton D. Adams wrote:
>>> I get the following output from the config tool v2.04
>>>
>>>
>>
> /cygdrive/C/MyDocuments/eCosRepository/ecos/packages/io/eth/current/src/
>>> net/eth_drv.c
>>>
>>
> /cygdrive/C/MyDocuments/eCosRepository/ecos/packages/io/eth/current/src/
>>> net/eth_drv.c: In function `eth_drv_send':
>>>
>>
> /cygdrive/C/MyDocuments/eCosRepository/ecos/packages/io/eth/current/src/
>>> net/eth_drv.c:617: parse error before `static'
>>> make[1]: Leaving directory
>>> `/c/MyDocuments/eCosBuilds/RAMSupport_build/io/eth/current'
>>>
>>
> /cygdrive/C/MyDocuments/eCosRepository/ecos/packages/io/eth/current/src/
>>> net/eth_drv.c:629: `mbuf_key' undeclared (first use in this function)
>>> make: Leaving directory `/c/MyDocuments/eCosBuilds/RAMSupport_build'
>>>
>>
> /cygdrive/C/MyDocuments/eCosRepository/ecos/packages/io/eth/current/src/
>>> net/eth_drv.c:629: (Each undeclared identifier is reported only once
>>>
>>
> /cygdrive/C/MyDocuments/eCosRepository/ecos/packages/io/eth/current/src/
>>> net/eth_drv.c:629: for each function it appears in.)
>>>
>>
> /cygdrive/C/MyDocuments/eCosRepository/ecos/packages/io/eth/current/src/
>>> net/eth_drv.c:504: warning: `m0' might be used uninitialized in this
>>> function
>>>
>>
> /cygdrive/C/MyDocuments/eCosRepository/ecos/packages/io/eth/current/src/
>>> net/eth_drv.c: At top level:
>>>
>>
> /cygdrive/C/MyDocuments/eCosRepository/ecos/packages/io/eth/current/src/
>>> net/eth_drv.c:284: warning: `eth_drv_tx_done' used but never defined
>>> make[1]: *** [src/net/eth_drv.o.d] Error 1
>>>
>>>
>>> It would appear that the #ifdef _LOCK_WITH_ROM_MONITOR begins before
>> the
>>> end of the while loop and ends after the while loop. So, this would
>>> naturally cause a parse error!
>>>
>>> It appears to have been done on 2001-08-14 which is probably why I
>> never
>>> seen it before.
>>>
>>> So, just move the "#endif // _LOCK_WITH_ROM_MONITOR" and "#undef
>>> _LOCK_WITH_ROM_MONITOR" up above the end of the while loop.
>>>
>>> Works for me now! :)
>>>
>>>
>>>#ifdef _LOCK_WITH_ROM_MONITOR
>>> // Unlock the driver & hardware. It can once again be safely
>>> shared.
>>> if (need_lock) {
>>> cyg_drv_dsr_unlock();
>>>#ifdef _LOCK_USING_INTERRUPTS
>>> HAL_RESTORE_INTERRUPTS(ints);
>>>#endif
>>> }
>>> } // this here appears to me to be the end of the while loop
>>>#endif // _LOCK_WITH_ROM_MONITOR
>>>#undef _LOCK_WITH_ROM_MONITOR
>>>
>>>
>>>
>>> Trenton D. Adams
>>> Extreme Engineering
>>>#17, 6025 - 12 St. SE
>>> Calgary, Alberta, Canada
>>> T2H 2K1
>>>
>>> Phone: 403 640 9494 ext-208
>>> Fax: 403 640 9599
>>>
>>> http://www.extremeeng.com