On 10/7/2022 6:43 PM, Andrew Boyer wrote:
Pensando Systems has been acquired by AMD.
Update all copyright strings and email addresses.

Signed-off-by: Andrew Boyer <andrew.bo...@amd.com>
---
  MAINTAINERS                            |  4 ++--
  doc/guides/nics/ionic.rst              | 16 ++++++++--------
  doc/guides/rel_notes/release_22_11.rst |  5 +++++
  drivers/net/ionic/ionic.h              |  6 +++---
  drivers/net/ionic/ionic_dev.c          |  2 +-
  drivers/net/ionic/ionic_dev.h          |  2 +-
  drivers/net/ionic/ionic_ethdev.c       |  2 +-
  drivers/net/ionic/ionic_ethdev.h       |  2 +-
  drivers/net/ionic/ionic_if.h           |  2 +-
  drivers/net/ionic/ionic_lif.c          |  2 +-
  drivers/net/ionic/ionic_lif.h          |  2 +-
  drivers/net/ionic/ionic_logs.h         |  2 +-
  drivers/net/ionic/ionic_mac_api.c      |  2 +-
  drivers/net/ionic/ionic_mac_api.h      |  2 +-
  drivers/net/ionic/ionic_main.c         |  2 +-
  drivers/net/ionic/ionic_osdep.h        |  2 +-
  drivers/net/ionic/ionic_regs.h         |  2 +-
  drivers/net/ionic/ionic_rx_filter.c    |  2 +-
  drivers/net/ionic/ionic_rx_filter.h    |  2 +-
  drivers/net/ionic/ionic_rxtx.c         |  2 +-
  drivers/net/ionic/ionic_rxtx.h         |  2 +-
  drivers/net/ionic/meson.build          |  2 +-
  22 files changed, 36 insertions(+), 31 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 08e575faac..f2f7f33d1f 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -895,8 +895,8 @@ F: doc/guides/nics/pfe.rst
  F: drivers/net/pfe/
  F: doc/guides/nics/features/pfe.ini
-Pensando ionic
-M: Andrew Boyer <abo...@pensando.io>
+AMD Pensando ionic
+M: Andrew Boyer <andrew.bo...@amd.com>
  F: drivers/net/ionic/
  F: doc/guides/nics/ionic.rst
  F: doc/guides/nics/features/ionic.ini

Need to move the block up after other AMD driver, since it is sorted alphabetically.

diff --git a/doc/guides/nics/ionic.rst b/doc/guides/nics/ionic.rst
index 673700d6ce..24b57fc0f5 100644
--- a/doc/guides/nics/ionic.rst
+++ b/doc/guides/nics/ionic.rst
@@ -1,22 +1,22 @@
  ..  SPDX-License-Identifier: (BSD-3-Clause OR GPL-2.0)
-    Copyright(c) 2018-2020 Pensando Systems, Inc. All rights reserved.
+    Copyright 2018-2022 Advanced Micro Devices, Inc. All Rights Reserved.
IONIC Driver
  ============
-The ionic driver provides support for Pensando server adapters.
+The ionic driver provides support for AMD Pensando server adapters.
  It currently supports the below models:
-- DSC-25 dual-port 25G Distributed Services Card `(pdf) <https://pensando.io/wp-content/uploads/2020/03/Pensando-DSC-25-Product-Brief.pdf>`__
-- DSC-100 dual-port 100G Distributed Services Card `(pdf) 
<https://pensando.io/wp-content/uploads/2020/03/Pensando-DSC-100-Product-Brief.pdf>`__
+- DSC-25 dual-port 25G Distributed Services Card
+- DSC-100 dual-port 100G Distributed Services Card
+- DSC-200 dual-port 200G Distributed Services Card `(pdf) 
<https://www.amd.com/system/files/documents/pensando-dsc-200-product-brief.pdf>`__

This 'DSC-200' support seems an addition, can you please separate it from this patch.

-Please visit the Pensando web site at https://pensando.io for more information.
-The `Documents <https://pensando.io/documents/>`_ page contains Product Briefs 
and other product information.
+Please visit the AMD Pensando web site at 
https://www.amd.com/en/accelerators/pensando for more information.
Identifying the Adapter
  -----------------------
-To determine if one or more Pensando DSC Ethernet devices are installed
+To determine if one or more AMD Pensando DSC Ethernet devices are installed
  on the host, check for the PCI devices:
.. code-block:: console
@@ -30,7 +30,7 @@ Firmware Support
The ionic PMD requires firmware which supports 16 segment transmit SGLs.
  This support was added prior to version 1.0. For help upgrading older 
versions,
-please contact Pensando support.
+please contact AMD Pensando support.
Building DPDK
  -------------
diff --git a/doc/guides/rel_notes/release_22_11.rst 
b/doc/guides/rel_notes/release_22_11.rst
index 980c91d5d8..552cc5b62c 100644
--- a/doc/guides/rel_notes/release_22_11.rst
+++ b/doc/guides/rel_notes/release_22_11.rst
@@ -78,6 +78,11 @@ New Features
* Added support to set device link down/up. +* **Updated AMD Pensando ionic driver.**
+
+  Updated the ionic PMD with new features and improvements, including:
+
+  * Updated to reflect that Pensando has been acquired by AMD.


Need to move this block, to sort it alphabetically in the network drivers group.

  Removed Items
  -------------
diff --git a/drivers/net/ionic/ionic.h b/drivers/net/ionic/ionic.h
index a55202b178..31837f050d 100644
--- a/drivers/net/ionic/ionic.h
+++ b/drivers/net/ionic/ionic.h
@@ -1,5 +1,5 @@
  /* SPDX-License-Identifier: (BSD-3-Clause OR GPL-2.0)
- * Copyright(c) 2018-2019 Pensando Systems, Inc. All rights reserved.
+ * Copyright 2018-2022 Advanced Micro Devices, Inc. All Rights Reserved.
   */
#ifndef _IONIC_H_
@@ -15,8 +15,8 @@
  #include "ionic_osdep.h"
#define IONIC_DRV_NAME "ionic"
-#define IONIC_DRV_DESCRIPTION          "Pensando Ethernet NIC Driver"
-#define IONIC_DRV_VERSION              "0.11.0-49"
+#define IONIC_DRV_DESCRIPTION          "AMD Pensando Ethernet NIC Driver"
+#define IONIC_DRV_VERSION              "1.17.0-65"

If there is a driver version change, can you please make it in different patch that clarifies the change of the version, unless this version change is purely because of company change but I doubt that is the case.

/* Vendor ID */
  #define IONIC_PENSANDO_VENDOR_ID      0x1dd8
diff --git a/drivers/net/ionic/ionic_dev.c b/drivers/net/ionic/ionic_dev.c
index 5439b99b2d..a266ea7ddf 100644
--- a/drivers/net/ionic/ionic_dev.c
+++ b/drivers/net/ionic/ionic_dev.c
@@ -1,5 +1,5 @@
  /* SPDX-License-Identifier: (BSD-3-Clause OR GPL-2.0)
- * Copyright(c) 2018-2019 Pensando Systems, Inc. All rights reserved.
+ * Copyright 2018-2022 Advanced Micro Devices, Inc. All Rights Reserved.

I am not sure what is the additional benefit of the "All Rights Reserved." note when there is copyright and license in place, but it is common to have that note in separate line after Coypright line. Making it consistent can be helpful in the future if we have scripts to parse the copyright lines etc.. If not blocked by the legal, can you please move the "All Rights Reserved." to the next line below copyright?

And if possible, can you please question if "All Rights Reserved." line is required at all?

Reply via email to