From: Michael Ellerman <mpe@ellerman.id.au>
To: Ben Hutchings <ben@decadent.org.uk>,
Masahiro Yamada <yamada.masahiro@socionext.com>,
linux-kbuild@vger.kernel.org
Cc: Alessio Igor Bogani <alessio.bogani@elettra.eu>,
Arnd Bergmann <arnd@arndb.de>, Rob Herring <robh+dt@kernel.org>,
linux-c6x-dev@linux-c6x.org, Michal Simek <monstr@monstr.eu>,
Nicolas Pitre <nicolas.pitre@linaro.org>,
Marcin Nowakowski <marcin.nowakowski@imgtec.com>,
devicetree@vger.kernel.org,
Stefan Kristiansson <stefan.kristiansson@saunalahti.fi>,
Michal Marek <mmarek@suse.com>, Scott Wood <oss@buserror.net>,
Benjamin Herrenschmidt <benh@kernel.crashing.org>,
linuxppc-dev@lists.ozlabs.org, Paul Mackerras <paulus@samba.org>,
openrisc@lists.librecores.org, Mark Salter <msalter@redhat.com>,
Stafford Horne <shorne@gmail.com>,
Aurelien Jacquiot <a-jacquiot@ti.com>,
Jonas Bonn <jonas@southpole.se>,
linux-kernel@vger.kernel.org,
"Oliver O'Halloran" <oohall@gmail.com>,
Mark Rutland <mark.rutland@arm.com>,
Emese Revfy <re.emese@gmail.com>
Subject: Re: [PATCH] kbuild: avoid unrecognized option error for external DTC
Date: Tue, 28 Feb 2017 21:21:02 +1100 [thread overview]
Message-ID: <87tw7epqep.fsf@concordia.ellerman.id.au> (raw)
In-Reply-To: <1488204416.2819.9.camel@decadent.org.uk>
Ben Hutchings <ben@decadent.org.uk> writes:
> [ Unknown signature status ]
> On Mon, 2017-02-27 at 14:40 +0900, Masahiro Yamada wrote:
>> Since commit 6b22b3d1614a ("kbuild: Allow using host dtc instead of
>> kernel's copy"), it is possible to use an external dtc. In this
>> case, we do not know which options are supported on it.
>>
>> Commit bc553986a2f7 ("dtc: turn off dtc unit address warnings by
>> default") gives -Wno-unit_address_vs_reg, but this options is only
>> recognized by v1.4.2 or later.
>>
>> If an older version is specified, the build fails:
>
> But the option to use an external dtc was intended to allow testing of
> newer versions. If there's no reason to use this option to run an
> older version, why bother trying to support that?
+1
That's a lot of added complexity, when the answer could just be "use the
kernel dtc".
cheers
WARNING: multiple messages have this Message-ID (diff)
From: Michael Ellerman <mpe@ellerman.id.au>
To: Ben Hutchings <ben@decadent.org.uk>,
Masahiro Yamada <yamada.masahiro@socionext.com>,
linux-kbuild@vger.kernel.org
Cc: Nicolas Pitre <nicolas.pitre@linaro.org>,
Mark Rutland <mark.rutland@arm.com>,
Aurelien Jacquiot <a-jacquiot@ti.com>,
Paul Mackerras <paulus@samba.org>,
Jonas Bonn <jonas@southpole.se>,
linux-c6x-dev@linux-c6x.org,
Alessio Igor Bogani <alessio.bogani@elettra.eu>,
Emese Revfy <re.emese@gmail.com>,
Mark Salter <msalter@redhat.com>,
devicetree@vger.kernel.org, Arnd Bergmann <arnd@arndb.de>,
Stefan Kristiansson <stefan.kristiansson@saunalahti.fi>,
Scott Wood <oss@buserror.net>, Rob Herring <robh+dt@kernel.org>,
Stafford Horne <shorne@gmail.com>,
Michal Simek <monstr@monstr.eu>,
linux-kernel@vger.kernel.org, Michal Marek <mmarek@suse.com>,
Oliver O'Halloran <oohall@gmail.com>,
linuxppc-dev@lists.ozlabs.org,
Marcin Nowakowski <marcin.nowakowski@imgtec.com>,
openrisc@lists.librecores.org
Subject: Re: [PATCH] kbuild: avoid unrecognized option error for external DTC
Date: Tue, 28 Feb 2017 21:21:02 +1100 [thread overview]
Message-ID: <87tw7epqep.fsf@concordia.ellerman.id.au> (raw)
In-Reply-To: <1488204416.2819.9.camel@decadent.org.uk>
Ben Hutchings <ben@decadent.org.uk> writes:
> [ Unknown signature status ]
> On Mon, 2017-02-27 at 14:40 +0900, Masahiro Yamada wrote:
>> Since commit 6b22b3d1614a ("kbuild: Allow using host dtc instead of
>> kernel's copy"), it is possible to use an external dtc. In this
>> case, we do not know which options are supported on it.
>>
>> Commit bc553986a2f7 ("dtc: turn off dtc unit address warnings by
>> default") gives -Wno-unit_address_vs_reg, but this options is only
>> recognized by v1.4.2 or later.
>>
>> If an older version is specified, the build fails:
>
> But the option to use an external dtc was intended to allow testing of
> newer versions. If there's no reason to use this option to run an
> older version, why bother trying to support that?
+1
That's a lot of added complexity, when the answer could just be "use the
kernel dtc".
cheers
WARNING: multiple messages have this Message-ID (diff)
From: Michael Ellerman <mpe@ellerman.id.au>
To: Ben Hutchings <ben@decadent.org.uk>,
Masahiro Yamada <yamada.masahiro@socionext.com>,
linux-kbuild@vger.kernel.org
Cc: Alessio Igor Bogani <alessio.bogani@elettra.eu>,
Arnd Bergmann <arnd@arndb.de>, Rob Herring <robh+dt@kernel.org>,
linux-c6x-dev@linux-c6x.org, Michal Simek <monstr@monstr.eu>,
Nicolas Pitre <nicolas.pitre@linaro.org>,
Marcin Nowakowski <marcin.nowakowski@imgtec.com>,
devicetree@vger.kernel.org,
Stefan Kristiansson <stefan.kristiansson@saunalahti.fi>,
Michal Marek <mmarek@suse.com>, Scott Wood <oss@buserror.net>,
Benjamin Herrenschmidt <benh@kernel.crashing.org>,
linuxppc-dev@lists.ozlabs.org, Paul Mackerras <paulus@samba.org>,
openrisc@lists.librecores.org, Mark Salter <msalter@redhat.com>,
Stafford Horne <shorne@gmail.com>,
Aurelien Jacquiot <a-jacquiot@ti.com>,
Jonas Bonn <jonas@southpole.se>,
linux-kernel@vger.kernel.org,
Oliver O'Halloran <oohall@gmail.com>,
Mark Rutland <mark.rutland@arm.com>,
Emese Revfy <re.emese@gmail.com>
Subject: Re: [PATCH] kbuild: avoid unrecognized option error for external DTC
Date: Tue, 28 Feb 2017 21:21:02 +1100 [thread overview]
Message-ID: <87tw7epqep.fsf@concordia.ellerman.id.au> (raw)
In-Reply-To: <1488204416.2819.9.camel@decadent.org.uk>
Ben Hutchings <ben@decadent.org.uk> writes:
> [ Unknown signature status ]
> On Mon, 2017-02-27 at 14:40 +0900, Masahiro Yamada wrote:
>> Since commit 6b22b3d1614a ("kbuild: Allow using host dtc instead of
>> kernel's copy"), it is possible to use an external dtc.=C2=A0=C2=A0In th=
is
>> case, we do not know which options are supported on it.
>>=20
>> Commit bc553986a2f7 ("dtc: turn off dtc unit address warnings by
>> default") gives -Wno-unit_address_vs_reg, but this options is only
>> recognized by v1.4.2 or later.
>>=20
>> If an older version is specified, the build fails:
>
> But the option to use an external dtc was intended to allow testing of
> newer versions. If there's no reason to use this option to run an
> older version, why bother trying to support that?
+1
That's a lot of added complexity, when the answer could just be "use the
kernel dtc".
cheers
WARNING: multiple messages have this Message-ID (diff)
From: Michael Ellerman <mpe@ellerman.id.au>
To: openrisc@lists.librecores.org
Subject: [OpenRISC] [PATCH] kbuild: avoid unrecognized option error for external DTC
Date: Tue, 28 Feb 2017 21:21:02 +1100 [thread overview]
Message-ID: <87tw7epqep.fsf@concordia.ellerman.id.au> (raw)
In-Reply-To: <1488204416.2819.9.camel@decadent.org.uk>
Ben Hutchings <ben@decadent.org.uk> writes:
> [ Unknown signature status ]
> On Mon, 2017-02-27 at 14:40 +0900, Masahiro Yamada wrote:
>> Since commit 6b22b3d1614a ("kbuild: Allow using host dtc instead of
>> kernel's copy"), it is possible to use an external dtc. In this
>> case, we do not know which options are supported on it.
>>
>> Commit bc553986a2f7 ("dtc: turn off dtc unit address warnings by
>> default") gives -Wno-unit_address_vs_reg, but this options is only
>> recognized by v1.4.2 or later.
>>
>> If an older version is specified, the build fails:
>
> But the option to use an external dtc was intended to allow testing of
> newer versions. If there's no reason to use this option to run an
> older version, why bother trying to support that?
+1
That's a lot of added complexity, when the answer could just be "use the
kernel dtc".
cheers
next prev parent reply other threads:[~2017-02-28 10:28 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-27 5:40 [PATCH] kbuild: avoid unrecognized option error for external DTC Masahiro Yamada
2017-02-27 5:40 ` [OpenRISC] " Masahiro Yamada
2017-02-27 5:40 ` Masahiro Yamada
2017-02-27 14:06 ` Ben Hutchings
2017-02-27 14:06 ` [OpenRISC] " Ben Hutchings
2017-02-27 14:06 ` Ben Hutchings
2017-02-27 14:06 ` Ben Hutchings
2017-02-28 10:21 ` Michael Ellerman [this message]
2017-02-28 10:21 ` [OpenRISC] " Michael Ellerman
2017-02-28 10:21 ` Michael Ellerman
2017-02-28 10:21 ` Michael Ellerman
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=87tw7epqep.fsf@concordia.ellerman.id.au \
--to=mpe@ellerman.id.au \
--cc=a-jacquiot@ti.com \
--cc=alessio.bogani@elettra.eu \
--cc=arnd@arndb.de \
--cc=ben@decadent.org.uk \
--cc=benh@kernel.crashing.org \
--cc=devicetree@vger.kernel.org \
--cc=jonas@southpole.se \
--cc=linux-c6x-dev@linux-c6x.org \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=marcin.nowakowski@imgtec.com \
--cc=mark.rutland@arm.com \
--cc=mmarek@suse.com \
--cc=monstr@monstr.eu \
--cc=msalter@redhat.com \
--cc=nicolas.pitre@linaro.org \
--cc=oohall@gmail.com \
--cc=openrisc@lists.librecores.org \
--cc=oss@buserror.net \
--cc=paulus@samba.org \
--cc=re.emese@gmail.com \
--cc=robh+dt@kernel.org \
--cc=shorne@gmail.com \
--cc=stefan.kristiansson@saunalahti.fi \
--cc=yamada.masahiro@socionext.com \
/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.