Messages in this thread | | | Date | Tue, 27 Aug 2013 21:30:24 +0300 | Subject | Re: [PATCHv2 1/2] dev-core: fix build break when DEBUG is enabled | From | Dmitry Kasatkin <> |
| |
On Tue, Aug 27, 2013 at 9:16 PM, Joe Perches <joe@perches.com> wrote: > On Tue, 2013-08-27 at 13:32 -0400, Jason Baron wrote: >> On 08/27/2013 12:20 PM, Joe Perches wrote: >> > On Tue, 2013-08-27 at 17:47 +0300, Dmitry Kasatkin wrote: >> >> When DEBUG is defined, dev_dbg_ratelimited uses dynamic debug data >> >> structures even when CONFIG_DYNAMIC_DEBUG is not defined. >> >> It leads to build break. >> >> For example, when I try to use dev_dbg_ratelimited in USB code and >> >> CONFIG_USB_DEBUG is enabled, but CONFIG_DYNAMIC_DEBUG is not, I get: >> > Jason? >> > >> > Seems mostly sensible to me but I think the first check >> > needs to be >> > >> > #if defined(CONFIG_DYNAMIC_DEBUG) && defined(DEBUG) >> >> Why? All the other call-sites, do it the way Dmitry has done it. >
I followed the pattern..
> Fine. Originally I thought it useful to not > store the ratelimit state, but this way those > messages can be enabled via the dynamic_debug > control. >
Indeed.
Thanks.
- Dmitry
-- Thanks, Dmitry
| |