From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eduardo Valentin Subject: Re: [PATCH v3 00/23] Add multi-channel support to Armada thermal driver Date: Thu, 26 Jul 2018 14:35:18 -0700 Message-ID: <20180726213517.GA15690@localhost.localdomain> References: <20180716144206.30985-1-miquel.raynal@bootlin.com> <20180718172849.30b7cec0@xps13> <20180725095222.054f1e80@xps13> <1532570441.2358.27.camel@intel.com> <20180726211442.GB3487@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <20180726211442.GB3487@localhost.localdomain> 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: Zhang Rui Cc: Mark Rutland , Andrew Lunn , Jason Cooper , devicetree@vger.kernel.org, Antoine Tenart , Catalin Marinas , Gregory Clement , linux-pm@vger.kernel.org, Will Deacon , Maxime Chevallier , Nadav Haklai , David Sniatkiwicz , Rob Herring , Thomas Petazzoni , Miquel Raynal , linux-arm-kernel@lists.infradead.org, Sebastian Hesselbarth List-Id: linux-pm@vger.kernel.org T24gVGh1LCBKdWwgMjYsIDIwMTggYXQgMDI6MTQ6NDNQTSAtMDcwMCwgRWR1YXJkbyBWYWxlbnRp biB3cm90ZToKPiBIZWxsbywKPiAKPiBPbiBUaHUsIEp1bCAyNiwgMjAxOCBhdCAxMDowMDo0MUFN ICswODAwLCBaaGFuZyBSdWkgd3JvdGU6Cj4gPiBPbiDkuIksIDIwMTgtMDctMjUgYXQgMDk6NTIg KzAyMDAsIE1pcXVlbCBSYXluYWwgd3JvdGU6Cj4gPiA+IEhpIEVkdWFyZG8sIFpoYW5nLAo+ID4g PiAKPiA+ID4gTWlxdWVsIFJheW5hbCA8bWlxdWVsLnJheW5hbEBib290bGluLmNvbT4gd3JvdGUg b24gV2VkLCAxOCBKdWwgMjAxOAo+ID4gPiAxNzoyODo0OSArMDIwMDoKPiA+ID4gCj4gPiA+ID4g Cj4gPiA+ID4gSGkgRWR1YXJkbywKPiA+ID4gPiAKPiA+ID4gPiBNaXF1ZWwgUmF5bmFsIDxtaXF1 ZWwucmF5bmFsQGJvb3RsaW4uY29tPiB3cm90ZSBvbiBNb24sIDE2IEp1bCAyMDE4Cj4gPiA+ID4g MTY6NDE6NDMgKzAyMDA6Cj4gPiA+ID4gCj4gPiA+ID4gPiAKPiA+ID4gPiA+IFRoZSBvbmx5IGNh cGFiaWxpdHkgb2YgdGhlIEFybWFkYSB0aGVybWFsIGRyaXZlciBpcyBjdXJyZW50bHkKPiA+ID4g PiA+IGp1c3QgdG8KPiA+ID4gPiA+IHJlYWQgb25lIHNlbnNvciAodGhlIGRlZmF1bHQgb25lKSBw ZXIgQVAgYW5kIG9uZSBwZXIgQ1AuCj4gPiA+ID4gPiBBY3R1YWxseSwKPiA+ID4gPiA+IHRoZXJl IGlzIG9uZSBzZW5zb3IgcGVyIGNvcmUgaW4gdGhlIEFQODA2IHBsdXMgb25lIHNlbnNvciBpbiB0 aGUKPiA+ID4gPiA+IHRoZXJtYWwgSVAgaXRzZWxmLiBUaGUgQ1AxMTAganVzdCBmZWF0dXJlcyBv bmUgdGhlcm1hbCBzZW5zb3IgaW4KPiA+ID4gPiA+IGl0cwo+ID4gPiA+ID4gb3duIHRoZXJtYWwg SVAuCj4gPiA+ID4gPiAKPiA+ID4gPiA+IFRoaXMgc2VyaWVzIGZpcnN0IGltcHJvdmVzIHRoZSBy ZWFkYWJpbGl0eSBvZiB0aGlzIGRyaXZlciwgdGhlbgo+ID4gPiA+ID4gYWRkcwo+ID4gPiA+ID4g c3VwcG9ydCBmb3IgbXVsdGktY2hhbm5lbCB0aGVybWFsIElQcy4gVGhlIGJpbmRpbmdzIGFuZCB0 aGUKPiA+ID4gPiA+IGRldmljZS10cmVlcyBhcmUgdXBkYXRlZCBhY2NvcmRpbmdseS4KPiA+ID4g PiA+IAo+ID4gPiA+ID4gVGhhbmsgeW91LAo+ID4gPiA+ID4gTWlxdcOobMKgwqAKPiA+ID4gPiBJ IGtub3cgSSBzZW50IHRoaXMgc2VyaWVzIG9uIG1vbmRheSBidXQgdGhlcmUgd2FzIGFsbW9zdCBu byBjaGFuZ2UKPiA+ID4gPiBvbgo+ID4gPiA+IGl0IChzZWUgYmVsb3cpLCBkbyB5b3UgdGhpbmsg eW91IHdpbGwgaGF2ZSB0aGUgdGltZSB0byBsb29rIGF0IGl0Cj4gPiA+ID4gYmVmb3JlIGl0J3Mg dG9vIGxhdGUgZm9yIHRoZSBtZXJnZSB3aW5kb3c/Cj4gPiA+IEkga25vdyBhIGxpdHRsZSBiaXQg d2hhdCBpdCBpcyB0byBiZSBvbiB0aGUgbWFpbnRhaW5lciBzaWRlIG9mIGEKPiA+ID4gc3Vic3lz dGVtIGJ1dCB0aGVzZSBjaGFuZ2VzIGFyZSBsYXlpbmcgc2luY2UgQXByaWwsIDIxc3QgYW5kIEkg cmVhbGx5Cj4gPiA+IGV4cGVjdCB0byBnZXQgdGhlbSBtZXJnZWQuIFpoYW5nLCBhcyBFZHVhcmRv IHNlZW1zIHRvIGJlIG92ZXJsb2FkZWQKPiA+ID4gdGhpcyByZWxlYXNlLCBjb3VsZCB5b3UgcGxl YXNlIHRha2UgdGhlIHNlcmllcz8KPiA+ID4gCj4gPiA+IE9mIGNvdXJzZSBpZiB0aGVyZSBhcmUg dGhpbmdzIHRvIGFkZHJlc3MgSSdsbCBiZSBoYXBweSB0byBkbyBzby4KPiA+ID4gCj4gPiBJIGhh ZCBhIHN5bmMgd2l0aCBFZHVhcmRvIGR1cmluZyBsYXN0IG1lcmdlIHdpbmRvdywgYW5kIHdlIGFn cmVlZCB0bwo+ID4gc2VuZCBzZXBhcmF0ZSBnaXQgcHVsbCByZXF1ZXN0cyB0byBMaW51cyBzaW5j ZSB0aGlzIHJlbGVhc2UgYW5kIHNlZSBob3cKPiA+IGl0IHdvcmtzLgo+ID4gVGhpcyBtZWFucyB0 aGF0IGFsbCB0aGUgc29jIHRoZXJtYWwgZHJpdmVyIHBhdGNoZXMgc2hvdWxkIGdvIHRvCj4gPiBF ZHVhcmRvJyB0cmVlIGFuZCB0aGVuIExpbnVzJyB0cmVlIGRpcmVjdGx5Lgo+ID4gCj4gPiBBbnl3 YXksIGxldCdzIHBpbmcgRWR1YXJkbyBhbmQgc2VlIGlmIGhlIGNhbiByZXZpZXcvdGFrZSB0aGUg cGF0Y2hlcy4KPiAKPiBTb3JyeSBmZWxsb3dzLCBidXQgSSB3YXMgb2ZmIGNhbXBpbmcgaW4gYSBw bGFjZSB3aXRoIG5vIGVsZWN0cmljaXR5L25vCj4gaW50ZXJuZXQuIEkgbG9va2VkIGF0IHRoaXMg c2VyaWVzIGJlZm9yZSwgYW5kIEkgZG8gbm90IHJlbWVtYmVyIGFueXRoaW5nCj4gdGhhdCBjb3Vs ZCBibG9jayBpdCB0byBnby4gTGV0IG1lIHJlc3RhcnQgb24gd29ya2luZyBvbiBpdC4KCkxldCBt ZSB0YWtlIHRoYXQgYmFjay4gSSBnb3Qgc29tZSBjb21waWxhdGlvbiBlcnJvci4KCj4gCj4gPiAK PiA+IHRoYW5rcywKPiA+IHJ1aQo+ID4gPiBUaGFua3MsCj4gPiA+IE1pcXXDqGwKCl9fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmxpbnV4LWFybS1rZXJuZWwg bWFpbGluZyBsaXN0CmxpbnV4LWFybS1rZXJuZWxAbGlzdHMuaW5mcmFkZWFkLm9yZwpodHRwOi8v bGlzdHMuaW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LWFybS1rZXJuZWwK From mboxrd@z Thu Jan 1 00:00:00 1970 From: edubezval@gmail.com (Eduardo Valentin) Date: Thu, 26 Jul 2018 14:35:18 -0700 Subject: [PATCH v3 00/23] Add multi-channel support to Armada thermal driver In-Reply-To: <20180726211442.GB3487@localhost.localdomain> References: <20180716144206.30985-1-miquel.raynal@bootlin.com> <20180718172849.30b7cec0@xps13> <20180725095222.054f1e80@xps13> <1532570441.2358.27.camel@intel.com> <20180726211442.GB3487@localhost.localdomain> Message-ID: <20180726213517.GA15690@localhost.localdomain> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, Jul 26, 2018 at 02:14:43PM -0700, Eduardo Valentin wrote: > Hello, > > On Thu, Jul 26, 2018 at 10:00:41AM +0800, Zhang Rui wrote: > > On ?, 2018-07-25 at 09:52 +0200, Miquel Raynal wrote: > > > Hi Eduardo, Zhang, > > > > > > Miquel Raynal wrote on Wed, 18 Jul 2018 > > > 17:28:49 +0200: > > > > > > > > > > > Hi Eduardo, > > > > > > > > Miquel Raynal wrote on Mon, 16 Jul 2018 > > > > 16:41:43 +0200: > > > > > > > > > > > > > > The only capability of the Armada thermal driver is currently > > > > > just to > > > > > read one sensor (the default one) per AP and one per CP. > > > > > Actually, > > > > > there is one sensor per core in the AP806 plus one sensor in the > > > > > thermal IP itself. The CP110 just features one thermal sensor in > > > > > its > > > > > own thermal IP. > > > > > > > > > > This series first improves the readability of this driver, then > > > > > adds > > > > > support for multi-channel thermal IPs. The bindings and the > > > > > device-trees are updated accordingly. > > > > > > > > > > Thank you, > > > > > Miqu?l?? > > > > I know I sent this series on monday but there was almost no change > > > > on > > > > it (see below), do you think you will have the time to look at it > > > > before it's too late for the merge window? > > > I know a little bit what it is to be on the maintainer side of a > > > subsystem but these changes are laying since April, 21st and I really > > > expect to get them merged. Zhang, as Eduardo seems to be overloaded > > > this release, could you please take the series? > > > > > > Of course if there are things to address I'll be happy to do so. > > > > > I had a sync with Eduardo during last merge window, and we agreed to > > send separate git pull requests to Linus since this release and see how > > it works. > > This means that all the soc thermal driver patches should go to > > Eduardo' tree and then Linus' tree directly. > > > > Anyway, let's ping Eduardo and see if he can review/take the patches. > > Sorry fellows, but I was off camping in a place with no electricity/no > internet. I looked at this series before, and I do not remember anything > that could block it to go. Let me restart on working on it. Let me take that back. I got some compilation error. > > > > > thanks, > > rui > > > Thanks, > > > Miqu?l