Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Camelia Alexandra Groza <camelia.groza@nxp.com>
To:     Sean Anderson <sean.anderson@seco.com>,
        "David S . Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>
CC:     "linuxppc-dev@lists.ozlabs.org" <linuxppc-dev@lists.ozlabs.org>,
        open list <linux-kernel@vger.kernel.org>,
        Madalin Bucur <madalin.bucur@nxp.com>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        Sean Anderson <sean.anderson@seco.com>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Leo Li <leoyang.li@nxp.com>, Rob Herring <robh+dt@kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>
Subject: RE: [PATCH v4 00/25] net: dpaa: Cleanups in preparation for phylink
 conversion
Thread-Topic: [PATCH v4 00/25] net: dpaa: Cleanups in preparation for phylink
 conversion
Thread-Index: AQHYoDi/0FyeCMejy0mOPXZmSS1cz62VVleQ
Date:   Fri, 29 Jul 2022 13:05:31 +0000
Message-ID: <VI1PR04MB5807DB400C6DDDD31FF21746F2999@VI1PR04MB5807.eurprd04.prod.outlook.com>
References: <20220725151039.2581576-1-sean.anderson@seco.com>
In-Reply-To: <20220725151039.2581576-1-sean.anderson@seco.com>
Accept-Language: en-GB, ro-RO, en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?NGe1fH3n3m8EjRaXa+lAjMe0Tb75m4qXIuR1HvtIffz642AGDmYTV5hql7gD?=
 =?us-ascii?Q?XzugXKeZfe7v1rAYfROLdEmAcK6dkwB7BmtZy4uJA2rZOKie7J1w9nHIKzN2?=
 =?us-ascii?Q?j9DdaDwtMSkEC+Ngp1psA0WRi5CuXTzkhDid5WVVbVfO2qwZPWZ/jF7+yZgf?=
 =?us-ascii?Q?E0DtEzLtze3AM+w7Z3RXqM6D9hCSJGk5RNODBfn7NNA1F7XDZ1/8yrgSkD6o?=
 =?us-ascii?Q?1ASWKSaOxN4Pj++elnDHQnt+4HNdPhfE1kfikaOKY89yVkfFiD9CJ5Vb062H?=
 =?us-ascii?Q?zmFzAXCox5IZeczrrpkhmiDUQV0jq9eRIVLAPxF8D9hGXyz1kHUsUXokTryz?=
 =?us-ascii?Q?A28Onc1VGWcc0KYTnNkZt8nj0Dra+peSVSXQt+DrP1tdBTGbXX+dWnEeqxzp?=
 =?us-ascii?Q?kF+wbQa7j2GHOcMUhzOeUC4RH7EzlmtfOCTIT2hcasoY9Gk74z6QU94hO31Z?=
 =?us-ascii?Q?4ry2dxIQjeueuhVcJlMxSOJKjR8CdDt4+YOTMHHOm0/6mKdHcB16jgbAZKNS?=
 =?us-ascii?Q?d4PY2Nzf3Ugwf74NAAHmSdA5mcCBoJJ0kxqOdxCg+qQi/X2S0FYOYwIVgLbM?=
 =?us-ascii?Q?KYvTaLktLdD7+AOSS9X0y6rPF48+h4Bxf4rpqOPuZhP0Z0cUj2SabkPBMFY+?=
 =?us-ascii?Q?2vkMTkqhRpVDouRvdcbHDcSH/WsAcfHHvzeL0FaBg+7vyUpYfVmbj5Gtmcz7?=
 =?us-ascii?Q?XWYUnl/kGVptOSsNafflLY5o6vYEoId0wczbHHa73XowX1y2ozpu8PblWtjt?=
 =?us-ascii?Q?I1SXjEb3joRHoad9zmzrGVdymw50ns4icJm65GcSvP+0D6UmhTIONGHh+0up?=
 =?us-ascii?Q?236R7L+fvit0fq4UmqMRAeOgXru1/VJaH4+RHO0pdspC9V33Ua7aM/Wjk2xa?=
 =?us-ascii?Q?GH3+17L1p3bDNB+moNaMd3MRm75sLcvtfUEFXIEP1h/8tAi6TMXrqrqmya+j?=
 =?us-ascii?Q?u47BP5hhmEuQZGVa9q1nDXWQFpkYuI+WORFTkeZSKnRAQuJCETwZvNbTvGUW?=
 =?us-ascii?Q?W/lxyz5gwsGMBHFy5IcY8ji/1DGJqYRkOExDOzCNyQE+UO/RIzaumVALKU6K?=
 =?us-ascii?Q?uMRq6heqUJVf+snYDG0Z4/iqNGbvx12QNT0Ix5oJUCbIUjVJsXuNU3xn1qRH?=
 =?us-ascii?Q?d8dtfeBLBzbZeDZ0kNDNugmzSzZh0PkhRlr9qCkG3PwXifofbLmAiJve9GXG?=
 =?us-ascii?Q?i/cwfjJN53Loi3EgELbYD7RyICNBc5rDTKNZm0BNVrgjmUg+nRUMljR3ZXan?=
 =?us-ascii?Q?jpztP+USPRrlHT19mduLklSlmx6HeF2k2oAYutyN5qtn4KYbxAq0DpRixysr?=
 =?us-ascii?Q?eytMnl3lIwgkIg6r0qYIS2Apw+H3zaEqmtF0eofHth+0uUL4dqQ+MlXhyUE/?=
 =?us-ascii?Q?9To/DFPsltGe1vFR/2TV81VbD410nhrECx8AVuIedcR7hxGY3ykF4WXk/AlB?=
 =?us-ascii?Q?LpnJyLbMaOyM6hiVLwR8lgYlJA4Bcm0nniOVM0cTCaorlIAI2mlEjlPZAcfi?=
 =?us-ascii?Q?E60rXWLjgATb3Lpnrl8OBWL35d8sjzdWZB7DVJ62MIfzr3MovfmLui5OZdxi?=
 =?us-ascii?Q?oa4mrDBZTbnpw1dzGRDQczNKlQBo0oAhjQgTtKY6?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5807.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e4381cf4-75c6-428a-2840-08da71630482
