[RFCv2 3/3] ARM: dts: N900: Add SSI information

* Stephen Warren [off-list ref] [130924 13:03]:

On 09/23/2013 05:46 PM, Sebastian Reichel wrote:

quoted

Hi,

On Mon, Sep 23, 2013 at 02:35:35PM -0600, Stephen Warren wrote:

quoted

On 09/15/2013 02:44 PM, Sebastian Reichel wrote:

quoted

Add SSI device tree data for OMAP34xx and Nokia N900.

...

quoted

quoted

quoted

+- 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