All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v1 1/1] gpiolib: cdev: Fix kernel doc for struct line
@ 2022-05-20 16:47 Andy Shevchenko
  2022-05-20 17:10 ` Dipen Patel
  2022-06-09  9:34 ` Bartosz Golaszewski
  0 siblings, 2 replies; 7+ messages in thread
From: Andy Shevchenko @ 2022-05-20 16:47 UTC (permalink / raw)
  To: Dipen Patel, linux-gpio, linux-kernel
  Cc: Linus Walleij, Bartosz Golaszewski, Andy Shevchenko

Kernel doc validator is not happy:
  gpiolib-cdev.c:487: warning: Function parameter or member 'hdesc' not described in 'line'
  gpiolib-cdev.c:487: warning: Function parameter or member 'raw_level' not described in 'line'
  gpiolib-cdev.c:487: warning: Function parameter or member 'total_discard_seq' not described in 'line'
  gpiolib-cdev.c:487: warning: Function parameter or member 'last_seqno' not described in 'line'

Describe above mentioned parameters.

Fixes: 2068339a6c35 ("gpiolib: cdev: Add hardware timestamp clock type")
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/gpio/gpiolib-cdev.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/gpio/gpiolib-cdev.c b/drivers/gpio/gpiolib-cdev.c
index f5aa5f93342a..d0dc68d04059 100644
--- a/drivers/gpio/gpiolib-cdev.c
+++ b/drivers/gpio/gpiolib-cdev.c
@@ -421,6 +421,10 @@ static int linehandle_create(struct gpio_device *gdev, void __user *ip)
  * @work: the worker that implements software debouncing
  * @sw_debounced: flag indicating if the software debouncer is active
  * @level: the current debounced physical level of the line
+ * @hdesc: the Hardware Timestamp Engine (HTE) descriptor
+ * @raw_level: the line level at the time of event
+ * @total_discard_seq: the running counter of the discarded events
+ * @last_seqno: the last sequence number before debounce period expires
  */
 struct line {
 	struct gpio_desc *desc;
-- 
2.35.1


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH v1 1/1] gpiolib: cdev: Fix kernel doc for struct line
  2022-05-20 16:47 [PATCH v1 1/1] gpiolib: cdev: Fix kernel doc for struct line Andy Shevchenko
@ 2022-05-20 17:10 ` Dipen Patel
  2022-06-03 17:28   ` Andy Shevchenko
  2022-06-09  9:34 ` Bartosz Golaszewski
  1 sibling, 1 reply; 7+ messages in thread
From: Dipen Patel @ 2022-05-20 17:10 UTC (permalink / raw)
  To: Andy Shevchenko, linux-gpio, linux-kernel
  Cc: Linus Walleij, Bartosz Golaszewski, thierry.reding

Acked-by: Dipen Patel <dipenp@nvidia.com>

On 5/20/22 9:47 AM, Andy Shevchenko wrote:
> Kernel doc validator is not happy:
>   gpiolib-cdev.c:487: warning: Function parameter or member 'hdesc' not described in 'line'
>   gpiolib-cdev.c:487: warning: Function parameter or member 'raw_level' not described in 'line'
>   gpiolib-cdev.c:487: warning: Function parameter or member 'total_discard_seq' not described in 'line'
>   gpiolib-cdev.c:487: warning: Function parameter or member 'last_seqno' not described in 'line'
>
> Describe above mentioned parameters.
>
> Fixes: 2068339a6c35 ("gpiolib: cdev: Add hardware timestamp clock type")
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> ---
>  drivers/gpio/gpiolib-cdev.c | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/drivers/gpio/gpiolib-cdev.c b/drivers/gpio/gpiolib-cdev.c
> index f5aa5f93342a..d0dc68d04059 100644
> --- a/drivers/gpio/gpiolib-cdev.c
> +++ b/drivers/gpio/gpiolib-cdev.c
> @@ -421,6 +421,10 @@ static int linehandle_create(struct gpio_device *gdev, void __user *ip)
>   * @work: the worker that implements software debouncing
>   * @sw_debounced: flag indicating if the software debouncer is active
>   * @level: the current debounced physical level of the line
> + * @hdesc: the Hardware Timestamp Engine (HTE) descriptor
> + * @raw_level: the line level at the time of event
> + * @total_discard_seq: the running counter of the discarded events
> + * @last_seqno: the last sequence number before debounce period expires
>   */
>  struct line {
>  	struct gpio_desc *desc;

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v1 1/1] gpiolib: cdev: Fix kernel doc for struct line
  2022-05-20 17:10 ` Dipen Patel
