From: Jayachandran C <jnair@caviumnetworks.com>
To: Linus Walleij <linus.walleij@linaro.org>
Cc: "linux-gpio@vger.kernel.org" <linux-gpio@vger.kernel.org>,
Kamlakant.Patel@cavium.com
Subject: Re: [PATCH] gpio: xlp: Update for ARCH_THUNDER2
Date: Wed, 15 Mar 2017 10:48:00 +0000 [thread overview]
Message-ID: <20170315104759.GA2584@localhost> (raw)
In-Reply-To: <CACRpkdZ88ObS-eqT5tVmktXWy6HdS9hor9wjJ6Ktvb6efcNYGQ@mail.gmail.com>
On Wed, Mar 15, 2017 at 11:22:58AM +0100, Linus Walleij wrote:
> On Sun, Mar 12, 2017 at 12:14 PM, Jayachandran C
> <jnair@caviumnetworks.com> wrote:
>
> > ARCH_VULCAN arm64 platform (for Broadcom Vulcan ARM64 processors) has
> > been discontinued. Cavium's ThunderX2 CN99XX (ARCH_THUNDER2) will be
> > the next revision of this platform.
> >
> > Update compile dependencies and ACPI ID to reflect this change. There
> > is not need to retain ARCH_VULCAN since the Vulcan processor was never
> > in production.
> >
> > Signed-off-by: Jayachandran C <jnair@caviumnetworks.com>
>
> Patch applied.
>
> > - depends on OF_GPIO && (CPU_XLP || ARCH_VULCAN || COMPILE_TEST)
> > + depends on OF_GPIO && (CPU_XLP || ARCH_THUNDER2 || COMPILE_TEST)
>
> I kept ARCH_VULCAN.
>
> I will consider removing it when Vulcan DTS files etc get removed from
> arch/arm64/boot/dts/broadcom/*
I have a patch posted to move the dts files at:
https://www.spinics.net/lists/arm-kernel/msg568462.html
and I hope it will get merged in the next cycle. I should have added
this to the patch comments, so that you have the complete information.
> People may still be using preproduction boards and I do not want
> to screw things up for them.
I understand, I left the Broadcom ACPI and DT strings alone due
to similar concerns. For the Kconfig symbol, it is not much of an
issue, but I'll leave that to your preference.
Thanks,
JC.
prev parent reply other threads:[~2017-03-15 10:48 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-12 11:14 [PATCH] gpio: xlp: Update for ARCH_THUNDER2 Jayachandran C
2017-03-15 10:22 ` Linus Walleij
2017-03-15 10:48 ` Jayachandran C [this message]
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=20170315104759.GA2584@localhost \
--to=jnair@caviumnetworks.com \
--cc=Kamlakant.Patel@cavium.com \
--cc=linus.walleij@linaro.org \
--cc=linux-gpio@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).