X-MS-Exchange-CrossTenant-originalarrivaltime: 29 Jul 2022 13:05:31.7373
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: NghgMrmu6WBuZgZ0CMc8RbdDF+LHbP/A+nr48YfDPGjowAO/SQBNlhst08oskmfrjz8gOhFEJPxFujCtFStYmw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR04MB8449
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> -----Original Message-----
> From: Sean Anderson <sean.anderson@seco.com>
> Sent: Monday, July 25, 2022 18:10
> To: David S . Miller <davem@davemloft.net>; Eric Dumazet
> <edumazet@google.com>; Jakub Kicinski <kuba@kernel.org>; Paolo Abeni
> <pabeni@redhat.com>; netdev@vger.kernel.org
> Cc: linuxppc-dev@lists.ozlabs.org; Camelia Alexandra Groza
> <camelia.groza@nxp.com>; open list <linux-kernel@vger.kernel.org>;
> Madalin Bucur <madalin.bucur@nxp.com>; linux-arm-
> kernel@lists.infradead.org; Sean Anderson <sean.anderson@seco.com>;
> Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>; Leo Li
> <leoyang.li@nxp.com>; Rob Herring <robh+dt@kernel.org>;
> devicetree@vger.kernel.org
> Subject: [PATCH v4 00/25] net: dpaa: Cleanups in preparation for phylink
> conversion
>=20
> This series contains several cleanup patches for dpaa/fman. While they
> are intended to prepare for a phylink conversion, they stand on their
> own. This series was originally submitted as part of [1].
>=20
> [1] https://lore.kernel.org/netdev/20220715215954.1449214-1-sean.anderson=
@seco.com=20

