* [PATCH] soc: ti: k3-ringacc: Fixup documentation errors
@ 2023-06-21 2:24 Nishanth Menon
2023-06-21 2:27 ` Randy Dunlap
0 siblings, 1 reply; 3+ messages in thread
From: Nishanth Menon @ 2023-06-21 2:24 UTC (permalink / raw)
To: Santosh Shilimkar, Tero Kristo, Nishanth Menon
Cc: vigneshr, praneeth, u-kumar1, linux-kernel, linux-arm-kernel,
Peter Ujfalusi
Fixup couple of misses in documentation. This squashes the following
warnings:
drivers/soc/ti/k3-ringacc.c:135: warning: Function parameter or member 'tdown_complete' not described in 'k3_ring_state'
drivers/soc/ti/k3-ringacc.c:238: warning: expecting prototype for struct k3_ringacc. Prototype was for struct k3_ringacc_soc_data instead
Cc: Peter Ujfalusi <peter.ujfalusi@gmail.com>
Signed-off-by: Nishanth Menon <nm@ti.com>
---
drivers/soc/ti/k3-ringacc.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/soc/ti/k3-ringacc.c b/drivers/soc/ti/k3-ringacc.c
index 8f131368a758..548b9f605d86 100644
--- a/drivers/soc/ti/k3-ringacc.c
+++ b/drivers/soc/ti/k3-ringacc.c
@@ -125,6 +125,7 @@ struct k3_ring_ops {
* @occ: Occupancy
* @windex: Write index
* @rindex: Read index
+ * @tdown_complete: Tear down complete state
*/
struct k3_ring_state {
u32 free;
@@ -229,7 +230,7 @@ struct k3_ringacc {
};
/**
- * struct k3_ringacc - Rings accelerator SoC data
+ * struct k3_ringacc_soc_data - Rings accelerator SoC data
*
* @dma_ring_reset_quirk: DMA reset w/a enable
*/
--
2.40.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] soc: ti: k3-ringacc: Fixup documentation errors
2023-06-21 2:24 [PATCH] soc: ti: k3-ringacc: Fixup documentation errors Nishanth Menon
@ 2023-06-21 2:27 ` Randy Dunlap
2023-06-21 2:30 ` Nishanth Menon
0 siblings, 1 reply; 3+ messages in thread
From: Randy Dunlap @ 2023-06-21 2:27 UTC (permalink / raw)
To: Nishanth Menon, Santosh Shilimkar, Tero Kristo
Cc: vigneshr, praneeth, u-kumar1, linux-kernel, linux-arm-kernel,
Peter Ujfalusi
On 6/20/23 19:24, Nishanth Menon wrote:
> Fixup couple of misses in documentation. This squashes the following
> warnings:
> drivers/soc/ti/k3-ringacc.c:135: warning: Function parameter or member 'tdown_complete' not described in 'k3_ring_state'
> drivers/soc/ti/k3-ringacc.c:238: warning: expecting prototype for struct k3_ringacc. Prototype was for struct k3_ringacc_soc_data instead
>
> Cc: Peter Ujfalusi <peter.ujfalusi@gmail.com>
> Signed-off-by: Nishanth Menon <nm@ti.com>
Reviewed-by: Randy Dunlap <rdunlap@infradead.org>
(more below)
> ---
> drivers/soc/ti/k3-ringacc.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/soc/ti/k3-ringacc.c b/drivers/soc/ti/k3-ringacc.c
> index 8f131368a758..548b9f605d86 100644
> --- a/drivers/soc/ti/k3-ringacc.c
> +++ b/drivers/soc/ti/k3-ringacc.c
> @@ -125,6 +125,7 @@ struct k3_ring_ops {
> * @occ: Occupancy
> * @windex: Write index
> * @rindex: Read index
> + * @tdown_complete: Tear down complete state
> */
> struct k3_ring_state {
> u32 free;
> @@ -229,7 +230,7 @@ struct k3_ringacc {
> };
>
> /**
> - * struct k3_ringacc - Rings accelerator SoC data
> + * struct k3_ringacc_soc_data - Rings accelerator SoC data
> *
> * @dma_ring_reset_quirk: DMA reset w/a enable
What does "w/a" mean?
> */
Thanks.
--
~Randy
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] soc: ti: k3-ringacc: Fixup documentation errors
2023-06-21 2:27 ` Randy Dunlap
@ 2023-06-21 2:30 ` Nishanth Menon
0 siblings, 0 replies; 3+ messages in thread
From: Nishanth Menon @ 2023-06-21 2:30 UTC (permalink / raw)
To: Randy Dunlap
Cc: Santosh Shilimkar, Tero Kristo, vigneshr, praneeth, u-kumar1,
linux-kernel, linux-arm-kernel, Peter Ujfalusi
On 19:27-20230620, Randy Dunlap wrote:
>
>
> On 6/20/23 19:24, Nishanth Menon wrote:
> > Fixup couple of misses in documentation. This squashes the following
> > warnings:
> > drivers/soc/ti/k3-ringacc.c:135: warning: Function parameter or member 'tdown_complete' not described in 'k3_ring_state'
> > drivers/soc/ti/k3-ringacc.c:238: warning: expecting prototype for struct k3_ringacc. Prototype was for struct k3_ringacc_soc_data instead
> >
> > Cc: Peter Ujfalusi <peter.ujfalusi@gmail.com>
> > Signed-off-by: Nishanth Menon <nm@ti.com>
>
> Reviewed-by: Randy Dunlap <rdunlap@infradead.org>
>
> (more below)
>
> > ---
> > drivers/soc/ti/k3-ringacc.c | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/soc/ti/k3-ringacc.c b/drivers/soc/ti/k3-ringacc.c
> > index 8f131368a758..548b9f605d86 100644
> > --- a/drivers/soc/ti/k3-ringacc.c
> > +++ b/drivers/soc/ti/k3-ringacc.c
> > @@ -125,6 +125,7 @@ struct k3_ring_ops {
> > * @occ: Occupancy
> > * @windex: Write index
> > * @rindex: Read index
> > + * @tdown_complete: Tear down complete state
> > */
> > struct k3_ring_state {
> > u32 free;
> > @@ -229,7 +230,7 @@ struct k3_ringacc {
> > };
> >
> > /**
> > - * struct k3_ringacc - Rings accelerator SoC data
> > + * struct k3_ringacc_soc_data - Rings accelerator SoC data
> > *
> > * @dma_ring_reset_quirk: DMA reset w/a enable
>
> What does "w/a" mean?
I think it means work around.. (going by the "quirk" terminology).. Let
me respin this with w/a replaced with workaround as well.
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-06-21 2:30 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-06-21 2:24 [PATCH] soc: ti: k3-ringacc: Fixup documentation errors Nishanth Menon
2023-06-21 2:27 ` Randy Dunlap
2023-06-21 2:30 ` Nishanth Menon
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox