Re: [PATCH net-next v3 09/11] dt-bindings: add constants for Microsemi Ocelot SerDes driver
From: Florian Fainelli <f.fainelli@gmail.com>
Date: 2018-09-15 21:11:37
Also in:
linux-devicetree, linux-mips, lkml
On 09/14/18 01:16, Quentin Schulz wrote:
quoted hunk ↗ jump to hunk
The Microsemi Ocelot has multiple SerDes and requires that the SerDes be muxed accordingly to the hardware representation. Let's add a constant for each SerDes available in the Microsemi Ocelot. Signed-off-by: Quentin Schulz <redacted> --- include/dt-bindings/phy/phy-ocelot-serdes.h | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) create mode 100644 include/dt-bindings/phy/phy-ocelot-serdes.hdiff --git a/include/dt-bindings/phy/phy-ocelot-serdes.h b/include/dt-bindings/phy/phy-ocelot-serdes.h new file mode 100644 index 0000000..cf111ba --- /dev/null +++ b/include/dt-bindings/phy/phy-ocelot-serdes.h@@ -0,0 +1,19 @@ +/* SPDX-License-Identifier: (GPL-2.0 OR MIT) */ +/* Copyright (c) 2018 Microsemi Corporation */ +#ifndef __PHY_OCELOT_SERDES_H__ +#define __PHY_OCELOT_SERDES_H__ + +#define SERDES1G_0 0 +#define SERDES1G_1 1 +#define SERDES1G_2 2 +#define SERDES1G_3 3 +#define SERDES1G_4 4 +#define SERDES1G_5 5 +#define SERDES1G_MAX 6
Given you use the C preprocessor you could have done something like: #define SERDES1G(x) (x) #define SERDES1G_MAX 5 #define SERDES6G(x) ((x) + SERDES1G_MAX) etc. but this works for me as well.
+#define SERDES6G_0 SERDES1G_MAX +#define SERDES6G_1 (SERDES1G_MAX + 1) +#define SERDES6G_2 (SERDES1G_MAX + 2) +#define SERDES6G_MAX (SERDES1G_MAX + 3) +#define SERDES_MAX (SERDES1G_MAX + SERDES6G_MAX) + +#endif
-- Florian