From: Muzammil Ashraf <[email protected]>
kernel-doc: fixed erroneous warning generated by '__counted_by'
Signed-off-by: Muzammil Ashraf <[email protected]>
---
scripts/kernel-doc | 1 +
1 file changed, 1 insertion(+)
diff --git a/scripts/kernel-doc b/scripts/kernel-doc
index 1484127db104..ea9688df0e93 100755
--- a/scripts/kernel-doc
+++ b/scripts/kernel-doc
@@ -1661,6 +1661,7 @@ sub check_sections($$$$$) {
}
elsif (($decl_type eq "struct") or
($decl_type eq "union")) {
+ next if (index("@_", "__counted_by") != -1);
emit_warning("${file}:$.",
"Excess $decl_type member " .
"'$sects[$sx]' " .
--
2.27.0
Hi,
On 12/19/23 22:24, Muhammad Muzammil wrote:
> From: Muzammil Ashraf <[email protected]>
>
> kernel-doc: fixed erroneous warning generated by '__counted_by'
>
> Signed-off-by: Muzammil Ashraf <[email protected]>
> ---
> scripts/kernel-doc | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/scripts/kernel-doc b/scripts/kernel-doc
> index 1484127db104..ea9688df0e93 100755
> --- a/scripts/kernel-doc
> +++ b/scripts/kernel-doc
> @@ -1661,6 +1661,7 @@ sub check_sections($$$$$) {
> }
> elsif (($decl_type eq "struct") or
> ($decl_type eq "union")) {
> + next if (index("@_", "__counted_by") != -1);
> emit_warning("${file}:$.",
> "Excess $decl_type member " .
> "'$sects[$sx]' " .
One of both of these patches should be enough. Can you test these
instead of your patch, please?
https://lore.kernel.org/linux-doc/[email protected]/
https://lore.kernel.org/linux-doc/[email protected]/
Thanks.
--
#Randy
https://people.kernel.org/tglx/notes-about-netiquette
https://subspace.kernel.org/etiquette.html
On Wed, Dec 20, 2023 at 11:24:46AM +0500, Muhammad Muzammil wrote:
> From: Muzammil Ashraf <[email protected]>
>
> kernel-doc: fixed erroneous warning generated by '__counted_by'
>
> Signed-off-by: Muzammil Ashraf <[email protected]>
> ---
> scripts/kernel-doc | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/scripts/kernel-doc b/scripts/kernel-doc
> index 1484127db104..ea9688df0e93 100755
> --- a/scripts/kernel-doc
> +++ b/scripts/kernel-doc
> @@ -1661,6 +1661,7 @@ sub check_sections($$$$$) {
> }
> elsif (($decl_type eq "struct") or
> ($decl_type eq "union")) {
> + next if (index("@_", "__counted_by") != -1);
Please fix your editor to properly use tabs, and to set them to the
correct 8 column spacing.
thanks,
greg k-h
HI,
1) I already have the below patch but it does not work.
https://lore.kernel.org/linux-doc/[email protected]/
2) When I applied the below patch. It works.
https://lore.kernel.org/linux-doc/[email protected]/
On Wed, Dec 20, 2023 at 11:29 AM Randy Dunlap <[email protected]> wrote:
>
> Hi,
>
> On 12/19/23 22:24, Muhammad Muzammil wrote:
> > From: Muzammil Ashraf <[email protected]>
> >
> > kernel-doc: fixed erroneous warning generated by '__counted_by'
> >
> > Signed-off-by: Muzammil Ashraf <[email protected]>
> > ---
> > scripts/kernel-doc | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/scripts/kernel-doc b/scripts/kernel-doc
> > index 1484127db104..ea9688df0e93 100755
> > --- a/scripts/kernel-doc
> > +++ b/scripts/kernel-doc
> > @@ -1661,6 +1661,7 @@ sub check_sections($$$$$) {
> > }
> > elsif (($decl_type eq "struct") or
> > ($decl_type eq "union")) {
> > + next if (index("@_", "__counted_by") != -1);
> > emit_warning("${file}:$.",
> > "Excess $decl_type member " .
> > "'$sects[$sx]' " .
>
> One of both of these patches should be enough. Can you test these
> instead of your patch, please?
>
> https://lore.kernel.org/linux-doc/[email protected]/
>
> https://lore.kernel.org/linux-doc/[email protected]/
>
> Thanks.
>
> --
> #Randy
> https://people.kernel.org/tglx/notes-about-netiquette
> https://subspace.kernel.org/etiquette.html
On 12/19/23 22:35, Muhammad Muzammil wrote:
> HI,
>
> 1) I already have the below patch but it does not work.
> https://lore.kernel.org/linux-doc/[email protected]/
>
> 2) When I applied the below patch. It works.
> https://lore.kernel.org/linux-doc/[email protected]/
Thanks!
> On Wed, Dec 20, 2023 at 11:29 AM Randy Dunlap <[email protected]> wrote:
>>
>> Hi,
>>
>> On 12/19/23 22:24, Muhammad Muzammil wrote:
>>> From: Muzammil Ashraf <[email protected]>
>>>
>>> kernel-doc: fixed erroneous warning generated by '__counted_by'
>>>
>>> Signed-off-by: Muzammil Ashraf <[email protected]>
>>> ---
>>> scripts/kernel-doc | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff --git a/scripts/kernel-doc b/scripts/kernel-doc
>>> index 1484127db104..ea9688df0e93 100755
>>> --- a/scripts/kernel-doc
>>> +++ b/scripts/kernel-doc
>>> @@ -1661,6 +1661,7 @@ sub check_sections($$$$$) {
>>> }
>>> elsif (($decl_type eq "struct") or
>>> ($decl_type eq "union")) {
>>> + next if (index("@_", "__counted_by") != -1);
>>> emit_warning("${file}:$.",
>>> "Excess $decl_type member " .
>>> "'$sects[$sx]' " .
>>
>> One of both of these patches should be enough. Can you test these
>> instead of your patch, please?
>>
>> https://lore.kernel.org/linux-doc/[email protected]/
>>
>> https://lore.kernel.org/linux-doc/[email protected]/
>>
>> Thanks.
>>
>> --
>> #Randy
>> https://people.kernel.org/tglx/notes-about-netiquette
>> https://subspace.kernel.org/etiquette.html
--
#Randy
https://people.kernel.org/tglx/notes-about-netiquette
https://subspace.kernel.org/etiquette.html
Muhammad Muzammil <[email protected]> writes:
> From: Muzammil Ashraf <[email protected]>
>
> kernel-doc: fixed erroneous warning generated by '__counted_by'
>
> Signed-off-by: Muzammil Ashraf <[email protected]>
> ---
> scripts/kernel-doc | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/scripts/kernel-doc b/scripts/kernel-doc
> index 1484127db104..ea9688df0e93 100755
> --- a/scripts/kernel-doc
> +++ b/scripts/kernel-doc
> @@ -1661,6 +1661,7 @@ sub check_sections($$$$$) {
> }
> elsif (($decl_type eq "struct") or
> ($decl_type eq "union")) {
> + next if (index("@_", "__counted_by") != -1);
> emit_warning("${file}:$.",
> "Excess $decl_type member " .
> "'$sects[$sx]' " .
Could you give an example of the warnings that this patch addresses?
I've not seen any caused by __counted_by since f600c77aeaff was applied
to docs-next. What did it miss?
Thanks,
jon