Thread (9 messages) flat view 9 messages, 5 authors, 2016-06-02

[PATCH 01/32] arm64: Add platform selection for BCM2835.

From: f.fainelli@gmail.com (Florian Fainelli)
Date: 2016-06-02 17:36:35
Also in: lkml

On 06/02/2016 10:21 AM, Scott Branden wrote:

On 16-06-02 10:12 AM, Gerd Hoffmann wrote:
quoted
quoted
There is no need to add ARCH_BCM though?
There are Makefile(s) with

obj-$(CONFIG_ARCH_BCM)          += bcm/

in drivers/pinctrl for example, maybe more, didn't check the whole tree.

So without ARCH_BCM some drivers are dropped from the build ...
I see that now.  Quite bizarre how the NS2 pinctl driver (used on a
arm64 SoC) gets built right now....

Yes, adding the select with hidden option seems like what needs to be
done.  The ARM Maintainers did not like multi-level ARCH_BCM being added
in the arm64 Kconfig previously so we dropped it.
We should always recusrse into that directory and build the respective
pinctrl drivers within drivers/pinctrl/bcm, only for the enabled
platforms, it makes no sense to have this kind of config symbol gating that.

I can submit a patch doing just that unless somebody wants to do it as
part of this patch series.
-- 
Florian
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help