[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250422123842.GA896279-robh@kernel.org>
Date: Tue, 22 Apr 2025 07:38:42 -0500
From: Rob Herring <robh@...nel.org>
To: Mukesh Kumar Savaliya <quic_msavaliy@...cinc.com>
Cc: Nitin Rawat <quic_nitirawa@...cinc.com>, alim.akhtar@...sung.com,
avri.altman@....com, bvanassche@....org, krzk+dt@...nel.org,
mani@...nel.org, conor+dt@...nel.org,
James.Bottomley@...senpartnership.com, martin.petersen@...cle.com,
beanhuo@...ron.com, peter.wang@...iatek.com,
linux-arm-msm@...r.kernel.org, linux-scsi@...r.kernel.org,
linux-kernel@...r.kernel.org, devicetree@...r.kernel.org
Subject: Re: [PATCH V1 2/3] scsi: ufs: pltfrm: Add parsing support for
disable LPM property
On Fri, Apr 18, 2025 at 10:13:46AM +0530, Mukesh Kumar Savaliya wrote:
>
>
> On 4/17/2025 6:16 PM, Nitin Rawat wrote:
> [...]
> > +/**
> > + * ufshcd_parse_lpm_support - read from DT whether LPM modes should be disabled.
> > + * @hba: host controller instance
> > + */
> > +static void ufshcd_parse_lpm_support(struct ufs_hba *hba)
> > +{
> > + struct device *dev = hba->dev;
> > +
> > + hba->disable_lpm = of_property_read_bool(dev->of_node, "disable-lpm");
> > + if (hba->disable_lpm)
> > + dev_info(hba->dev, "UFS LPM is disabled\n");
> How about keeping as debug ?
Why print it at all. You can just read the DT from userspace.
Rob
Powered by blists - more mailing lists