[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <ee95c747-4e24-4cad-80d2-b13f2e704122@lechnology.com>
Date: Tue, 17 Jan 2017 12:31:56 -0600
From: David Lechner <david@...hnology.com>
To: Sekhar Nori <nsekhar@...com>,
Bartosz Golaszewski <bgolaszewski@...libre.com>
Cc: Kevin Hilman <khilman@...nel.org>,
Patrick Titiano <ptitiano@...libre.com>,
Michael Turquette <mturquette@...libre.com>,
Tejun Heo <tj@...nel.org>, Rob Herring <robh+dt@...nel.org>,
Mark Rutland <mark.rutland@....com>,
Russell King <linux@...linux.org.uk>,
linux-ide@...r.kernel.org,
linux-devicetree <devicetree@...r.kernel.org>,
LKML <linux-kernel@...r.kernel.org>,
arm-soc <linux-arm-kernel@...ts.infradead.org>
Subject: davinci common clock framework (was Re: [PATCH 03/10] devicetree:
bindings: add bindings for ahci-da850)
On 01/17/2017 06:00 AM, Sekhar Nori wrote:
> On Tuesday 17 January 2017 12:17 AM, David Lechner wrote:
>> On 01/16/2017 08:30 AM, Bartosz Golaszewski wrote:
>>> 2017-01-16 13:45 GMT+01:00 Sekhar Nori <nsekhar@...com>:
>>>> On Monday 16 January 2017 03:43 PM, Bartosz Golaszewski wrote:
>>>
>>> It's true that once davinci gets ported (is this planned?) to using
>>> the common clock framework, we could just create a fixed-clock node in
>>> da850-lcdk for the SATA oscillator, so the new property is redundant.
>>>
>>
>> I have some commits[1] where I started on converting da850 to use the
>> common clock framework. But, I don't know anything about other davinci
>> family devices, so I don't think I could really take that to completion
>> without lots of help.
>
> I can help with testing, reviewing and filling in any missing
> information. But I wont have time to write the code itself.
>
>>
>> [1]: https://github.com/dlech/ev3dev-kernel/commits/wip-20160509
>
> I see that you have made a copy of the keystone PSC driver. I think you
> will need pretty strong reasons to not use the same driver with some
> customization for DaVinci.
>
It has been a while since I looked at this, but as I recall, the device
tree bindings for keystone are horrible and make no sense. So, I made
new bindings that make more sense. But since we can't break backwards
compatibility in device tree, I made a new driver rather than having the
mess of supporting two very different bindings in one driver. I don't
know if that is a strong enough reason, but that is why I did it. :-)
Powered by blists - more mailing lists