lkml.org 
[lkml]   [2012]   [Mar]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: [PATCH] ti-st: Enhange logging for Shared Transport - TI driver
    From
    On 15 March 2012 23:10, Alexandra Sava <alexandrasava18@gmail.com> wrote:
    > On 15 March 2012 22:29, Savoy, Pavan <pavan_savoy@ti.com> wrote:
    >> Daniel Baluta wrote:
    >>> Adding Greg's new email address.
    >>>
    >>> On Thu, Mar 15, 2012 at 8:26 PM,  <alexandrasava18@gmail.com> wrote:
    >>>> From: Alexandra Sava <alexandrasava18@gmail.com>
    >>>>
    >>>> * added new module parameter named debug_mask to control displayed
    >>>> messages verbosity
    >>>> * the logging mechanism is enabled via CONFIG_TI_ST_DEBUG Kconfig
    >>>> option
    >>
    >> Any specific reason you want to have this ? Custom logging ?
    >> Also please fix the spelling in the subject line.
    >>
    >
    > The logging for this driver is too verbose, and I thought it would be
    > better to implement a configurable mechanism which can enable/disable
    > the log messages (not only via the global CONFIG_DYNAMIC_DEBUG).
    > So, I think there are two ways:

    The logging for this driver is too verbose, and I thought it would be
    better to implement a configurable mechanism which can enable/disable
    the log messages (not only via the global CONFIG_DYNAMIC_DEBUG).
    So, I think there are two ways:
    1. keep pr_debug but reduce the amount of messages
    2. make the logs configurabile via CONFIG_TI_SI_DEBUG


    Alexandra
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2012-03-15 22:19    [W:0.063 / U:0.112 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site