Re: [PATCH 2/2] arm64: dts: ti: Add k3-am62l3-beaglebadge

[Judith Mendez <jm@ti.com>]

On 5/5/26 11:01 AM, Andrew Davis wrote:
> On 5/5/26 10:48 AM, Judith Mendez wrote:
>> Hi Bryan, all,
>>
>> On 5/4/26 11:19 AM, Bryan Brattlof wrote:
>>> On May  1, 2026 thus sayeth Judith Mendez:
>>>> BeagleBoard.org BeagleBadge is a compact, affordable open source
>>>> hardware single board computer based on the Texas Instruments AM62L3
>>>> SoC designed for IoT and embedded applications with low power 
>>>> consumption.
>>>> Expansion is provided over open standards based headers including
>>>> QWIIC and GPIO interfaces.
>>>>
>>>> https://beagleboard.org/
>>>> https://github.com/beagleboard/BeagleBadge
>>>>
>>>> Co-developed-by: Andrew Davis <afd@ti.com>
>>>> Signed-off-by: Judith Mendez <jm@ti.com>
>>>> ---
>>>>   arch/arm64/boot/dts/ti/Makefile            |   1 +
>>>>   arch/arm64/boot/dts/ti/k3-am62l3-badge.dts | 655 +++++++++++++++++ 
>>>> ++++
>>>
>>> ...
>>>
>>>> diff --git a/arch/arm64/boot/dts/ti/k3-am62l3-badge.dts
>>>> b/arch/arm64/boot/dts/ti/k3-am62l3-badge.dts
>>>> new file mode 100644
>>>> index 000000000000..0125d3fec5e9
>>>> --- /dev/null
>>>> +++ b/arch/arm64/boot/dts/ti/k3-am62l3-badge.dts
>>>> @@ -0,0 +1,655 @@
>>>> +// SPDX-License-Identifier: GPL-2.0
>>>> +/*
>>>> + * https://www.beagleboard.org/boards/beaglebadge
>>>> + *
>>>> + * Copyright (C) 2026 Texas Instruments Incorporated - https:// 
>>>> www.ti.com/
>>>> + */
>>>> +
>>>> +/dts-v1/;
>>>> +
>>>> +#include <dt-bindings/gpio/gpio.h>
>>>> +#include <dt-bindings/input/input.h>
>>>> +#include <dt-bindings/leds/common.h>
>>>> +#include <dt-bindings/net/ti-dp83867.h>
>>>> +#include "k3-am62l3.dtsi"
>>>> +#include "k3-pinctrl.h"
>>>> +
>>>> +/ {
>>>> +    compatible = "beagle,am62l3-badge", "ti,am62l3";
>>>
>>> I noticed the $subject calls it the k3-am62l3-beaglebadge do we want to
>>> name the DT that as well?
>>>
>>>> +    model = "BeagleBoard.org BeagleBadge";
>>>> +
>>>
>>
>> So I see other beagle boards have the following:
>>
>> beagle,am67a-beagley-ai in k3-am67a-beagley-ai.dts
>> beagle,am625-beagleplay in k3-am625-beagleplay.dts
>> beagle,am62-pocketbeagle2 in k3-am62-pocketbeagle2.dts
>> beagle,j721e-beagleboneai64 in k3-j721e-beagleboneai64.dts
>>
>> so I think we should be using:
>>
>> beagle,am62l3-beaglebadge in k3-am62l3-beaglebadge.dts
>>
>> so if it is fine with everyone will update compatible
>> & DT filename accordingly.
>>
> 
> Looks correct to me.
> 
>>>
>>>> +&sdhci2 {
>>>> +    vmmc-supply = <&wlan_en>;
>>>> +    bus-width = <4>;
>>>> +    non-removable;
>>>> +    cap-power-off-card;
>>>> +    keep-power-in-suspend;
>>>> +    #address-cells = <1>;
>>>> +    #size-cells = <0>;
>>>> +    pinctrl-names = "default";
>>>> +    pinctrl-0 = <&mmc2_pins_default>;
>>>> +    ti,driver-strength-ohm = <50>;
>>>> +    ti,fails-without-test-cd;
>>>> +    status = "okay";
>>>> +
>>>> +    wlcore: wlcore@2 {
>>>> +        compatible = "ti,cc3300";
>>>
>>> Sadly this only exists on our evil vendor tree :/
>>
>> So sure, I saw that the WIFI driver is not in mainline,
>> but I also referenced beagleplay series & it seems
>> like as long as the hardware is described properly,
>> the node can be upstreamed. Please correct me if I
>> am wrong here..
>>
> 
> Only in some special cases, but in general the driver needs
> to be upstreamed first. And even in those special cases a
> DT binding is always needed first. Otherwise how would we know
> if the node is described properly.
> 
> For Beagleplay I only see the CC1352P7 radio in DT upstream,
> and it has a proper binding[0]. For now drop the CC3300, we
> can simply add it when the driver gets upstream.
> 

got it, will drop wlcore node, thanks.

~ Judith