Re: [RFC PATCH] net: stmicro: eQOS IP Core

[Joao Pinto]

Às 4:20 PM de 1/20/2017, David Miller escreveu:
> From: Joao Pinto <Joao.Pinto@xxxxxxxxxxxx>
> Date: Fri, 20 Jan 2017 12:27:33 +0000
> 
>> For historical reasons the Designware eQOS IP Core started in version 4.x,
>> instead of starting in 1.x. This caused some misunderstanding, which resulted
>> in calling this IP as GMAC4. If you go to Synopsys website and check the
>> Ethernet portfolio there is no GMAC4 IP:
>> https://www.synopsys.com/designware-ip/interface-ip/ethernet.html
>>
>> There is a new version of eQOS comming out (5.x) that will have the same
>> mac, dma registers and flow, with an extra set of registers to enable
>> features like TSN. The creation of a dwmac5 or gmac5 would be wrong, and
>> including those new features related to eQOS 5.x in dwmac4 would also not
>> be very correct.
>>
>> This patch suggests to rename dwmac4_* driver to eqos_*, turning it the
>> official driver for eQOS IP. The registers definitions are also now
>> called QOS_ instead of GMAC4_.
>>
>> For more information about eQOS IP please check:
>> https://www.synopsys.com/dw/ipdir.php?ds=dwc_ether_qos
>>
>> Signed-off-by: Joao Pinto <jpinto@xxxxxxxxxxxx>
> 
> Sorry, you're not renaming every function in the driver that meets this
> qualification.
> 
> I understand you want to work hard on the driver, but part of that is
> _accepting_ things how they have been for years and making code
> improvements and additions rather than messing around playing
> rename-this/rename-that games until it looks how you want it to.
> 
> Please stop this charade of renaming, I'm getting really tired of it and
> I think you really have no concept what kind of effect it has on people
> who have to review, apply, and backport this code.
> 
> Kconfig text changes and clarifications, that's fine.  But all of these
> function and variable name changes to the driver, and comment sweeps, no
> it is not acceptable to me.
> 

Hello David.

Charade of renames? It is a bit strong, don't you agree?

So you suggest to keep the driver called as dwmac4 when the IP is in fact eQOS?
Where do you suggest to put the eQOS 5.x features?

Thanks,
Joao