From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: Minimal support for dm814x Date: Wed, 11 Nov 2015 09:40:29 -0800 Message-ID: <20151111174028.GF3218@atomide.com> References: <4999BB3D-4BB5-4F7C-96D7-FB687725CDCC@audioscience.com> <20151109150602.GR3199@atomide.com> <7933071D-84FD-4A85-8CD0-CC504CE1D9D1@audioscience.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <7933071D-84FD-4A85-8CD0-CC504CE1D9D1@audioscience.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Delio Brignoli Cc: "linux-omap@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , Matthijs van Duin List-Id: linux-omap@vger.kernel.org KiBEZWxpbyBCcmlnbm9saSA8ZGJyaWdub2xpQGF1ZGlvc2NpZW5jZS5jb20+IFsxNTExMTAgMDI6 MjRdOgo+IE9uIDEwIE5vdiAyMDE1LCBhdCAwOTo1MCwgTWF0dGhpanMgdmFuIER1aW4gPG1hdHRo aWpzdmFuZHVpbkBnbWFpbC5jb20+IHdyb3RlOgo+ID4gT24gOSBOb3ZlbWJlciAyMDE1IGF0IDE2 OjA2LCBUb255IExpbmRncmVuIDx0b255QGF0b21pZGUuY29tPiB3cm90ZToKPiA+PiBUaGUgUExM IHN1cHBvcnQgaXMgc3RpbGwgbWlzc2luZywgc28gaXQgcmVsaWVzIG9uIHRoZSBib290bG9hZGVy Cj4gPj4gY29uZmlndXJlZCBQTEwgdmFsdWVzIGZvciBub3cuIEknbSBob3BpbmcgdG8gcG9zdCBQ TEwgc3VwcG9ydCBwYXRjaGVzIG92ZXIKPiA+PiBuZXh0IGZldyB3ZWVrcyBhbmQgdGhlbiB3ZSBj YW4gaGF2ZSB0aGF0IGFuZCBtb3JlIGRldmljZXMgd29ya2luZyBmb3IgdjQuNS4KPiA+IAo+ID4g QWgsIHllcywgY29uZmlndXJpbmcgYSBEUExMLUxKIGlzIGZ1bi4uIGZpZ3VyaW5nIG91dCBob3cg dG8gd3JpdGUgdGhlCj4gPiBkZXNpcmVkIHJhdGlvIGFzIE0vKE0yKigxK04pKSB3aGlsZSBzaW11 bHRhbmVvdXNseSBzYXRpc2Z5aW5nIGFsbAo+ID4gY29uc3RyYWludHMgb24gTSwgTiwgTTIsIHJl ZmNsaywgYW5kIGRjby4gOi0pCj4gCj4gWWVzLCBpbmRlZWQuIFdlIGhhdmUgdGhlIGFkZGl0aW9u YWwgcmVxdWlyZW1lbnQgb2YgYmVpbmcgYWJsZSB0byBhZGp1c3QgdGhlIGZyZXF1ZW5jeSAoYnkg YSByZWxhdGl2ZWx5IHNtYWxsIGFtb3VudCkgd2l0aG91dCBsb3NzIG9mIGxvY2suIFJlY2FsY3Vs YXRpbmcgdGhlIERDTyBtb2RlIGFuZCBNLE4sTTIgZnJvbSBzY3JhdGNoIGVhY2ggdGltZSBiYXNl ZCBvbiB0aGUgdGFyZ2V0IGZyZXF1ZW5jeSwgbGlrZSB3YXMgZG9uZSBpbiB0aGUgMi42LjM3IGJh c2VkIHRyZWUgZnJvbSBUSSB3YXMgbm90IGFjY2VwdGFibGUsIHNvIHdlIHRyeSB0byBjaGFuZ2Ug bTIgZmlyc3QgdG8gc2VlIGlmIHdlIGNhbiByZWFjaCB0aGUgdGFyZ2V0IGZyZXF1ZW5jeSBhbmQg ZmFsbCBiYWNrIHRvIHJlY2FsY3VsYXRlIHBhcmFtZXRlcnMgZnJvbSBzY3JhdGNoIGlmIHRoYXQg ZmFpbHMuCgpXZWxsIHdlIGRvIGZpcnN0IHRyeSB0byBzZXQgdGhlIHJhdGUgdXNpbmcgdGhlIGRp dmlkZXIgb25seSBhdCBsZWFzdCBmb3IKZHJpdmVycy9jbGsvdGkvZmFwbGwuYyB1c2VkIG9uIGRt ODE2eC4gSSdtIHRoaW5raW5nIGFib3V0IGRvaW5nIGEgc2ltaWxhcgpkcml2ZXIgZm9yIHRoZSBk bTgxNHggYWRwbGwgd2hlcmUgd2UgaGF2ZSBhIFBMTCBhbmQgc2VwYXJhdGUgb3V0cHV0IGNsb2Nr cwppbiBhIHNpbmdsZSBkcml2ZXIgYXMgdGhlIFBMTCBhbmQgb3V0cHV0IGNvbnRyb2wgcmVnaXN0 ZXJzIGFyZSBhbGwgbWl4ZWQKaW4uCgo+IEJUVywgYXJlIHlvdSBhd2FyZSBvZiBzZWN0aW9uIDIu MS4yIG9mIOKAnFRNUzMyMERNODE0eCBEYVZpbmNpIERpZ2l0YWwgTWVkaWEgUHJvY2Vzc29ycyBT aWxpY29uIFJldmlzaW9ucyAzLjAsIDIuMeKAnT8gPGh0dHA6Ly93d3cudGkuY29tL2xpdC9lci9z cHJ6MzQzYy9zcHJ6MzQzYy5wZGY+CgpPSyBnb29kIHRvIGtub3cgOikKClRvbnkKCl9fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmxpbnV4LWFybS1rZXJuZWwg bWFpbGluZyBsaXN0CmxpbnV4LWFybS1rZXJuZWxAbGlzdHMuaW5mcmFkZWFkLm9yZwpodHRwOi8v bGlzdHMuaW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LWFybS1rZXJuZWwK From mboxrd@z Thu Jan 1 00:00:00 1970 From: tony@atomide.com (Tony Lindgren) Date: Wed, 11 Nov 2015 09:40:29 -0800 Subject: Minimal support for dm814x In-Reply-To: <7933071D-84FD-4A85-8CD0-CC504CE1D9D1@audioscience.com> References: <4999BB3D-4BB5-4F7C-96D7-FB687725CDCC@audioscience.com> <20151109150602.GR3199@atomide.com> <7933071D-84FD-4A85-8CD0-CC504CE1D9D1@audioscience.com> Message-ID: <20151111174028.GF3218@atomide.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org * Delio Brignoli [151110 02:24]: > On 10 Nov 2015, at 09:50, Matthijs van Duin wrote: > > On 9 November 2015 at 16:06, Tony Lindgren wrote: > >> The PLL support is still missing, so it relies on the bootloader > >> configured PLL values for now. I'm hoping to post PLL support patches over > >> next few weeks and then we can have that and more devices working for v4.5. > > > > Ah, yes, configuring a DPLL-LJ is fun.. figuring out how to write the > > desired ratio as M/(M2*(1+N)) while simultaneously satisfying all > > constraints on M, N, M2, refclk, and dco. :-) > > Yes, indeed. We have the additional requirement of being able to adjust the frequency (by a relatively small amount) without loss of lock. Recalculating the DCO mode and M,N,M2 from scratch each time based on the target frequency, like was done in the 2.6.37 based tree from TI was not acceptable, so we try to change m2 first to see if we can reach the target frequency and fall back to recalculate parameters from scratch if that fails. Well we do first try to set the rate using the divider only at least for drivers/clk/ti/fapll.c used on dm816x. I'm thinking about doing a similar driver for the dm814x adpll where we have a PLL and separate output clocks in a single driver as the PLL and output control registers are all mixed in. > BTW, are you aware of section 2.1.2 of ?TMS320DM814x DaVinci Digital Media Processors Silicon Revisions 3.0, 2.1?? OK good to know :) Tony