mbox series

[v2,0/7] Remove some uses of PMD logtype

Message ID 20200331173110.25746-1-stephen@networkplumber.org (mailing list archive)
Headers
Series Remove some uses of PMD logtype |

Message

Stephen Hemminger March 31, 2020, 5:31 p.m. UTC
  While working on netvsc driver, noticed that there are still some places
in some drivers using the old PMD static logtype.  Would like to
eradicate all of these and even mark it as deprecated in a future
release.

v2 - fix checkpatch whitespace complaints

Stephen Hemminger (7):
  net/mvneta: do not use PMD logtype
  net/virtio: do not use PMD logtype
  net/tap: do not use PMD logtype
  net/pfe: do not use PMD logtype
  net/bnxt: don't use PMD logtype
  net/dpaa: use dynamic logtype
  net/thunderx: use dynamic logtype

 drivers/net/bnxt/bnxt_ring.c            |  2 +-
 drivers/net/dpaa/dpaa_ethdev.c          | 23 ++++++++---------------
 drivers/net/mvneta/mvneta_ethdev.c      |  2 +-
 drivers/net/pfe/pfe_ethdev.c            |  4 ++--
 drivers/net/tap/rte_eth_tap.c           |  6 ++++--
 drivers/net/thunderx/nicvf_ethdev.c     | 17 +++++++++--------
 drivers/net/virtio/virtio_rxtx.c        |  6 +++---
 drivers/net/virtio/virtio_user_ethdev.c |  2 +-
 8 files changed, 29 insertions(+), 33 deletions(-)
  

Comments

Ferruh Yigit March 31, 2020, 5:43 p.m. UTC | #1
On 3/31/2020 6:31 PM, Stephen Hemminger wrote:
> While working on netvsc driver, noticed that there are still some places
> in some drivers using the old PMD static logtype.  Would like to
> eradicate all of these and even mark it as deprecated in a future
> release.
> 
> v2 - fix checkpatch whitespace complaints

Hi Stephen,

I have applied the v1 and fixed the whitespace complaints while merging, is
there anything else changed in v2?

> 
> Stephen Hemminger (7):
>   net/mvneta: do not use PMD logtype
>   net/virtio: do not use PMD logtype
>   net/tap: do not use PMD logtype
>   net/pfe: do not use PMD logtype
>   net/bnxt: don't use PMD logtype
>   net/dpaa: use dynamic logtype
>   net/thunderx: use dynamic logtype
> 
>  drivers/net/bnxt/bnxt_ring.c            |  2 +-
>  drivers/net/dpaa/dpaa_ethdev.c          | 23 ++++++++---------------
>  drivers/net/mvneta/mvneta_ethdev.c      |  2 +-
>  drivers/net/pfe/pfe_ethdev.c            |  4 ++--
>  drivers/net/tap/rte_eth_tap.c           |  6 ++++--
>  drivers/net/thunderx/nicvf_ethdev.c     | 17 +++++++++--------
>  drivers/net/virtio/virtio_rxtx.c        |  6 +++---
>  drivers/net/virtio/virtio_user_ethdev.c |  2 +-
>  8 files changed, 29 insertions(+), 33 deletions(-)
>
  
Ferruh Yigit March 31, 2020, 5:49 p.m. UTC | #2
On 3/31/2020 6:43 PM, Ferruh Yigit wrote:
> On 3/31/2020 6:31 PM, Stephen Hemminger wrote:
>> While working on netvsc driver, noticed that there are still some places
>> in some drivers using the old PMD static logtype.  Would like to
>> eradicate all of these and even mark it as deprecated in a future
>> release.
>>
>> v2 - fix checkpatch whitespace complaints
> 
> Hi Stephen,
> 
> I have applied the v1 and fixed the whitespace complaints while merging, is
> there anything else changed in v2?

I double checked, there was only one line merging diff in dpaa and applied that
too, so practically v2 has been merged.

> 
>>
>> Stephen Hemminger (7):
>>   net/mvneta: do not use PMD logtype
>>   net/virtio: do not use PMD logtype
>>   net/tap: do not use PMD logtype
>>   net/pfe: do not use PMD logtype
>>   net/bnxt: don't use PMD logtype
>>   net/dpaa: use dynamic logtype
>>   net/thunderx: use dynamic logtype
>>
>>  drivers/net/bnxt/bnxt_ring.c            |  2 +-
>>  drivers/net/dpaa/dpaa_ethdev.c          | 23 ++++++++---------------
>>  drivers/net/mvneta/mvneta_ethdev.c      |  2 +-
>>  drivers/net/pfe/pfe_ethdev.c            |  4 ++--
>>  drivers/net/tap/rte_eth_tap.c           |  6 ++++--
>>  drivers/net/thunderx/nicvf_ethdev.c     | 17 +++++++++--------
>>  drivers/net/virtio/virtio_rxtx.c        |  6 +++---
>>  drivers/net/virtio/virtio_user_ethdev.c |  2 +-
>>  8 files changed, 29 insertions(+), 33 deletions(-)
>>
>