Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Petlozu Pravareshwar <petlozup@nvidia.com>
To:     Thierry Reding <thierry.reding@gmail.com>
CC:     Jonathan Hunter <jonathanh@nvidia.com>,
        "p.zabel@pengutronix.de" <p.zabel@pengutronix.de>,
        "dmitry.osipenko@collabora.com" <dmitry.osipenko@collabora.com>,
        "ulf.hansson@linaro.org" <ulf.hansson@linaro.org>,
        "Kartik ." <kkartik@nvidia.com>,
        "cai.huoqing@linux.dev" <cai.huoqing@linux.dev>,
        Sandipan Patra <spatra@nvidia.com>,
        "linux-tegra@vger.kernel.org" <linux-tegra@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH] soc/tegra: pmc: Print reset info during probe
Thread-Topic: [PATCH] soc/tegra: pmc: Print reset info during probe
Thread-Index: AQHYsmmGR8fp18152ESxKlaB5UYTsq20xbIAgAm3/4A=
Date:   Wed, 24 Aug 2022 19:39:46 +0000
Message-ID: <DM6PR12MB316434A9BA61A01251E3BA3CB5739@DM6PR12MB3164.namprd12.prod.outlook.com>
References: <20220817184456.1184459-1-petlozup@nvidia.com>
 <Yv5XarzacTOkTL1k@orome>
In-Reply-To: <Yv5XarzacTOkTL1k@orome>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?JD9E61RlYFFcYPLToOGC0sH96I2EzqxhUcgCi/9GCwLzAdpBzcV+/BWAVgn+?=
 =?us-ascii?Q?WMUOrVMwrk6CeoT6BG+DGNkz5zQs9VUPd4LM6PfQ4yp5M3KUj5A6mRPi864g?=
 =?us-ascii?Q?d1m54j3+X0sTgKLLlUROavIo1VT7+fhNOQC5lZhGvXpSbRM7YR25uPwHdwCk?=
 =?us-ascii?Q?GgR9jtZ4jv0KSQpVCVSlsRFmw3sHvE4JLQTRGVoOgfKjBK2yqrufNtA894UR?=
 =?us-ascii?Q?zbOmOHQa+OgehRoRFsOmHg+V+PPfyXHBfJHNQRajj8rn2p/cgQ3+c0eG4N0L?=
 =?us-ascii?Q?KH8ywAuGY9C/l5HHP3CvpKiE4kenXvH1g/DCK+9gj+CPWD8zBPc1V3D2k5P7?=
 =?us-ascii?Q?Vj822W3A+OH4NWdz8io9NVeNm38uJQ6Zd1wRtmkn0nBJttIrtvLpUCkl/G8p?=
 =?us-ascii?Q?cCQ8UXy3zYOsat2h7iAWvNFsMXkLM2QbkMtUoTwtNDaiCwUu3MmkSlRI7031?=
 =?us-ascii?Q?fUcfH+JVLQHKfM4mAgMX/A9KWWNyf29hhe1bKQR2Go3S1SOCKxrxIJJpNitB?=
 =?us-ascii?Q?feo1atTzsSr4/K1DR5JqxUBRzquHcMa0M87B5V3RgDxnowVDocTKq4e/r2jY?=
 =?us-ascii?Q?wpOCHY9v+ea4WiwwN9pw6ohJrQT5LQXfPTnjRRUHmz2IBxPg8ndeR4dAznty?=
 =?us-ascii?Q?FIfrnOFMmC1Kt+pm400EDCu1p5QRHoMtXs8kOJfgvi4D0HvolxJ/fieWX6SE?=
 =?us-ascii?Q?rBdjveNfgXxgpcTsPyi2YXeruK4gRjhKACNBrouLE5kFws9wTmzw1ApfYu2F?=
 =?us-ascii?Q?Jg0ndto12fndeaUbdx28n6eb5gZC5HdAxyTPdI65dJfvEK5OFHZ6AvJ/pg3A?=
 =?us-ascii?Q?5uYBzu74jq1tkDT151xNPkFlQeSO1wUdsqAVd6wn3GPsEbHlqhHkaDoh8Z2D?=
 =?us-ascii?Q?aIe14LwPkp643LOl42rxhZpFr2gS7GR9fVGt3W1jLsXSLqC2dUd1u80dwSXZ?=
 =?us-ascii?Q?uR4BTU+tdaBUTpUK8tdNo6ZWFYrdfvKkyZU9AoJc7ZtgJMPLoRUEgH2UxU4m?=
 =?us-ascii?Q?c3Qv9kA04BVTPjSyIh31XR3/iT77fkRNkHX0WuPB07nrb4M8vHhhKWkNwEzx?=
 =?us-ascii?Q?BAzhv58WgKeIkLZH9+MTH3erZ87faXsDRasZ7fstUqhAnZyK0CWscASfOeBN?=
 =?us-ascii?Q?WFXq6B6bp11eD/sv4nvZhPsd9aB4GJ/LBiztCtIKTRdFnFEecUIZ+PZ+ATU7?=
 =?us-ascii?Q?9Ay84jgpERbJqSFcQIlVOATyQah1WzxaJCOZiW0F+I0ZXW4iLBxqk4OAM+4v?=
 =?us-ascii?Q?vTX1RoNtwcnKMGW1vG9cwXSlJKpF1CymotB+umN1MVXkW9yPtsqt4duGMFtZ?=
 =?us-ascii?Q?JSG0J3dpy5lv7H4sN+oOHxg5mJdPuBFeq6IlKHwZ9YvhebXC/Wn9OtmAK89y?=
 =?us-ascii?Q?simWdRy9iNoEDvEePnWu98fG+F6nzZIcBOh/wkzknHumMXrnkFIQfAN9Q2Zf?=
 =?us-ascii?Q?3j3Z+UO+9r54ctK5HMSINnnRynsqbRHsXosb/lQmG7qZMvKfjOMGGJ4xPrzJ?=
 =?us-ascii?Q?h0p6GmKqi0/7QIRzhWTRX+yFZoUo94ceq3A7ZgfjeehblbiZwT/ROZWgbno/?=
 =?us-ascii?Q?hPB+Z6fuwvgJDV8+CFXp5GkgKxZc3oopNSug34oZ?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DM6PR12MB3164.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 046b2372-328c-464e-c1d5-08da86086686
X-MS-Exchange-CrossTenant-originalarrivaltime: 24 Aug 2022 19:39:46.4001
 (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: L9aMdgMx8ULHExe0wZE9PvFJ/NeqmLfaiADC6SQgmowxLKtcLrJp/wS+7XfJoJQAzuwmt6I1OgjQPSO7ef4DwQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR12MB5747
X-Spam-Status: No, score=-1.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_NONE,
        T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=no autolearn_force=no
        version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

>=20
> On Wed, Aug 17, 2022 at 06:44:56PM +0000, Petlozu Pravareshwar wrote:
> > During PMC probe, print previous reset related info such as reset
> > reason and reset level.
> >
> > Signed-off-by: Petlozu Pravareshwar <petlozup@nvidia.com>
> > ---
> >  drivers/soc/tegra/pmc.c | 27 +++++++++++++++++++++++++++
> >  1 file changed, 27 insertions(+)
>=20
> We already expose this information in sysfs, why do we need to print it
> during boot?
>=20
> Thierry
There are some tests which would parse the boot log for this information.
But I agree, since we are already providing this information through sysfs,
there is no need to print it during boot.

Thanks.