Messages in this thread | | | Subject | Re: [PATCH net-next v3 09/11] dt-bindings: add constants for Microsemi Ocelot SerDes driver | From | Florian Fainelli <> | Date | Fri, 14 Sep 2018 19:31:31 -0700 |
| |
On 09/14/18 01:16, Quentin Schulz wrote: > 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 <quentin.schulz@bootlin.com> > --- > include/dt-bindings/phy/phy-ocelot-serdes.h | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) > create mode 100644 include/dt-bindings/phy/phy-ocelot-serdes.h > > diff --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
| |