Received-SPF: pass (google.com: domain of linux-kernel+bounces-29164-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
From: Petlozu Pravareshwar <petlozup@nvidia.com>
To: Rob Herring <robh@kernel.org>
CC: "conor+dt@kernel.org" <conor+dt@kernel.org>, "cai.huoqing@linux.dev"
	<cai.huoqing@linux.dev>, "devicetree@vger.kernel.org"
	<devicetree@vger.kernel.org>, "linux-kernel@vger.kernel.org"
	<linux-kernel@vger.kernel.org>, "ulf.hansson@linaro.org"
	<ulf.hansson@linaro.org>, "thierry.reding@gmail.com"
	<thierry.reding@gmail.com>, "krzysztof.kozlowski+dt@linaro.org"
	<krzysztof.kozlowski+dt@linaro.org>, "p.zabel@pengutronix.de"
	<p.zabel@pengutronix.de>, Sandipan Patra <spatra@nvidia.com>,
	"linux-tegra@vger.kernel.org" <linux-tegra@vger.kernel.org>, Kartik Rajput
	<kkartik@nvidia.com>, "robh+dt@kernel.org" <robh+dt@kernel.org>,
	"dmitry.osipenko@collabora.com" <dmitry.osipenko@collabora.com>, Jonathan
 Hunter <jonathanh@nvidia.com>
Subject: RE: [PATCH 2/3] dt-bindings: tegra: pmc: Update scratch as an
 optional aperture
Thread-Topic: [PATCH 2/3] dt-bindings: tegra: pmc: Update scratch as an
 optional aperture
Thread-Index: AQHaQHVXi8lSHewdikO4T3WqC1nA87DMiwUAgBGpZsA=
Date: Wed, 17 Jan 2024 15:41:23 +0000
Message-ID:
 <IA0PR12MB89066B43B0646AFA4AD8200DB5722@IA0PR12MB8906.namprd12.prod.outlook.com>
References: <20240106075134.3933491-1-petlozup@nvidia.com>
 <20240106075134.3933491-2-petlozup@nvidia.com>
 <170453471019.703179.18182038720980471680.robh@kernel.org>
In-Reply-To: <170453471019.703179.18182038720980471680.robh@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?I0jOyTnkg+OttuN+L75ub1RToNniwBb4Kk0qcW24GoxglX+pVdNKPrqzDAfK?=
 =?us-ascii?Q?Hd3ipSmV0lpdUFn0XVOBQ2NfT5DMLOl0I2O5wNEgAJoihp7Hum4SwJEaP49G?=
 =?us-ascii?Q?4DXWIUKYdfgL4hyH/yP+pe/+CDoC46DSeCTUsJwNLWrhwKFvDYWnMYpiRoLs?=
 =?us-ascii?Q?y1fwoK5Eor8SxMIf9N1C1Mcupq0zBBaTS/lkG+4W98GahZBKokOZr94Pm6Xt?=
 =?us-ascii?Q?sXtcQ++XBqPJcFBM/KDAtoHYjG0nQ05Zfn6461Cpddu/nrm6gBxi16QKGuYs?=
 =?us-ascii?Q?dTaBNLsuWM1QZfr6KEa6micDTtNPMw4z8o6sY628Ztm5uVa6B6qLrw7gIaDE?=
 =?us-ascii?Q?XXcBn6kt24MilgOSU764OgNTbRGIjtElNgkYVhhebBbZrd3MuUi9OHGp9myb?=
 =?us-ascii?Q?w5kP1yyooFxNx3l/I9aPGkE1wIU49C4vhe3ny1/x2JEL4t7gWZH3007rc2oP?=
 =?us-ascii?Q?MXUFuGdliiM+KTjkOU2RXUdpAHvd4FMOv8yf3IMLCCx1ceET5UO0a07eh3hU?=
 =?us-ascii?Q?Ba46Um01CuuG2qgRNIjlWKgwIAsKiLo8aMOPyNcKK9vTHszMd+mXcqAfg2zB?=
 =?us-ascii?Q?fz8yJdRb9E2p8W55+LiS5CS2w1W9VL2FMho4LGkCG4VdDEs7TSldaE5mLUqf?=
 =?us-ascii?Q?+iWjNNhZCqfXqpWXxBUtoDmO0mlfiYNCTDwGFqgVWd62FxHgz1RfrS+zDW3m?=
 =?us-ascii?Q?8kUbkFRQgnu9SONyTmhuopVZFS0MJs003kHl228/o0rXEVMnW7QNr3z1WbPK?=
 =?us-ascii?Q?FZT1Kxa58sJrlF8RG0PGBNaj7mwVRD/SOZQMDRZR7SRknKdd2XdvbDX18Sm6?=
 =?us-ascii?Q?ApPXKDtq3GYX92c9jl5TYuOHcaOIJD5Pl/D86LM3kVozX6R3NsqgTLZqlmeg?=
 =?us-ascii?Q?SHuuDPzUi1VoTL/C/tY4CEzr4WaffRPgYQRHVRcHAlRn/smJOQdCSTk1UNeH?=
 =?us-ascii?Q?eSQxcXapv+5qs0mB57QUgtIGnkro2YTt0XR2gr/jW4kpgMewaaL+KxuJpfWx?=
 =?us-ascii?Q?Wi1MQNf/PN4Z7suh9heauBXSCz+/+sV7nwyo5YxM8R/s7f90LweTbc05IHLB?=
 =?us-ascii?Q?8r7imujv35vR/6Yq/DT7cypSVT1NlfVsl6eA4Qhs4LXJiiJKpeOQ8EsvAlpJ?=
 =?us-ascii?Q?Wx5UWWq6sntvu/2p+nJK069XZOlrLARLUBqpDpKoKR3fuW5ZNQmG4RANu3w/?=
 =?us-ascii?Q?NSMLKsvEmfAH2jStiO0dN0ghst1lk2nyaxcgmElvG7eruHeUg6/RDA6PjxKi?=
 =?us-ascii?Q?fyLu+5TBrr2I7dhYVrfbb3OkqCph+b9Lfr6aKob0LkqyXOBO7/DGdqIUJMkz?=
 =?us-ascii?Q?R3gPmyMVZWMNjUHmbm93piI1b0Fe+Vp/MSVBzjqi2JCLGXo0XetKPe/j8idP?=
 =?us-ascii?Q?HElJqF0B8zA/+TYfuK39xTD0Q/QLWaP97J6rJZnlbaihLaeveko/g0IAwjGB?=
 =?us-ascii?Q?24ubyHcEqJ7MO6eXXNToeDscfOTOyPRq9uTbtF8yUmrLjnkmL9hLfq0d8R93?=
 =?us-ascii?Q?ekrEngbOEQQFP/VgV3A1/uP9nqfWJEwgPJQoXoxh+7qf/pBbYxrYPzxVZcKb?=
 =?us-ascii?Q?VBWZ53M+IDjblTOSAva8Py8SZYe+174Qt4G8puqz?=
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: Nvidia.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: IA0PR12MB8906.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 51e096b2-4cf1-4d2b-88e0-08dc1772c246
X-MS-Exchange-CrossTenant-originalarrivaltime: 17 Jan 2024 15:41:23.2743
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: WTgZELy8aQk9MrNX0tYKzpTTi0f6aZG9hQvty/b5WtvRpbVRjF3BjwX7SGzdBKAKBU1tgnHa0I/M36p2Ql7HKA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR12MB5403

 > On Sat, 06 Jan 2024 07:51:33 +0000, Petlozu Pravareshwar wrote:
> > Scratch address space register is used to store reboot reason. For
> > some Tegra234 systems, the scratch space is not available to store the
> > reboot reason. This is because scratch region on these systems is not
> > accessible by the kernel as restricted by the Hypervisor.
> > Such systems would delist scratch aperture from PMC DT node.
> >
> > Accordingly, this change makes "scratch" as an optional aperture for
> > Tegra234 in PMC dt-binding document.
> >
> > Signed-off-by: Petlozu Pravareshwar <petlozup@nvidia.com>
> > ---
> >  .../arm/tegra/nvidia,tegra186-pmc.yaml        | 83 +++++++++++++------
> >  1 file changed, 58 insertions(+), 25 deletions(-)
> >
>=20
> My bot found errors running 'make DT_CHECKER_FLAGS=3D-m
> dt_binding_check'
> on your patch (DT_CHECKER_FLAGS is new in v5.13):
>=20
> yamllint warnings/errors:
> ./Documentation/devicetree/bindings/arm/tegra/nvidia,tegra186-
> pmc.yaml:85:12: [warning] wrong indentation: expected 12 but found 11
> (indentation)
>=20
> dtschema/dtc warnings/errors:
>=20
> doc reference errors (make refcheckdocs):
>=20
> See https://patchwork.ozlabs.org/project/devicetree-
> bindings/patch/20240106075134.3933491-2-petlozup@nvidia.com
>=20
> The base for the series is generally the latest rc1. A different dependen=
cy
> should be noted in *this* patch.
>=20
> If you already ran 'make dt_binding_check' and didn't see the above error=
(s),
> then make sure 'yamllint' is installed and dt-schema is up to
> date:
>=20
> pip3 install dtschema --upgrade
>=20
> Please check and re-submit after running the above command yourself. Note
> that DT_SCHEMA_FILES can be set to your schema file to speed up checking
> your schema. However, it must be unset to test all examples with your
> schema.
I could reproduce the above warning after installing 'yamllint'. I will fix=
 this warning in the next patch.
Thanks.