* Stephen Warren <swarren@xxxxxxxxxxxxx> [130924 13:03]: > On 09/23/2013 05:46 PM, Sebastian Reichel wrote: > > Hi, > > > > On Mon, Sep 23, 2013 at 02:35:35PM -0600, Stephen Warren wrote: > >> On 09/15/2013 02:44 PM, Sebastian Reichel wrote: > >>> Add SSI device tree data for OMAP34xx and Nokia N900. > > ... > >>> +- ti,hwmods: Name of the hwmod associated to the controller, > >>> which + is "ssi". > >> > >> I don't think we should add any more of that, for new bindings. > > > > That basically means not adding new drivers until hwmod is > > completly removed, since no new drivers not using DT are accepted > > anymore. > > > > hwmod still holds some information, which are not yet mapped to > > DT. > > Tony, is defining hwmod properties for new OMAP bindings what everyone > is currently doing? I'm not sure how that will work with a stable DT > ABI... > > I wonder if it makes sense not to define the ti,hwmods property in the > binding document (so it doesn't become part of the ABI), but put it > into the DTS file simply to make the current Linux code work? I'm not > sure if that's any better though. There are still pieces of information in hwmod that is needed for PM to work for things like device autoidle and reset. The plan is to describe those with Linux generic bindings, but that's going to take a while. Clearly once we have those bindings, ti,hwmods is not needed and then the problem is how to support old .dtb files.. > It would be best to completely define the DT binding so that all > required clocks etc. are already present in the DT. That way, the DT > ABI won't change once people stop using hwmods. Tony, is that possible > on OMAP at present, irrespective of whether those e.g. clock > properties will actually be used by Linux? Unfortunately not doable yet. For v3.13 we should have the clock bindings in place, but quite a bit of work still remains for the remaining hwmod pieces. Considering all this, we may be best off dropping ti,hwmods binding completely, and populate it based on DT auxdata or something similar. That means that old .dtb files won't work once we populate things from DT, but at least the binding issue would be sorted out. If anybody has better suggestions, let me know. Regards, Tony -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html