@ 2022-06-03 17:28   ` Andy Shevchenko
  2022-06-03 17:28     ` Andy Shevchenko
  0 siblings, 1 reply; 7+ messages in thread
From: Andy Shevchenko @ 2022-06-03 17:28 UTC (permalink / raw)
  To: Dipen Patel
  Cc: linux-gpio, linux-kernel, Linus Walleij, Bartosz Golaszewski,
	thierry.reding

On Fri, May 20, 2022 at 10:10:49AM -0700, Dipen Patel wrote:
> Acked-by: Dipen Patel <dipenp@nvidia.com>

Thanks!

Bart, this can be applied, I think.

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v1 1/1] gpiolib: cdev: Fix kernel doc for struct line
  2022-06-03 17:28   ` Andy Shevchenko
@ 2022-06-03 17:28     ` Andy Shevchenko
       [not found]       ` <CA+PwDYdf7JEimfYG9ChTHp=Dds_Ljx1Y2KK-H5zfeu52=NxXFA@mail.gmail.com>
  0 siblings, 1 reply; 7+ messages in thread
From: Andy Shevchenko @ 2022-06-03 17:28 UTC (permalink / raw)
  To: Dipen Patel
  Cc: linux-gpio, linux-kernel, Linus Walleij, Bartosz Golaszewski,
	thierry.reding

On Fri, Jun 03, 2022 at 08:28:09PM +0300, Andy Shevchenko wrote:
> On Fri, May 20, 2022 at 10:10:49AM -0700, Dipen Patel wrote:
> > Acked-by: Dipen Patel <dipenp@nvidia.com>
> 
> Thanks!
> 
> Bart, this can be applied, I think.

Or do you want it in my PR?

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v1 1/1] gpiolib: cdev: Fix kernel doc for struct line
       [not found]       ` <CA+PwDYdf7JEimfYG9ChTHp=Dds_Ljx1Y2KK-H5zfeu52=NxXFA@mail.gmail.com>
@ 2022-06-06 10:05         ` Andy Shevchenko
  2022-06-08 16:32           ` Dipen Patel
  0 siblings, 1 reply; 7+ messages in thread
From: Andy Shevchenko @ 2022-06-06 10:05 UTC (permalink / raw)
  To: Thierry Reding
  Cc: Dipen Patel, open list:GPIO SUBSYSTEM, Linux Kernel Mailing List,
	Linus Walleij, Bartosz Golaszewski

On Sun, Jun 05, 2022 at 01:11:54AM +0200, Thierry Reding wrote:
> On Fri, 3 Jun 2022, 19:28 Andy Shevchenko, <
> andriy.shevchenko@linux.intel.com> wrote:
> 
> > On Fri, Jun 03, 2022 at 08:28:09PM +0300, Andy Shevchenko wrote:
> > > On Fri, May 20, 2022 at 10:10:49AM -0700, Dipen Patel wrote:
> > > > Acked-by: Dipen Patel <dipenp@nvidia.com>
> > >
> > > Thanks!
> > >
> > > Bart, this can be applied, I think.
> >
> > Or do you want it in my PR?
> >
> 
> This would need to go in through the HTE tree.
> 
> I can pick it up, but it looks like Linus isn't happy with this yet anyway.

It's merged as far as I can see. Feel free to pick it up, I'll drop it from my
tree.

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v1 1/1] gpiolib: cdev: Fix kernel doc for struct line
  2022-06-06 10:05         ` Andy Shevchenko
@ 2022-06-08 16:32           ` Dipen Patel
  0 siblings, 0 replies; 7+ messages in thread
From: Dipen Patel @ 2022-06-08 16:32 UTC (permalink / raw)
  To: Andy Shevchenko, Thierry Reding
  Cc: open list:GPIO SUBSYSTEM, Linux Kernel Mailing List,
	Linus Walleij, Bartosz Golaszewski

