| Message ID | 20240630073605.2164346-4-jacobe.zang@wesion.com (mailing list archive) |
|---|---|
| State | Not Applicable |
| Headers | show |
| Series | Add AP6275P wireless support | expand |
| Context | Check | Description |
|---|---|---|
| netdev/tree_selection | success | Not a local patch |
On 30/06/2024 09:36, Jacobe Zang wrote: > Khadas Edge2 uses the PCI-e Ampak AP6275P 2T2R Wi-Fi 6 module. The > pcie@0 node can be used as Bridge1, so the wifi@0 node is used as a > device under the Bridge1. As a PCIe device the wifi@0 can be probed > without compatible. > > Co-developed-by: Muhammed Efe Cetin <efectn@protonmail.com> > Signed-off-by: Muhammed Efe Cetin <efectn@protonmail.com> > Signed-off-by: Jacobe Zang <jacobe.zang@wesion.com> > --- > .../boot/dts/rockchip/rk3588s-khadas-edge2.dts | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > > diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-khadas-edge2.dts b/arch/arm64/boot/dts/rockchip/rk3588s-khadas-edge2.dts > index dbddfc3bb4641..8c152d587aefc 100644 > --- a/arch/arm64/boot/dts/rockchip/rk3588s-khadas-edge2.dts > +++ b/arch/arm64/boot/dts/rockchip/rk3588s-khadas-edge2.dts > @@ -283,6 +283,21 @@ &pcie2x1l2 { > reset-gpios = <&gpio3 RK_PD1 GPIO_ACTIVE_HIGH>; > vpcie3v3-supply = <&vcc3v3_pcie_wl>; > status = "okay"; > + > + pcie@0,0 { > + reg = <0x400000 0 0 0 0>; > + #address-cells = <3>; > + #size-cells = <2>; > + ranges; > + device_type = "pci"; > + bus-range = <0x40 0x4f>; > + > + wifi: wifi@0,0 { Where is the compatible (again!)? Test your code - you will see your binding is a no-op. Best regards, Krzysztof
>> + pcie@0,0 { >> + reg = <0x400000 0 0 0 0>; >> + #address-cells = <3>; >> + #size-cells = <2>; >> + ranges; >> + device_type = "pci"; >> + bus-range = <0x40 0x4f>; >> + >> + wifi: wifi@0,0 { > Where is the compatible (again!)? Test your code - you will see your > binding is a no-op. I tried to build kernel with CHECK_DTBS=1. And didn't get any message like 'compatible' is a required property in wifi node. But when I check the bindings that do required the compatible... So I will add it next time. --- Best Regards Jacobe
On 01/07/2024 12:38, Jacobe Zang wrote: >>> + pcie@0,0 { >>> + reg = <0x400000 0 0 0 0>; >>> + #address-cells = <3>; >>> + #size-cells = <2>; >>> + ranges; >>> + device_type = "pci"; >>> + bus-range = <0x40 0x4f>; >>> + >>> + wifi: wifi@0,0 { >> Where is the compatible (again!)? Test your code - you will see your >> binding is a no-op. > > I tried to build kernel with CHECK_DTBS=1. And didn't get any message > like 'compatible' is a required property in wifi node. But when I check > the bindings that do required the compatible... So I will add it next time. Yep, use different clock name and then test. You should see errors, right? But there are not, because schema is not applied to this node at all. Look how Apple is doing this. I have doubts that your code works at all in the first place. If there is no compatible, how your platform device gets of_node? Best regards, Krzysztof
>>> + pcie@0,0 { >>> + reg = <0x400000 0 0 0 0>; >>> + #address-cells = <3>; >>> + #size-cells = <2>; >>> + ranges; >>> + device_type = "pci"; >>> + bus-range = <0x40 0x4f>; >>> + >>> + wifi: wifi@0,0 { >> Where is the compatible (again!)? Test your code - you will see your >> binding is a no-op. > > I tried to build kernel with CHECK_DTBS=1. And didn't get any message > like 'compatible' is a required property in wifi node. But when I check > the bindings that do required the compatible... So I will add it next time. > Yep, use different clock name and then test. You should see errors, > right? But there are not, because schema is not applied to this node at all. > > Look how Apple is doing this. > > I have doubts that your code works at all in the first place. If there > is no compatible, how your platform device gets of_node? In file brcm_hw_ids.h and pcie.c has added Device ID and Vendor ID for bus to enumerate the device when board bootup, so I didn't add specific compatible in DTS. And by doing so, it can probe successfully. --- Best Regards Jacobe
On 01/07/2024 13:31, Jacobe Zang wrote: >>>> + pcie@0,0 { >>>> + reg = <0x400000 0 0 0 0>; >>>> + #address-cells = <3>; >>>> + #size-cells = <2>; >>>> + ranges; >>>> + device_type = "pci"; >>>> + bus-range = <0x40 0x4f>; >>>> + >>>> + wifi: wifi@0,0 { >>> Where is the compatible (again!)? Test your code - you will see your >>> binding is a no-op. >> >> I tried to build kernel with CHECK_DTBS=1. And didn't get any message >> like 'compatible' is a required property in wifi node. But when I check >> the bindings that do required the compatible... So I will add it next time. > >> Yep, use different clock name and then test. You should see errors, >> right? But there are not, because schema is not applied to this node at all. >> >> Look how Apple is doing this. >> >> I have doubts that your code works at all in the first place. If there >> is no compatible, how your platform device gets of_node? > > In file brcm_hw_ids.h and pcie.c has added Device ID and Vendor ID > for bus to enumerate the device when board bootup, so I didn't add > specific compatible in DTS. And by doing so, it can probe successfully. I did not ask about probe()... that's quite obvious. How the device's of_node is assigned? Best regards, Krzysztof
diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-khadas-edge2.dts b/arch/arm64/boot/dts/rockchip/rk3588s-khadas-edge2.dts index dbddfc3bb4641..8c152d587aefc 100644 --- a/arch/arm64/boot/dts/rockchip/rk3588s-khadas-edge2.dts +++ b/arch/arm64/boot/dts/rockchip/rk3588s-khadas-edge2.dts @@ -283,6 +283,21 @@ &pcie2x1l2 { reset-gpios = <&gpio3 RK_PD1 GPIO_ACTIVE_HIGH>; vpcie3v3-supply = <&vcc3v3_pcie_wl>; status = "okay"; + + pcie@0,0 { + reg = <0x400000 0 0 0 0>; + #address-cells = <3>; + #size-cells = <2>; + ranges; + device_type = "pci"; + bus-range = <0x40 0x4f>; + + wifi: wifi@0,0 { + reg = <0x410000 0 0 0 0>; + clocks = <&hym8563>; + clock-names = "lpo"; + }; + }; }; &pwm11 {