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;
Date:   Thu, 10 Feb 2022 19:14:49 +0530
From:   Vinod Koul <vkoul@kernel.org>
To:     Bjorn Andersson <bjorn.andersson@linaro.org>
Cc:     linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] soc: qcom: socinfo: Add some more PMICs and SoCs
Message-ID: <YgUW0SrMgIKWGdqL@matsya>
References: <20220210051043.748275-1-bjorn.andersson@linaro.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20220210051043.748275-1-bjorn.andersson@linaro.org>
Precedence: bulk

On 09-02-22, 21:10, Bjorn Andersson wrote:
> Add SM8350, SC8280XP, SA8540P and one more SM8450 and various PMICs
> found on boards on these platforms to the socinfo driver.

This fixes the unknown ids reported by socinfo debugfs for SM8450 HDK

Tested-by: Vinod Koul <vkoul@kernel.org>
Reviewed-by: Vinod Koul <vkoul@kernel.org>


> 
> Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> ---
>  drivers/soc/qcom/socinfo.c | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/drivers/soc/qcom/socinfo.c b/drivers/soc/qcom/socinfo.c
> index 6dc0f39c0ec3..8b38d134720a 100644
> --- a/drivers/soc/qcom/socinfo.c
> +++ b/drivers/soc/qcom/socinfo.c
> @@ -104,6 +104,14 @@ static const char *const pmic_models[] = {
>  	[36] = "PM8009",
>  	[38] = "PM8150C",
>  	[41] = "SMB2351",
> +	[47] = "PMK8350",
> +	[48] = "PM8350",
> +	[49] = "PM8350C",
> +	[50] = "PM8350B",
> +	[51] = "PMR735A",
> +	[52] = "PMR735B",
> +	[58] = "PM8450",
> +	[65] = "PM8010",
>  };
>  #endif /* CONFIG_DEBUG_FS */
>  
> @@ -314,10 +322,14 @@ static const struct soc_id soc_id[] = {
>  	{ 422, "IPQ6010" },
>  	{ 425, "SC7180" },
>  	{ 434, "SM6350" },
> +	{ 439, "SM8350" },
> +	{ 449, "SC8280XP" },
>  	{ 453, "IPQ6005" },
>  	{ 455, "QRB5165" },
>  	{ 457, "SM8450" },
>  	{ 459, "SM7225" },
> +	{ 460, "SA8540P" },
> +	{ 480, "SM8450" },
>  };
>  
>  static const char *socinfo_machine(struct device *dev, unsigned int id)
> -- 
> 2.33.1

-- 
~Vinod