Re: [PATCH] gpio: xlp: Update for ARCH_THUNDER2

[Jayachandran C <jnair@xxxxxxxxxxxxxxxxxx>]

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@xxxxxxxxxxxxxxxxxx> 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@xxxxxxxxxxxxxxxxxx>
> 
> 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.
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html