On Sun, 13 Jan 2019 at 13:35, @lex <alex.mob...@gmail.com> 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 <alex....@gmail.com> 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.
>> > 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.

-- 
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.

Reply via email to