Received-SPF: pass (google.com: domain of linux-kernel+bounces-135256-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Precedence: bulk
MIME-Version: 1.0
References: <20240408075806.1194297-1-mirela.rabulea@nxp.com>
In-Reply-To: <20240408075806.1194297-1-mirela.rabulea@nxp.com>
From: Fabio Estevam <festevam@gmail.com>
Date: Mon, 8 Apr 2024 08:14:35 -0300
Message-ID: <CAOMZO5BRY5YamT9bxPPMQo8A5msZ-kCUcxn9eLCJEEJwxD-J9A@mail.gmail.com>
Subject: Re: [PATCH v5 1/2] media: dt-bindings: nxp,imx8-jpeg:: Add clocks entries
To: Mirela Rabulea <mirela.rabulea@nxp.com>
Cc: shawnguo@kernel.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, 
	festevam@denx.de, alexander.stein@ew.tq-group.com, Frank.li@nxp.com, 
	ming.qian@nxp.com, conor+dt@kernel.org, devicetree@vger.kernel.org, 
	linux-arm-kernel@lists.infradead.org, s.hauer@pengutronix.de, 
	kernel@pengutronix.de, mchehab@kernel.org, hverkuil@xs4all.nl, 
	linux-media@vger.kernel.org, imx@lists.linux.dev, 
	linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hi Mirela,

On Mon, Apr 8, 2024 at 4:58=E2=80=AFAM Mirela Rabulea <mirela.rabulea@nxp.c=
om> wrote:
>
> From: Fabio Estevam <festevam@gmail.com>
>
> The JPEG decoder/encoder present in iMX8QXP and iMX8QM SoCs need
> the PER and IPG clocks to be functional, so add the clock entries.
>
> This also fixes the following schema warning:
>
> imx8qm-apalis-eval.dtb: jpegdec@58400000: 'assigned-clock-rates', 'assign=
ed-clocks', 'clock-names', 'clocks' do not match any of the regexes: 'pinct=
rl-[0-9]+'
>         from schema $id: http://devicetree.org/schemas/media/nxp,imx8-jpe=
g.yaml#
>
> Signed-off-by: Fabio Estevam <festevam@denx.de>
> Signed-off-by: Mirela Rabulea <mirela.rabulea@nxp.com>

Thanks for submitting this series.

In the Subject, you added an extra : "..nxp,imx8-jpeg:: Add". Please
remove the extra :.

My From and Signed-off-by tags e-mails do not match.

Please use the email from the Signed-off-by in the From field as well.

The same applies to patch 2/2.

Thanks