From:   Yangtao Li <frank.li@vivo.com>
To:     "Rafael J. Wysocki" <rafael@kernel.org>,
        Daniel Lezcano <daniel.lezcano@linaro.org>,
        Amit Kucheria <amitk@kernel.org>,
        Zhang Rui <rui.zhang@intel.com>,
        Shawn Guo <shawnguo@kernel.org>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        Pengutronix Kernel Team <kernel@pengutronix.de>,
        Fabio Estevam <festevam@gmail.com>,
        NXP Linux Team <linux-imx@nxp.com>
Cc:     Yangtao Li <frank.li@vivo.com>, linux-pm@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: [PATCH 4/9] thermal/drivers/imx: remove redundant msg in imx8mm_tmu_probe() and imx_sc_thermal_probe()
Date:   Sat, 18 Mar 2023 22:44:07 +0800
Message-Id: <20230318144412.75046-4-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?cm2Z5MFsbNQrliM68Ke9j+ozrhqfGQrUUmHwlFkw9oTxXKQYmQJ4pENXE5Rr?=
 =?us-ascii?Q?KHWkuv3+25BDmnBnrtjgfwoe85Aa3Vsn1v8tBRLSVeUQNartR4DWE4/1NtDu?=
 =?us-ascii?Q?TYThGcTxPST9C7DPTFiDmZYo7QKEqOkQ87Ycjvqf70bFXk8EFwnz+3Mf4T3p?=
 =?us-ascii?Q?TA4ZYy1yyRIlU4Qcln/meM0cVffEJfRHi9yaZKulzIyLetS/swtIgVuuLSyc?=
 =?us-ascii?Q?yJT1SY2b8V+Ba1oiq/8AK1GTX4ONr7r6WpPaqS6KWDyejwJtbzxOX/eC2o6O?=
 =?us-ascii?Q?VAcR3RzyACoeoXSIuMdOrkJd1CNnTO5g8vTJ/BI+Vuc1RT7q505BTrZL+quI?=
 =?us-ascii?Q?ow8qJsO3NuogwdBEFTHE0eeecE6ISCgmA7MW402FaVwJuJAtggbs3kuTnaGM?=
 =?us-ascii?Q?16oDQIjx5tA3T4rZrnV96RSmnab23d+mU/OaSWGfDOWdn+KT+3Y+59apF/xR?=
 =?us-ascii?Q?XMxYy/WmkEjSddauPZlAimsep7GOqZmZ3kbhZ5bltefzpVmRgr08cvCx1XAr?=
 =?us-ascii?Q?rYMq2XEA1gJbZjPcsQEq0C15XIdp/F+QdVd8m/sG7jDmxUNtYQEn4fqiiWik?=
 =?us-ascii?Q?OZXy8/VmYiYC5v0owV7czaztn3+4XxIEOEjyJIlOBNGSmznAZXdJenA21bx+?=
 =?us-ascii?Q?4RMthvV6kswnZ3RTKbSZHA8iolaHHIsLuEAYmqu/IvSp9JXHFeomXp5neoI7?=
 =?us-ascii?Q?DCWPfXq8ldwcQlYpkT/2XRHE5Qspa8V70DPXPUZGjogIsA+ryIH1B56/jM0j?=
 =?us-ascii?Q?rr24wHHrDfjeAgb7hJ4p13yUkxYJJPriaLK2MLiD7mIHes208m4IzJRhlQ9P?=
 =?us-ascii?Q?vm4pEeE2DRYXILoKHw8NJYHmYrxcawE+650eFXkGUijXG9ZQXeTa9ekhJy5/?=
 =?us-ascii?Q?vRaxG01vtzw4YIUEkJCrPMP6eJwmSHT0vv96SPw1LgmVpI7voV2gs7IvFFw3?=
 =?us-ascii?Q?SOH/cZOnu1Exq7EMWTHYN/KTd5UsdXqiYrXBsRUTn3urPTffX391bvEmt29F?=
 =?us-ascii?Q?asZB5dZFRYm+1ZJDkbrSM835cy6TzY90HkL98iL7m7fqlyEqldV+LXGhdhKu?=
 =?us-ascii?Q?IL/PmNCmmCQvNlsVEvoaGU+jAro3JopP1i8e/vm9Vk8oBEyPPLrzPMOZXxMs?=
 =?us-ascii?Q?GYELGyap0v+jU2pEKtOaYj8lYNQxht+PaAKw0McNEZvDvgNzSUV/wRUHcl/6?=
 =?us-ascii?Q?vOXgNIBIL7BOaPbTNTwN0GGwCEGCWQTigSkcgee2hMafptV+g9AG025KCI8Q?=
 =?us-ascii?Q?oPU2f0kLNYEiSvukLsjaHkgctAY7SiDbY8lfjTOgIoktIlS5ps9tD4iLYHdO?=
 =?us-ascii?Q?xpdVW6JFXsBW8g5gu4nmub/1zn30l80iejWooxjwn6eNlW7/PFs/ZDepky6t?=
 =?us-ascii?Q?OpP2F/VTcym9J+NqF0ofuQuOX3chEAeWbnUSRU1xkXqyGZ79Cgc/MIFBfnv/?=
 =?us-ascii?Q?qfkolYSxNv9lhCQ2n256HF4cE01+Uu2zYMEGY1cmiZApDTIzSCRyBJigg3xt?=
 =?us-ascii?Q?trw2tII0ze9+YBdbG8isedgl8vliwj6IJzUyyUm1h3gvTNGzLMHgtQ/8AboO?=
 =?us-ascii?Q?xdih9Lvs1qnzVCt09BTT7V/wYyCxgo7jAVDEb5Ca?=
