On Fri, Apr 15, 2022 at 07:39:22PM +0530, Vignesh Raghavendra wrote: > From: Suman Anna <s-a...@ti.com> > > Add pinctrl macros for AM62x SoCs. These macro definitions are similar > to that of previous platforms, but adding new definitions to avoid any > naming confusions in the SoC dts files. > > checkpatch insists the following error exists: > ERROR: Macros with complex values should be enclosed in parentheses > > However, we do not need parentheses enclosing the values for this > macro as we do intend it to generate two separate values as has been > done for other similar platforms. > > Signed-off-by: Suman Anna <s-a...@ti.com> > Signed-off-by: Vignesh Raghavendra <vigne...@ti.com> > --- > include/dt-bindings/pinctrl/k3.h | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/include/dt-bindings/pinctrl/k3.h > b/include/dt-bindings/pinctrl/k3.h > index 63e038e36c..a5204ab91d 100644 > --- a/include/dt-bindings/pinctrl/k3.h > +++ b/include/dt-bindings/pinctrl/k3.h > @@ -41,4 +41,7 @@ > #define J721S2_IOPAD(pa, val, muxmode) (((pa) & 0x1fff)) > ((val) | (muxmode)) > #define J721S2_WKUP_IOPAD(pa, val, muxmode) (((pa) & 0x1fff)) ((val) | > (muxmode)) > > +#define AM62X_IOPAD(pa, val, muxmode) (((pa) & 0x1fff)) > ((val) | (muxmode)) > +#define AM62X_MCU_IOPAD(pa, val, muxmode) (((pa) & 0x1fff)) ((val) | > (muxmode)) > + > #endif
I assume this is being submitted to upstream linux as well? -- Tom
signature.asc
Description: PGP signature