[Bug 1001219] Ethernet driver for STM32 connectivity line with port on MMstm32f107 board.

[bugzilla-daemon at bugs dot ecos dot sourceware dot org]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001219

--- Comment #27 from Ilija Kocho <ilijak@siva.com.mk> 2011-12-21 09:33:14 GMT ---
Created an attachment (id=1475)
 --> (http://bugs.ecos.sourceware.org/attachment.cgi?id=1475)
CDL Snippet: Eth/lwIP Hardware TCP/IP checksums

Hi Jerzy

Thank you for update.

I I haven't reviewed complete code yet, but I tried HW checksum gen/check part.
The behavior of HW checksum CDL is not as expected:
    - Following addition of lwIP and Common Ethernet Support HW checksum
remains unselected. It seems that configtool conflict resolution in this case
produces an opposite result than what we want.
    - When HW checksums are manually selected lwIP checksums are cleared as
they should but the opposite action is missing:
      - Unselect of HW checksum does not re-activate lwIP checksums;
      - Activation of some lwIP checksum de-selects HW checksum, but other lwIP
checksums remain unselected.

Attached CDL snippet should provide correct behavior for both. It may look
complex, but it seem there are some aspects of configtool's conflict resolution
machine that we have to circumvent somehow.

The submitted code is not fully tested, but stems from code that works for
other hardware so I expect it to be close to what we need for this driver.
Following test cases should be executed as a minimum:
    - eCos
        - lwIP default settings (should use HW checksum)
        - lwIP HW checksum disabled (should use lwIP checksum)
    - RedBoot - HW checksum should be inactive -> SW checksum.

It would be also good to test BSDstack but I guess that memory resources are
too low.

I'm looking forward to some feedback.

Ilija

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.