Re: [PATCH 00/17] net: stmmac: misc fix

From	Joao Pinto <Joao.Pinto@synopsys.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH 00/17] net: stmmac: misc fix
Date	2017-01-31 11:50 +0100
Message-ID	<t5EaD-84y-37@gated-at.bofh.it> (permalink)
References	<t5CLw-7mj-19@gated-at.bofh.it> <t5DxV-7RQ-43@gated-at.bofh.it> <t5E0W-81e-19@gated-at.bofh.it> <t5E0W-81e-35@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Às 10:33 AM de 1/31/2017, Giuseppe CAVALLARO escreveu:
> On 1/31/2017 11:23 AM, Joao Pinto wrote:
>>
>> Hi Peppe,
>>
>> Às 10:00 AM de 1/31/2017, Giuseppe CAVALLARO escreveu:
>>> Hello Corentin
>>>
>>> On 1/31/2017 10:11 AM, Corentin Labbe wrote:
>>>> Hello
>>>>
>>>> I am currently working on dwmac-sun8i glue driver for Allwinner H3/A83T/A64.
>>>> This serie is the result of all minor problem found in the stmmac driver.
>>>
>>> thank for this effort, many changes are to tidy up some part of the code
>>> so  I wonder if these should stay on top of net-next.
>>> There is on-going a thread to change something in the stmmac (where I
>>> missed more details and I tried to refresh all asap); so added on copy
>>> Joao.
>>
>> I submitted 2 suggestions that would rename folder stmicro to dwc and dwmac4*
>> files to eqos. The second one is important, since there is no dwmc4/gmac4 core,
>> in fact it is called eqos that will have 5.00 version very soon. Both
>> suggestions were rejected and so I turned to developing features.
>> I am available to discuss with you any improvement on these aspects if you think
>> it makes sense.
>>
>> Currently I am developing the support for multi-queues / multi-channels which
>> will enable in the future stmmac to be used in QAv and TSN applications.
> 
> this is a really good point, so I am happy that you are looking for this
> supports that can stay on high priority and do not generate conflicts
> with this patch series.
> 
> We will reopen the renaming story later... I hope to be more reactive
> on that and sorry for my absence.

No problem Peppe. When you feel that makes sense I am available.
You can check here the patches I submited:
https://patchwork.ozlabs.org/patch/715682/
https://patchwork.ozlabs.org/patch/717620/

Regards,
Joao

