[U-Boot] [PATCH 4/4] sunxi: board: fixup the BT address for Orange Pi 3

Chen-Yu Tsai wens at kernel.org
Sat Nov 23 03:24:52 UTC 2019


On Fri, Nov 22, 2019 at 9:05 PM Andre Heider <a.heider at gmail.com> wrote:
>
> The BCM4345C5 of the Orange Pi 3 ships with the controller default
> address. Fix it up so it can function properly.
>
> The used address is "ethaddr" with the LSB flipped.
>
> Signed-off-by: Andre Heider <a.heider at gmail.com>
> ---
>
> NOTE:
>  "local-bd-address" is a universal property, the kernel patch for
>  btbcm to use that is in bluetooth-next.
>
>  board/sunxi/board.c | 28 ++++++++++++++++++++++++++++
>  1 file changed, 28 insertions(+)
>
> diff --git a/board/sunxi/board.c b/board/sunxi/board.c
> index bb35d6b66e..2897bf45e1 100644
> --- a/board/sunxi/board.c
> +++ b/board/sunxi/board.c
> @@ -856,6 +856,32 @@ int misc_init_r(void)
>         return 0;
>  }
>
> +static void fixup_bd_address(void *blob)
> +{
> +#if defined(CONFIG_MACH_SUN50I_H6)
> +       /* Some devices ship with the controller default address.
> +        * Set a valid address through the device tree.
> +        */
> +       uchar mac[ETH_ALEN], bdaddr[ETH_ALEN];
> +       int i;
> +
> +       if (!of_machine_is_compatible("xunlong,orangepi-3"))
> +               return;
> +
> +       if (!eth_env_get_enetaddr("ethaddr", mac))
> +               return;
> +
> +       /* Addresses need to be in the binary format of the corresponding stack */
> +       for (i = 0; i < ETH_ALEN; ++i)
> +               bdaddr[i] = mac[ETH_ALEN - i - 1];
> +
> +       bdaddr[0] ^= 1;

IIRC someone mentioned before that unlike Ethernet and WiFi,
Bluetooth does not have a "locally administered" address space,
so any address used could possibly collide with other devices.

ChenYu


> +
> +       do_fixup_by_compat(blob, "brcm,bcm4345c5",
> +                          "local-bd-address", bdaddr, ETH_ALEN, 1);
> +#endif
> +}
> +
>  int ft_board_setup(void *blob, bd_t *bd)
>  {
>         int __maybe_unused r;
> @@ -866,6 +892,8 @@ int ft_board_setup(void *blob, bd_t *bd)
>          */
>         setup_environment(blob);
>
> +       fixup_bd_address(blob);
> +
>  #ifdef CONFIG_VIDEO_DT_SIMPLEFB
>         r = sunxi_simplefb_setup(blob);
>         if (r)
> --
> 2.24.0
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot


More information about the U-Boot mailing list