Hi Marcus, Thank you for the input. The patch seems to be verbatim. The message is clear about the size mismatch, but where it should be fixed?
*#thermal-sensor-cells = <2>; <= this gives the warning about the wronng size* The bootlog below is when *#thermal-sensor-cells = <1>;* and *no* ths calibration. Here i provide the complete DTS for the NanoPi A4 / BananaPi M64. Tested on NanoPi A64 with the following working: * ethernet * wifi * mali (3D) * hdmi Not tested yet: sunxi_cedrus Not working (yet I presume): * sound * thermal *Modules loaded* https://gist.github.com/avafinger/59a44d3429627939ac026268b8f2fef3 *.config* https://gist.github.com/avafinger/b86ab9474401991796d2d6f3be447cc6 *boot log* https://gist.github.com/avafinger/e50e084c09124b567f9ba66fb35903f7 *DTS* https://gist.github.com/avafinger/c326ebf61d5bb87423a9541f785b77c6 Thanks On Sunday, January 13, 2019 at 7:10:41 AM UTC-2, CodeKipper wrote: > > s > s > > On Sat, 12 Jan 2019 at 20:33, @lex <alex....@gmail.com <javascript:>> > wrote: > > > > I've tried to apply this patch in a hope to get the thermal sensor > working but things got hairy, can anyone please explain what is wrong with > this patch: > > https://patchwork.kernel.org/patch/10555615/ > Hi Alex, > check the patch after applying git show BLAH to check that it lsooks > the same as the original. For debugging the below and also the sound > card could you paste the following somewhere > arch/arm64/boot/dts/allwinner/sun50i-a64-bananapi-m64.dts > arch/arm64/boot/dts/allwinner/sun50i-a64-nanopi-a64.dts > arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi > .config > serial console/dmesg > BR, > CK > > > > Warnings: > > > > DTC arch/arm64/boot/dts/allwinner/sun50i-a64-bananapi-m64.dtb > > arch/arm64/boot/dts/allwinner/sun50i-a64-bananapi-m64.dtb: Warning > (reg_format): /soc/eeprom@1c14000/calib@234:reg: property has invalid > length (8 bytes) (#address-cells == 2, #size-cells == 1) > > arch/arm64/boot/dts/allwinner/sun50i-a64-bananapi-m64.dtb: Warning > (pci_device_bus_num): Failed prerequisite 'reg_format' > > arch/arm64/boot/dts/allwinner/sun50i-a64-bananapi-m64.dtb: Warning > (simple_bus_reg): Failed prerequisite 'reg_format' > > arch/arm64/boot/dts/allwinner/sun50i-a64-bananapi-m64.dtb: Warning > (i2c_bus_reg): Failed prerequisite 'reg_format' > > arch/arm64/boot/dts/allwinner/sun50i-a64-bananapi-m64.dtb: Warning > (spi_bus_reg): Failed prerequisite 'reg_format' > > arch/arm64/boot/dts/allwinner/sun50i-a64-bananapi-m64.dtb: Warning > (avoid_default_addr_size): /soc/eeprom@1c14000/calib@234: Relying on > default #address-cells value > > arch/arm64/boot/dts/allwinner/sun50i-a64-bananapi-m64.dtb: Warning > (avoid_default_addr_size): /soc/eeprom@1c14000/calib@234: Relying on > default #size-cells value > > arch/arm64/boot/dts/allwinner/sun50i-a64-bananapi-m64.dtb: Warning > (thermal_sensors_property): /thermal-zones/cpu0-thermal:thermal-sensors: > property size (8) too small for cell size 2 > > arch/arm64/boot/dts/allwinner/sun50i-a64-bananapi-m64.dtb: Warning > (thermal_sensors_property): /thermal-zones/gpu0-thermal:thermal-sensors: > property size (8) too small for cell size 2 > > arch/arm64/boot/dts/allwinner/sun50i-a64-bananapi-m64.dtb: Warning > (thermal_sensors_property): /thermal-zones/gpu1-thermal:thermal-sensors: > property size (8) too small for cell size 2 > > DTC arch/arm64/boot/dts/allwinner/sun50i-a64-nanopi-a64.dtb > > arch/arm64/boot/dts/allwinner/sun50i-a64-nanopi-a64.dtb: Warning > (reg_format): /soc/eeprom@1c14000/calib@234:reg: property has invalid > length (8 bytes) (#address-cells == 2, #size-cells == 1) > > arch/arm64/boot/dts/allwinner/sun50i-a64-nanopi-a64.dtb: Warning > (pci_device_bus_num): Failed prerequisite 'reg_format' > > arch/arm64/boot/dts/allwinner/sun50i-a64-nanopi-a64.dtb: Warning > (simple_bus_reg): Failed prerequisite 'reg_format' > > arch/arm64/boot/dts/allwinner/sun50i-a64-nanopi-a64.dtb: Warning > (i2c_bus_reg): Failed prerequisite 'reg_format' > > arch/arm64/boot/dts/allwinner/sun50i-a64-nanopi-a64.dtb: Warning > (spi_bus_reg): Failed prerequisite 'reg_format' > > arch/arm64/boot/dts/allwinner/sun50i-a64-nanopi-a64.dtb: Warning > (avoid_default_addr_size): /soc/eeprom@1c14000/calib@234: Relying on > default #address-cells value > > arch/arm64/boot/dts/allwinner/sun50i-a64-nanopi-a64.dtb: Warning > (avoid_default_addr_size): /soc/eeprom@1c14000/calib@234: Relying on > default #size-cells value > > arch/arm64/boot/dts/allwinner/sun50i-a64-nanopi-a64.dtb: Warning > (thermal_sensors_property): /thermal-zones/cpu0-thermal:thermal-sensors: > property size (8) too small for cell size 2 > > arch/arm64/boot/dts/allwinner/sun50i-a64-nanopi-a64.dtb: Warning > (thermal_sensors_property): /thermal-zones/gpu0-thermal:thermal-sensors: > property size (8) too small for cell size 2 > > arch/arm64/boot/dts/allwinner/sun50i-a64-nanopi-a64.dtb: Warning > (thermal_sensors_property): /thermal-zones/gpu1-thermal:thermal-sensors: > property size (8) too small for cell size 2 > > > > > > > > On Friday, January 11, 2019 at 6:38:05 PM UTC-2, @lex wrote: > >> > >> Hi, > >> I am running stable kernel 4.20.1 on NanoPi A64 with the linux-sunxi > community patches and it works really well thanks for the hard work however > i am struggling to get the sound card (any) detected and the upper USB > receptacle is not working (for some reason). > >> > >> Regarding the USB upper slot i have in my dtsi: > >> > >> usbphy: phy@1c19400 { > >> compatible = "allwinner,sun50i-a64-usb-phy"; > >> reg = <0x01c19400 0x14>, > >> <0x01c1a800 0x4>, > >> <0x01c1b800 0x4>; > >> reg-names = "phy_ctrl", > >> "pmu0", > >> "pmu1"; > >> clocks = <&ccu CLK_USB_PHY0>, > >> <&ccu CLK_USB_PHY1>; > >> clock-names = "usb0_phy", > >> "usb1_phy"; > >> resets = <&ccu RST_USB_PHY0>, > >> <&ccu RST_USB_PHY1>; > >> reset-names = "usb0_reset", > >> "usb1_reset"; > >> status = "disabled"; > >> #phy-cells = <1>; > >> }; > >> > >> > >> ehci0: usb@1c1a000 { > >> compatible = "allwinner,sun50i-a64-ehci", "generic-ehci"; > >> reg = <0x01c1a000 0x100>; > >> interrupts = <GIC_SPI 72 IRQ_TYPE_LEVEL_HIGH>; > >> clocks = <&ccu CLK_BUS_OHCI0>, > >> <&ccu CLK_BUS_EHCI0>, > >> <&ccu CLK_USB_OHCI0>; > >> resets = <&ccu RST_BUS_OHCI0>, > >> <&ccu RST_BUS_EHCI0>; > >> status = "disabled"; > >> }; > >> > >> > >> ohci0: usb@1c1a400 { > >> compatible = "allwinner,sun50i-a64-ohci", "generic-ohci"; > >> reg = <0x01c1a400 0x100>; > >> interrupts = <GIC_SPI 73 IRQ_TYPE_LEVEL_HIGH>; > >> clocks = <&ccu CLK_BUS_OHCI0>, > >> <&ccu CLK_USB_OHCI0>; > >> resets = <&ccu RST_BUS_OHCI0>; > >> status = "disabled"; > >> }; > >> > >> > >> ehci1: usb@1c1b000 { > >> compatible = "allwinner,sun50i-a64-ehci", "generic-ehci"; > >> reg = <0x01c1b000 0x100>; > >> interrupts = <GIC_SPI 74 IRQ_TYPE_LEVEL_HIGH>; > >> clocks = <&ccu CLK_BUS_OHCI1>, > >> <&ccu CLK_BUS_EHCI1>, > >> <&ccu CLK_USB_OHCI1>; > >> resets = <&ccu RST_BUS_OHCI1>, > >> <&ccu RST_BUS_EHCI1>; > >> phys = <&usbphy 1>; > >> phy-names = "usb"; > >> status = "disabled"; > >> }; > >> > >> > >> ohci1: usb@1c1b400 { > >> compatible = "allwinner,sun50i-a64-ohci", "generic-ohci"; > >> reg = <0x01c1b400 0x100>; > >> interrupts = <GIC_SPI 75 IRQ_TYPE_LEVEL_HIGH>; > >> clocks = <&ccu CLK_BUS_OHCI1>, > >> <&ccu CLK_USB_OHCI1>; > >> resets = <&ccu RST_BUS_OHCI1>; > >> phys = <&usbphy 1>; > >> phy-names = "usb"; > >> status = "disabled"; > >> }; > >> > >> > >> and my dts: > >> > >> &ehci0 { > >> status = "okay"; > >> }; > >> > >> > >> &ehci1 { > >> status = "okay"; > >> }; > >> > >> > >> &ohci0 { > >> status = "okay"; > >> }; > >> > >> > >> &ohci1 { > >> status = "okay"; > >> }; > >> > >> > >> But the upper slot is dead (but works with legacy kernel, so no > hardware issue) > >> Can someone pinpoint what is wrong with this setup or what i missed? > >> > >> > >> Regarding sound card, i have: > >> &codec { > >> status = "okay"; > >> }; > >> > >> > >> &codec_analog { > >> status = "okay"; > >> }; > >> > >> > >> &sound { > >> status = "okay"; > >> }; > >> > >> > >> but no sound card detected (audiocodec or hdmicodec), any advice? > >> > >> Another question, i cannot have thermal readout even with: > >> thermal-zones { > >> cpu_thermal: cpu0-thermal { > >> /* milliseconds */ > >> polling-delay-passive = <250>; > >> polling-delay = <1000>; > >> thermal-sensors = <&ths 0>; > >> > >> > >> cooling-maps { > >> map0 { > >> trip = <&cpu_alert0>; > >> cooling-device = <&cpu0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>; > >> }; > >> map1 { > >> trip = <&cpu_alert1>; > >> cooling-device = <&cpu0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>; > >> }; > >> }; > >> trips { > >> cpu_alert0: cpu_alert0 { > >> /* milliCelsius */ > >> temperature = <75000>; > >> hysteresis = <2000>; > >> type = "passive"; > >> }; > >> > >> > >> cpu_alert1: cpu_alert1 { > >> /* milliCelsius */ > >> temperature = <90000>; > >> hysteresis = <2000>; > >> type = "hot"; > >> }; > >> > >> > >> cpu_crit: cpu_crit { > >> /* milliCelsius */ > >> temperature = <110000>; > >> hysteresis = <2000>; > >> type = "critical"; > >> }; > >> }; > >> }; > >> gpu0_thermal: gpu0-thermal { > >> /* milliseconds */ > >> polling-delay-passive = <250>; > >> polling-delay = <1000>; > >> thermal-sensors = <&ths 1>; > >> }; > >> gpu1_thermal: gpu1-thermal { > >> /* milliseconds */ > >> polling-delay-passive = <250>; > >> polling-delay = <1000>; > >> thermal-sensors = <&ths 1>; > >> }; > >> }; > >> > >> > >> I can't see any ths driver for the A64 or any entity that reads it, is > it working on the A64? > >> > >> Jagan, you are extensively working on BananaPii M64 and NanoPi A64, do > you have this working and can you give some directions? > >> > >> Thank you. > >> > >> > > -- > > You received this message because you are subscribed to the Google > Groups "linux-sunxi" group. > > To unsubscribe from this group and stop receiving emails from it, send > an email to linux-sunxi...@googlegroups.com <javascript:>. > > For more options, visit https://groups.google.com/d/optout. > -- You received this message because you are subscribed to the Google Groups "linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.