Add multi-channel support to Armada thermal driver
Get series with b4
b4 am 20180727005928.6bbe021f@xps13Versions newest first · linked by author and title
- v3 Latest posting 5/50 of 23 patches 0 repliesNew
- v3 posting 4/50 of 23 patches 0 repliesSuperseded
- v3 posting 3/50 of 23 patches 0 repliesSuperseded
Earlier versions · 4
| Version | Posted | Patches | Replies | Status |
|---|---|---|---|---|
| v3 posting 2/5 | 23 patches | 0 replies | Superseded | |
| v3 posting 1/5 | 23 patches | 0 replies | Superseded | |
| v2 | 23 patches | 0 replies | Superseded | |
| v1 | 0 of 27 patches | 0 replies | Superseded |
Patches in v3 posting 5/5 No applies recorded yet
None of the 23 patches the cover letter announces are in this archive.
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v3 yet.
Cover letter [PATCH v3 00/23] Add multi-channel support to Armada thermal driver
Hi Eduardo,
Eduardo Valentin <[email protected]> wrote on Thu, 26 Jul 2018 15:14:24 -0700:
Quoted 51 lines · Eduardo Valentin
Quoted 50 lines · Eduardo Valentin
Quoted 49 lines
Hello,
Quoted 46 lines · Zhang Rui
Quoted 32 lines · Miquel Raynal
Hi Eduardo, Zhang,
Miquel Raynal <[email protected]> wrote on Wed, 18 Jul 2018 17:28:49 +0200:
Quoted 21 lines
Hi Eduardo,
Miquel Raynal <[email protected]> wrote on Mon, 16 Jul 2018 16:41:43 +0200:
Quoted 15 lines
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.Quoted 6 lines
Quoted 2 lines
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?Quoted 7 lines
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.
Quoted 5 lines
Quoted 5 lines
Quoted 5 lines
Quoted 1 line
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.
No problem.
Quoted 7 lines
Quoted 2 lines
Let me take that back. I got some compilation error.
Given that I let this one fall into the cracks and that these are minor issues, I fixed them manually, and applied up to patch 19. But next at least compile test your series and run checkpatch on them.
I was really surprised of your remark but indeed there was a build issue in the middle of the series that I did not see because I always test with a last patch fixing it (this patch adds interrupt support, I plan to send it for the next release). I'm very sorry for this mistake, thanks for fixing it by yourself.
Quoted 2 lines
Patches 20-23 need to go via your arch tree.
Sure, they will enter in the next release now.
Thanks,
Miqu?l