[PATCH] clocksource: Remove redundant casts

Tang Bin posted 1 patch 2 weeks, 2 days ago
drivers/clocksource/timer-gxp.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] clocksource: Remove redundant casts
Posted by Tang Bin 2 weeks, 2 days ago
In the function gxp_timer_init, the 'int' type cast
in front of the PTR_ERR() macro is redundant, thus
remove it.

Signed-off-by: Tang Bin <tangbin@cmss.chinamobile.com>
---
 drivers/clocksource/timer-gxp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clocksource/timer-gxp.c b/drivers/clocksource/timer-gxp.c
index 57aa2e2cc..48a73c101 100644
--- a/drivers/clocksource/timer-gxp.c
+++ b/drivers/clocksource/timer-gxp.c
@@ -85,7 +85,7 @@ static int __init gxp_timer_init(struct device_node *node)
 
 	clk = of_clk_get(node, 0);
 	if (IS_ERR(clk)) {
-		ret = (int)PTR_ERR(clk);
+		ret = PTR_ERR(clk);
 		pr_err("%pOFn clock not found: %d\n", node, ret);
 		goto err_free;
 	}
-- 
2.33.0
Re: [PATCH] clocksource: Remove redundant casts
Posted by Daniel Lezcano 1 week, 3 days ago
On 07/11/2024 08:46, Tang Bin wrote:
> In the function gxp_timer_init, the 'int' type cast
> in front of the PTR_ERR() macro is redundant, thus
> remove it.
> 
> Signed-off-by: Tang Bin <tangbin@cmss.chinamobile.com>
> ---

Applied, thanks


-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
[tip: timers/core] clocksource/drivers/gpx: Remove redundant casts
Posted by tip-bot2 for Tang Bin 1 week, 3 days ago
The following commit has been merged into the timers/core branch of tip:

Commit-ID:     5569d7348b4a927eb5a2449ddc175ec7c3930c4d
Gitweb:        https://git.kernel.org/tip/5569d7348b4a927eb5a2449ddc175ec7c3930c4d
Author:        Tang Bin <tangbin@cmss.chinamobile.com>
AuthorDate:    Thu, 07 Nov 2024 15:46:19 +08:00
Committer:     Daniel Lezcano <daniel.lezcano@linaro.org>
CommitterDate: Wed, 13 Nov 2024 13:49:33 +01:00

clocksource/drivers/gpx: Remove redundant casts

In the function gxp_timer_init, the 'int' type cast in front of the
PTR_ERR() macro is redundant, thus remove it.

Signed-off-by: Tang Bin <tangbin@cmss.chinamobile.com>
Link: https://lore.kernel.org/r/20241107074619.2714-1-tangbin@cmss.chinamobile.com
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
 drivers/clocksource/timer-gxp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clocksource/timer-gxp.c b/drivers/clocksource/timer-gxp.c
index 57aa2e2..48a73c1 100644
--- a/drivers/clocksource/timer-gxp.c
+++ b/drivers/clocksource/timer-gxp.c
@@ -85,7 +85,7 @@ static int __init gxp_timer_init(struct device_node *node)
 
 	clk = of_clk_get(node, 0);
 	if (IS_ERR(clk)) {
-		ret = (int)PTR_ERR(clk);
+		ret = PTR_ERR(clk);
 		pr_err("%pOFn clock not found: %d\n", node, ret);
 		goto err_free;
 	}