From:   Yangtao Li <frank.li@vivo.com>
To:     Amit Kucheria <amitk@kernel.org>,
        Thara Gopinath <thara.gopinath@gmail.com>,
        Andy Gross <agross@kernel.org>,
        Bjorn Andersson <andersson@kernel.org>,
        Konrad Dybcio <konrad.dybcio@linaro.org>,
        "Rafael J. Wysocki" <rafael@kernel.org>,
        Daniel Lezcano <daniel.lezcano@linaro.org>,
        Zhang Rui <rui.zhang@intel.com>
Cc:     Yangtao Li <frank.li@vivo.com>, linux-arm-msm@vger.kernel.org,
        linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: [PATCH 9/9] thermal/drivers/qcom: remove redundant msg
Date:   Sat, 18 Mar 2023 22:44:12 +0800
Message-Id: <20230318144412.75046-9-frank.li@vivo.com>
In-Reply-To: <20230318144412.75046-1-frank.li@vivo.com>
References: <20230318144412.75046-1-frank.li@vivo.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?W+NwwH8ekpB6ioEa9aZX+XzzjFFt/HjaraFKbxWHORcRYUE7SArOewx172mY?=
 =?us-ascii?Q?1HiLGAmcgysI785O5Ft+z3fJvGYiG0MI9T+f5/V2Tfcd7YdIYRsdrpK30P1F?=
 =?us-ascii?Q?U0vQWX8YsI/Fvrb2P0hL9PFBSGRWbJrkl3HRDFi0zXKM1VoflXZOYv32qbIx?=
 =?us-ascii?Q?h8QT/ined9Z4eJkUlcAWmO2sMIc0SuNzoXgPcEylsGWVumDfyvXAYTtC9UQX?=
 =?us-ascii?Q?xjtm3cfsZHHU0I3pBps3T9/TKieQLEDpLPevq8YgrmqdEZVk0Wg3UdfJYgrk?=
 =?us-ascii?Q?AHHlPOmfLPiRilQQE3+XydRdQAMscnHDDuSBO1cWHMjV5kn3/yI+9u/+m195?=
 =?us-ascii?Q?zpHqcxOA2N4VvQVHQS2XaIuYHr1uz8ysW/qQp5LUeqz22Vn5BlSnEYxt/POG?=
 =?us-ascii?Q?Da6LE+UvbXQAddpyh9sb+O0COpg8l/KLi0tCM2DY2xo6rMQFm6zKpztdTZM3?=
 =?us-ascii?Q?+1irfvlzbBhqtih81i2dtHl1u5wx0EMErDE6HXl+KM1v+dCGGuLgbjdxgIr7?=
 =?us-ascii?Q?sPz7WP9Wful00LRVp1rFB57TAuFWy83rRXhIpjOOZ8kbnc9uW5Z5X4rYtGJ8?=
 =?us-ascii?Q?5PJ9tHgpvotKhOcg0T72Cf3+9k8Qwxw5QX3kqRTBrltRvQYUjoe782a32F0D?=
 =?us-ascii?Q?2FG74eXDqWM+8cEnw3TvDxwKZZ3VVkCy6+YGaYyX0kCBow59XTjBTFb6Ti2d?=
 =?us-ascii?Q?9HnB2Fe9vw/QWR9bSgMdzRy9hEk0PvuHzlhUU0lADgcxPieOhM1nhQYwgmQG?=
 =?us-ascii?Q?DyhpO84XKQqzwyJOIaMk1T1vfvM41Fn692vwc168gwuIaYGKrI883kkZC+BF?=
 =?us-ascii?Q?jqPOpHYhfMxaXctpejAHEMg/6tcjXMjtQBpu8QJYXHxfmb+Rgtd4v4ENoHNj?=
 =?us-ascii?Q?FPQRlFI8POhpUAcRdWqO6Lke+5i6S6yvL8lsdR//EE8tkty3d4jK5BeOe3vB?=
 =?us-ascii?Q?v+Wocm3jtp+Z4UIlsgBqNNmPu3vmkp6mPnG5MpkI+eyASo61CXT3/ueTB/A7?=
 =?us-ascii?Q?gORo3T3Rr/slJu2A0dsx8/9LYyvKZfukA7JuTY5o6Yusy6ZYrlZUxJcxkIyy?=
 =?us-ascii?Q?jEb8+uGDj8AYK49fMk3kwFNrvInMVVPTSDJVZ2Zl16B2ClkD+QWxK2zW6Uet?=
 =?us-ascii?Q?tVkxFIQkVo5y7tKzukb0Ir/C05G1nIkI1iYKmJXU6rvQKSKgNVTNykuHy+CS?=
 =?us-ascii?Q?wMUtQkF6jc4F7uSqeLnumU9Km/tTnzp2OSaVPbCaDlbXOYRhrfXXpOok0QG8?=
 =?us-ascii?Q?4asasrISmuz7UGC0x5socgP7pUERDBE9oA8UgIk0MMNI1CU9BURaVKsb3kRz?=
 =?us-ascii?Q?Ci3Ii0m7ZAqQLelVJI3Y2pFUs20po9AP72y4UhiEOMeWRpxiXGPiHuaTmZHa?=
 =?us-ascii?Q?t4Qs8T4+hH8xmGRUY+CVN6pgYeknJCmYYgKfUW7MRtxLQsyVp84Sr699d+0+?=
 =?us-ascii?Q?4tPhEDlemRO4wizocIR4LjOE4AtF6xFoXtuEkcTWGNzxqG1tNUNnqeHCVXJ9?=
 =?us-ascii?Q?EwtBZy8SydGOPowaURuj8uKsqw3j7D0iPLA7gTJK7af7FX5Ii4TYIeC302j7?=
 =?us-ascii?Q?773Vjb0kWvxmFyc+6UTBfECkDc/LeOpkh6Ccnseu?=
