[lkml]   [2012]   [Feb]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRE: [PATCH 1/3] pinctrl: Introduce PINCTRL_STATE_DEFAULT define, and use it
    Dong Aisheng wrote at Thursday, February 23, 2012 8:25 PM:
    > On Fri, Feb 24, 2012 at 08:04:38AM +0800, Stephen Warren wrote:
    > > This provides a single centralized name for the default state.
    > >
    > > Update PIN_MAP_* macros to use this state name, instead of requiring the
    > > user to pass a state name in.
    > > diff --git a/Documentation/pinctrl.txt b/Documentation/pinctrl.txt
    > > - PIN_MAP("I2CMAP", "pinctrl-foo", "i2c0", "foo-i2c.0"),
    > > + PIN_MAP(PINCTRL_STATE_DEFAULT, "pinctrl-foo", "i2c0", "foo-i2c.0"),
    > To keep align with the following
    > PIN_MAP_SYS_HOG("pinmux-u300", "power"),
    > Maybe PIN_MAP("pinctrl-foo", "i2c0", "foo-i2c.0") is better, right?
    > And we may want to add a PIN_MAP_STAT macro, as well as PIN_MAP_SYS_HOG_STAT.

    I don't think so.

    PIN_MAP_SYS_HOG hard-codes the state name inside the macro because its
    sole purpose is to support drivers/pinctrl/core.c's pinctrl_get() call
    for pin hogging purposes, so that state name is always "default", or

    The same isn't true of PIN_MAP; it's a general purpose macro intended for
    any device/situation/... I'd rather not keep inventing tons of macros in
    consumer.h and clutter it up.

    I guess if you really want, you can submit a patch to add a macro e.g.
    PIN_MAP_MUX_GROUP_DEFAULT() which hard-codes the default state name if
    you wish, following on from my patch to extend the mapping table to
    support pin config.

    BTW, did you have any further thoughts on (not) allowing NULL state names?
    I'd like to repost the final version of that patch, or rework it so that
    the code actually does allow NULL state names ASAP (i.e. early Friday for
    me), since it blocks some of the later patches in the series. Thanks.


     \ /
      Last update: 2012-02-24 06:29    [W:0.023 / U:4.800 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site