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@xxxxxxxxxxx> > --- > 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