X-OriginatorOrg: vivo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: fbf9b63d-707e-4ad4-fa8f-08db27bf6546
X-MS-Exchange-CrossTenant-AuthSource: SEZPR06MB5269.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Mar 2023 14:45:19.5694
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 923e42dc-48d5-4cbe-b582-1a797a6412ed
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: lJffMyIrAL1HdutmE5dYT1W5VNCMEw0ryki/gMgt2ncHFcPZ7eBcHUP7OIoiZcSxl747FPpZfAgIfuk09CJv2A==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: TY0PR06MB5283
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

The upper-layer devm_thermal_add_hwmon_sysfs() function can directly
print error information.

Signed-off-by: Yangtao Li <frank.li@vivo.com>
---
 drivers/thermal/qcom/qcom-spmi-adc-tm5.c    | 4 +---
 drivers/thermal/qcom/qcom-spmi-temp-alarm.c | 4 +---
 drivers/thermal/qcom/tsens.c                | 4 +---
 3 files changed, 3 insertions(+), 9 deletions(-)

diff --git a/drivers/thermal/qcom/qcom-spmi-adc-tm5.c b/drivers/thermal/qcom/qcom-spmi-adc-tm5.c
index 5749149ae2e4..5ddc39b2be32 100644
--- a/drivers/thermal/qcom/qcom-spmi-adc-tm5.c
+++ b/drivers/thermal/qcom/qcom-spmi-adc-tm5.c
@@ -689,9 +689,7 @@ static int adc_tm5_register_tzd(struct adc_tm5_chip *adc_tm)
 			return PTR_ERR(tzd);
 		}
 		adc_tm->channels[i].tzd = tzd;
-		if (devm_thermal_add_hwmon_sysfs(adc_tm->dev, tzd))
-			dev_warn(adc_tm->dev,
-				 "Failed to add hwmon sysfs attributes\n");
+		devm_thermal_add_hwmon_sysfs(adc_tm->dev, tzd);
 	}
 
 	return 0;
diff --git a/drivers/thermal/qcom/qcom-spmi-temp-alarm.c b/drivers/thermal/qcom/qcom-spmi-temp-alarm.c
index 0f88e98428ac..2a3b3e21260f 100644
--- a/drivers/thermal/qcom/qcom-spmi-temp-alarm.c
+++ b/drivers/thermal/qcom/qcom-spmi-temp-alarm.c
@@ -459,9 +459,7 @@ static int qpnp_tm_probe(struct platform_device *pdev)
 		return ret;
 	}
 
-	if (devm_thermal_add_hwmon_sysfs(&pdev->dev, chip->tz_dev))
-		dev_warn(&pdev->dev,
-			 "Failed to add hwmon sysfs attributes\n");
+	devm_thermal_add_hwmon_sysfs(&pdev->dev, chip->tz_dev);
 
 	ret = devm_request_threaded_irq(&pdev->dev, irq, NULL, qpnp_tm_isr,
 					IRQF_ONESHOT, node->name, chip);
diff --git a/drivers/thermal/qcom/tsens.c b/drivers/thermal/qcom/tsens.c
index d3218127e617..f99b0539468b 100644
--- a/drivers/thermal/qcom/tsens.c
+++ b/drivers/thermal/qcom/tsens.c
@@ -1189,9 +1189,7 @@ static int tsens_register(struct tsens_priv *priv)
 		if (priv->ops->enable)
 			priv->ops->enable(priv, i);
 
-		if (devm_thermal_add_hwmon_sysfs(priv->dev, tzd))
-			dev_warn(priv->dev,
-				 "Failed to add hwmon sysfs attributes\n");
+		devm_thermal_add_hwmon_sysfs(priv->dev, tzd);
 	}
 
 	/* VER_0 require to set MIN and MAX THRESH
-- 
2.35.1