From: Randy Dunlap on
On 03/23/10 08:31, William Cohen wrote:
>
> Function definitions may include __init_or_module. kernel-doc needs to
> filter those out to avoid confusing __init_or_module for the function name.
>
> Signed-off-by: William cohen <wcohen(a)redhat.com>
> ---
> scripts/kernel-doc | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/scripts/kernel-doc b/scripts/kernel-doc
> index c7865c3..1d8db58 100755
> --- a/scripts/kernel-doc
> +++ b/scripts/kernel-doc
> @@ -1728,6 +1728,7 @@ sub dump_function($$) {
> $prototype =~ s/^noinline +//;
> $prototype =~ s/__devinit +//;
> $prototype =~ s/__init +//;
> + $prototype =~ s/__init_or_module +//;
> $prototype =~ s/^#\s*define\s+//; #ak added
> $prototype =~ s/__attribute__\s*\(\([a-z,]*\)\)//;
>

Hi,

Yes, I know. I've already sent this patch to Linus more than once
and now Andrew has it for merging...

--
~Randy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: William Cohen on
On 03/23/2010 11:40 AM, Randy Dunlap wrote:
> On 03/23/10 08:31, William Cohen wrote:
>>
>> Function definitions may include __init_or_module. kernel-doc needs to
>> filter those out to avoid confusing __init_or_module for the function name.
>>
>> Signed-off-by: William cohen <wcohen(a)redhat.com>
>> ---
>> scripts/kernel-doc | 1 +
>> 1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/scripts/kernel-doc b/scripts/kernel-doc
>> index c7865c3..1d8db58 100755
>> --- a/scripts/kernel-doc
>> +++ b/scripts/kernel-doc
>> @@ -1728,6 +1728,7 @@ sub dump_function($$) {
>> $prototype =~ s/^noinline +//;
>> $prototype =~ s/__devinit +//;
>> $prototype =~ s/__init +//;
>> + $prototype =~ s/__init_or_module +//;
>> $prototype =~ s/^#\s*define\s+//; #ak added
>> $prototype =~ s/__attribute__\s*\(\([a-z,]*\)\)//;
>>
>
> Hi,
>
> Yes, I know. I've already sent this patch to Linus more than once
> and now Andrew has it for merging...
>

Thanks, Randy. -Will
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/