Hi Krzysztof,
First bad commit (maybe != root cause):
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 76c057c84d286140c6c416c3b4ba832cd1d8984e
commit: e96c0ff4b1e013a4e9174344b0fcda0d566d3689 pwm: Enable compile testing for some of drivers
date: 10 months ago
config: sh-randconfig-c003-20210128 (attached as .config)
compiler: sh4-linux-gcc (GCC) 9.3.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <[email protected]>
"coccinelle warnings: (new ones prefixed by >>)"
>> drivers/pwm/pwm-lpc18xx-sct.c:292:2-3: Unneeded semicolon
Please review and possibly fold the followup patch.
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/[email protected]
From: kernel test robot <[email protected]>
drivers/pwm/pwm-lpc18xx-sct.c:292:2-3: Unneeded semicolon
Remove unneeded semicolon.
Generated by: scripts/coccinelle/misc/semicolon.cocci
Fixes: e96c0ff4b1e0 ("pwm: Enable compile testing for some of drivers")
CC: Krzysztof Kozlowski <[email protected]>
Reported-by: kernel test robot <[email protected]>
Signed-off-by: kernel test robot <[email protected]>
---
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 76c057c84d286140c6c416c3b4ba832cd1d8984e
commit: e96c0ff4b1e013a4e9174344b0fcda0d566d3689 pwm: Enable compile testing for some of drivers
pwm-lpc18xx-sct.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- a/drivers/pwm/pwm-lpc18xx-sct.c
+++ b/drivers/pwm/pwm-lpc18xx-sct.c
@@ -289,7 +289,7 @@ static int lpc18xx_pwm_request(struct pw
dev_err(lpc18xx_pwm->dev,
"maximum number of simultaneous channels reached\n");
return -EBUSY;
- };
+ }
set_bit(event, &lpc18xx_pwm->event_map);
lpc18xx_data->duty_event = event;
On 1/28/21 3:45 PM, kernel test robot wrote:
> From: kernel test robot <[email protected]>
>
> drivers/pwm/pwm-lpc18xx-sct.c:292:2-3: Unneeded semicolon
>
>
> Remove unneeded semicolon.
>
> Generated by: scripts/coccinelle/misc/semicolon.cocci
>
> Fixes: e96c0ff4b1e0 ("pwm: Enable compile testing for some of drivers")
> CC: Krzysztof Kozlowski <[email protected]>
> Reported-by: kernel test robot <[email protected]>
> Signed-off-by: kernel test robot <[email protected]>
Acked-by: Vladimir Zapolskiy <[email protected]>
--
Best wishes,
Vladimir
Hello,
On Thu, Jan 28, 2021 at 09:45:37PM +0800, kernel test robot wrote:
> From: kernel test robot <[email protected]>
>
> drivers/pwm/pwm-lpc18xx-sct.c:292:2-3: Unneeded semicolon
>
>
> Remove unneeded semicolon.
>
> Generated by: scripts/coccinelle/misc/semicolon.cocci
>
> Fixes: e96c0ff4b1e0 ("pwm: Enable compile testing for some of drivers")
This looks wrong. e96c0ff4b1e0 only touches drivers/pwm/Kconfig.
The ; was introduced by commit 841e6f90bb78 ("pwm: NXP LPC18xx PWM/SCT
driver")
Best regards
Uwe
> CC: Krzysztof Kozlowski <[email protected]>
> Reported-by: kernel test robot <[email protected]>
> Signed-off-by: kernel test robot <[email protected]>
> ---
>
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head: 76c057c84d286140c6c416c3b4ba832cd1d8984e
> commit: e96c0ff4b1e013a4e9174344b0fcda0d566d3689 pwm: Enable compile testing for some of drivers
>
> pwm-lpc18xx-sct.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- a/drivers/pwm/pwm-lpc18xx-sct.c
> +++ b/drivers/pwm/pwm-lpc18xx-sct.c
> @@ -289,7 +289,7 @@ static int lpc18xx_pwm_request(struct pw
> dev_err(lpc18xx_pwm->dev,
> "maximum number of simultaneous channels reached\n");
> return -EBUSY;
> - };
> + }
>
> set_bit(event, &lpc18xx_pwm->event_map);
> lpc18xx_data->duty_event = event;
>
--
Pengutronix e.K. | Uwe Kleine-K?nig |
Industrial Linux Solutions | https://www.pengutronix.de/ |
On 1/28/21 10:57 PM, Uwe Kleine-König wrote:
> Hello,
>
> On Thu, Jan 28, 2021 at 09:45:37PM +0800, kernel test robot wrote:
>> From: kernel test robot <[email protected]>
>>
>> drivers/pwm/pwm-lpc18xx-sct.c:292:2-3: Unneeded semicolon
>>
>>
>> Remove unneeded semicolon.
>>
>> Generated by: scripts/coccinelle/misc/semicolon.cocci
>>
>> Fixes: e96c0ff4b1e0 ("pwm: Enable compile testing for some of drivers")
>
> This looks wrong. e96c0ff4b1e0 only touches drivers/pwm/Kconfig.
>
> The ; was introduced by commit 841e6f90bb78 ("pwm: NXP LPC18xx PWM/SCT
> driver")
Right, thank you for the correction, Uwe.
Since the patch has been composed by the robot, it has to be fixed
in the first place.
And regarding this particular change and in general fixes to this type
of issues detected by the robot, I don't think that it earns a Fixes tag.
>> CC: Krzysztof Kozlowski <[email protected]>
>> Reported-by: kernel test robot <[email protected]>
>> Signed-off-by: kernel test robot <[email protected]>
--
Best wishes,
Vladimir