From: "Rafael J. Wysocki" <rjw@rjwysocki.net>
To: Linus Walleij <linus.walleij@linaro.org>
Cc: Ulf Hansson <ulf.hansson@linaro.org>,
Arnd Bergmann <arnd@arndb.de>,
"linux-pm@vger.kernel.org" <linux-pm@vger.kernel.org>,
Viresh Kumar <viresh.kumar@linaro.org>,
"Rafael J . Wysocki" <rafael@kernel.org>,
Lee Jones <lee.jones@linaro.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 2/3 v3] mfd: db8500-prcmu: Get rid of cpufreq dependency
Date: Tue, 22 Aug 2017 15:43:11 +0200 [thread overview]
Message-ID: <22468782.O1RHkarIGQ@aspire.rjw.lan> (raw)
In-Reply-To: <CACRpkdZ91zUPE8Oyf+ZALekCUbAakM=PX6DiYTO-VZewGGNFmg@mail.gmail.com>
On Tuesday, August 22, 2017 3:44:18 PM CEST Linus Walleij wrote:
> On Tue, Aug 22, 2017 at 3:25 PM, Rafael J. Wysocki <rjw@rjwysocki.net> wrote:
> > On Tuesday, August 22, 2017 3:17:40 PM CEST Linus Walleij wrote:
> >> The ARMSS clock, also known as the operating point of the
> >> CPU, should not cross-depend on cpufreq like this. Move
> >> the code to use just frequencies and remove the false
> >> frequency (1GHz) and put in the actual frequency provided
> >> by the ARMSS clock (998400000 Hz) as part of the process.
> >>
> >> After this and the related cpufreq patch, the DB8500 will
> >> simply use the standard DT cpufreq driver to change the
> >> operating points through the common clock framework using
> >> the ARMSS clock.
> >>
> >> Acked-by: Lee Jones <lee.jones@linaro.org>
> >> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> >
> > Well, I've already applied the v2, so do you want me to replace the
> > commit with this patch?
>
> If you can, else tell me and I can send a fixup patch.
I can. :-)
Done already, but the replacement will show up in linux-next when I re-merge
the branches, which may be in a few days.
> I want to keep Lee happy!
Fair enough.
WARNING: multiple messages have this Message-ID (diff)
From: rjw@rjwysocki.net (Rafael J. Wysocki)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/3 v3] mfd: db8500-prcmu: Get rid of cpufreq dependency
Date: Tue, 22 Aug 2017 15:43:11 +0200 [thread overview]
Message-ID: <22468782.O1RHkarIGQ@aspire.rjw.lan> (raw)
In-Reply-To: <CACRpkdZ91zUPE8Oyf+ZALekCUbAakM=PX6DiYTO-VZewGGNFmg@mail.gmail.com>
On Tuesday, August 22, 2017 3:44:18 PM CEST Linus Walleij wrote:
> On Tue, Aug 22, 2017 at 3:25 PM, Rafael J. Wysocki <rjw@rjwysocki.net> wrote:
> > On Tuesday, August 22, 2017 3:17:40 PM CEST Linus Walleij wrote:
> >> The ARMSS clock, also known as the operating point of the
> >> CPU, should not cross-depend on cpufreq like this. Move
> >> the code to use just frequencies and remove the false
> >> frequency (1GHz) and put in the actual frequency provided
> >> by the ARMSS clock (998400000 Hz) as part of the process.
> >>
> >> After this and the related cpufreq patch, the DB8500 will
> >> simply use the standard DT cpufreq driver to change the
> >> operating points through the common clock framework using
> >> the ARMSS clock.
> >>
> >> Acked-by: Lee Jones <lee.jones@linaro.org>
> >> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> >
> > Well, I've already applied the v2, so do you want me to replace the
> > commit with this patch?
>
> If you can, else tell me and I can send a fixup patch.
I can. :-)
Done already, but the replacement will show up in linux-next when I re-merge
the branches, which may be in a few days.
> I want to keep Lee happy!
Fair enough.
next prev parent reply other threads:[~2017-08-22 13:43 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-22 13:17 [PATCH 1/3 v3] cpufreq: enable the DT cpufreq driver on the Ux500 Linus Walleij
2017-08-22 13:17 ` Linus Walleij
2017-08-22 13:17 ` [PATCH 2/3 v3] mfd: db8500-prcmu: Get rid of cpufreq dependency Linus Walleij
2017-08-22 13:17 ` Linus Walleij
2017-08-22 13:25 ` Rafael J. Wysocki
2017-08-22 13:25 ` Rafael J. Wysocki
2017-08-22 13:44 ` Linus Walleij
2017-08-22 13:44 ` Linus Walleij
2017-08-22 13:43 ` Rafael J. Wysocki [this message]
2017-08-22 13:43 ` Rafael J. Wysocki
2017-08-22 13:17 ` [PATCH 3/3 v3] cpufreq: dbx500: Delete obsolete driver Linus Walleij
2017-08-22 13:17 ` Linus Walleij
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=22468782.O1RHkarIGQ@aspire.rjw.lan \
--to=rjw@rjwysocki.net \
--cc=arnd@arndb.de \
--cc=lee.jones@linaro.org \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-pm@vger.kernel.org \
--cc=rafael@kernel.org \
--cc=ulf.hansson@linaro.org \
--cc=viresh.kumar@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.