Thread (117 messages) 117 messages, 5 authors, 2022-01-31

Re: [PATCH v2 10/35] brcmfmac: firmware: Allow platform to override macaddr

From: Hector Martin <hidden>
Date: 2022-01-17 06:39:13
Also in: linux-acpi, linux-devicetree, linux-wireless, lkml

On 09/01/2022 05.14, Arend van Spriel wrote:
On 1/4/2022 8:26 AM, Hector Martin wrote:
quoted
On Device Tree platforms, it is customary to be able to set the MAC
address via the Device Tree, as it is often stored in system firmware.
This is particularly relevant for Apple ARM64 platforms, where this
information comes from system configuration and passed through by the
bootloader into the DT.

Implement support for this by fetching the platform MAC address and
adding or replacing the macaddr= property in nvram. This becomes the
dongle's default MAC address.

On platforms with an SROM MAC address, this overrides it. On platforms
without one, such as Apple ARM64 devices, this is required for the
firmware to boot (it will fail if it does not have a valid MAC at all).
What overrides what. Can you elaborate a bit?
The behavior seems to be:

- Use the NVRAM MAC address, if any
- Use the SROM MAC address, if any
- Fail to boot

So a platform with a module containing a MAC address may choose to
override it using the DT mechanism with this patch. This is consistent
with the behavior of other drivers implementing platform MAC support.

-- 
Hector Martin (marcan@marcan.st)
Public Key: https://mrcn.st/pub
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help