allwinner: Convert to extres clk/hwreset APIs

[jmcneill]

Summary

Convert the Allwinner port from custom clock drivers to the new extres clk/hwreset APIs.

This removes the need for the custom "ccm" node in the Allwinner dts files, allowing us to use more or less unmodified upstream dts files.

As of now, upstream dts does not support the A20 HDMI or display engine, so these nodes are provided in sun7i-a20-hdmi.dtsi.

This patch has been verified across A10, A20, A31, and A31s boards.

[jmcneill]

sys/arm/allwinner/a10_hdmi.c

198 ↗

(On Diff #14637)

#define<tab>

[andrew]

sys/arm/allwinner/a10_ahci.c

329 ↗

(On Diff #14637)

It wold be safer to check if both clocks exist in the dtb before trying to enable them.

358 ↗

(On Diff #14637)

This might leave one or both clocks enabled which may not be desirable.

sys/arm/allwinner/a10_codec.c

809–819 ↗

(On Diff #14637)

It feels like we need a high level API to optionally set the frequency & enable the clock, it's a common operation. Even better if it can handle multiple clocks & cleanup correctly on failure of one.

[manu_bidouilliste.com]

sys/arm/conf/A10

61 ↗

(On Diff #14637)

Missing uart_snps

sys/arm/conf/A20

81 ↗

(On Diff #14637)

This is not needed

sys/boot/fdt/dts/arm/bananapi.dts

29 ↗

(On Diff #14637)

Could you integrate the change from D5718 with it ?
The disabled nodes aren't needed.

#include "sun7i-a20-hdmi.dtsi"

/ {

soc@01c00000 {
        hdmi@01c16000 {
                status = "okay";
        };

        hdmiaudio {
                status = "okay";
        };
};

};

&mmc0_pins_a {

allwinner,pull = <SUN4I_PINCTRL_PULL_UP>;

};

sys/boot/fdt/dts/arm/cubieboard.dts

29 ↗

(On Diff #14637)

#include "sun7i-a20-hdmi.dtsi"

/ {

soc@01c00000 {
        hdmi@01c16000 {
                status = "okay";
        };

        hdmiaudio {
                status = "okay";
        };
};

};

sys/boot/fdt/dts/arm/olinuxino-lime.dts

29 ↗

(On Diff #14637)

#include "sun7i-a20-hdmi.dtsi"

/ {

soc@01c00000 {
        hdmi@01c16000 {
                status = "okay";
        };

        hdmiaudio {
                status = "okay";
        };
};

};

[jmcneill]

sys/boot/fdt/dts/arm/cubieboard.dts

29 ↗

(On Diff #14637)

Do we know if the HDMI code works on sun4i-a10?

[jmcneill]

sys/arm/conf/A10

61 ↗

(On Diff #14637)

Should get sorted out when D5749 is merged.

sys/arm/conf/A20

81 ↗

(On Diff #14637)

This should get cleaned up when D5749 gets merged.

[mmel]

sys/arm/allwinner/a10_codec.c

819 ↗

(On Diff #14637)

Settings frequency in consumer driver is not frequent operation, mainly if is fixed. Imho, it's always better to setup base frequency plan for whole SoC at one place (in this case probably in driver for "/clocks" node). Typical consumers of clk_set_freq() are MMC, HDMI or UART - where frequency must be changed at runtime by user or connected device request.

[manu_bidouilliste.com]

sys/boot/fdt/dts/arm/cubieboard.dts

29 ↗

(On Diff #14637)

You're right I forgot, just tested it and it doesn't work.

[jmcneill]

sys/boot/fdt/dts/arm/olinuxino-lime.dts

29 ↗

(On Diff #14637)

Not supported on sun4i-a10 yet.

[jmcneill]

• a10_hdmi.c: style fix
• a10_ahci.c: cleanup clocks on failure paths

[jmcneill]

Fix some off-by-ones when validating mux input values, and add expected clock indices for sun4i usb clocks.

[jmcneill]

• aw_usbphy: Use regulator framework instead of poking GPIOs directly.
• dts: add HDMI, HDMI audio, and MMC pin setting fixes to bananapi.dts
• dwc: Split clock initialization between generic dwc and a20 dwc glue based on bindings documents (stmmac.txt, allwinner,sun7i-a20-gmac.txt)