Re: [PATCH 1/5 v10] dt/bindings: Add binding for the DA8xx MUSB driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 






On 11.03.2016 17:06, Bin Liu wrote:
Hi,

On Fri, Mar 11, 2016 at 04:58:52PM +0100, Petr Kulhavy wrote:

On 11.03.2016 16:51, Bin Liu wrote:
Hi,

On Fri, Mar 11, 2016 at 09:29:45AM +0100, Petr Kulhavy wrote:
DT binding for the TI DA8xx/OMAP-L1x/AM17xx/AM18xx MUSB driver.
It seems the dts(i) file is not touched in this patch set, so DA8xx MUSB
still does not use DT with these patches?  Or did I miss anything?

Thanks,
-Bin.
Hi Bin,

I have not touched the DA8xx dtsi file as I have these changes in my
own board specific dts.
But I can create another patch if you find it sensible. For me it
makes sense.
Okay, that is up to you, you can submit that patch whenever you are
ready. I will take this patch set when all comments are sorted out.
I will submit that patch once the DT is there.

I am having 2nd thought on parsing the clock prop, Sergei's comment
might be better. I will look more on this over this weekend. (DT is not
in my expertise...)

Regards,
-Bin.

I like Sergei's comment as well, but cannot see (yet) how the clock input selection would be done. I mean, it makes sense to do the clock abstraction only if it can be done properly and the clock input selection can be covered as well. The DA8xx platform is missing the real clock framework and therefore the different clocks cannot be referenced in DT. There is a fake clock framework in arch/arm/mach-davinci/clock.c - I've already been through that and then gave up.

So if you come up with some good idea it's appreciated.

Regards
Petr
--
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



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux