Received-SPF: pass (google.com: domain of linux-kernel+bounces-174114-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
From: Vabhav Sharma <vabhav.sharma@nxp.com>
To: Krzysztof Kozlowski <krzk@kernel.org>, Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Conor Dooley
	<conor+dt@kernel.org>, Pengutronix Kernel Team <kernel@pengutronix.de>, Fabio
 Estevam <festevam@gmail.com>, dl-linux-imx <linux-imx@nxp.com>, Ulf Hansson
	<ulf.hansson@linaro.org>, "linux-kernel@vger.kernel.org"
	<linux-kernel@vger.kernel.org>, "linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>, Shawn Guo <shawnguo@kernel.org>,
	Sascha Hauer <s.hauer@pengutronix.de>, Iuliana Prodan
	<iuliana.prodan@nxp.com>
CC: Silvano Di Ninno <silvano.dininno@nxp.com>, Varun Sethi <V.Sethi@nxp.com>,
	Pankaj Gupta <pankaj.gupta@nxp.com>, Peng Fan <peng.fan@nxp.com>, Aisheng
 Dong <aisheng.dong@nxp.com>, Frank Li <frank.li@nxp.com>, Daniel Baluta
	<daniel.baluta@nxp.com>, Franck Lenormand <franck.lenormand@nxp.com>
Subject: RE: [EXT] Re: [PATCH 1/4] dt-bindings: firmware: secvio: Add
 documentation
Thread-Topic: [EXT] Re: [PATCH 1/4] dt-bindings: firmware: secvio: Add
 documentation
Thread-Index: AQHam4j/tJCTK9tQiUWZU25vCayiarGCIcAAgAwypBA=
Date: Thu, 9 May 2024 04:04:29 +0000
Message-ID:
 <AS1PR04MB9358617BF0A5BBFA886F8CDDF3E62@AS1PR04MB9358.eurprd04.prod.outlook.com>
References: <20240501053205.1737248-1-vabhav.sharma@nxp.com>
 <20240501053205.1737248-2-vabhav.sharma@nxp.com>
 <bc812026-bfe7-4733-ae9c-444f1a64382d@kernel.org>
In-Reply-To: <bc812026-bfe7-4733-ae9c-444f1a64382d@kernel.org>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0:
 =?us-ascii?Q?gwMsP/kKL51qMBOFIUaHvezXk3ZLPMDDpGTANLV3nJq+f4CTiLij8tUbV7bU?=
 =?us-ascii?Q?wnCIyTFLitoEr4qLbJM6fiU0tRKA5M5eRYR+BzeubncpB+U9yV9sl0k1cWwQ?=
 =?us-ascii?Q?10NfYv03rbOoNtP72doBGxRMoy94v+IAR17eAtgh+A5ehOStc96K7O+m2L3L?=
 =?us-ascii?Q?OoG05hk3etQi2InmgKXozRG/m20lCdiirPGddoBtI9kGBIDLfqN31JgG0V2T?=
 =?us-ascii?Q?UA+5t+MJF8AxqwsjqFwx7smEJ6x8o+FeaeVtcCV+KKGdn5pQNKPXURZx46e3?=
 =?us-ascii?Q?Z7yD4cbgNzKhMFi/LgR+oxNnPzzu4nxdmemGOUlZUvYO90HlYnd2YhsryOaF?=
 =?us-ascii?Q?bzW/IGDtTgn43b/UnunWdLrKdpLhwMVfZvy/Ud8SCiMg3hjROSt+CQCPgaR0?=
 =?us-ascii?Q?6Ig1bJi6qgvg6bG5tuHnpu+3Xujz+wZzCPvcHqO+z62cNwXEdeU6PNU26H6y?=
 =?us-ascii?Q?CN7qUZk2eXNDzZhQEoNZyMN25/G3GZQPo7uPmy3PD3TmSOWTEs3z7DSFJRkx?=
 =?us-ascii?Q?1BV+6KQ5f+nJkbo2TFuzUA+/nhOwN5vCnBrTQTTy8I5gXtP6kYQwJ0yYuLw8?=
 =?us-ascii?Q?EF9CLkJb1sPbfqfyBMjDcm57xFN44fnvfHRYJdHng2XhXSFlbLvOnE4Q02cC?=
 =?us-ascii?Q?zR+QyAUfuO7dnnATkuHirovdsHqyPAA1HTm9V3fmGHiERwrDINruQeMyNk61?=
 =?us-ascii?Q?btEQtJEhyBAsjPr6LVi6NEBCemFd1H6FauEagkgJ9waDEsaLR0QTtez4IwUa?=
 =?us-ascii?Q?BOE9OJG9mqg84WKMvqTlnU2r/B8ML7T31FblUMqtHGGyqJHnpJIEuR8gwUiF?=
 =?us-ascii?Q?Mt71EuKVDWqzg2Q4tizwkWO00HQMka/ENJ5UKqppHourrSD93muiCK6aOUPI?=
 =?us-ascii?Q?i5InbJkF+8nBPaupsV+uyMUCYaAhpALZUQrZX2fr/sBQs7l6OPqsEHPw9DeG?=
 =?us-ascii?Q?A8rqOUHrTCX5OEpTOATMA8fvkomXgXwIlszXSHkyyoPsHIeyFBkiSKgoyB63?=
 =?us-ascii?Q?R+kj3eGsJcuZfSSuKpINMHxLhfDAOFJ0Kj3HQg4kwQHVtA2ziEpehcrKy/ku?=
 =?us-ascii?Q?EqCRrgjCze57yZfGjhZ90NkmcYMwzi1Qo6/T/JLD9L90picGioBit6eGdM94?=
 =?us-ascii?Q?6dAhNPAOcyP+xIcNLOeakCzlWCBeC74FKIVEGSZt6+3GwiOVzFt2ASDFzzbR?=
 =?us-ascii?Q?F5A44CTfSyXpdCCfXkFAM/WWTFZ0Imhxuar+I2PcQDVHaUj2Jp4rMO25rWZn?=
 =?us-ascii?Q?Jy+5cTBelfvgq+tTZFtuoZqOSEEATA1kP0nO2QKrcnaeg77NoSGnWbY8KFc3?=
 =?us-ascii?Q?ImzLok+qH3NnBoobt2/8H5ddySK0pgifDRoKThFRb2Ihld2oy+5e7E0kgufl?=
 =?us-ascii?Q?nlgol8eMZPOWHdxvPsTRaglP+VkzByZuXUan05fIbnwGi5ws3ufFFKnEoghX?=
 =?us-ascii?Q?S5zREnmwgZrpOEcrDEIiXn2CBPv87DeV7qN3vAzU4mOzFgc5DjsQe2j/bElA?=
 =?us-ascii?Q?TxexS3UPuQ7qKCOrIrAFPMIF5KhJ1ChrGebUNkO0CJ/HuEvBo9n4n44TM7Yn?=
 =?us-ascii?Q?P1cdcodbIXBAmhSIbwgp4TIkop00IpBmfyim3H8W?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: AS1PR04MB9358.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: be80a322-9a8e-42b4-6072-08dc6fdd1fe0
X-MS-Exchange-CrossTenant-originalarrivaltime: 09 May 2024 04:04:29.2882
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: AzZR3R9CYqRpbFdeyxd477xAdfZVOmrBithChcsNXtslqI7j3ZYxyvyEZZ4B11uCqgbHWbkYyFxyYJ4yNMU+Yg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM9PR04MB8843


> -----Original Message-----
> From: Krzysztof Kozlowski <krzk@kernel.org>
> Sent: Wednesday, May 1, 2024 3:16 PM
> To: Vabhav Sharma <vabhav.sharma@nxp.com>; Rob Herring
> <robh+dt@kernel.org>; Krzysztof Kozlowski
> <krzysztof.kozlowski+dt@linaro.org>; Conor Dooley <conor+dt@kernel.org>;
> Pengutronix Kernel Team <kernel@pengutronix.de>; Fabio Estevam
> <festevam@gmail.com>; dl-linux-imx <linux-imx@nxp.com>; Ulf Hansson
> <ulf.hansson@linaro.org>; linux-kernel@vger.kernel.org; linux-arm-
> kernel@lists.infradead.org; Shawn Guo <shawnguo@kernel.org>; Sascha
> Hauer <s.hauer@pengutronix.de>; Iuliana Prodan
> <iuliana.prodan@nxp.com>
> Cc: Silvano Di Ninno <silvano.dininno@nxp.com>; Varun Sethi
> <V.Sethi@nxp.com>; Pankaj Gupta <pankaj.gupta@nxp.com>; Peng Fan
> <peng.fan@nxp.com>; Aisheng Dong <aisheng.dong@nxp.com>; Frank Li
> <frank.li@nxp.com>; Daniel Baluta <daniel.baluta@nxp.com>; Franck
> Lenormand <franck.lenormand@nxp.com>
> Subject: [EXT] Re: [PATCH 1/4] dt-bindings: firmware: secvio: Add
> documentation
>
> Caution: This is an external email. Please take care when clicking links =
or
> opening attachments. When in doubt, report the message using the 'Report
> this email' button
>
>
> On 01/05/2024 07:32, Vabhav Sharma wrote:
> > This patch adds the documentation for the SECVIO driver.
>
> Please do not use "This commit/patch/change", but imperative mood. See
> longer explanation here:
> https://elixir.b/
> ootlin.com%2Flinux%2Fv5.17.1%2Fsource%2FDocumentation%2Fprocess%2F
> submitting-
> patches.rst%23L95&data=3D05%7C02%7Cvabhav.sharma%40nxp.com%7C4e698
> f57f2be4e4b419c08dc69c39540%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C
> 0%7C0%7C638501535958415388%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiM
> C4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C
> %7C%7C&sdata=3D7NB%2FbjnqJStL2QBVTwGflgSSViD9n8TL78X85OtQX54%3D&
> reserved=3D0
>
> Describe hardware, not driver.
Ok
>
>
> >
> > Signed-off-by: Franck LENORMAND <franck.lenormand@nxp.com>
> > Signed-off-by: Vabhav Sharma <vabhav.sharma@nxp.com>
>
>
> Please use scripts/get_maintainers.pl to get a list of necessary people a=
nd lists
> to CC. It might happen, that command when run on an older kernel, gives y=
ou
> outdated entries. Therefore please be sure you base your patches on recen=
t
> Linux kernel.
>
> Tools like b4 or scripts/get_maintainer.pl provide you proper list of peo=
ple, so
> fix your workflow. Tools might also fail if you work on some ancient tree
> (don't, instead use mainline), work on fork of kernel (don't, instead use
> mainline) or you ignore some maintainers (really don't). Just use b4 and
> everything should be fine, although remember about `b4 prep --auto-to-cc`=
 if
> you added new patches to the patchset.
Thank you for feedback and used b4 as recommended
>
> You missed at least devicetree list (maybe more), so this won't be tested=
 by
> automated tooling. Performing review on untested code might be a waste of
> time, thus I will skip this patch entirely till you follow the process al=
lowing the
> patch to be tested.
Added the required list
>
> Please kindly resend and include all necessary To/Cc entries.
Ok, done
>
>
> Best regards,
> Krzysztof