network configuration was: [PATCH 4/4] RTEMS_USE_LWIP option to use lwip in RTEMS

[Chris Johns]

On 21/4/21 4:17 pm, Sebastian Huber wrote:
> On 20/04/2021 19:37, Gedare Bloom wrote:
> 
>> On Tue, Apr 20, 2021 at 11:09 AM Sebastian Huber
>> <sebastian.huber at embedded-brains.de> wrote:
>>> On 12/04/2021 11:56, Robin.Mueller wrote:
>>>
>>>> This adds a configuration option if lwIP is used instead of libbsd.
>>>> It works by specifying RTEMS_USE_LWIP in the config.ini file.
>>> The RTEMS_* namespace is reserved for RTEMS API parts. Since this is
>>> BSP-specific, I suggest to rename this option to
>>> STM32H7_ENABLE_LWIP_SUPPORT.
>>>
>> [...]
>>
>> This will obviate any need for BSP-specific configuration variables
>> for networking stacks, they can just inherit from the top-level config
>> option.
> 
> For this particular BSP, the option changes the behaviour of the HAL Ethernet
> support code. STM ships lwIP support so it seems that the HAL was written to
> support this. However, the libbsd Ethernet driver uses a different DMA
> descriptor handling which better fits for the FreeBSD interfaces. This is a
> low-level issue.
> 
> I don't like this option. I think it would be better to just provide functions
> for both network stacks. 

+1

> However, I don't have time for this at the moment.

Oh come on ... sleep is overrated! :)

Chris