Sounds good. I think i'd better restart with linux-next as a reference. On Sunday, January 13, 2019 at 4:51:20 PM UTC-2, CodeKipper wrote: > > On Sun, 13 Jan 2019 at 18:42, @lex <[email protected] <javascript:>> > wrote: > > > > Hmm, i thought you would know about the wrong size, or where to look > for. > > The patches are maybe from Icenowy, Jagan and some others, maybe they > can give some directions. > > I have seen a showcase of Pine64SO with sound working but have yet to > see a public DTS for kernel 4.20 as i have provided here, but if someone > can point me to a complete DTS and dtsi please do so. > the changes have now made it into linux-next > > https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/log/arch/arm64/boot/dts/allwinner?h=next-20190111 > > > > > > my little understanding is that A64 has 2 sensors, it is certainly not 0 > in #thermal-sensor-cells. > > > > Thermal sensor devices are nodes providing temperature sensing > capabilities on > > thermal zones. Typical devices are I2C ADC converters and bandgaps. > These are > > nodes providing temperature data to thermal zones. Thermal sensor > devices may > > control one or more internal sensors. > > > > Required property: > > - #thermal-sensor-cells: Used to provide sensor device specific > information > > Type: unsigned while referring to it. Typically 0 on thermal sensor > > Size: one cell nodes with only one sensor, and at least 1 on nodes > > with several internal sensors, in order > > to identify uniquely the sensor instances within > > the IC. See thermal zone binding for more details > > on how consumers refer to sensor devices. > > > > > > /* > > * An IC with several temperature sensor. > > */ > > adc_dummy: sensor@50 { > > ... > > #thermal-sensor-cells = <1>; /* sensor internal ID */ > > }; > > > > > > Thanks > > > > On Sunday, January 13, 2019 at 12:48:43 PM UTC-2, CodeKipper wrote: > >> > >> On Sun, 13 Jan 2019 at 13:35, @lex <[email protected]> wrote: > >> > > >> > 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 > >> just doing a comparison with a fresh armbian install. Audio doesn't > >> seem to work for you as sun8i-codec, sun4i-i2s, sun50i_codec_analog > >> and sun8i_adda_pr_regmap haven't been probed. They all seem to be in > >> the config file so should be available to you. > >> Not sure about the thermal side but the Documentation states that > >> thernal-sensor-cells should be 0. The patchset that you've > >> cherry-picked had a lot of comments, maybe contact the author to see > >> if he's updated it. > >> BR, > >> CK > >> > 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 <[email protected]> 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 [email protected]. > >> >> > 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 [email protected]. > >> > 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 [email protected] <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 [email protected]. For more options, visit https://groups.google.com/d/optout.
