Messages in this thread | | | From | "Rafael J. Wysocki" <> | Date | Tue, 16 Oct 2018 10:44:45 +0200 | Subject | Re: [RFC PATCH 0/5] device property: Introducing software nodes |
| |
On Tue, Oct 16, 2018 at 10:40 AM Heikki Krogerus <heikki.krogerus@linux.intel.com> wrote: > > On Tue, Oct 16, 2018 at 09:36:33AM +0200, Rafael J. Wysocki wrote: > > On Tue, Oct 16, 2018 at 9:35 AM Linus Walleij <linus.walleij@linaro.org> wrote: > > > > > > On Fri, Oct 12, 2018 at 1:39 PM Heikki Krogerus > > > <heikki.krogerus@linux.intel.com> wrote: > > > > > > > To continue the discussion started by Dmitry [1], this is my proposal > > > > that I mentioned in my last mail. In short, the idea is that instead > > > > of trying to extend the support for the currently used struct > > > > property_set, I'm proposing that we introduce a completely new, > > > > independent type of fwnode, and replace the struct property_set with > > > > it. I'm calling the type "software node" here. > > > > > > I'm a big fan of this approach. > > > Acked-by: Linus Walleij <linus.walleij@linaro.org> > > > for all patches. > > > > > > I don't know who can finally review and merge this though, > > > I guess Rafael? > > > > Yes, that would be me. :-) > > > > I no one speaks up against them, I'll pick them up. > > Let me send a final version of these. > > I need to add one more patch to the series where I remove an extra > device_remove_properties() call from platform_device_del(). > > It's unnecessary in any case as device_del() calls > device_remove_properties() for every device, but as the properties are > removed there before the device is removed, we're unable to deduct > the final ref count in the "remove" platform notification since our > node is no longer bind to the device.
OK, I'll wait for an update, then.
Thanks, Rafael
| |