Re: [PATCH 1/1] DEBUGFS: Fixing name for debug fs

Michal Simek
2008-05-04T08:55:38Z

Hi Arnd,

This is question up to PowerPC guys not to me.
of_debugfs_root sounds good.

M

>> This change is made because powerpc_debugfs_root is arch
>> specific name.
> ...
>>
>> static int powerpc_debugfs_init(void)
>> {
>> - powerpc_debugfs_root = debugfs_create_dir("powerpc", NULL);
>> + debugfs_root = debugfs_create_dir("powerpc", NULL);
>>
>
> You left the name as "powerpc", which sort of defeats the purpose of your
> patch. Files in debugfs are by definition not part of the ABI, so it's
> actually possible to change this as well. How about making this "of"
> instead of "powerpc"?
>
> Arnd <><
>