? 2017-05-19 15:19?Maxime Ripard ???

On Fri, May 19, 2017 at 11:03:33AM +0800, Icenowy Zheng wrote:

quoted

quoted

The patch looks OK, but given the module is removable, I think it
should be
an overlay. The overlay would enable WiFi + Bluetooth, and all the
peripherals needed to connect them.

That way, if the module is not installed, we avoid confusing the user
with
unused peripherals, and avoid having live voltage on an unoccupied
header.

But as this header have specific usage, we will also confuse user
if it's not enabled as its target usage.

Yet you would enable unused resources.

This was already pointed out here:
https://patchwork.kernel.org/patch/9670235/

I think, before we have an official repository of device tree
overlays (either directly in torvalds/linux or maintained by
linux-sunxi), we shouldn't rely on device tree overlay
functionality.

Maxime

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel at lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel