Hi,
Roger Quadros <rogerq@xxxxxx> writes:
>>>> +extern const char *usb_decode_ctrl(char *str, size_t size, __u8 bRequestType,
>>>> + __u8 bRequest, __u16 wValue, __u16 wIndex,
>>>> + __u16 wLength);
>>>> +
>>>
>>> where's the stub when !TRACING?
>>
>> Right, I will add
>> #ifdef CONFIG_TRACING
>> .....
>> #endif
>
> Can usb_decode_ctrl() be used even when CONFIG_TRACING is not set?
> If yes then above #ifdefe is not sufficient.
>
> You might need to do something like
>
> #if defined(CONFIG_TRACING)
>
> extern const char *usb_decode_ctrl(..)
>
> #else
>
> static inline const char *usb_decode_ctrl(..) {
> return NULL;
> }
>
> #endif
This is what I mean. They shouldn't be used outside of TRACING, but it's
far safer to have the stubs.
--
balbi