> 
> Regards
> Peppe
> 
>>
>> Thanks,
>> Joao
>>
>>>
>>> I will give you my feedback on some patches too.
>>>
>>> peppe
>>>
>>>> Regards
>>>>
>>>> Corentin Labbe (17):
>>>>   net: stmmac: fix the typo on MAC_RNABLE_RX
>>>>   net: stmmac: Remove the bus_setup function pointer
>>>>   net: stmmac: fix some typos in comments
>>>>   net: stmmac: remove freesoftware address
>>>>   net: stmmac: remplace asm/io.h by linux/io.h
>>>>   net: stmmac: fix some code style problem
>>>>   net: stmmac: replace stmmac_mdio_busy_wait by readl_poll_timeout
>>>>   net: stmmac: Use readl_poll_timeout
>>>>   net: stmmac: replace ENOSYS by EINVAL
>>>>   net: stmmac: Correct the error message about invalid speed
>>>>   net: stmmac: Rewrite two test against NULL value
>>>>   net: stmmac: rename rx_crc to rx_crc_errors
>>>>   net: stmmac: Implement NAPI for TX
>>>>   net: stmmac: print phy information
>>>>   net: stmmac: remove dead code in stmmac_tx_clean
>>>>   net: stmmac: remove unused variable in sysfs_display_ring
>>>>   net: stmmac: replace unsigned by u32
>>>>
>>>>  drivers/net/ethernet/stmicro/stmmac/chain_mode.c   |  4 --
>>>>  drivers/net/ethernet/stmicro/stmmac/common.h       |  8 +--
>>>>  drivers/net/ethernet/stmicro/stmmac/descs.h        |  4 --
>>>>  drivers/net/ethernet/stmicro/stmmac/descs_com.h    |  4 --
>>>>  drivers/net/ethernet/stmicro/stmmac/dwmac100.h     |  4 --
>>>>  drivers/net/ethernet/stmicro/stmmac/dwmac1000.h    |  4 --
>>>>  .../net/ethernet/stmicro/stmmac/dwmac1000_core.c   |  4 --
>>>>  .../net/ethernet/stmicro/stmmac/dwmac1000_dma.c    |  4 --
>>>>  .../net/ethernet/stmicro/stmmac/dwmac100_core.c    |  4 --
>>>>  drivers/net/ethernet/stmicro/stmmac/dwmac100_dma.c |  4 --
>>>>  drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c |  2 +-
>>>>  drivers/net/ethernet/stmicro/stmmac/dwmac_dma.h    |  4 --
>>>>  drivers/net/ethernet/stmicro/stmmac/dwmac_lib.c    | 28 ++++------
>>>>  drivers/net/ethernet/stmicro/stmmac/enh_desc.c     |  6 +--
>>>>  drivers/net/ethernet/stmicro/stmmac/mmc.h          |  4 --
>>>>  drivers/net/ethernet/stmicro/stmmac/mmc_core.c     |  4 --
>>>>  drivers/net/ethernet/stmicro/stmmac/norm_desc.c    |  6 +--
>>>>  drivers/net/ethernet/stmicro/stmmac/ring_mode.c    |  4 --
>>>>  drivers/net/ethernet/stmicro/stmmac/stmmac.h       |  4 --
>>>>  .../net/ethernet/stmicro/stmmac/stmmac_ethtool.c   |  6 +--
>>>>  .../net/ethernet/stmicro/stmmac/stmmac_hwtstamp.c  |  4 --
>>>>  drivers/net/ethernet/stmicro/stmmac/stmmac_main.c  | 63 ++++++++++------------
>>>>  drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c  | 52 +++++++-----------
>>>>  drivers/net/ethernet/stmicro/stmmac/stmmac_pci.c   |  4 --
>>>>  .../net/ethernet/stmicro/stmmac/stmmac_platform.c  |  6 +--
>>>>  drivers/net/ethernet/stmicro/stmmac/stmmac_ptp.c   |  4 --
>>>>  drivers/net/ethernet/stmicro/stmmac/stmmac_ptp.h   |  4 --
>>>>  include/linux/stmmac.h                             |  1 -
>>>>  28 files changed, 65 insertions(+), 185 deletions(-)
>>>>
>>>
>>
>>
>

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH 00/17] net: stmmac: misc fix Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-01-31 10:30 +0100
  [PATCH 08/17] net: stmmac: Use readl_poll_timeout Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-01-31 10:30 +0100
  [PATCH 07/17] net: stmmac: replace stmmac_mdio_busy_wait by readl_poll_timeout Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-01-31 10:30 +0100
    Re: [PATCH 07/17] net: stmmac: replace stmmac_mdio_busy_wait by  readl_poll_timeout Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-01-31 11:40 +0100
      Re: [PATCH 07/17] net: stmmac: replace stmmac_mdio_busy_wait by  readl_poll_timeout Giuseppe CAVALLARO <peppe.cavallaro@st.com> - 2017-01-31 11:50 +0100
    Re: [PATCH 07/17] net: stmmac: replace stmmac_mdio_busy_wait by  readl_poll_timeout Giuseppe CAVALLARO <peppe.cavallaro@st.com> - 2017-01-31 11:50 +0100
  [PATCH 10/17] net: stmmac: Correct the error message about invalid speed Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-01-31 10:30 +0100
    Re: [PATCH 10/17] net: stmmac: Correct the error message about  invalid speed Giuseppe CAVALLARO <peppe.cavallaro@st.com> - 2017-01-31 11:10 +0100
  [PATCH 02/17] net: stmmac: Remove the bus_setup function pointer Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-01-31 10:30 +0100
    Re: [PATCH 02/17] net: stmmac: Remove the bus_setup function pointer Giuseppe CAVALLARO <peppe.cavallaro@st.com> - 2017-01-31 11:10 +0100
  [PATCH 11/17] net: stmmac: Rewrite two test against NULL value Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-01-31 10:30 +0100
    Re: [PATCH 11/17] net: stmmac: Rewrite two test against NULL value Giuseppe CAVALLARO <peppe.cavallaro@st.com> - 2017-01-31 11:10 +0100
  [PATCH 01/17] net: stmmac: fix the typo on MAC_RNABLE_RX Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-01-31 10:30 +0100
  [PATCH 14/17] net: stmmac: print phy information Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-01-31 10:50 +0100
    Re: [PATCH 14/17] net: stmmac: print phy information Giuseppe CAVALLARO <peppe.cavallaro@st.com> - 2017-01-31 11:20 +0100
      Re: [PATCH 14/17] net: stmmac: print phy information Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-02-03 14:20 +0100
  [PATCH 06/17] net: stmmac: fix some code style problem Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-01-31 10:50 +0100
    Re: [PATCH 06/17] net: stmmac: fix some code style problem Giuseppe CAVALLARO <peppe.cavallaro@st.com> - 2017-01-31 11:10 +0100
  [PATCH 05/17] net: stmmac: remplace asm/io.h by linux/io.h Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-01-31 10:50 +0100
  [PATCH 15/17] net: stmmac: remove dead code in stmmac_tx_clean Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-01-31 10:50 +0100
  Re: [PATCH 00/17] net: stmmac: misc fix Giuseppe CAVALLARO <peppe.cavallaro@st.com> - 2017-01-31 11:10 +0100
    Re: [PATCH 00/17] net: stmmac: misc fix Joao Pinto <Joao.Pinto@synopsys.com> - 2017-01-31 11:40 +0100
      Re: [PATCH 00/17] net: stmmac: misc fix Giuseppe CAVALLARO <peppe.cavallaro@st.com> - 2017-01-31 11:40 +0100
        Re: [PATCH 00/17] net: stmmac: misc fix Joao Pinto <Joao.Pinto@synopsys.com> - 2017-01-31 11:50 +0100

csiph-web