* [PATCH v2] soc: ti: pruss: remove misplaced semicolon from pruss_cfg_xfr_enable
@ 2024-11-09 3:26 guanjing
2024-11-11 12:32 ` Roger Quadros
0 siblings, 1 reply; 3+ messages in thread
From: guanjing @ 2024-11-09 3:26 UTC (permalink / raw)
To: rogerq, s-anna, nm, horms, p-mohan; +Cc: linux-kernel, guanjing
Remove misplaced colon in inline variant of pruss_cfg_xfr_enable()
which results in a syntax error when the code is compiled without
CONFIG_TI_PRUSS.
Fixes: 0211cc1e4fbb ("soc: ti: pruss: Add helper functions to set GPI mode, MII_RT_event and XFR")
Signed-off-by: guanjing <guanjing@cmss.chinamobile.com>
---
include/linux/pruss_driver.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/pruss_driver.h b/include/linux/pruss_driver.h
index c9a31c567e85..29a76a60869c 100644
--- a/include/linux/pruss_driver.h
+++ b/include/linux/pruss_driver.h
@@ -167,7 +167,7 @@ static inline int pruss_cfg_miirt_enable(struct pruss *pruss, bool enable)
static inline int pruss_cfg_xfr_enable(struct pruss *pruss,
enum pru_type pru_type,
- bool enable);
+ bool enable)
{
return ERR_PTR(-EOPNOTSUPP);
}
--
2.33.0
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] soc: ti: pruss: remove misplaced semicolon from pruss_cfg_xfr_enable
2024-11-09 3:26 [PATCH v2] soc: ti: pruss: remove misplaced semicolon from pruss_cfg_xfr_enable guanjing
@ 2024-11-11 12:32 ` Roger Quadros
2024-11-11 12:39 ` Nishanth Menon
0 siblings, 1 reply; 3+ messages in thread
From: Roger Quadros @ 2024-11-11 12:32 UTC (permalink / raw)
To: guanjing, s-anna, nm, horms, p-mohan; +Cc: linux-kernel
On 09/11/2024 05:26, guanjing wrote:
> Remove misplaced colon in inline variant of pruss_cfg_xfr_enable()
s/colon/semicolon
> which results in a syntax error when the code is compiled without
> CONFIG_TI_PRUSS.
>
> Fixes: 0211cc1e4fbb ("soc: ti: pruss: Add helper functions to set GPI mode, MII_RT_event and XFR")
> Signed-off-by: guanjing <guanjing@cmss.chinamobile.com>
Reviewed-by: Roger Quadros <rogerq@kernel.org>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] soc: ti: pruss: remove misplaced semicolon from pruss_cfg_xfr_enable
2024-11-11 12:32 ` Roger Quadros
@ 2024-11-11 12:39 ` Nishanth Menon
0 siblings, 0 replies; 3+ messages in thread
From: Nishanth Menon @ 2024-11-11 12:39 UTC (permalink / raw)
To: Roger Quadros; +Cc: guanjing, s-anna, horms, p-mohan, linux-kernel
On 14:32-20241111, Roger Quadros wrote:
>
>
> On 09/11/2024 05:26, guanjing wrote:
> > Remove misplaced colon in inline variant of pruss_cfg_xfr_enable()
>
> s/colon/semicolon
>
> > which results in a syntax error when the code is compiled without
> > CONFIG_TI_PRUSS.
> >
> > Fixes: 0211cc1e4fbb ("soc: ti: pruss: Add helper functions to set GPI mode, MII_RT_event and XFR")
> > Signed-off-by: guanjing <guanjing@cmss.chinamobile.com>
>
> Reviewed-by: Roger Quadros <rogerq@kernel.org>
https://lore.kernel.org/all/Zy3nBW3XvFy-Y1Fe@kernel.org/
Please also add to the commit message what Simon suggested (Found using ..)
- how was this found? That helps other folks learn better practises.
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-11-11 12:39 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-11-09 3:26 [PATCH v2] soc: ti: pruss: remove misplaced semicolon from pruss_cfg_xfr_enable guanjing
2024-11-11 12:32 ` Roger Quadros
2024-11-11 12:39 ` Nishanth Menon
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®