lkml.org 
[lkml]   [2010]   [Mar]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] driver core: Early dev_name() support.
    On Thu, Mar 11, 2010 at 08:59:18PM +0100, Kay Sievers wrote:
    > On Tue, Mar 9, 2010 at 07:57, Paul Mundt <lethal@linux-sh.org> wrote:
    > > This implements early init_name construction from the matched name/id
    > > pair following the semantics of the late device/driver match. As a
    > > result, matched IDs (inclusive of requested ones) are preserved when the
    > > handoff from the early platform code happens at kobject initialization
    > > time.
    >
    > > + ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? if (!match->dev.init_name) {
    > > + ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? char buf[32];
    > > +
    > > + ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? if (match->id != -1)
    > > + ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? snprintf(buf, sizeof(buf), "%s.%d",
    > > + ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ??match->name, match->id);
    > > + ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? else
    > > + ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? snprintf(buf, sizeof(buf), "%s",
    > > + ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ??match->name);
    > > +
    > > + ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? match->dev.init_name = kstrdup(buf, GFP_KERNEL);
    > > + ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? if (!match->dev.init_name)
    > > + ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? return -ENOMEM;
    > > + ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? }
    >
    > This can be kasprintf() I guess.
    >
    Yeah, that looks fine. I'll update it.

    > > ??static inline const char *dev_name(const struct device *dev)
    > > ??{
    > > + ?? ?? ?? /* Use the init name until the kobject becomes available */
    >
    > This should probably state that it's used for getting names out of
    > unregistered devices. Otherwise it sounds confusing.
    >
    It's not clear that that distinction makes things any less confusing.
    Early devices are registered, they just haven't been fully initialized
    yet.


    \
     
     \ /
      Last update: 2010-03-15 09:07    [W:0.041 / U:29.520 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site