X-OriginatorOrg: vivo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 524a2dd3-1af0-4aea-be11-08db27bf5893
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:44:58.3558
 (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: vkcmH+lOKvOfvVlAn2yjRXrwMGjzTUA2WhVqHdbfOSGfZXqqWjPLC7jszy8YTLxgPI0lC2+Ghjeo/0pnANFpsA==
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/imx8mm_thermal.c | 3 +--
 drivers/thermal/imx_sc_thermal.c | 3 +--
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/thermal/imx8mm_thermal.c b/drivers/thermal/imx8mm_thermal.c
index e0de6ac49469..d837e2f96397 100644
--- a/drivers/thermal/imx8mm_thermal.c
+++ b/drivers/thermal/imx8mm_thermal.c
@@ -343,8 +343,7 @@ static int imx8mm_tmu_probe(struct platform_device *pdev)
 		}
 		tmu->sensors[i].hw_id = i;
 
-		if (devm_thermal_add_hwmon_sysfs(&pdev->dev, tmu->sensors[i].tzd))
-			dev_warn(&pdev->dev, "failed to add hwmon sysfs attributes\n");
+		devm_thermal_add_hwmon_sysfs(&pdev->dev, tmu->sensors[i].tzd);
 	}
 
 	platform_set_drvdata(pdev, tmu);
diff --git a/drivers/thermal/imx_sc_thermal.c b/drivers/thermal/imx_sc_thermal.c
index 839bb9958f60..8d6b4ef23746 100644
--- a/drivers/thermal/imx_sc_thermal.c
+++ b/drivers/thermal/imx_sc_thermal.c
@@ -116,8 +116,7 @@ static int imx_sc_thermal_probe(struct platform_device *pdev)
 			return ret;
 		}
 
-		if (devm_thermal_add_hwmon_sysfs(&pdev->dev, sensor->tzd))
-			dev_warn(&pdev->dev, "failed to add hwmon sysfs attributes\n");
+		devm_thermal_add_hwmon_sysfs(&pdev->dev, sensor->tzd);
 	}
 
 	return 0;
-- 
2.35.1