From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kishon Vijay Abraham I Subject: Re: [PATCH v9 0/8] Generic PHY Framework Date: Thu, 4 Jul 2013 15:25:32 +0530 Message-ID: <51D54694.20203@ti.com> References: <1372247257-30186-1-git-send-email-kishon@ti.com> <780E789C2E067A4BB8F69D0BB9EC4F253E975B5E@DBDE04.ent.ti.com> <51D3F773.9000209@ti.com> <20130703132038.GI15056@arwen.pp.htv.fi> <780E789C2E067A4BB8F69D0BB9EC4F253E975D6E@DBDE04.ent.ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <780E789C2E067A4BB8F69D0BB9EC4F253E975D6E@DBDE04.ent.ti.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: "Patel, Satish" Cc: "mchehab@redhat.com" , "linux-doc@vger.kernel.org" , "tony@atomide.com" , "Nori, Sekhar" , "Mankad, Maulik Ojas" , "swarren@nvidia.com" , "grant.likely@linaro.org" , "cesarb@cesarb.net" , "Cherian, George" , "arnd@arndb.de" , "devicetree-discuss@lists.ozlabs.org" , "Nayak, Rajendra" , "rob.herring@calxeda.com" , "linux@arm.linux.org.uk" , "sylvester.nawrocki@gmail.com" , "linux-omap@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "Krishnamoorthy, Balaji T" List-Id: linux-omap@vger.kernel.org T24gVGh1cnNkYXkgMDQgSnVseSAyMDEzIDAyOjUxIFBNLCBQYXRlbCwgU2F0aXNoIHdyb3RlOgo+ IEhpLAo+Cj4+IC0tLS0tT3JpZ2luYWwgTWVzc2FnZS0tLS0tCj4+IEZyb206IEJhbGJpLCBGZWxp cGUKPj4gU2VudDogV2VkbmVzZGF5LCBKdWx5IDAzLCAyMDEzIDY6NTEgUE0KPj4gVG86IEFCUkFI QU0sIEtJU0hPTiBWSUpBWQo+PiBDYzogUGF0ZWwsIFNhdGlzaDsgZ3JhbnQubGlrZWx5QGxpbmFy by5vcmc7IHRvbnlAYXRvbWlkZS5jb207IEJhbGJpLAo+PiBGZWxpcGU7IGFybmRAYXJuZGIuZGU7 IHN3YXJyZW5AbnZpZGlhLmNvbTsKPj4gc3lsdmVzdGVyLm5hd3JvY2tpQGdtYWlsLmNvbTsgbGlu dXgta2VybmVsQHZnZXIua2VybmVsLm9yZzsgbGludXgtCj4+IG9tYXBAdmdlci5rZXJuZWwub3Jn OyBsaW51eC1hcm0ta2VybmVsQGxpc3RzLmluZnJhZGVhZC5vcmc7IGxpbnV4LQo+PiB1c2JAdmdl ci5rZXJuZWwub3JnOyBncmVna2hAbGludXhmb3VuZGF0aW9uLm9yZzsgYWtwbUBsaW51eC0KPj4g Zm91bmRhdGlvbi5vcmc7IHJvYi5oZXJyaW5nQGNhbHhlZGEuY29tOyByb2JAbGFuZGxleS5uZXQ7 Cj4+IGxpbnV4QGFybS5saW51eC5vcmcudWs7IGJlbm9pdC5jb3Vzc29uQGxpbmFyby5vcmc7IG1j aGVoYWJAcmVkaGF0LmNvbTsKPj4gY2VzYXJiQGNlc2FyYi5uZXQ7IGRhdmVtQGRhdmVtbG9mdC5u ZXQ7IE5heWFrLCBSYWplbmRyYTsKPj4gc2hhd24uZ3VvQGxpbmFyby5vcmc7IFNoaWxpbWthciwg U2FudG9zaDsgZGV2aWNldHJlZS0KPj4gZGlzY3Vzc0BsaXN0cy5vemxhYnMub3JnOyBsaW51eC1k b2NAdmdlci5rZXJuZWwub3JnOyBOb3JpLCBTZWtoYXI7Cj4+IEtyaXNobmFtb29ydGh5LCBCYWxh amkgVDsgQ2hlcmlhbiwgR2VvcmdlCj4+IFN1YmplY3Q6IFJlOiBbUEFUQ0ggdjkgMC84XSBHZW5l cmljIFBIWSBGcmFtZXdvcmsKPj4KPj4gSGksCj4+Cj4+IE9uIFdlZCwgSnVsIDAzLCAyMDEzIGF0 IDAzOjM1OjM5UE0gKzA1MzAsIEtpc2hvbiBWaWpheSBBYnJhaGFtIEkKPj4gd3JvdGU6Cj4+PiBP biBXZWRuZXNkYXkgMDMgSnVseSAyMDEzIDAzOjAyIFBNLCBQYXRlbCwgU2F0aXNoIHdyb3RlOgo+ Pj4+IEhpIEtpc2hvbiwKPj4+Pgo+Pj4+PiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQo+Pj4+ PiBGcm9tOiBBQlJBSEFNLCBLSVNIT04gVklKQVkKPj4+Pj4gU2VudDogV2VkbmVzZGF5LCBKdW5l IDI2LCAyMDEzIDU6MTcgUE0KPj4+Pj4gVG86IGdyYW50Lmxpa2VseUBsaW5hcm8ub3JnOyB0b255 QGF0b21pZGUuY29tOyBCYWxiaSwgRmVsaXBlOwo+PiBBQlJBSEFNLAo+Pj4+PiBLSVNIT04gVklK QVk7IGFybmRAYXJuZGIuZGU7IHN3YXJyZW5AbnZpZGlhLmNvbTsKPj4+Pj4gc3lsdmVzdGVyLm5h d3JvY2tpQGdtYWlsLmNvbTsgbGludXgta2VybmVsQHZnZXIua2VybmVsLm9yZzsgbGludXgtCj4+ Pj4+IG9tYXBAdmdlci5rZXJuZWwub3JnOyBsaW51eC1hcm0ta2VybmVsQGxpc3RzLmluZnJhZGVh ZC5vcmc7IGxpbnV4LQo+Pj4+PiB1c2JAdmdlci5rZXJuZWwub3JnOyBncmVna2hAbGludXhmb3Vu ZGF0aW9uLm9yZzsgYWtwbUBsaW51eC0KPj4+Pj4gZm91bmRhdGlvbi5vcmcKPj4+Pj4gQ2M6IHJv Yi5oZXJyaW5nQGNhbHhlZGEuY29tOyByb2JAbGFuZGxleS5uZXQ7Cj4+IGxpbnV4QGFybS5saW51 eC5vcmcudWs7Cj4+Pj4+IGJlbm9pdC5jb3Vzc29uQGxpbmFyby5vcmc7IG1jaGVoYWJAcmVkaGF0 LmNvbTsgY2VzYXJiQGNlc2FyYi5uZXQ7Cj4+Pj4+IGRhdmVtQGRhdmVtbG9mdC5uZXQ7IE5heWFr LCBSYWplbmRyYTsgc2hhd24uZ3VvQGxpbmFyby5vcmc7Cj4+IFNoaWxpbWthciwKPj4+Pj4gU2Fu dG9zaDsgZGV2aWNldHJlZS1kaXNjdXNzQGxpc3RzLm96bGFicy5vcmc7IGxpbnV4LQo+Pj4+PiBk b2NAdmdlci5rZXJuZWwub3JnOyBOb3JpLCBTZWtoYXI7IEtyaXNobmFtb29ydGh5LCBCYWxhamkg VDsKPj4gQ2hlcmlhbiwKPj4+Pj4gR2VvcmdlCj4+Pj4+IFN1YmplY3Q6IFtQQVRDSCB2OSAwLzhd IEdlbmVyaWMgUEhZIEZyYW1ld29yawo+Pj4+Pgo+Pj4+PiBBZGRlZCBhIGdlbmVyaWMgUEhZIGZy YW1ld29yayB0aGF0IHByb3ZpZGVzIGEgc2V0IG9mIEFQSXMgZm9yIHRoZQo+PiBQSFkKPj4+Pj4g ZHJpdmVycwo+Pj4+PiB0byBjcmVhdGUvZGVzdHJveSBhIFBIWSBhbmQgQVBJcyBmb3IgdGhlIFBI WSB1c2VycyB0byBvYnRhaW4gYQo+Pj4+PiByZWZlcmVuY2UgdG8KPj4+Pj4gdGhlIFBIWSB3aXRo IG9yIHdpdGhvdXQgdXNpbmcgcGhhbmRsZS4KPj4+Pj4KPj4+Pj4gVGhpcyBmcmFtZXdvcmsgd2ls bCBiZSBvZiB1c2Ugb25seSB0byBkZXZpY2VzIHRoYXQgdXNlcyBleHRlcm5hbAo+PiBQSFkKPj4+ Pj4gKFBIWQo+Pj4+PiBmdW5jdGlvbmFsaXR5IGlzIG5vdCBlbWJlZGRlZCB3aXRoaW4gdGhlIGNv bnRyb2xsZXIpLgo+Pj4+Pgo+Pj4+PiBUaGUgaW50ZW50aW9uIG9mIGNyZWF0aW5nIHRoaXMgZnJh bWV3b3JrIGlzIHRvIGJyaW5nIHRoZSBwaHkKPj4gZHJpdmVycwo+Pj4+PiBzcHJlYWQKPj4+Pj4g YWxsIG92ZXIgdGhlIExpbnV4IGtlcm5lbCB0byBkcml2ZXJzL3BoeSB0byBpbmNyZWFzZSBjb2Rl IHJlLXVzZQo+PiBhbmQKPj4+Pj4gdG8KPj4+Pj4gaW5jcmVhc2UgY29kZSBtYWludGFpbmFiaWxp dHkuCj4+Pj4KPj4+PiBJIHdvdWxkIGxpa2UgdG8gdXNlIHRoaXMgZnJhbWV3b3JrIGZvciBhIHNt YXJ0LWNhcmQgY29udHJvbGxlcgo+PiBjb25uZWN0ZWQgdG8gYQo+Pj4+IHNtYXJ0LWNhcmQgcGh5 LiBJIGhhdmUgc29tZSBxdWVzdGlvbnMgYW5kIHdvdWxkIGxpa2UgdG8gZ2V0Cj4+IGZlZWRiYWNr IG9uIHRoZSBzYW1lLgo+Pj4KPj4+IGdsYWQgdG8ga25vdyB0aGF0IDotKQo+Pj4+Cj4+Pj4gSSBh bSB1c2luZyDigJxUREE4MDI2IiBTbWFydGNhcmQgUEhZIGZyb20gTlhQLiBIZXJlIGlzIHRoZSBs aW5rIGZvcgo+PiBkYXRhc2hlZXQKPj4+PiBhbmQgYXBwIG5vdGUgZm9yIHRoZSBzYW1lLiBUaGUg c21hcnQgY2FyZCBjb250cm9sbGVyIGlzIGluc2lkZSB0aGUKPj4gVEkgU29DCj4+Pj4gSSBhbSB3 b3JraW5nIHdpdGguCj4+Pj4KPj4+PiBEYXRhc2hlZXQgOgo+Pj4+IHd3dy5ueHAuY29tL2RvY3Vt ZW50cy9kYXRhX3NoZWV0L1REQTgwMjYucGRmPwo+Pj4+Cj4+Pj4gQXBwbm90ZSA6Cj4+Pj4gaHR0 cDovL3d3dy5ueHAuY29tL2RvY3VtZW50cy9hcHBsaWNhdGlvbl9ub3RlL0FOMTA3MjQucGRmCj4+ Pj4KPj4+PiBUaGUgVEkgU29DIGRldGFpbHMgYXJlIG5vdCBwdWJsaWMgKHlldCkuIEkgY2FuIHBy b3ZpZGUgZGV0YWlscyB0bwo+PiB5b3Ugb2ZmbGluZS4KPj4+Pgo+Pj4+IEJyaWVmIGFib3V0IG9w ZXJhdGlvbjoKPj4+PiAtCVRoZSBjb250cm9sbGVyIGNhbiB3b3JrIHdpdGggYW5kIHdpdGhvdXQg YSBQSFkKPj4+PiAtCVdoZW4gbm90IHVzaW5nIFBIWSwgaXQgaXMgbGltaXRlZCB0byB0YWxraW5n IHRvIGEgc2luZ2xlCj4+Pj4gCXNtYXJ0IGNhcmQuIFRoZXJlIGlzIGFsc28gYSBuZWVkIHRvIHB1 dCBleHRlcm5hbCBkZS1hY3RpdmF0aW9uCj4+IGxvZ2ljCj4+Pj4gCW9uIGNhcmQgcmVtb3ZhbCBm b3IgdGhpcyBjYXNlLgo+Pj4+IC0JV2l0aCBhIFBIWSB5b3UgY2FuIHVzZSBtb3JlIHRoYW4gb25l IHNtYXJ0IGNhcmQuCj4+Pj4gLQlQaHkgaGFzIDUgc2xvdHMgOiAgMSBmb3Igc21hcnQgY2FyZCAo Y3JlZGl0L2RlYml0L290aGVyIGNhcmQKPj4gd2l0aCBjaGlwKQo+Pj4+ICAgICAgICBhbmQgb3Ro ZXJzIGZvciBTQU0g4oCTIFNJTSBsaWtlIG1vZHVsZXMKPj4+PiAtIAlPbmNlIHRoZSBQSFkgaXMg aW5pdGlhbGl6ZWQsIHRoZXJlIGFyZSBzb21lIG9wZXJhdGlvbnMgdGhhdCB0aGUKPj4gY29udHJv bGxlcgo+Pj4+IAljYW4gcmVxdWVzdCBvZiB0aGUgUEhZIGxpa2U6Cj4+Pj4gCS0gQ2FyZCBjb25m aWd1cmF0aW9ucyAgLSBzZXQgdm9sdGFnZQo+Pj4+IAktIEFjdGl2YXRpb24gb2YgY2FyZAo+Pj4+ IAktIEFUUiDigJMgQW5zd2VyIHRvIHJlc2V0Cj4+Pj4gCS0gV2FybSByZXNldAo+Pj4+IAktIEFE UFUgZXhjaGFuZ2UKPj4+PiAJLSBEZWFjdGl2YXRpb24gKCBOb3JtYWwvRW1lcmdlbmN5KQo+Pj4K Pj4+IGhtbS4uIFdlIHNob3VsZCB0aGluayBhYm91dCBleHRlbmRpbmcgdGhlIHBoeV9vcHMgdG8g aW5jbHVkZSB0aGVzZQo+Pj4gb3BlcmF0aW9ucyAoc29tZXRoaW5nIGxpa2UgcGh5X3NtYXJ0X2Nh cmRfb3BzIHNvIHRoYXQgb3RoZXIKPj4+IHNtYXJ0X2NhcmQgUEhZcyB3aWxsIGFsc28gYmUgYWJs ZSB0byB1c2UgaXQpLgo+Pgo+PiBsZXQncyB0cnkgdG8gYXZvaWQgdXNlLWNhc2Ugc3BlY2lmaWMg YWRkaXRpb25zLiBzZXRfdm9sdGFnZSBzb3VuZHMKPj4gbGlrZQo+PiBhIHJlZ3VsYXRvciB0aGlu ZywgYnV0IHRoZSByZWd1bGF0b3IgaXMgY29udHJvbGxlZCB0aHJvdWdoIHRoZSBQSFkuIEkKPj4g Z3Vlc3MgaXQgbWFrZXMgc2Vuc2UgdG8gaGF2ZSBhIGdlbmVyaWMgcGh5X3NldF92b2x0YWdlKCkg Y2FsbCBzaW5jZQo+PiBldmVuCj4+IFVTQiBjYW4gbWFrZSB1c2Ugb2YgdGhhdC4KPj4KPj4gRm9y IGNhcmQgYWN0aXZhdGlvbiwgaXQgc291bmRzIGxpa2UgcGh5X2luaXQoKS9waHlfc2h1dGRvd24o KSB3b3VsZAo+PiBjb3ZlciBpdC4KPj4KPj4gRm9yIHdhcm0gcmVzZXQgcGVyaGFwcyBhIHBoeV9y ZXNldCgpIGNhbGxiYWNrID8gQWx0aG91Z2ggdGhhdCBjb3VsZCwKPj4gZWFzaWx5LCBnZXQgYWJ1 c2VkLgo+Pgo+PiBGb3IgZGVhY3RpdmF0aW9uLCB0aGF0J3MgcGh5X3NodXRkb3duKCkuCj4+Cj4+ IEFUUiBhbmQgQURQVSBuZWVkcyBtb3JlIHRob3VnaHQsIEkgZ3Vlc3MuCj4+Cj4+Pj4gLSAJSW4g dGhlIG1vZGUgd2hlbiBzbWFydGNhcmQgY29udHJvbGxlciB0YWxrcyBkaXJlY3RseSB0byB0aGUK Pj4gY2FyZCB3aXRob3V0IHRoZSBuZWVkCj4+Pj4gCWZvciBhIFBIWSwgYWxsIHRoZSBhYm92ZSBv cGVyYXRpb25zIHdpbGwgYmUgY2FycmllZCBvdXQgYnkgdGhlCj4+IGNvbnRyb2xsZXIgaXRzZWxm Cj4+Pj4KPj4+PiBNeSBjdXJyZW50IHRob3VnaHQgcHJvY2VzcyBpcyB0byBtYWtlIHRoZSBjb250 cm9sbGVyIGRyaXZlciBwcm92aWRlCj4+IHRoZSB1c2VyIGludGVyZmFjZQo+Pj4+IGFuZCB0YWxr IHRvIHRoZSBQSFkgdXNpbmcgdGhlIGdlbmVyaWMgUEhZIGZyYW1ld29yayB5b3UgcHJvcG9zZWQu Cj4+IEluIHRoZSBjYXNlIHdoZXJlIHRoZXJlCj4+Pj4gaXMgbm8gUEhZLCBteSBpZGVhIGlzIHRv IGNyZWF0ZSBhICJkdW1teSIgUEhZIHdoaWNoIHVzZXMgdGhlCj4+IGNvbnRyb2xsZXIgZnVuY3Rp b25hbGl0eSBpdHNlbGYuCj4+Pgo+Pj4gcmlnaHQuIEFuZCBpbiB0aGUgY2FzZSB3aGVyZSB5b3Ug YWN0dWFsbHkgaGF2ZSBhIFBIWSwgY3JlYXRlIGEgUEhZCj4+PiBkcml2ZXIgYW5kIGltcGxlbWVu dCB0aGUgcGh5X3NtYXJ0X2NhcmRfb3BzIGFuZCByZWdpc3RlciB3aXRoIHRoZQo+PiBQSFkKPj4+ IGZyYW1ld29yay4KPj4KPj4gSSB3b3VsZCB0cnkgdG8gYXZvaWQgdGhhdC4gT3RoZXJ3aXNlIHdl IHdpbGwgaGF2ZSBwaHlfdXNiX29wcywKPj4gcGh5X3NhdGFfb3BzLCBwaHlfbmV0d29ya19vcHMs IHBoeV9wY2lfb3BzLCBldGMgZXRjIGV0Yy4gSXQgd291bGQKPj4gZWFzaWx5Cj4+IGJsb3cgdXAu Cj4+Cj4KPiAtIEkgZG8gYWdyZWUgd2l0aCB5b3UuIENyZWF0aW5nIFBoeSBzcGVjaWZpYyBvcHMg d2lsbCBibG93IHVwIHdob2xlCj4gICAgY29uY2VwdCBvZiBnZW5lcmljIHBoeSBmL3cuCj4gLSBD YW4gd2UgaGF2ZSBpbnRlcmZhY2UgbGlrZSBwaHlfc2V0Y29uZmlnIC0gd2l0aCBwYXJhbWV0ZXIg bGlrZQo+ICAgIHBoeV9zZXRjb25maWcoaW50IHBhcmFtLCB2b2lkICp2YWx1ZSkKPiAJLSBIZXJl IHBhcmFtIGNhbiBiZSBlbnVtIG9mIGF2YWlsYWJsZSBjb25maWcgcGFyYW1ldGVycyBmb3IKPiAJ ICBzcGVjaWZpYyBwaHkuCj4gUGh5IGNhbiBwZXJmb3JtIGRpZmZlcmVudCBvcGVyYXRpb24vc2V0 IGludGVybmFsIHN0YXRlIGJhc2VkIG9uCj4gcGFyYW0gc2VsZWN0aW9uIGFuZCB2YWx1ZSBwYXNz ZWQgYnkuCj4gCQo+IGUueCBpbiBjYXNlIG9mIHNtYXJ0Y2FyZAo+IAllbnVtIHNldF9jb25maWcg ewo+IAkJU0VUX1ZPTEFUQUdFLAo+IAkJU0VUX0FDVElWQVRFLAo+IAkJU0VUX1dBUk1SRVNFVCwK PiAJCVNFVF9BVFIsCj4gCQlTRVRfREVBQ1RJVkUsCj4gCQkuLi4uCj4gCX07CgpobW0uLiB0aGlz IGxvb2tzIHNpbWlsYXIgdG8gaW9jdGwgYW5kIGNhbiBiZSBhYnVzZWQgZWFzaWx5IElNTyA6cwoK LUtpc2hvbgoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18K bGludXgtYXJtLWtlcm5lbCBtYWlsaW5nIGxpc3QKbGludXgtYXJtLWtlcm5lbEBsaXN0cy5pbmZy YWRlYWQub3JnCmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4vbGlzdGluZm8vbGlu dXgtYXJtLWtlcm5lbAo= From mboxrd@z Thu Jan 1 00:00:00 1970 From: kishon@ti.com (Kishon Vijay Abraham I) Date: Thu, 4 Jul 2013 15:25:32 +0530 Subject: [PATCH v9 0/8] Generic PHY Framework In-Reply-To: <780E789C2E067A4BB8F69D0BB9EC4F253E975D6E@DBDE04.ent.ti.com> References: <1372247257-30186-1-git-send-email-kishon@ti.com> <780E789C2E067A4BB8F69D0BB9EC4F253E975B5E@DBDE04.ent.ti.com> <51D3F773.9000209@ti.com> <20130703132038.GI15056@arwen.pp.htv.fi> <780E789C2E067A4BB8F69D0BB9EC4F253E975D6E@DBDE04.ent.ti.com> Message-ID: <51D54694.20203@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thursday 04 July 2013 02:51 PM, Patel, Satish wrote: > Hi, > >> -----Original Message----- >> From: Balbi, Felipe >> Sent: Wednesday, July 03, 2013 6:51 PM >> To: ABRAHAM, KISHON VIJAY >> Cc: Patel, Satish; grant.likely at linaro.org; tony at atomide.com; Balbi, >> Felipe; arnd at arndb.de; swarren at nvidia.com; >> sylvester.nawrocki at gmail.com; linux-kernel at vger.kernel.org; linux- >> omap at vger.kernel.org; linux-arm-kernel at lists.infradead.org; linux- >> usb at vger.kernel.org; gregkh at linuxfoundation.org; akpm at linux- >> foundation.org; rob.herring at calxeda.com; rob at landley.net; >> linux at arm.linux.org.uk; benoit.cousson at linaro.org; mchehab at redhat.com; >> cesarb at cesarb.net; davem at davemloft.net; Nayak, Rajendra; >> shawn.guo at linaro.org; Shilimkar, Santosh; devicetree- >> discuss at lists.ozlabs.org; linux-doc at vger.kernel.org; Nori, Sekhar; >> Krishnamoorthy, Balaji T; Cherian, George >> Subject: Re: [PATCH v9 0/8] Generic PHY Framework >> >> Hi, >> >> On Wed, Jul 03, 2013 at 03:35:39PM +0530, Kishon Vijay Abraham I >> wrote: >>> On Wednesday 03 July 2013 03:02 PM, Patel, Satish wrote: >>>> Hi Kishon, >>>> >>>>> -----Original Message----- >>>>> From: ABRAHAM, KISHON VIJAY >>>>> Sent: Wednesday, June 26, 2013 5:17 PM >>>>> To: grant.likely at linaro.org; tony at atomide.com; Balbi, Felipe; >> ABRAHAM, >>>>> KISHON VIJAY; arnd at arndb.de; swarren at nvidia.com; >>>>> sylvester.nawrocki at gmail.com; linux-kernel at vger.kernel.org; linux- >>>>> omap at vger.kernel.org; linux-arm-kernel at lists.infradead.org; linux- >>>>> usb at vger.kernel.org; gregkh at linuxfoundation.org; akpm at linux- >>>>> foundation.org >>>>> Cc: rob.herring at calxeda.com; rob at landley.net; >> linux at arm.linux.org.uk; >>>>> benoit.cousson at linaro.org; mchehab at redhat.com; cesarb at cesarb.net; >>>>> davem at davemloft.net; Nayak, Rajendra; shawn.guo at linaro.org; >> Shilimkar, >>>>> Santosh; devicetree-discuss at lists.ozlabs.org; linux- >>>>> doc at vger.kernel.org; Nori, Sekhar; Krishnamoorthy, Balaji T; >> Cherian, >>>>> George >>>>> Subject: [PATCH v9 0/8] Generic PHY Framework >>>>> >>>>> Added a generic PHY framework that provides a set of APIs for the >> PHY >>>>> drivers >>>>> to create/destroy a PHY and APIs for the PHY users to obtain a >>>>> reference to >>>>> the PHY with or without using phandle. >>>>> >>>>> This framework will be of use only to devices that uses external >> PHY >>>>> (PHY >>>>> functionality is not embedded within the controller). >>>>> >>>>> The intention of creating this framework is to bring the phy >> drivers >>>>> spread >>>>> all over the Linux kernel to drivers/phy to increase code re-use >> and >>>>> to >>>>> increase code maintainability. >>>> >>>> I would like to use this framework for a smart-card controller >> connected to a >>>> smart-card phy. I have some questions and would like to get >> feedback on the same. >>> >>> glad to know that :-) >>>> >>>> I am using ?TDA8026" Smartcard PHY from NXP. Here is the link for >> datasheet >>>> and app note for the same. The smart card controller is inside the >> TI SoC >>>> I am working with. >>>> >>>> Datasheet : >>>> www.nxp.com/documents/data_sheet/TDA8026.pdf? >>>> >>>> Appnote : >>>> http://www.nxp.com/documents/application_note/AN10724.pdf >>>> >>>> The TI SoC details are not public (yet). I can provide details to >> you offline. >>>> >>>> Brief about operation: >>>> - The controller can work with and without a PHY >>>> - When not using PHY, it is limited to talking to a single >>>> smart card. There is also a need to put external de-activation >> logic >>>> on card removal for this case. >>>> - With a PHY you can use more than one smart card. >>>> - Phy has 5 slots : 1 for smart card (credit/debit/other card >> with chip) >>>> and others for SAM ? SIM like modules >>>> - Once the PHY is initialized, there are some operations that the >> controller >>>> can request of the PHY like: >>>> - Card configurations - set voltage >>>> - Activation of card >>>> - ATR ? Answer to reset >>>> - Warm reset >>>> - ADPU exchange >>>> - Deactivation ( Normal/Emergency) >>> >>> hmm.. We should think about extending the phy_ops to include these >>> operations (something like phy_smart_card_ops so that other >>> smart_card PHYs will also be able to use it). >> >> let's try to avoid use-case specific additions. set_voltage sounds >> like >> a regulator thing, but the regulator is controlled through the PHY. I >> guess it makes sense to have a generic phy_set_voltage() call since >> even >> USB can make use of that. >> >> For card activation, it sounds like phy_init()/phy_shutdown() would >> cover it. >> >> For warm reset perhaps a phy_reset() callback ? Although that could, >> easily, get abused. >> >> For deactivation, that's phy_shutdown(). >> >> ATR and ADPU needs more thought, I guess. >> >>>> - In the mode when smartcard controller talks directly to the >> card without the need >>>> for a PHY, all the above operations will be carried out by the >> controller itself >>>> >>>> My current thought process is to make the controller driver provide >> the user interface >>>> and talk to the PHY using the generic PHY framework you proposed. >> In the case where there >>>> is no PHY, my idea is to create a "dummy" PHY which uses the >> controller functionality itself. >>> >>> right. And in the case where you actually have a PHY, create a PHY >>> driver and implement the phy_smart_card_ops and register with the >> PHY >>> framework. >> >> I would try to avoid that. Otherwise we will have phy_usb_ops, >> phy_sata_ops, phy_network_ops, phy_pci_ops, etc etc etc. It would >> easily >> blow up. >> > > - I do agree with you. Creating Phy specific ops will blow up whole > concept of generic phy f/w. > - Can we have interface like phy_setconfig - with parameter like > phy_setconfig(int param, void *value) > - Here param can be enum of available config parameters for > specific phy. > Phy can perform different operation/set internal state based on > param selection and value passed by. > > e.x in case of smartcard > enum set_config { > SET_VOLATAGE, > SET_ACTIVATE, > SET_WARMRESET, > SET_ATR, > SET_DEACTIVE, > .... > }; hmm.. this looks similar to ioctl and can be abused easily IMO :s -Kishon