From: Arnd Bergmann <arnd@arndb.de>
To: Stephen Boyd <sboyd@codeaurora.org>
Cc: Daniel Lezcano <daniel.lezcano@linaro.org>,
Andy Gross <agross@codeaurora.org>,
linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 3/3] ARM: qcom: Drop ARCH_MSM* configs
Date: Wed, 25 Nov 2015 21:30:41 +0100 [thread overview]
Message-ID: <5774027.3ZvxSC0ub5@wuerfel> (raw)
In-Reply-To: <20151125193447.GD11298@codeaurora.org>
On Wednesday 25 November 2015 11:34:47 Stephen Boyd wrote:
> On 11/25, Arnd Bergmann wrote:
> > On Wednesday 25 November 2015 13:27:55 Daniel Lezcano wrote:
> > >
> > > What about:
> > >
> > > textofs-$(CONFIG_ARCH_MSM8X60) := 0x00208000
> > > textofs-$(CONFIG_ARCH_MSM8960) := 0x00208000
> > >
> > > in arch/arm/Makefile
> >
> > Good point, we need to do something about these still.
> >
>
> Yeah good catch! My brain must be shutting down due to the
> holiday coming up.
>
> The nice thing is that CLKSRC_QCOM and this text offset stuff are
> always used on the same SoCs, so we should be able to make one
> config in mach-qcom/Kconfig that selects the qcom clksrc driver
> and adjusts the text offset. So is the suggestion to move
> CLKSRC_QCOM to mach-qcom/Kconfig, or to make a new config that
> selects it?
I was thinking of a new option that selects it, which I think is
more in line with what Daniel wants.
Arnd
WARNING: multiple messages have this Message-ID (diff)
From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 3/3] ARM: qcom: Drop ARCH_MSM* configs
Date: Wed, 25 Nov 2015 21:30:41 +0100 [thread overview]
Message-ID: <5774027.3ZvxSC0ub5@wuerfel> (raw)
In-Reply-To: <20151125193447.GD11298@codeaurora.org>
On Wednesday 25 November 2015 11:34:47 Stephen Boyd wrote:
> On 11/25, Arnd Bergmann wrote:
> > On Wednesday 25 November 2015 13:27:55 Daniel Lezcano wrote:
> > >
> > > What about:
> > >
> > > textofs-$(CONFIG_ARCH_MSM8X60) := 0x00208000
> > > textofs-$(CONFIG_ARCH_MSM8960) := 0x00208000
> > >
> > > in arch/arm/Makefile
> >
> > Good point, we need to do something about these still.
> >
>
> Yeah good catch! My brain must be shutting down due to the
> holiday coming up.
>
> The nice thing is that CLKSRC_QCOM and this text offset stuff are
> always used on the same SoCs, so we should be able to make one
> config in mach-qcom/Kconfig that selects the qcom clksrc driver
> and adjusts the text offset. So is the suggestion to move
> CLKSRC_QCOM to mach-qcom/Kconfig, or to make a new config that
> selects it?
I was thinking of a new option that selects it, which I think is
more in line with what Daniel wants.
Arnd
next prev parent reply other threads:[~2015-11-25 20:31 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-25 1:08 [PATCH 0/3] Remove ARCH_MSM* configs Stephen Boyd
2015-11-25 1:08 ` Stephen Boyd
2015-11-25 1:08 ` [PATCH 1/3] clocksource: defbool CLKSRC_QCOM=y on ARCH_QCOM and make it visible Stephen Boyd
2015-11-25 1:08 ` Stephen Boyd
2015-11-25 2:07 ` Stephen Boyd
2015-11-25 2:07 ` Stephen Boyd
2015-11-25 9:03 ` Arnd Bergmann
2015-11-25 9:03 ` Arnd Bergmann
2015-11-25 10:10 ` Daniel Lezcano
2015-11-25 10:10 ` Daniel Lezcano
2015-11-25 10:17 ` Arnd Bergmann
2015-11-25 10:17 ` Arnd Bergmann
2015-11-25 12:37 ` Daniel Lezcano
2015-11-25 12:37 ` Daniel Lezcano
2015-11-25 12:49 ` Arnd Bergmann
2015-11-25 12:49 ` Arnd Bergmann
2015-11-25 13:22 ` Daniel Lezcano
2015-11-25 13:22 ` Daniel Lezcano
2015-11-25 1:08 ` [PATCH 2/3] ARM: qcom_defconfig: Enable HAVE_ARM_ARCH_TIMER Stephen Boyd
2015-11-25 1:08 ` Stephen Boyd
2015-11-25 1:08 ` [PATCH 3/3] ARM: qcom: Drop ARCH_MSM* configs Stephen Boyd
2015-11-25 1:08 ` Stephen Boyd
2015-11-25 12:27 ` Daniel Lezcano
2015-11-25 12:27 ` Daniel Lezcano
2015-11-25 12:51 ` Arnd Bergmann
2015-11-25 12:51 ` Arnd Bergmann
2015-11-25 19:34 ` Stephen Boyd
2015-11-25 19:34 ` Stephen Boyd
2015-11-25 20:30 ` Arnd Bergmann [this message]
2015-11-25 20:30 ` Arnd Bergmann
2015-11-25 20:39 ` Stephen Boyd
2015-11-25 20:39 ` Stephen Boyd
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=5774027.3ZvxSC0ub5@wuerfel \
--to=arnd@arndb.de \
--cc=agross@codeaurora.org \
--cc=daniel.lezcano@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=sboyd@codeaurora.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.