[PATCH] counter: linux/counter.h: fix Excess kernel-doc description warning

[PATCH] counter: linux/counter.h: fix Excess kernel-doc description warning

[Randy Dunlap]

Remove the @priv: line to prevent the kernel-doc warning:

include/linux/counter.h:400: warning: Excess struct member 'priv' description in 'counter_device'

Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Cc: William Breathitt Gray <william.gray@linaro.org>
Cc: linux-iio@vger.kernel.org
---
 include/linux/counter.h |    1 -
 1 file changed, 1 deletion(-)

diff -- a/include/linux/counter.h b/include/linux/counter.h
--- a/include/linux/counter.h
+++ b/include/linux/counter.h
@@ -359,7 +359,6 @@ struct counter_ops {
  * @num_counts:		number of Counts specified in @counts
  * @ext:		optional array of Counter device extensions
  * @num_ext:		number of Counter device extensions specified in @ext
- * @priv:		optional private data supplied by driver
  * @dev:		internal device structure
  * @chrdev:		internal character device structure
  * @events_list:	list of current watching Counter events

Re: [PATCH] counter: linux/counter.h: fix Excess kernel-doc description warning

[William Breathitt Gray]

[-- Attachment #1: Type: text/plain, Size: 1289 bytes --]

On Fri, Dec 22, 2023 at 09:05:11PM -0800, Randy Dunlap wrote:
> Remove the @priv: line to prevent the kernel-doc warning:
> 
> include/linux/counter.h:400: warning: Excess struct member 'priv' description in 'counter_device'
> 
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> Cc: William Breathitt Gray <william.gray@linaro.org>
> Cc: linux-iio@vger.kernel.org
> ---
>  include/linux/counter.h |    1 -
>  1 file changed, 1 deletion(-)
> 
> diff -- a/include/linux/counter.h b/include/linux/counter.h
> --- a/include/linux/counter.h
> +++ b/include/linux/counter.h
> @@ -359,7 +359,6 @@ struct counter_ops {
>   * @num_counts:		number of Counts specified in @counts
>   * @ext:		optional array of Counter device extensions
>   * @num_ext:		number of Counter device extensions specified in @ext
> - * @priv:		optional private data supplied by driver
>   * @dev:		internal device structure
>   * @chrdev:		internal character device structure
>   * @events_list:	list of current watching Counter events

Hi Randy,

Would you provide a Fixes tag for the commit that removed the 'priv'
member so we can track when this warning appeared? You can respond with
it to this thread and I'll add it in when I merge your patch.

Thanks,

William Breathitt Gray

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

Re: [PATCH] counter: linux/counter.h: fix Excess kernel-doc description warning

[Randy Dunlap]

On 12/27/23 06:35, William Breathitt Gray wrote:
> On Fri, Dec 22, 2023 at 09:05:11PM -0800, Randy Dunlap wrote:
>> Remove the @priv: line to prevent the kernel-doc warning:
>>
>> include/linux/counter.h:400: warning: Excess struct member 'priv' description in 'counter_device'
>>
>> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
>> Cc: William Breathitt Gray <william.gray@linaro.org>
>> Cc: linux-iio@vger.kernel.org
>> ---
>>  include/linux/counter.h |    1 -
>>  1 file changed, 1 deletion(-)
>>
>> diff -- a/include/linux/counter.h b/include/linux/counter.h
>> --- a/include/linux/counter.h
>> +++ b/include/linux/counter.h
>> @@ -359,7 +359,6 @@ struct counter_ops {
>>   * @num_counts:		number of Counts specified in @counts
>>   * @ext:		optional array of Counter device extensions
>>   * @num_ext:		number of Counter device extensions specified in @ext
>> - * @priv:		optional private data supplied by driver
>>   * @dev:		internal device structure
>>   * @chrdev:		internal character device structure
>>   * @events_list:	list of current watching Counter events
> 
> Hi Randy,
> 
> Would you provide a Fixes tag for the commit that removed the 'priv'
> member so we can track when this warning appeared? You can respond with
> it to this thread and I'll add it in when I merge your patch.

Fixes: f2ee4759fb70 ("counter: remove old and now unused registration API")

Thank you.

-- 
#Randy

Re: [PATCH] counter: linux/counter.h: fix Excess kernel-doc description warning

[William Breathitt Gray]

[-- Attachment #1: Type: text/plain, Size: 1767 bytes --]

On Wed, Dec 27, 2023 at 01:40:01PM -0800, Randy Dunlap wrote:
> 
> 
> On 12/27/23 06:35, William Breathitt Gray wrote:
> > On Fri, Dec 22, 2023 at 09:05:11PM -0800, Randy Dunlap wrote:
> >> Remove the @priv: line to prevent the kernel-doc warning:
> >>
> >> include/linux/counter.h:400: warning: Excess struct member 'priv' description in 'counter_device'
> >>
> >> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> >> Cc: William Breathitt Gray <william.gray@linaro.org>
> >> Cc: linux-iio@vger.kernel.org
> >> ---
> >>  include/linux/counter.h |    1 -
> >>  1 file changed, 1 deletion(-)
> >>
> >> diff -- a/include/linux/counter.h b/include/linux/counter.h
> >> --- a/include/linux/counter.h
> >> +++ b/include/linux/counter.h
> >> @@ -359,7 +359,6 @@ struct counter_ops {
> >>   * @num_counts:		number of Counts specified in @counts
> >>   * @ext:		optional array of Counter device extensions
> >>   * @num_ext:		number of Counter device extensions specified in @ext
> >> - * @priv:		optional private data supplied by driver
> >>   * @dev:		internal device structure
> >>   * @chrdev:		internal character device structure
> >>   * @events_list:	list of current watching Counter events
> > 
> > Hi Randy,
> > 
> > Would you provide a Fixes tag for the commit that removed the 'priv'
> > member so we can track when this warning appeared? You can respond with
> > it to this thread and I'll add it in when I merge your patch.
> 
> Fixes: f2ee4759fb70 ("counter: remove old and now unused registration API")
> 
> Thank you.
> 
> -- 
> #Randy

Applied, thanks!

[1/1] counter: linux/counter.h: fix Excess kernel-doc description warning
      commit: 49ca40f8d6ceb76bff83713f79215d8e6252f045

William Breathitt Gray

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]