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.
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] <javascript:>>
> 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] <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.