Hi Thierry,

We need this patch to get rid off document warnings.

Best Regards,

Dipen Patel

On 6/6/22 3:05 AM, Andy Shevchenko wrote:
> On Sun, Jun 05, 2022 at 01:11:54AM +0200, Thierry Reding wrote:
>> On Fri, 3 Jun 2022, 19:28 Andy Shevchenko, <
>> andriy.shevchenko@linux.intel.com> wrote:
>>
>>> On Fri, Jun 03, 2022 at 08:28:09PM +0300, Andy Shevchenko wrote:
>>>> On Fri, May 20, 2022 at 10:10:49AM -0700, Dipen Patel wrote:
>>>>> Acked-by: Dipen Patel <dipenp@nvidia.com>
>>>> Thanks!
>>>>
>>>> Bart, this can be applied, I think.
>>> Or do you want it in my PR?
>>>
>> This would need to go in through the HTE tree.
>>
>> I can pick it up, but it looks like Linus isn't happy with this yet anyway.
> It's merged as far as I can see. Feel free to pick it up, I'll drop it from my
> tree.
>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v1 1/1] gpiolib: cdev: Fix kernel doc for struct line
  2022-05-20 16:47 [PATCH v1 1/1] gpiolib: cdev: Fix kernel doc for struct line Andy Shevchenko
  2022-05-20 17:10 ` Dipen Patel
@ 2022-06-09  9:34 ` Bartosz Golaszewski
  1 sibling, 0 replies; 7+ messages in thread
From: Bartosz Golaszewski @ 2022-06-09  9:34 UTC (permalink / raw)
  To: Andy Shevchenko
  Cc: Dipen Patel, open list:GPIO SUBSYSTEM, Linux Kernel Mailing List,
	Linus Walleij

On Fri, May 20, 2022 at 6:47 PM Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
>
> Kernel doc validator is not happy:
>   gpiolib-cdev.c:487: warning: Function parameter or member 'hdesc' not described in 'line'
>   gpiolib-cdev.c:487: warning: Function parameter or member 'raw_level' not described in 'line'
>   gpiolib-cdev.c:487: warning: Function parameter or member 'total_discard_seq' not described in 'line'
>   gpiolib-cdev.c:487: warning: Function parameter or member 'last_seqno' not described in 'line'
>
> Describe above mentioned parameters.
>
> Fixes: 2068339a6c35 ("gpiolib: cdev: Add hardware timestamp clock type")
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> ---
>  drivers/gpio/gpiolib-cdev.c | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/drivers/gpio/gpiolib-cdev.c b/drivers/gpio/gpiolib-cdev.c
> index f5aa5f93342a..d0dc68d04059 100644
> --- a/drivers/gpio/gpiolib-cdev.c
> +++ b/drivers/gpio/gpiolib-cdev.c
> @@ -421,6 +421,10 @@ static int linehandle_create(struct gpio_device *gdev, void __user *ip)
>   * @work: the worker that implements software debouncing
>   * @sw_debounced: flag indicating if the software debouncer is active
>   * @level: the current debounced physical level of the line
> + * @hdesc: the Hardware Timestamp Engine (HTE) descriptor
> + * @raw_level: the line level at the time of event
> + * @total_discard_seq: the running counter of the discarded events
> + * @last_seqno: the last sequence number before debounce period expires
>   */
>  struct line {
>         struct gpio_desc *desc;
> --
> 2.35.1
>

Acked-by: Bartosz Golaszewski <brgl@bgdev.pl>

Feel free to take it through whatever tree is appropriate.

Bart

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2022-06-09  9:34 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-20 16:47 [PATCH v1 1/1] gpiolib: cdev: Fix kernel doc for struct line Andy Shevchenko
2022-05-20 17:10 ` Dipen Patel
2022-06-03 17:28   ` Andy Shevchenko
2022-06-03 17:28     ` Andy Shevchenko
     [not found]       ` <CA+PwDYdf7JEimfYG9ChTHp=Dds_Ljx1Y2KK-H5zfeu52=NxXFA@mail.gmail.com>
2022-06-06 10:05         ` Andy Shevchenko
2022-06-08 16:32           ` Dipen Patel
2022-06-09  9:34 ` Bartosz Golaszewski

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.