From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrzej Hajda Subject: Re: [RFC 04/15] regulator: add restrack support Date: Fri, 12 Dec 2014 09:22:29 +0100 Message-ID: <548AA5C5.4060204@samsung.com> References: <1418226513-14105-1-git-send-email-a.hajda@samsung.com> <1418226513-14105-5-git-send-email-a.hajda@samsung.com> <20141210160720.GS11764@sirena.org.uk> <548976D2.7080005@samsung.com> <20141211125837.GV11764@sirena.org.uk> <20141211134327.GO11285@n2100.arm.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-reply-to: <20141211134327.GO11285@n2100.arm.linux.org.uk> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Russell King - ARM Linux , Mark Brown Cc: Alexandre Courbot , "moderated list:ARM/S5P EXYNOS AR..." , Mike Turquette , Liam Girdwood , Greg Kroah-Hartman , open list , Rob Herring , Kishon Vijay Abraham I , "open list:GPIO SUBSYSTEM" , "open list:OPEN FIRMWARE AND..." , "open list:DRM PANEL DRIVERS" , Grant Likely , "moderated list:ARM/CLKDEV SUPPORT" , Marek Szyprowski List-Id: linux-gpio@vger.kernel.org T24gMTIvMTEvMjAxNCAwMjo0MyBQTSwgUnVzc2VsbCBLaW5nIC0gQVJNIExpbnV4IHdyb3RlOgo+ IE9uIFRodSwgRGVjIDExLCAyMDE0IGF0IDEyOjU4OjM3UE0gKzAwMDAsIE1hcmsgQnJvd24gd3Jv dGU6Cj4+IEknZCBleHBlY3Qgc29tZW9uZSByZWFkaW5nIHRoZSBjaGFuZ2UgaW4gdGhlIHJlZ3Vs YXRvciBBUEkgdG8gaGF2ZSBhdAo+PiBsZWFzdCBzb21lIGlkZWEgaG93IHRoaXMgZml0cyBpbiB3 aXRoIHRoZSByZXN0IG9mIHRoZSBBUEkgYW5kIGhvdyB0byB1c2UKPj4gaXQsIGFuZCBwcm9iYWJs eSBtb3JlIGltcG9ydGFudGx5IEknZCBleHBlY3QgdG8gYmUgYWJsZSB0byB1bmRlcnN0YW5kCj4+ IHdoeSB0aGlzIGlzIERUIG9ubHkuCj4gCj4gWWVwLgo+IAo+IFRoaXMgaXMgYSByZXBldGl0aXZl IHByb2JsZW0sIGFuZCBJIGZ1bGx5IGFncmVlIHdpdGggeW91ciBjb25jZXJuIGFib3V0Cj4gc3R1 ZmYgd2hpY2ggaXMgc3VwcG9zZWQgdG8gYmUgYXJjaC1pbmRlcGVuZGVudCBiZWluZyBkZXNpZ25l ZCB3aXRoIG9ubHkKPiBEVCBpbiBtaW5kLgo+IAo+IE5ldyBjb3JlIGtlcm5lbCBmZWF0dXJlcyBz aG91bGQgKm5vdCogYmUgZGVzaWduZWQgd2l0aCBvbmx5IERUIGluIG1pbmQgLQo+IERUIGlzIG5v dCB0aGUgb25seSBmaXJtd2FyZSBkZXNjcmlwdGlvbiBsYW5ndWFnZSB3aGljaCB0aGUga2VybmVs Cj4gc3VwcG9ydHMuICBGb2xrIG5lZWQgdG8gdW5kZXJzdGFuZCB0aGF0IGlmIHRoZXkgZGVzaWdu IGEgbmV3IGFyY2gKPiBpbmRlcGVuZGVudCBrZXJuZWwgZmVhdHVyZSB3aGVyZSB0aGUgc29sZSB1 c2UgY2FzZSBpcyB3aXRoIERULCB0aGF0IG5ldwo+IGZlYXR1cmUgaXMgcHJvYmFibHkgZ29pbmcg dG8gZ2V0IHJlamVjdGVkLCBlc3BlY2lhbGx5IHdoZW4gaXQncwo+IHNvbWV0aGluZyBhcyBnZW5l cmljIGFzIHJlc291cmNlIHRyYWNraW5nLgo+IAo+IFRoZSB3b3JsZCBpcyBub3QgRFQgb25seS4K PiAKCk9LLiBJIHdpbGwgcG9zdCBuZXh0IHZlcnNpb24gb2YgcGF0Y2hzZXQgd2l0aCByZXNvdXJj ZS9wcm92aWRlciBsb29rdXAKbGVmdCB0byBmcmFtZXdvcmtzIChyZWd1bGF0b3JzLCBjbG9jaywg ZXRjKSwgdGhpcyB3YXkgaXQgd2lsbCBiZSBmdWxseQpmaXJtd2FyZSBhZ25vc3RpYy4gSSB3aWxs IGFkZCBhbHNvIGJldHRlciBkZXNjcmlwdGlvbiBvZiB0aGUgZnJhbWV3b3JrLgoKUmVnYXJkcwpB bmRyemVqCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmRy aS1kZXZlbCBtYWlsaW5nIGxpc3QKZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRw Oi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: a.hajda@samsung.com (Andrzej Hajda) Date: Fri, 12 Dec 2014 09:22:29 +0100 Subject: [RFC 04/15] regulator: add restrack support In-Reply-To: <20141211134327.GO11285@n2100.arm.linux.org.uk> References: <1418226513-14105-1-git-send-email-a.hajda@samsung.com> <1418226513-14105-5-git-send-email-a.hajda@samsung.com> <20141210160720.GS11764@sirena.org.uk> <548976D2.7080005@samsung.com> <20141211125837.GV11764@sirena.org.uk> <20141211134327.GO11285@n2100.arm.linux.org.uk> Message-ID: <548AA5C5.4060204@samsung.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 12/11/2014 02:43 PM, Russell King - ARM Linux wrote: > On Thu, Dec 11, 2014 at 12:58:37PM +0000, Mark Brown wrote: >> I'd expect someone reading the change in the regulator API to have at >> least some idea how this fits in with the rest of the API and how to use >> it, and probably more importantly I'd expect to be able to understand >> why this is DT only. > > Yep. > > This is a repetitive problem, and I fully agree with your concern about > stuff which is supposed to be arch-independent being designed with only > DT in mind. > > New core kernel features should *not* be designed with only DT in mind - > DT is not the only firmware description language which the kernel > supports. Folk need to understand that if they design a new arch > independent kernel feature where the sole use case is with DT, that new > feature is probably going to get rejected, especially when it's > something as generic as resource tracking. > > The world is not DT only. > OK. I will post next version of patchset with resource/provider lookup left to frameworks (regulators, clock, etc), this way it will be fully firmware agnostic. I will add also better description of the framework. Regards Andrzej From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965852AbaLLIWg (ORCPT ); Fri, 12 Dec 2014 03:22:36 -0500 Received: from mailout3.w1.samsung.com ([210.118.77.13]:57091 "EHLO mailout3.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933414AbaLLIWe (ORCPT ); Fri, 12 Dec 2014 03:22:34 -0500 X-AuditID: cbfec7f5-b7fc86d0000066b7-83-548aa5c7c7c1 Message-id: <548AA5C5.4060204@samsung.com> Date: Fri, 12 Dec 2014 09:22:29 +0100 From: Andrzej Hajda User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 MIME-version: 1.0 To: Russell King - ARM Linux , Mark Brown Cc: open list , Marek Szyprowski , Greg Kroah-Hartman , Mike Turquette , Linus Walleij , Alexandre Courbot , Thierry Reding , Inki Dae , Kishon Vijay Abraham I , Liam Girdwood , Grant Likely , Rob Herring , "moderated list:ARM/CLKDEV SUPPORT" , "open list:GPIO SUBSYSTEM" , "open list:DRM PANEL DRIVERS" , "moderated list:ARM/S5P EXYNOS AR..." , "open list:OPEN FIRMWARE AND..." Subject: Re: [RFC 04/15] regulator: add restrack support References: <1418226513-14105-1-git-send-email-a.hajda@samsung.com> <1418226513-14105-5-git-send-email-a.hajda@samsung.com> <20141210160720.GS11764@sirena.org.uk> <548976D2.7080005@samsung.com> <20141211125837.GV11764@sirena.org.uk> <20141211134327.GO11285@n2100.arm.linux.org.uk> In-reply-to: <20141211134327.GO11285@n2100.arm.linux.org.uk> Content-type: text/plain; charset=windows-1252 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFprAIsWRmVeSWpSXmKPExsVy+t/xq7rHl3aFGNybIWkx9eETNov5R86x Wlz5+p7N4tyrRywWB/7sYLRoXryezWLS/QksFhee9rBZfLvSwWQx5c9yJotNj6+xWmye/4fR 4vKuOWwWM87vY7K4fZnXYu2Ru+wWTydcZLNo3XuE3eLnrnksDsIeLc09bB47Z91l99i0qpPN 4861PWwe++euYfe4332cyWPzknqPvi2rGD2O39jO5PF5k1wAVxSXTUpqTmZZapG+XQJXxppn X9kLLnJWnF02jbWB8TR7FyMnh4SAicSaK++hbDGJC/fWs3UxcnEICSxllDgxZRmU84lR4uPq JcwgVbwCWhJNO86C2SwCqhITp18B62YT0JT4u/kmG4gtKhAh8WHVVzaIekGJH5PvsYDYIkDx Q0/mgsWZBc6zSayZnQViCwtYSMw+sYgdYtkCJon2k41gRZwCNhKL7rUDNXMANehJ3L+oBdEr L7F5zVvmCYwCs5CsmIVQNQtJ1QJG5lWMoqmlyQXFSem5RnrFibnFpXnpesn5uZsYIbH4dQfj 0mNWhxgFOBiVeHhfpHaFCLEmlhVX5h5ilOBgVhLh/RsFFOJNSaysSi3Kjy8qzUktPsTIxMEp 1cDo1SL/8rjHqp6eS8crxb9scFobXTjbo1TrW4m9zdXI/7t0LR9/+NLLHSJVIngvRuCOpuKk 54ZWTpNnhd74+3+5a0jJHye2Zw0KwW1/IxZyLjl5QrA57dGBVWs1bP5Olb549YHV40dL+liO TnCwupObdmN16gwRHVOxmYxi61cdrnObF2F7YEawEktxRqKhFnNRcSIAj78xKKMCAAA= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 12/11/2014 02:43 PM, Russell King - ARM Linux wrote: > On Thu, Dec 11, 2014 at 12:58:37PM +0000, Mark Brown wrote: >> I'd expect someone reading the change in the regulator API to have at >> least some idea how this fits in with the rest of the API and how to use >> it, and probably more importantly I'd expect to be able to understand >> why this is DT only. > > Yep. > > This is a repetitive problem, and I fully agree with your concern about > stuff which is supposed to be arch-independent being designed with only > DT in mind. > > New core kernel features should *not* be designed with only DT in mind - > DT is not the only firmware description language which the kernel > supports. Folk need to understand that if they design a new arch > independent kernel feature where the sole use case is with DT, that new > feature is probably going to get rejected, especially when it's > something as generic as resource tracking. > > The world is not DT only. > OK. I will post next version of patchset with resource/provider lookup left to frameworks (regulators, clock, etc), this way it will be fully firmware agnostic. I will add also better description of the framework. Regards Andrzej