Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1501359 > unrolled thread
| Started by | Robert Jarzmik <robert.jarzmik@free.fr> |
|---|---|
| First post | 2016-10-15 22:10 +0200 |
| Last post | 2016-10-15 23:40 +0200 |
| Articles | 4 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH v2 0/4] PXA cpufreq conversion to clock API Robert Jarzmik <robert.jarzmik@free.fr> - 2016-10-15 22:10 +0200
Re: [PATCH v2 0/4] PXA cpufreq conversion to clock API Russell King - ARM Linux <linux@armlinux.org.uk> - 2016-10-15 22:40 +0200
Re: [PATCH v2 0/4] PXA cpufreq conversion to clock API Robert Jarzmik <robert.jarzmik@free.fr> - 2016-10-15 23:20 +0200
WARNING: SOMEONE RECEIVING THIS HAS BEEN HACKED (was: Re: [PATCH v2 0/4] PXA cpufreq conversion to clock API) Russell King - ARM Linux <linux@armlinux.org.uk> - 2016-10-15 23:40 +0200
| From | Robert Jarzmik <robert.jarzmik@free.fr> |
|---|---|
| Date | 2016-10-15 22:10 +0200 |
| Subject | [PATCH v2 0/4] PXA cpufreq conversion to clock API |
| Message-ID | <ssDrj-45V-5@gated-at.bofh.it> |
Hi, This serie is a preparation to shift the cpufreq of pxa2xx platforms to clocks API. The first 3 patches are review and merge material : - patch 1/4 for Viresh and Rafael - patches 2/4 and 3/4 for me The 4th on is for review but not merge, as the clock changes must be fully reviewed and go in first as a prequisite Robert Jarzmik (4): cpufreq: pxa: use generic platdev driver for device-tree ARM: dts: pxa: add pxa25x cpu operating points ARM: dts: pxa: add pxa27x cpu operating points cpufreq: pxa: convert to clock API arch/arm/boot/dts/pxa25x.dtsi | 25 +++++ arch/arm/boot/dts/pxa27x.dtsi | 40 ++++++++ drivers/cpufreq/Kconfig.arm | 2 +- drivers/cpufreq/cpufreq-dt-platdev.c | 2 + drivers/cpufreq/pxa2xx-cpufreq.c | 191 +++++++---------------------------- 5 files changed, 107 insertions(+), 153 deletions(-) -- 2.1.4
[toc] | [next] | [standalone]
| From | Russell King - ARM Linux <linux@armlinux.org.uk> |
|---|---|
| Date | 2016-10-15 22:40 +0200 |
| Message-ID | <ssDUl-4j7-3@gated-at.bofh.it> |
| In reply to | #1501359 |
On Sat, Oct 15, 2016 at 09:57:26PM +0200, Robert Jarzmik wrote: > Hi, > > This serie is a preparation to shift the cpufreq of pxa2xx platforms to clocks > API. > > The first 3 patches are review and merge material : > - patch 1/4 for Viresh and Rafael > - patches 2/4 and 3/4 for me > > The 4th on is for review but not merge, as the clock changes must be fully > reviewed and go in first as a prequisite Have you tested whether this patch set results in functional cpufreq support - looking at drivers/clk/pxa, the CPU clock is read-only - it can't be used to change the clock rate. So, I very much doubt this has been functionally tested. Don't forget that changing the CPU clock rate needs other changes (like the memory clock rate) so all that code you're removing in patch 4 (which does the actual clock change) needs to go somewhere. -- RMK's Patch system: http://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up according to speedtest.net.
[toc] | [prev] | [next] | [standalone]
| From | Robert Jarzmik <robert.jarzmik@free.fr> |
|---|---|
| Date | 2016-10-15 23:20 +0200 |
| Message-ID | <ssEx3-4LR-5@gated-at.bofh.it> |
| In reply to | #1501360 |
Russell King - ARM Linux <linux@armlinux.org.uk> writes: > On Sat, Oct 15, 2016 at 09:57:26PM +0200, Robert Jarzmik wrote: >> Hi, >> >> This serie is a preparation to shift the cpufreq of pxa2xx platforms to clocks >> API. >> >> The first 3 patches are review and merge material : >> - patch 1/4 for Viresh and Rafael >> - patches 2/4 and 3/4 for me >> >> The 4th on is for review but not merge, as the clock changes must be fully >> reviewed and go in first as a prequisite > > Have you tested whether this patch set results in functional cpufreq > support - looking at drivers/clk/pxa, the CPU clock is read-only - > it can't be used to change the clock rate. So, I very much doubt this > has been functionally tested. Oh yes I did, on lubbock and mainstone ... but this requires another patchset under review, here : https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1247123.html > Don't forget that changing the CPU clock rate needs other changes > (like the memory clock rate) so all that code you're removing in > patch 4 (which does the actual clock change) needs to go somewhere. Sure, here : https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1247130.html I would have been wiser to quote these URLs instead of speaking of "clock changes must be fully review and go in first as a prequisite". Cheers. -- Robert
[toc] | [prev] | [next] | [standalone]
| From | Russell King - ARM Linux <linux@armlinux.org.uk> |
|---|---|
| Date | 2016-10-15 23:40 +0200 |
| Subject | WARNING: SOMEONE RECEIVING THIS HAS BEEN HACKED (was: Re: [PATCH v2 0/4] PXA cpufreq conversion to clock API) |
| Message-ID | <ssEQq-4Ss-13@gated-at.bofh.it> |
| In reply to | #1501360 |
Guys, Looks like gmail's been hacked (or something). Within 50 minutes of sending the reply below, I've received at least 10 replies with the following headers: In-Reply-To: <20161015203421.GG1041@n2100.armlinux.org.uk> References: <1476561450-28407-1-git-send-email-robert.jarzmik@free.fr> <20161015203421.GG1041@n2100.armlinux.org.uk> from various gmail.com addresses, each and every one containing some kind of spam. Each reply is from a victim of spam - the replies themselves are people replying to the spam that they have received. The quoted message is allegedly from me, and even includes the line: "On 15 Oct 2016, at 21:34, Russell King - ARM Linux <linux@armlinux.org.uk> wrote:" which is the time I sent the original message. It's either that gmail has been hacked, or one of the mailing lists in the recipients of this message has been hacked. On Sat, Oct 15, 2016 at 09:34:21PM +0100, Russell King - ARM Linux wrote: > On Sat, Oct 15, 2016 at 09:57:26PM +0200, Robert Jarzmik wrote: > > Hi, > > > > This serie is a preparation to shift the cpufreq of pxa2xx platforms to clocks > > API. > > > > The first 3 patches are review and merge material : > > - patch 1/4 for Viresh and Rafael > > - patches 2/4 and 3/4 for me > > > > The 4th on is for review but not merge, as the clock changes must be fully > > reviewed and go in first as a prequisite > > Have you tested whether this patch set results in functional cpufreq > support - looking at drivers/clk/pxa, the CPU clock is read-only - > it can't be used to change the clock rate. So, I very much doubt this > has been functionally tested. > > Don't forget that changing the CPU clock rate needs other changes > (like the memory clock rate) so all that code you're removing in > patch 4 (which does the actual clock change) needs to go somewhere. > > -- > RMK's Patch system: http://www.armlinux.org.uk/developer/patches/ > FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up > according to speedtest.net. > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- RMK's Patch system: http://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up according to speedtest.net.
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web