#1833 closed defect (fixed)

ne2000 driver does not accept broadcasts; need to fix endianness, too

Reported by: strauman Owned by: Joel Sherrill
Priority: normal Milestone: 4.11
Component: bsps Version: 4.10
Severity: normal Keywords:
Cc: Blocked By:
Blocking:

Description

A recent change which added multicast support to the ne2000 driver introduced a bug where reception of broadcasts was always disabled.

Also, the ne2000 driver only worked on little-endian CPUs.

Both (unrelated) issues are fixed by the same patch (sorry, I just don't have
the time to file 2 separate reports).

NOTE: the multicast support apparently was added on rtems-4-9-branch, rtems-4-10-branch and the HEAD. Hence, all these three branches need to be patched.

(for starters: the file c/src/lib/libbsp/i386/pc386/ne2000/ne2000.c needs to be patched; the driver should really be in libchip. With the endianness bug fixed it should work on big-endian CPUs, too (tested under prep-ppc-qemu).

Attachments (1)

ne2000_bc_and_endian_fix.diff (2.4 KB) - added by strauman on Jul 14, 2011 at 1:19:01 AM.
fix bug that disabled broadcast reception; also fix another (endianness) bug

Download all attachments as: .zip

Change History (2)

Changed on Jul 14, 2011 at 1:19:01 AM by strauman

fix bug that disabled broadcast reception; also fix another (endianness) bug

comment:1 Changed on Jul 14, 2011 at 1:49:57 PM by Joel Sherrill

Resolution: fixed
Status: newclosed

Really closing. Address moving to libcpu on head separately. Doesn't require a PR.

Note: See TracTickets for help on using tickets.