netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Joao Pinto <Joao.Pinto@synopsys.com>
To: Alexandre Torgue <alexandre.torgue@st.com>,
	Joao Pinto <Joao.Pinto@synopsys.com>,
	Florian Fainelli <f.fainelli@gmail.com>, <davem@davemloft.net>
Cc: <lars.persson@axis.com>, <niklass@axis.com>,
	<peppe.cavallaro@st.com>, <netdev@vger.kernel.org>
Subject: Re: [PATCH v2 2/2] stmmac: rename it to synopsys
Date: Thu, 12 Jan 2017 11:09:47 +0000	[thread overview]
Message-ID: <dbb59d07-4ab4-37d2-5e70-073091749445@synopsys.com> (raw)
In-Reply-To: <57d96ccb-7fd4-0707-4902-a5bdd2a6ef39@st.com>

Hi Alex, good morning!

Às 10:11 AM de 1/12/2017, Alexandre Torgue escreveu:
>>
>> Lets not name it synopsys, for me it is totally fine, but naming it
>> stmicro/stmmac is not the right way because it seems like it is a driver just
>> for stmicro products, which is not, is for products that use Designware Ethernet
>> IPs.
>>
>> I am volunteering to do this work, let's discuss this.
> 
> For me it makes no sens to rename only folder (stmicro/stmmac by synopsys) and
> keep stmmac* inside a synopsys folder (that is very confusing). If you propose
> that you have to change all.
> 
> BUT doing that, we will lose all stmmac driver story and we don't want that.

Totally understand your point. Do you agree on this approach?
rename "stmicro" to "dwc" (designware controllers) and leave stmmac as it is
today. This small change is enough in my point of view and sole the problems you
refer. We would have net/ethernet/dwc/stmmac/.

I can also rename the dwmac4 files and functions to eqos, since soon we will
have a new eqos version.

dwmac4.h -> eqos.h
dwmac4_core.c -> eqos_core.c
dwmac4_descs.c -> eqos_descs.c
dwmac4_descs.h -> eqos_descs.h
dwmac4_dma.c -> eqos_dma.c
dwmac4_dma.h -> eqos_dma.h
dwmac4_lib.c -> eqos_lib.c

What do you think about this approach?

Thanks,
Joao

> 
> 
> 
>>
>> Thanks,
>> Joao
>>
>>

  reply	other threads:[~2017-01-12 11:14 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-01-10 14:52 [PATCH v2 0/2] remove dwc_eth_qos and rename stmicro/stmmac Joao Pinto
2017-01-10 14:52 ` [PATCH v2 1/2] synopsys: remove dwc_eth_qos driver Joao Pinto
2017-01-10 15:15   ` Lars Persson
2017-01-10 14:52 ` [PATCH v2 2/2] stmmac: rename it to synopsys Joao Pinto
2017-01-11  4:00   ` Jie Deng
2017-01-11 10:35     ` Joao Pinto
2017-01-12  3:44       ` Jie Deng
2017-01-12  5:05         ` Florian Fainelli
2017-01-11 10:00   ` kbuild test robot
2017-01-11 21:14   ` Florian Fainelli
2017-01-11 21:32     ` David Miller
2017-01-12  9:43     ` Joao Pinto
2017-01-12 10:11       ` Alexandre Torgue
2017-01-12 11:09         ` Joao Pinto [this message]
2017-01-12 15:26           ` David Miller
2017-01-12 15:39             ` Joao Pinto
2017-01-12 15:45               ` David Miller
2017-01-12 16:04                 ` Joao Pinto
2017-01-12 19:42       ` Florian Fainelli
2017-01-13 11:20         ` Joao Pinto
2017-01-11 11:39 ` [PATCH v2 0/2] remove dwc_eth_qos and rename stmicro/stmmac Alexandre Torgue
2017-01-11 11:54   ` Joao Pinto
2017-01-11 14:25     ` Alexandre Torgue
2017-01-11 14:29       ` Joao Pinto
2017-01-11 14:39         ` Alexandre Torgue
2017-01-11 14:54           ` Joao Pinto
2017-01-11 14:52   ` David Miller
2017-01-11 15:06     ` Joao Pinto
2017-01-11 15:20       ` David Miller
2017-01-11 16:17         ` Joao Pinto
2017-01-11 16:19         ` Joao Pinto

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=dbb59d07-4ab4-37d2-5e70-073091749445@synopsys.com \
    --to=joao.pinto@synopsys.com \
    --cc=alexandre.torgue@st.com \
    --cc=davem@davemloft.net \
    --cc=f.fainelli@gmail.com \
    --cc=lars.persson@axis.com \
    --cc=netdev@vger.kernel.org \
    --cc=niklass@axis.com \
    --cc=peppe.cavallaro@st.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).