Hi Mathieu, On Tue, May 23, 2017 at 11:39:16AM -0600, Mathieu Poirier wrote: [...] > > +static bool debug_enable; > > +module_param_named(enable, debug_enable, bool, 0600); > > +MODULE_PARM_DESC(enable, "Knob to enable debug functionality " > > + "(default is 0, which means is disabled by default)"); > > Checkpatch complains about a quoted string split across multiple > lines. You can probably removes the second one since information > about the default value is already detailed in patch 03/09. In this > case I suggest modifiying the description to: > > "Control to enable coresight CPU debug functionality" It's shame for checkpatch.pl warning. Although everytime I used script before sending out patch set, should think a bit more to handle them better. Have sent new patch set to dismiss them. Thanks, Leo Yan -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html