[PATCH v1 1/2] net: cortina_ni: Addd eth support for Cortina Access CAxxxx SoCs

Alex Nemirovsky Alex.Nemirovsky at cortina-access.com
Wed Jun 3 03:35:39 CEST 2020


Abbie/Aaron,
please remove legacy mode code so that we only support DM mode per Tom’s request.
Thanks
-AN

> On Jun 2, 2020, at 5:38 PM, Tom Rini <trini at konsulko.com> wrote:
> 
> On Wed, Jun 03, 2020 at 12:18:18AM +0000, Alex Nemirovsky wrote:
>> Hi Tom,
>> 
>>> On Jun 2, 2020, at 6:02 AM, Tom Rini <trini at konsulko.com> wrote:
>>> 
>>> On Mon, Jun 01, 2020 at 07:44:25PM -0700, Alex Nemirovsky wrote:
>>> 
>>>> From: Aaron Tseng <aaron.tseng at cortina-access.com>
>>>> 
>>>> Add Cortina Access Ethernet device driver for CAxxxx SoCs.
>>>> This driver supports both legacy and DM_ETH network models.
>>>> 
>>>> Signed-off-by: Aaron Tseng <aaron.tseng at cortina-access.com>
>>>> Signed-off-by: Alex Nemirovsky <alex.nemirovsky at cortina-access.com>
>>>> 
>>>> CC: Joe Hershberger <joe.hershberger at ni.com>
>>>> CC: Abbie Chang <abbie.chang at Cortina-Access.com>
>>>> CC: Tom Rini <trini at konsulko.com>
>>> [snip]
>>>> +#define HEADER_A_SIZE	8
>>>> +#define CORTINA_NI_DBG 1
>>>> +/*define CORTINA_NI_DBG if individual rx,tx,init needs to be called */
>>> 
>>> We have pr_debug, etc, please use those and not a custom debug defiine.
>>> 
>>>> +#ifdef CONFIG_DM_ETH
>>>> +#if CORTINA_NI_DBG
>>>> +static struct udevice	*dbg_dev;
>>>> +#endif
>>>> +static struct udevice   *curr_dev;
>>>> +#else
>>> 
>>> The deadline for drivers to convert to DM for ethernet is v2020.07.
>>> Please rework to not introduce any legacy mode support.  Thanks!
>> 
>> For clarity, we are providing BOTH DM and Legacy mode support in our driver.
>> Is the request to REMOVE Legacy mode support, even though DM support is already provided to meet the 2020.07 requirement?
> 
> Yes, remove the legacy code as there's no non-legacy users in upstream
> to support.  Thanks!
> 
> -- 
> Tom



More information about the U-Boot mailing list