Date:   Mon, 29 Nov 2021 18:11:39 -0600
From:   Rob Herring <robh@kernel.org>
To:     Aswath Govindraju <a-govindraju@ti.com>
Cc:     Rob Herring <robh+dt@kernel.org>, Nishanth Menon <nm@ti.com>,
        Kishon Vijay Abraham I <kishon@ti.com>,
        linux-kernel@vger.kernel.org,
        Vignesh Raghavendra <vigneshr@ti.com>,
        linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
        Tero Kristo <kristo@kernel.org>
Subject: Re: [PATCH 2/5] dt-bindings: pinctrl: k3: Introduce pinmux
 definitions for J721S2
Message-ID: <YaVsO9qbX5J8uuEn@robh.at.kernel.org>
References: <20211119133052.17793-1-a-govindraju@ti.com>
 <20211119133052.17793-3-a-govindraju@ti.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20211119133052.17793-3-a-govindraju@ti.com>
Precedence: bulk

On Fri, 19 Nov 2021 19:00:47 +0530, Aswath Govindraju wrote:
> Add pinctrl macros for J721S2 SoC. These macro definitions are
> similar to that of J721E, 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: Aswath Govindraju <a-govindraju@ti.com>
> ---
>  include/dt-bindings/pinctrl/k3.h | 3 +++
>  1 file changed, 3 insertions(+)
> 

Acked-by: Rob Herring <robh@kernel.org>