For the series:
Acked-by: Camelia Groza <camelia.groza@nxp.com>
=20
> Changes in v4:
> - Clarify commit message
> - weer -> were
> - tricy -> tricky
> - Use mac_dev for calling change_addr
> - qman_cgr_create -> qman_create_cgr
>=20
> Changes in v3:
> - Incorperate some minor changes into the first FMan binding commit
>=20
> Changes in v2:
> - Convert FMan MAC bindings to yaml
> - Remove some unused variables
> - Fix prototype for dtsec_initialization
> - Fix warning if sizeof(void *) !=3D sizeof(resource_size_t)
> - Specify type of mac_dev for exception_cb
> - Add helper for sanity checking cgr ops
> - Add CGR update function
> - Adjust queue depth on rate change
>=20
> Sean Anderson (25):
>   dt-bindings: net: Convert FMan MAC bindings to yaml
>   net: fman: Convert to SPDX identifiers
>   net: fman: Don't pass comm_mode to enable/disable
>   net: fman: Store en/disable in mac_device instead of mac_priv_s
>   net: fman: dtsec: Always gracefully stop/start
>   net: fman: Get PCS node in per-mac init
>   net: fman: Store initialization function in match data
>   net: fman: Move struct dev to mac_device
>   net: fman: Configure fixed link in memac_initialization
>   net: fman: Export/rename some common functions
>   net: fman: memac: Use params instead of priv for max_speed
>   net: fman: Move initialization to mac-specific files
>   net: fman: Mark mac methods static
>   net: fman: Inline several functions into initialization
>   net: fman: Remove internal_phy_node from params
>   net: fman: Map the base address once
>   net: fman: Pass params directly to mac init
>   net: fman: Use mac_dev for some params
>   net: fman: Specify type of mac_dev for exception_cb
>   net: fman: Clean up error handling
>   net: fman: Change return type of disable to void
>   net: dpaa: Use mac_dev variable in dpaa_netdev_init
>   soc: fsl: qbman: Add helper for sanity checking cgr ops
>   soc: fsl: qbman: Add CGR update function
>   net: dpaa: Adjust queue depth on rate change
>=20
>  .../bindings/net/fsl,fman-dtsec.yaml          | 145 +++++
>  .../devicetree/bindings/net/fsl-fman.txt      | 128 +----
>  .../net/ethernet/freescale/dpaa/dpaa_eth.c    |  59 ++-
>  .../ethernet/freescale/dpaa/dpaa_eth_sysfs.c  |   2 +-
>  drivers/net/ethernet/freescale/fman/fman.c    |  31 +-
>  drivers/net/ethernet/freescale/fman/fman.h    |  31 +-
>  .../net/ethernet/freescale/fman/fman_dtsec.c  | 325 ++++++------
>  .../net/ethernet/freescale/fman/fman_dtsec.h  |  58 +-
>  .../net/ethernet/freescale/fman/fman_keygen.c |  29 +-
>  .../net/ethernet/freescale/fman/fman_keygen.h |  29 +-
>  .../net/ethernet/freescale/fman/fman_mac.h    |  24 +-
>  .../net/ethernet/freescale/fman/fman_memac.c  | 240 +++++----
>  .../net/ethernet/freescale/fman/fman_memac.h  |  57 +-
>  .../net/ethernet/freescale/fman/fman_muram.c  |  31 +-
>  .../net/ethernet/freescale/fman/fman_muram.h  |  32 +-
>  .../net/ethernet/freescale/fman/fman_port.c   |  29 +-
>  .../net/ethernet/freescale/fman/fman_port.h   |  29 +-
>  drivers/net/ethernet/freescale/fman/fman_sp.c |  29 +-
>  drivers/net/ethernet/freescale/fman/fman_sp.h |  28 +-
>  .../net/ethernet/freescale/fman/fman_tgec.c   | 163 +++---
>  .../net/ethernet/freescale/fman/fman_tgec.h   |  54 +-
>  drivers/net/ethernet/freescale/fman/mac.c     | 497 ++----------------
>  drivers/net/ethernet/freescale/fman/mac.h     |  45 +-
>  drivers/soc/fsl/qbman/qman.c                  |  76 ++-
>  include/soc/fsl/qman.h                        |   9 +
>  25 files changed, 739 insertions(+), 1441 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/net/fsl,fman-
> dtsec.yaml
>=20
> --
> 2.35.1.1320.gc452695387.dirty