On 11/22/2017 02:04 PM, Randy Dunlap wrote:
> On 11/22/2017 01:15 PM, Andy Shevchenko wrote:
>> This macro deduplicates a lot of similar code across the hci_debugfs.c
>> module. Targeting to be moved to seq_file.h eventually.
>>
>> Signed-off-by: Andy Shevchenko <andriy.shevche...@linux.intel.com>
>> ---
>> net/bluetooth/hci_debugfs.c | 184
>> +++++---------------------------------------
>> 1 file changed, 18 insertions(+), 166 deletions(-)
>
> Looks like a good idea, but below, there is a use of
> DEFINE_SHOW_ATTRIBUTE() before it is #defined.
>
>> diff --git a/net/bluetooth/hci_debugfs.c b/net/bluetooth/hci_debugfs.c
>> index 63df63ebfb24..d4174d508cbf 100644
>> --- a/net/bluetooth/hci_debugfs.c
>> +++ b/net/bluetooth/hci_debugfs.c
>> @@ -88,6 +88,9 @@ static int __name ## _show(struct seq_file *f, void *ptr)
>> \
>> return 0; \
>> } \
>> \
>> +DEFINE_SHOW_ATTRIBUTE(__name)
>
> eh?
OK, it's a continuation of the macro above it.
Sorry about that.
>> +> +#define DEFINE_SHOW_ATTRIBUTE(__name)
>> \
>> static int __name ## _open(struct inode *inode, struct file *file) \
>> { \
>> return single_open(file, __name ## _show, inode->i_private); \
>
>
--
~Randy
