lkml.org 
[lkml]   [2021]   [Dec]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH v2 net-next 4/5] net: mscc: ocelot: split register definitions to a separate file
Date
On Fri, Dec 03, 2021 at 05:00:49PM -0800, Colin Foster wrote:
> Move these to a separate file will allow them to be shared to other
> drivers.
>
> Signed-off-by: Colin Foster <colin.foster@in-advantage.com>
> Reported-by: kernel test robot <lkp@intel.com>

I wouldn't keep the "Reported-by" tag in this case.

> ---
> drivers/net/ethernet/mscc/Makefile | 3 +-
> drivers/net/ethernet/mscc/ocelot_vsc7514.c | 520 +-------------------
> drivers/net/ethernet/mscc/vsc7514_regs.c | 522 +++++++++++++++++++++
> include/soc/mscc/vsc7514_regs.h | 27 ++
> 4 files changed, 562 insertions(+), 510 deletions(-)
> create mode 100644 drivers/net/ethernet/mscc/vsc7514_regs.c
> create mode 100644 include/soc/mscc/vsc7514_regs.h
>
> diff --git a/drivers/net/ethernet/mscc/vsc7514_regs.c b/drivers/net/ethernet/mscc/vsc7514_regs.c
> new file mode 100644
> index 000000000000..b041756d1b78
> --- /dev/null
> +++ b/drivers/net/ethernet/mscc/vsc7514_regs.c
> @@ -0,0 +1,522 @@
> +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> +/*
> + * Microsemi Ocelot Switch driver
> + *
> + * Copyright (c) 2017 Microsemi Corporation
> + * Copyright (c) 2021 Innovative Advantage
> + */
> +#include <soc/mscc/ocelot_vcap.h>

With Jakub's comment to add an include here (which is to resolve these
warnings):
https://patchwork.hopto.org/static/nipa/590203/12656181/build_32bit/stderr
it looks fine, so please change this and then add my:

Reviewed-by: Vladimir Oltean <vladimir.oltean@nxp.com>
\
 
 \ /
  Last update: 2021-12-04 13:00    [W:0.101 / U:1.020 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site