Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1275134 > unrolled thread
| Started by | Maxime Ripard <maxime.ripard@free-electrons.com> |
|---|---|
| First post | 2015-11-23 10:00 +0100 |
| Last post | 2015-11-23 17:50 +0100 |
| Articles | 4 — 4 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH v4 5/6] ARM: dts: sunxi: Add Allwinner H3 DTSI Maxime Ripard <maxime.ripard@free-electrons.com> - 2015-11-23 10:00 +0100
Re: [linux-sunxi] Re: [PATCH v4 5/6] ARM: dts: sunxi: Add Allwinner H3 DTSI Hans de Goede <hdegoede@redhat.com> - 2015-11-23 12:00 +0100
Re: [linux-sunxi] Re: [PATCH v4 5/6] ARM: dts: sunxi: Add Allwinner H3 DTSI Jens Kuske <jenskuske@gmail.com> - 2015-11-23 17:30 +0100
Re: [linux-sunxi] Re: [PATCH v4 5/6] ARM: dts: sunxi: Add Allwinner H3 DTSI Chen-Yu Tsai <wens@csie.org> - 2015-11-23 17:50 +0100
| From | Maxime Ripard <maxime.ripard@free-electrons.com> |
|---|---|
| Date | 2015-11-23 10:00 +0100 |
| Subject | Re: [PATCH v4 5/6] ARM: dts: sunxi: Add Allwinner H3 DTSI |
| Message-ID | <qxV8C-7TP-19@gated-at.bofh.it> |
[Multipart message — attachments visible in raw view] — view raw
Hi,
On Sun, Nov 01, 2015 at 02:33:23PM +0100, Jens Kuske wrote:
> >> + bus_gates: clk@01c20060 {
> >> + #clock-cells = <1>;
> >> + compatible = "allwinner,sun8i-h3-bus-gates-clk";
> >> + reg = <0x01c20060 0x14>;
> >> + clocks = <&ahb1>, <&ahb2>, <&apb1>, <&apb2>;
> >> + clock-names = "ahb1", "ahb2", "apb1", "apb2";
> >> + clock-indices = <5>, <6>, <8>,
> >> + <9>, <10>, <13>,
> >> + <14>, <17>, <18>,
> >> + <19>, <20>,
> >> + <21>, <23>,
> >> + <24>, <25>,
> >> + <26>, <27>,
> >> + <28>, <29>,
> >> + <30>, <31>, <32>,
> >> + <35>, <36>, <37>,
> >> + <40>, <41>, <43>,
> >> + <44>, <52>, <53>,
> >> + <54>, <64>,
> >> + <65>, <69>, <72>,
> >> + <76>, <77>, <78>,
> >> + <96>, <97>, <98>,
> >> + <112>, <113>,
> >> + <114>, <115>, <116>,
> >> + <128>, <135>;
> >> + clock-output-names = "ahb1_ce", "ahb1_dma", "ahb1_mmc0",
> >> + "ahb1_mmc1", "ahb1_mmc2", "ahb1_nand",
> >> + "ahb1_sdram", "ahb2_gmac", "ahb1_ts",
> >> + "ahb1_hstimer", "ahb1_spi0",
> >> + "ahb1_spi1", "ahb1_otg",
> >> + "ahb1_otg_ehci0", "ahb1_ehic1",
> >
> > ahb1_ehci1? Same for the following 3 lines.
> I'll fix them...
> >
> >> + "ahb1_ehic2", "ahb1_ehic3",
> >> + "ahb1_otg_ohci0", "ahb2_ohic1",
> >> + "ahb2_ohic2", "ahb2_ohic3", "ahb1_ve",
> >> + "ahb1_lcd0", "ahb1_lcd1", "ahb1_deint",
> >> + "ahb1_csi", "ahb1_tve", "ahb1_hdmi",
> >> + "ahb1_de", "ahb1_gpu", "ahb1_msgbox",
> >> + "ahb1_spinlock", "apb1_codec",
> >> + "apb1_spdif", "apb1_pio", "apb1_ths",
> >> + "apb1_i2s0", "apb1_i2s1", "apb1_i2s2",
> >> + "apb2_i2c0", "apb2_i2c1", "apb2_i2c2",
> >> + "apb2_uart0", "apb2_uart1",
> >> + "apb2_uart2", "apb2_uart3", "apb2_scr",
> >> + "ahb1_ephy", "ahb1_dbg";
> >
> > If it weren't for the last 2 clocks, we could cleanly split out apb1 and apb2
> > gates. Having a separate AHB clock gate taking 2 addresses seems messy
> > as well. :(
>
> Well, maybe we still should do that, if we split the resets too at least
> apb[12] would line up again.
>
> I don't know what to do with these bus things any more, all variants I
> sent had issues somewhere...
AFAIK, Arnd had some objections, but he never got back to us when we
explained how the hardware was laid out, so I don't know if they still
apply.
> >> + };
> >> +
> >> + mmc0_clk: clk@01c20088 {
> >> + #clock-cells = <1>;
> >> + compatible = "allwinner,sun4i-a10-mmc-clk";
> >> + reg = <0x01c20088 0x4>;
> >> + clocks = <&osc24M>, <&pll6 0>, <&pll8 0>;
> >> + clock-output-names = "mmc0",
> >> + "mmc0_output",
> >> + "mmc0_sample";
> >> + };
> >> +
> >> + mmc1_clk: clk@01c2008c {
> >> + #clock-cells = <1>;
> >> + compatible = "allwinner,sun4i-a10-mmc-clk";
> >> + reg = <0x01c2008c 0x4>;
> >> + clocks = <&osc24M>, <&pll6 0>, <&pll8 0>;
> >> + clock-output-names = "mmc1",
> >> + "mmc1_output",
> >> + "mmc1_sample";
> >> + };
> >> +
> >> + mmc2_clk: clk@01c20090 {
> >> + #clock-cells = <1>;
> >> + compatible = "allwinner,sun4i-a10-mmc-clk";
> >> + reg = <0x01c20090 0x4>;
> >> + clocks = <&osc24M>, <&pll6 0>, <&pll8 0>;
> >> + clock-output-names = "mmc2",
> >> + "mmc2_output",
> >> + "mmc2_sample";
> >> + };
> >> +
> >> + mbus_clk: clk@01c2015c {
> >> + #clock-cells = <0>;
> >> + compatible = "allwinner,sun8i-a23-mbus-clk";
> >> + reg = <0x01c2015c 0x4>;
> >> + clocks = <&osc24M>, <&pll6 1>, <&pll5>;
> >> + clock-output-names = "mbus";
> >> + };
> >> + };
> >> +
> >> + soc {
> >> + compatible = "simple-bus";
> >> + #address-cells = <1>;
> >> + #size-cells = <1>;
> >> + ranges;
> >> +
> >> + dma: dma-controller@01c02000 {
> >> + compatible = "allwinner,sun8i-h3-dma";
> >> + reg = <0x01c02000 0x1000>;
> >> + interrupts = <GIC_SPI 50 IRQ_TYPE_LEVEL_HIGH>;
> >> + clocks = <&bus_gates 6>;
> >> + resets = <&bus_rst 6>;
> >> + #dma-cells = <1>;
> >> + };
> >> +
> >> + mmc0: mmc@01c0f000 {
> >> + compatible = "allwinner,sun5i-a13-mmc";
> >> + reg = <0x01c0f000 0x1000>;
> >> + clocks = <&bus_gates 8>,
> >> + <&mmc0_clk 0>,
> >> + <&mmc0_clk 1>,
> >> + <&mmc0_clk 2>;
> >> + clock-names = "ahb",
> >> + "mmc",
> >> + "output",
> >> + "sample";
> >> + resets = <&bus_rst 8>;
> >> + reset-names = "ahb";
> >> + interrupts = <GIC_SPI 60 IRQ_TYPE_LEVEL_HIGH>;
> >> + status = "disabled";
> >> + #address-cells = <1>;
> >> + #size-cells = <0>;
> >> + };
> >> +
> >> + mmc1: mmc@01c10000 {
> >> + compatible = "allwinner,sun5i-a13-mmc";
> >> + reg = <0x01c10000 0x1000>;
> >> + clocks = <&bus_gates 9>,
> >> + <&mmc1_clk 0>,
> >> + <&mmc1_clk 1>,
> >> + <&mmc1_clk 2>;
> >> + clock-names = "ahb",
> >> + "mmc",
> >> + "output",
> >> + "sample";
> >> + resets = <&bus_rst 9>;
> >> + reset-names = "ahb";
> >> + interrupts = <GIC_SPI 61 IRQ_TYPE_LEVEL_HIGH>;
> >> + status = "disabled";
> >> + #address-cells = <1>;
> >> + #size-cells = <0>;
> >> + };
> >> +
> >> + mmc2: mmc@01c11000 {
> >> + compatible = "allwinner,sun5i-a13-mmc";
> >> + reg = <0x01c11000 0x1000>;
> >> + clocks = <&bus_gates 10>,
> >> + <&mmc2_clk 0>,
> >> + <&mmc2_clk 1>,
> >> + <&mmc2_clk 2>;
> >> + clock-names = "ahb",
> >> + "mmc",
> >> + "output",
> >> + "sample";
> >> + resets = <&bus_rst 10>;
> >> + reset-names = "ahb";
> >> + interrupts = <GIC_SPI 62 IRQ_TYPE_LEVEL_HIGH>;
> >> + status = "disabled";
> >> + #address-cells = <1>;
> >> + #size-cells = <0>;
> >> + };
> >> +
> >> + pio: pinctrl@01c20800 {
> >> + compatible = "allwinner,sun8i-h3-pinctrl";
> >> + reg = <0x01c20800 0x400>;
> >> + interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>,
> >> + <GIC_SPI 17 IRQ_TYPE_LEVEL_HIGH>;
> >> + clocks = <&bus_gates 69>;
> >> + gpio-controller;
> >> + #gpio-cells = <3>;
> >> + interrupt-controller;
> >> + #interrupt-cells = <2>;
> >> +
> >> + uart0_pins_a: uart0@0 {
> >> + allwinner,pins = "PA4", "PA5";
> >> + allwinner,function = "uart0";
> >> + allwinner,drive = <SUN4I_PINCTRL_10_MA>;
> >> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
> >> + };
> >> +
> >> + mmc0_pins_a: mmc0@0 {
> >> + allwinner,pins = "PF0", "PF1", "PF2", "PF3",
> >> + "PF4", "PF5";
> >> + allwinner,function = "mmc0";
> >> + allwinner,drive = <SUN4I_PINCTRL_30_MA>;
> >> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
> >> + };
> >> +
> >> + mmc0_cd_pin: mmc0_cd_pin@0 {
> >> + allwinner,pins = "PF6";
> >> + allwinner,function = "gpio_in";
> >> + allwinner,drive = <SUN4I_PINCTRL_10_MA>;
> >> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>;
> >> + };
> >
> > This should be in the board DTS, unless this is the reference design,
> > in which case you should name the label like "mmc0_cd_pin_reference_design".
> >
>
> The datasheet mentions SDC0_DET function on PF6, so I thought this is
> sort of fixed to this pin now. All designs I've seen use this pin.
Why is it set as a gpio then if it is a separate function?
Thanks!
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
[toc] | [next] | [standalone]
| From | Hans de Goede <hdegoede@redhat.com> |
|---|---|
| Date | 2015-11-23 12:00 +0100 |
| Subject | Re: [linux-sunxi] Re: [PATCH v4 5/6] ARM: dts: sunxi: Add Allwinner H3 DTSI |
| Message-ID | <qxX0K-Ev-9@gated-at.bofh.it> |
| In reply to | #1275134 |
HI,
On 23-11-15 09:57, Maxime Ripard wrote:
> Hi,
>
> On Sun, Nov 01, 2015 at 02:33:23PM +0100, Jens Kuske wrote:
>>>> + bus_gates: clk@01c20060 {
>>>> + #clock-cells = <1>;
>>>> + compatible = "allwinner,sun8i-h3-bus-gates-clk";
>>>> + reg = <0x01c20060 0x14>;
>>>> + clocks = <&ahb1>, <&ahb2>, <&apb1>, <&apb2>;
>>>> + clock-names = "ahb1", "ahb2", "apb1", "apb2";
>>>> + clock-indices = <5>, <6>, <8>,
>>>> + <9>, <10>, <13>,
>>>> + <14>, <17>, <18>,
>>>> + <19>, <20>,
>>>> + <21>, <23>,
>>>> + <24>, <25>,
>>>> + <26>, <27>,
>>>> + <28>, <29>,
>>>> + <30>, <31>, <32>,
>>>> + <35>, <36>, <37>,
>>>> + <40>, <41>, <43>,
>>>> + <44>, <52>, <53>,
>>>> + <54>, <64>,
>>>> + <65>, <69>, <72>,
>>>> + <76>, <77>, <78>,
>>>> + <96>, <97>, <98>,
>>>> + <112>, <113>,
>>>> + <114>, <115>, <116>,
>>>> + <128>, <135>;
>>>> + clock-output-names = "ahb1_ce", "ahb1_dma", "ahb1_mmc0",
>>>> + "ahb1_mmc1", "ahb1_mmc2", "ahb1_nand",
>>>> + "ahb1_sdram", "ahb2_gmac", "ahb1_ts",
>>>> + "ahb1_hstimer", "ahb1_spi0",
>>>> + "ahb1_spi1", "ahb1_otg",
>>>> + "ahb1_otg_ehci0", "ahb1_ehic1",
>>>
>>> ahb1_ehci1? Same for the following 3 lines.
>> I'll fix them...
>>>
>>>> + "ahb1_ehic2", "ahb1_ehic3",
>>>> + "ahb1_otg_ohci0", "ahb2_ohic1",
>>>> + "ahb2_ohic2", "ahb2_ohic3", "ahb1_ve",
>>>> + "ahb1_lcd0", "ahb1_lcd1", "ahb1_deint",
>>>> + "ahb1_csi", "ahb1_tve", "ahb1_hdmi",
>>>> + "ahb1_de", "ahb1_gpu", "ahb1_msgbox",
>>>> + "ahb1_spinlock", "apb1_codec",
>>>> + "apb1_spdif", "apb1_pio", "apb1_ths",
>>>> + "apb1_i2s0", "apb1_i2s1", "apb1_i2s2",
>>>> + "apb2_i2c0", "apb2_i2c1", "apb2_i2c2",
>>>> + "apb2_uart0", "apb2_uart1",
>>>> + "apb2_uart2", "apb2_uart3", "apb2_scr",
>>>> + "ahb1_ephy", "ahb1_dbg";
>>>
>>> If it weren't for the last 2 clocks, we could cleanly split out apb1 and apb2
>>> gates. Having a separate AHB clock gate taking 2 addresses seems messy
>>> as well. :(
>>
>> Well, maybe we still should do that, if we split the resets too at least
>> apb[12] would line up again.
>>
>> I don't know what to do with these bus things any more, all variants I
>> sent had issues somewhere...
>
> AFAIK, Arnd had some objections, but he never got back to us when we
> explained how the hardware was laid out, so I don't know if they still
> apply.
>
>>>> + };
>>>> +
>>>> + mmc0_clk: clk@01c20088 {
>>>> + #clock-cells = <1>;
>>>> + compatible = "allwinner,sun4i-a10-mmc-clk";
>>>> + reg = <0x01c20088 0x4>;
>>>> + clocks = <&osc24M>, <&pll6 0>, <&pll8 0>;
>>>> + clock-output-names = "mmc0",
>>>> + "mmc0_output",
>>>> + "mmc0_sample";
>>>> + };
>>>> +
>>>> + mmc1_clk: clk@01c2008c {
>>>> + #clock-cells = <1>;
>>>> + compatible = "allwinner,sun4i-a10-mmc-clk";
>>>> + reg = <0x01c2008c 0x4>;
>>>> + clocks = <&osc24M>, <&pll6 0>, <&pll8 0>;
>>>> + clock-output-names = "mmc1",
>>>> + "mmc1_output",
>>>> + "mmc1_sample";
>>>> + };
>>>> +
>>>> + mmc2_clk: clk@01c20090 {
>>>> + #clock-cells = <1>;
>>>> + compatible = "allwinner,sun4i-a10-mmc-clk";
>>>> + reg = <0x01c20090 0x4>;
>>>> + clocks = <&osc24M>, <&pll6 0>, <&pll8 0>;
>>>> + clock-output-names = "mmc2",
>>>> + "mmc2_output",
>>>> + "mmc2_sample";
>>>> + };
>>>> +
>>>> + mbus_clk: clk@01c2015c {
>>>> + #clock-cells = <0>;
>>>> + compatible = "allwinner,sun8i-a23-mbus-clk";
>>>> + reg = <0x01c2015c 0x4>;
>>>> + clocks = <&osc24M>, <&pll6 1>, <&pll5>;
>>>> + clock-output-names = "mbus";
>>>> + };
>>>> + };
>>>> +
>>>> + soc {
>>>> + compatible = "simple-bus";
>>>> + #address-cells = <1>;
>>>> + #size-cells = <1>;
>>>> + ranges;
>>>> +
>>>> + dma: dma-controller@01c02000 {
>>>> + compatible = "allwinner,sun8i-h3-dma";
>>>> + reg = <0x01c02000 0x1000>;
>>>> + interrupts = <GIC_SPI 50 IRQ_TYPE_LEVEL_HIGH>;
>>>> + clocks = <&bus_gates 6>;
>>>> + resets = <&bus_rst 6>;
>>>> + #dma-cells = <1>;
>>>> + };
>>>> +
>>>> + mmc0: mmc@01c0f000 {
>>>> + compatible = "allwinner,sun5i-a13-mmc";
>>>> + reg = <0x01c0f000 0x1000>;
>>>> + clocks = <&bus_gates 8>,
>>>> + <&mmc0_clk 0>,
>>>> + <&mmc0_clk 1>,
>>>> + <&mmc0_clk 2>;
>>>> + clock-names = "ahb",
>>>> + "mmc",
>>>> + "output",
>>>> + "sample";
>>>> + resets = <&bus_rst 8>;
>>>> + reset-names = "ahb";
>>>> + interrupts = <GIC_SPI 60 IRQ_TYPE_LEVEL_HIGH>;
>>>> + status = "disabled";
>>>> + #address-cells = <1>;
>>>> + #size-cells = <0>;
>>>> + };
>>>> +
>>>> + mmc1: mmc@01c10000 {
>>>> + compatible = "allwinner,sun5i-a13-mmc";
>>>> + reg = <0x01c10000 0x1000>;
>>>> + clocks = <&bus_gates 9>,
>>>> + <&mmc1_clk 0>,
>>>> + <&mmc1_clk 1>,
>>>> + <&mmc1_clk 2>;
>>>> + clock-names = "ahb",
>>>> + "mmc",
>>>> + "output",
>>>> + "sample";
>>>> + resets = <&bus_rst 9>;
>>>> + reset-names = "ahb";
>>>> + interrupts = <GIC_SPI 61 IRQ_TYPE_LEVEL_HIGH>;
>>>> + status = "disabled";
>>>> + #address-cells = <1>;
>>>> + #size-cells = <0>;
>>>> + };
>>>> +
>>>> + mmc2: mmc@01c11000 {
>>>> + compatible = "allwinner,sun5i-a13-mmc";
>>>> + reg = <0x01c11000 0x1000>;
>>>> + clocks = <&bus_gates 10>,
>>>> + <&mmc2_clk 0>,
>>>> + <&mmc2_clk 1>,
>>>> + <&mmc2_clk 2>;
>>>> + clock-names = "ahb",
>>>> + "mmc",
>>>> + "output",
>>>> + "sample";
>>>> + resets = <&bus_rst 10>;
>>>> + reset-names = "ahb";
>>>> + interrupts = <GIC_SPI 62 IRQ_TYPE_LEVEL_HIGH>;
>>>> + status = "disabled";
>>>> + #address-cells = <1>;
>>>> + #size-cells = <0>;
>>>> + };
>>>> +
>>>> + pio: pinctrl@01c20800 {
>>>> + compatible = "allwinner,sun8i-h3-pinctrl";
>>>> + reg = <0x01c20800 0x400>;
>>>> + interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>,
>>>> + <GIC_SPI 17 IRQ_TYPE_LEVEL_HIGH>;
>>>> + clocks = <&bus_gates 69>;
>>>> + gpio-controller;
>>>> + #gpio-cells = <3>;
>>>> + interrupt-controller;
>>>> + #interrupt-cells = <2>;
>>>> +
>>>> + uart0_pins_a: uart0@0 {
>>>> + allwinner,pins = "PA4", "PA5";
>>>> + allwinner,function = "uart0";
>>>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>;
>>>> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
>>>> + };
>>>> +
>>>> + mmc0_pins_a: mmc0@0 {
>>>> + allwinner,pins = "PF0", "PF1", "PF2", "PF3",
>>>> + "PF4", "PF5";
>>>> + allwinner,function = "mmc0";
>>>> + allwinner,drive = <SUN4I_PINCTRL_30_MA>;
>>>> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
>>>> + };
>>>> +
>>>> + mmc0_cd_pin: mmc0_cd_pin@0 {
>>>> + allwinner,pins = "PF6";
>>>> + allwinner,function = "gpio_in";
>>>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>;
>>>> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>;
>>>> + };
>>>
>>> This should be in the board DTS, unless this is the reference design,
>>> in which case you should name the label like "mmc0_cd_pin_reference_design".
>>>
>>
>> The datasheet mentions SDC0_DET function on PF6
Hmm, not in my version, I've "Allwinner_H3_Datasheet_V1.0.pdf" and there
PF6 only has generic input / output functionality.
>> so I thought this is
>> sort of fixed to this pin now. All designs I've seen use this pin.
>
> Why is it set as a gpio then if it is a separate function?
I guess because we do not support this in the mmc driver yet. Also on
older devices the mmc controller has build-in card-detection features
(using the data lines in that case) but we've never supported this since
none of the boards sofar have been using it.
For now we can just treat PF6 as a gpio, until someone figures out how
to do this inside the mmc driver.
Regards,
Hans
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Jens Kuske <jenskuske@gmail.com> |
|---|---|
| Date | 2015-11-23 17:30 +0100 |
| Subject | Re: [linux-sunxi] Re: [PATCH v4 5/6] ARM: dts: sunxi: Add Allwinner H3 DTSI |
| Message-ID | <qy2a6-4i0-23@gated-at.bofh.it> |
| In reply to | #1275239 |
On 23/11/15 11:50, Hans de Goede wrote:
> HI,
>
> On 23-11-15 09:57, Maxime Ripard wrote:
>> Hi,
>>
>> On Sun, Nov 01, 2015 at 02:33:23PM +0100, Jens Kuske wrote:
>>>>> + bus_gates: clk@01c20060 {
>>>>> + #clock-cells = <1>;
>>>>> + compatible = "allwinner,sun8i-h3-bus-gates-clk";
>>>>> + reg = <0x01c20060 0x14>;
>>>>> + clocks = <&ahb1>, <&ahb2>, <&apb1>, <&apb2>;
>>>>> + clock-names = "ahb1", "ahb2", "apb1", "apb2";
>>>>> + clock-indices = <5>, <6>, <8>,
>>>>> + <9>, <10>, <13>,
>>>>> + <14>, <17>, <18>,
>>>>> + <19>, <20>,
>>>>> + <21>, <23>,
>>>>> + <24>, <25>,
>>>>> + <26>, <27>,
>>>>> + <28>, <29>,
>>>>> + <30>, <31>, <32>,
>>>>> + <35>, <36>, <37>,
>>>>> + <40>, <41>, <43>,
>>>>> + <44>, <52>, <53>,
>>>>> + <54>, <64>,
>>>>> + <65>, <69>, <72>,
>>>>> + <76>, <77>, <78>,
>>>>> + <96>, <97>, <98>,
>>>>> + <112>, <113>,
>>>>> + <114>, <115>, <116>,
>>>>> + <128>, <135>;
>>>>> + clock-output-names = "ahb1_ce", "ahb1_dma", "ahb1_mmc0",
>>>>> + "ahb1_mmc1", "ahb1_mmc2", "ahb1_nand",
>>>>> + "ahb1_sdram", "ahb2_gmac", "ahb1_ts",
>>>>> + "ahb1_hstimer", "ahb1_spi0",
>>>>> + "ahb1_spi1", "ahb1_otg",
>>>>> + "ahb1_otg_ehci0", "ahb1_ehic1",
>>>>
>>>> ahb1_ehci1? Same for the following 3 lines.
>>> I'll fix them...
>>>>
>>>>> + "ahb1_ehic2", "ahb1_ehic3",
>>>>> + "ahb1_otg_ohci0", "ahb2_ohic1",
>>>>> + "ahb2_ohic2", "ahb2_ohic3", "ahb1_ve",
>>>>> + "ahb1_lcd0", "ahb1_lcd1", "ahb1_deint",
>>>>> + "ahb1_csi", "ahb1_tve", "ahb1_hdmi",
>>>>> + "ahb1_de", "ahb1_gpu", "ahb1_msgbox",
>>>>> + "ahb1_spinlock", "apb1_codec",
>>>>> + "apb1_spdif", "apb1_pio", "apb1_ths",
>>>>> + "apb1_i2s0", "apb1_i2s1", "apb1_i2s2",
>>>>> + "apb2_i2c0", "apb2_i2c1", "apb2_i2c2",
>>>>> + "apb2_uart0", "apb2_uart1",
>>>>> + "apb2_uart2", "apb2_uart3", "apb2_scr",
>>>>> + "ahb1_ephy", "ahb1_dbg";
>>>>
>>>> If it weren't for the last 2 clocks, we could cleanly split out apb1 and apb2
>>>> gates. Having a separate AHB clock gate taking 2 addresses seems messy
>>>> as well. :(
>>>
>>> Well, maybe we still should do that, if we split the resets too at least
>>> apb[12] would line up again.
>>>
>>> I don't know what to do with these bus things any more, all variants I
>>> sent had issues somewhere...
>>
>> AFAIK, Arnd had some objections, but he never got back to us when we
>> explained how the hardware was laid out, so I don't know if they still
>> apply.
>>
>>>>> + };
>>>>> +
>>>>> + mmc0_clk: clk@01c20088 {
>>>>> + #clock-cells = <1>;
>>>>> + compatible = "allwinner,sun4i-a10-mmc-clk";
>>>>> + reg = <0x01c20088 0x4>;
>>>>> + clocks = <&osc24M>, <&pll6 0>, <&pll8 0>;
>>>>> + clock-output-names = "mmc0",
>>>>> + "mmc0_output",
>>>>> + "mmc0_sample";
>>>>> + };
>>>>> +
>>>>> + mmc1_clk: clk@01c2008c {
>>>>> + #clock-cells = <1>;
>>>>> + compatible = "allwinner,sun4i-a10-mmc-clk";
>>>>> + reg = <0x01c2008c 0x4>;
>>>>> + clocks = <&osc24M>, <&pll6 0>, <&pll8 0>;
>>>>> + clock-output-names = "mmc1",
>>>>> + "mmc1_output",
>>>>> + "mmc1_sample";
>>>>> + };
>>>>> +
>>>>> + mmc2_clk: clk@01c20090 {
>>>>> + #clock-cells = <1>;
>>>>> + compatible = "allwinner,sun4i-a10-mmc-clk";
>>>>> + reg = <0x01c20090 0x4>;
>>>>> + clocks = <&osc24M>, <&pll6 0>, <&pll8 0>;
>>>>> + clock-output-names = "mmc2",
>>>>> + "mmc2_output",
>>>>> + "mmc2_sample";
>>>>> + };
>>>>> +
>>>>> + mbus_clk: clk@01c2015c {
>>>>> + #clock-cells = <0>;
>>>>> + compatible = "allwinner,sun8i-a23-mbus-clk";
>>>>> + reg = <0x01c2015c 0x4>;
>>>>> + clocks = <&osc24M>, <&pll6 1>, <&pll5>;
>>>>> + clock-output-names = "mbus";
>>>>> + };
>>>>> + };
>>>>> +
>>>>> + soc {
>>>>> + compatible = "simple-bus";
>>>>> + #address-cells = <1>;
>>>>> + #size-cells = <1>;
>>>>> + ranges;
>>>>> +
>>>>> + dma: dma-controller@01c02000 {
>>>>> + compatible = "allwinner,sun8i-h3-dma";
>>>>> + reg = <0x01c02000 0x1000>;
>>>>> + interrupts = <GIC_SPI 50 IRQ_TYPE_LEVEL_HIGH>;
>>>>> + clocks = <&bus_gates 6>;
>>>>> + resets = <&bus_rst 6>;
>>>>> + #dma-cells = <1>;
>>>>> + };
>>>>> +
>>>>> + mmc0: mmc@01c0f000 {
>>>>> + compatible = "allwinner,sun5i-a13-mmc";
>>>>> + reg = <0x01c0f000 0x1000>;
>>>>> + clocks = <&bus_gates 8>,
>>>>> + <&mmc0_clk 0>,
>>>>> + <&mmc0_clk 1>,
>>>>> + <&mmc0_clk 2>;
>>>>> + clock-names = "ahb",
>>>>> + "mmc",
>>>>> + "output",
>>>>> + "sample";
>>>>> + resets = <&bus_rst 8>;
>>>>> + reset-names = "ahb";
>>>>> + interrupts = <GIC_SPI 60 IRQ_TYPE_LEVEL_HIGH>;
>>>>> + status = "disabled";
>>>>> + #address-cells = <1>;
>>>>> + #size-cells = <0>;
>>>>> + };
>>>>> +
>>>>> + mmc1: mmc@01c10000 {
>>>>> + compatible = "allwinner,sun5i-a13-mmc";
>>>>> + reg = <0x01c10000 0x1000>;
>>>>> + clocks = <&bus_gates 9>,
>>>>> + <&mmc1_clk 0>,
>>>>> + <&mmc1_clk 1>,
>>>>> + <&mmc1_clk 2>;
>>>>> + clock-names = "ahb",
>>>>> + "mmc",
>>>>> + "output",
>>>>> + "sample";
>>>>> + resets = <&bus_rst 9>;
>>>>> + reset-names = "ahb";
>>>>> + interrupts = <GIC_SPI 61 IRQ_TYPE_LEVEL_HIGH>;
>>>>> + status = "disabled";
>>>>> + #address-cells = <1>;
>>>>> + #size-cells = <0>;
>>>>> + };
>>>>> +
>>>>> + mmc2: mmc@01c11000 {
>>>>> + compatible = "allwinner,sun5i-a13-mmc";
>>>>> + reg = <0x01c11000 0x1000>;
>>>>> + clocks = <&bus_gates 10>,
>>>>> + <&mmc2_clk 0>,
>>>>> + <&mmc2_clk 1>,
>>>>> + <&mmc2_clk 2>;
>>>>> + clock-names = "ahb",
>>>>> + "mmc",
>>>>> + "output",
>>>>> + "sample";
>>>>> + resets = <&bus_rst 10>;
>>>>> + reset-names = "ahb";
>>>>> + interrupts = <GIC_SPI 62 IRQ_TYPE_LEVEL_HIGH>;
>>>>> + status = "disabled";
>>>>> + #address-cells = <1>;
>>>>> + #size-cells = <0>;
>>>>> + };
>>>>> +
>>>>> + pio: pinctrl@01c20800 {
>>>>> + compatible = "allwinner,sun8i-h3-pinctrl";
>>>>> + reg = <0x01c20800 0x400>;
>>>>> + interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>,
>>>>> + <GIC_SPI 17 IRQ_TYPE_LEVEL_HIGH>;
>>>>> + clocks = <&bus_gates 69>;
>>>>> + gpio-controller;
>>>>> + #gpio-cells = <3>;
>>>>> + interrupt-controller;
>>>>> + #interrupt-cells = <2>;
>>>>> +
>>>>> + uart0_pins_a: uart0@0 {
>>>>> + allwinner,pins = "PA4", "PA5";
>>>>> + allwinner,function = "uart0";
>>>>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>;
>>>>> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
>>>>> + };
>>>>> +
>>>>> + mmc0_pins_a: mmc0@0 {
>>>>> + allwinner,pins = "PF0", "PF1", "PF2", "PF3",
>>>>> + "PF4", "PF5";
>>>>> + allwinner,function = "mmc0";
>>>>> + allwinner,drive = <SUN4I_PINCTRL_30_MA>;
>>>>> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
>>>>> + };
>>>>> +
>>>>> + mmc0_cd_pin: mmc0_cd_pin@0 {
>>>>> + allwinner,pins = "PF6";
>>>>> + allwinner,function = "gpio_in";
>>>>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>;
>>>>> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>;
>>>>> + };
>>>>
>>>> This should be in the board DTS, unless this is the reference design,
>>>> in which case you should name the label like "mmc0_cd_pin_reference_design".
>>>>
>>>
>>> The datasheet mentions SDC0_DET function on PF6
>
> Hmm, not in my version, I've "Allwinner_H3_Datasheet_V1.0.pdf" and there
> PF6 only has generic input / output functionality.
Hm, indeed, it isn't mentioned in the Port Controller documentation, but
the table on page 76 (3.2. GPIO Multiplexing Functions) has it.
>
> >> so I thought this is
>>> sort of fixed to this pin now. All designs I've seen use this pin.
>>
>> Why is it set as a gpio then if it is a separate function?
>
> I guess because we do not support this in the mmc driver yet. Also on
> older devices the mmc controller has build-in card-detection features
> (using the data lines in that case) but we've never supported this since
> none of the boards sofar have been using it.
>
> For now we can just treat PF6 as a gpio, until someone figures out how
> to do this inside the mmc driver.
I could not find any documentation or reference how to use it, probably
because it doesn't exist...
Jens
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Chen-Yu Tsai <wens@csie.org> |
|---|---|
| Date | 2015-11-23 17:50 +0100 |
| Subject | Re: [linux-sunxi] Re: [PATCH v4 5/6] ARM: dts: sunxi: Add Allwinner H3 DTSI |
| Message-ID | <qy2ts-4pB-39@gated-at.bofh.it> |
| In reply to | #1275239 |
On Mon, Nov 23, 2015 at 6:50 PM, Hans de Goede <hdegoede@redhat.com> wrote:
> HI,
>
>
> On 23-11-15 09:57, Maxime Ripard wrote:
>>
>> Hi,
>>
>> On Sun, Nov 01, 2015 at 02:33:23PM +0100, Jens Kuske wrote:
>>>>>
>>>>> + bus_gates: clk@01c20060 {
>>>>> + #clock-cells = <1>;
>>>>> + compatible =
>>>>> "allwinner,sun8i-h3-bus-gates-clk";
>>>>> + reg = <0x01c20060 0x14>;
>>>>> + clocks = <&ahb1>, <&ahb2>, <&apb1>, <&apb2>;
>>>>> + clock-names = "ahb1", "ahb2", "apb1", "apb2";
>>>>> + clock-indices = <5>, <6>, <8>,
>>>>> + <9>, <10>, <13>,
>>>>> + <14>, <17>, <18>,
>>>>> + <19>, <20>,
>>>>> + <21>, <23>,
>>>>> + <24>, <25>,
>>>>> + <26>, <27>,
>>>>> + <28>, <29>,
>>>>> + <30>, <31>, <32>,
>>>>> + <35>, <36>, <37>,
>>>>> + <40>, <41>, <43>,
>>>>> + <44>, <52>, <53>,
>>>>> + <54>, <64>,
>>>>> + <65>, <69>, <72>,
>>>>> + <76>, <77>, <78>,
>>>>> + <96>, <97>, <98>,
>>>>> + <112>, <113>,
>>>>> + <114>, <115>, <116>,
>>>>> + <128>, <135>;
>>>>> + clock-output-names = "ahb1_ce", "ahb1_dma",
>>>>> "ahb1_mmc0",
>>>>> + "ahb1_mmc1", "ahb1_mmc2",
>>>>> "ahb1_nand",
>>>>> + "ahb1_sdram", "ahb2_gmac",
>>>>> "ahb1_ts",
>>>>> + "ahb1_hstimer", "ahb1_spi0",
>>>>> + "ahb1_spi1", "ahb1_otg",
>>>>> + "ahb1_otg_ehci0", "ahb1_ehic1",
>>>>
>>>>
>>>> ahb1_ehci1? Same for the following 3 lines.
>>>
>>> I'll fix them...
>>>>
>>>>
>>>>> + "ahb1_ehic2", "ahb1_ehic3",
>>>>> + "ahb1_otg_ohci0", "ahb2_ohic1",
>>>>> + "ahb2_ohic2", "ahb2_ohic3",
>>>>> "ahb1_ve",
>>>>> + "ahb1_lcd0", "ahb1_lcd1",
>>>>> "ahb1_deint",
>>>>> + "ahb1_csi", "ahb1_tve",
>>>>> "ahb1_hdmi",
>>>>> + "ahb1_de", "ahb1_gpu",
>>>>> "ahb1_msgbox",
>>>>> + "ahb1_spinlock", "apb1_codec",
>>>>> + "apb1_spdif", "apb1_pio",
>>>>> "apb1_ths",
>>>>> + "apb1_i2s0", "apb1_i2s1",
>>>>> "apb1_i2s2",
>>>>> + "apb2_i2c0", "apb2_i2c1",
>>>>> "apb2_i2c2",
>>>>> + "apb2_uart0", "apb2_uart1",
>>>>> + "apb2_uart2", "apb2_uart3",
>>>>> "apb2_scr",
>>>>> + "ahb1_ephy", "ahb1_dbg";
>>>>
>>>>
>>>> If it weren't for the last 2 clocks, we could cleanly split out apb1 and
>>>> apb2
>>>> gates. Having a separate AHB clock gate taking 2 addresses seems messy
>>>> as well. :(
>>>
>>>
>>> Well, maybe we still should do that, if we split the resets too at least
>>> apb[12] would line up again.
>>>
>>> I don't know what to do with these bus things any more, all variants I
>>> sent had issues somewhere...
>>
>>
>> AFAIK, Arnd had some objections, but he never got back to us when we
>> explained how the hardware was laid out, so I don't know if they still
>> apply.
>>
>>>>> + };
>>>>> +
>>>>> + mmc0_clk: clk@01c20088 {
>>>>> + #clock-cells = <1>;
>>>>> + compatible = "allwinner,sun4i-a10-mmc-clk";
>>>>> + reg = <0x01c20088 0x4>;
>>>>> + clocks = <&osc24M>, <&pll6 0>, <&pll8 0>;
>>>>> + clock-output-names = "mmc0",
>>>>> + "mmc0_output",
>>>>> + "mmc0_sample";
>>>>> + };
>>>>> +
>>>>> + mmc1_clk: clk@01c2008c {
>>>>> + #clock-cells = <1>;
>>>>> + compatible = "allwinner,sun4i-a10-mmc-clk";
>>>>> + reg = <0x01c2008c 0x4>;
>>>>> + clocks = <&osc24M>, <&pll6 0>, <&pll8 0>;
>>>>> + clock-output-names = "mmc1",
>>>>> + "mmc1_output",
>>>>> + "mmc1_sample";
>>>>> + };
>>>>> +
>>>>> + mmc2_clk: clk@01c20090 {
>>>>> + #clock-cells = <1>;
>>>>> + compatible = "allwinner,sun4i-a10-mmc-clk";
>>>>> + reg = <0x01c20090 0x4>;
>>>>> + clocks = <&osc24M>, <&pll6 0>, <&pll8 0>;
>>>>> + clock-output-names = "mmc2",
>>>>> + "mmc2_output",
>>>>> + "mmc2_sample";
>>>>> + };
>>>>> +
>>>>> + mbus_clk: clk@01c2015c {
>>>>> + #clock-cells = <0>;
>>>>> + compatible = "allwinner,sun8i-a23-mbus-clk";
>>>>> + reg = <0x01c2015c 0x4>;
>>>>> + clocks = <&osc24M>, <&pll6 1>, <&pll5>;
>>>>> + clock-output-names = "mbus";
>>>>> + };
>>>>> + };
>>>>> +
>>>>> + soc {
>>>>> + compatible = "simple-bus";
>>>>> + #address-cells = <1>;
>>>>> + #size-cells = <1>;
>>>>> + ranges;
>>>>> +
>>>>> + dma: dma-controller@01c02000 {
>>>>> + compatible = "allwinner,sun8i-h3-dma";
>>>>> + reg = <0x01c02000 0x1000>;
>>>>> + interrupts = <GIC_SPI 50 IRQ_TYPE_LEVEL_HIGH>;
>>>>> + clocks = <&bus_gates 6>;
>>>>> + resets = <&bus_rst 6>;
>>>>> + #dma-cells = <1>;
>>>>> + };
>>>>> +
>>>>> + mmc0: mmc@01c0f000 {
>>>>> + compatible = "allwinner,sun5i-a13-mmc";
>>>>> + reg = <0x01c0f000 0x1000>;
>>>>> + clocks = <&bus_gates 8>,
>>>>> + <&mmc0_clk 0>,
>>>>> + <&mmc0_clk 1>,
>>>>> + <&mmc0_clk 2>;
>>>>> + clock-names = "ahb",
>>>>> + "mmc",
>>>>> + "output",
>>>>> + "sample";
>>>>> + resets = <&bus_rst 8>;
>>>>> + reset-names = "ahb";
>>>>> + interrupts = <GIC_SPI 60 IRQ_TYPE_LEVEL_HIGH>;
>>>>> + status = "disabled";
>>>>> + #address-cells = <1>;
>>>>> + #size-cells = <0>;
>>>>> + };
>>>>> +
>>>>> + mmc1: mmc@01c10000 {
>>>>> + compatible = "allwinner,sun5i-a13-mmc";
>>>>> + reg = <0x01c10000 0x1000>;
>>>>> + clocks = <&bus_gates 9>,
>>>>> + <&mmc1_clk 0>,
>>>>> + <&mmc1_clk 1>,
>>>>> + <&mmc1_clk 2>;
>>>>> + clock-names = "ahb",
>>>>> + "mmc",
>>>>> + "output",
>>>>> + "sample";
>>>>> + resets = <&bus_rst 9>;
>>>>> + reset-names = "ahb";
>>>>> + interrupts = <GIC_SPI 61 IRQ_TYPE_LEVEL_HIGH>;
>>>>> + status = "disabled";
>>>>> + #address-cells = <1>;
>>>>> + #size-cells = <0>;
>>>>> + };
>>>>> +
>>>>> + mmc2: mmc@01c11000 {
>>>>> + compatible = "allwinner,sun5i-a13-mmc";
>>>>> + reg = <0x01c11000 0x1000>;
>>>>> + clocks = <&bus_gates 10>,
>>>>> + <&mmc2_clk 0>,
>>>>> + <&mmc2_clk 1>,
>>>>> + <&mmc2_clk 2>;
>>>>> + clock-names = "ahb",
>>>>> + "mmc",
>>>>> + "output",
>>>>> + "sample";
>>>>> + resets = <&bus_rst 10>;
>>>>> + reset-names = "ahb";
>>>>> + interrupts = <GIC_SPI 62 IRQ_TYPE_LEVEL_HIGH>;
>>>>> + status = "disabled";
>>>>> + #address-cells = <1>;
>>>>> + #size-cells = <0>;
>>>>> + };
>>>>> +
>>>>> + pio: pinctrl@01c20800 {
>>>>> + compatible = "allwinner,sun8i-h3-pinctrl";
>>>>> + reg = <0x01c20800 0x400>;
>>>>> + interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>,
>>>>> + <GIC_SPI 17 IRQ_TYPE_LEVEL_HIGH>;
>>>>> + clocks = <&bus_gates 69>;
>>>>> + gpio-controller;
>>>>> + #gpio-cells = <3>;
>>>>> + interrupt-controller;
>>>>> + #interrupt-cells = <2>;
>>>>> +
>>>>> + uart0_pins_a: uart0@0 {
>>>>> + allwinner,pins = "PA4", "PA5";
>>>>> + allwinner,function = "uart0";
>>>>> + allwinner,drive =
>>>>> <SUN4I_PINCTRL_10_MA>;
>>>>> + allwinner,pull =
>>>>> <SUN4I_PINCTRL_NO_PULL>;
>>>>> + };
>>>>> +
>>>>> + mmc0_pins_a: mmc0@0 {
>>>>> + allwinner,pins = "PF0", "PF1", "PF2",
>>>>> "PF3",
>>>>> + "PF4", "PF5";
>>>>> + allwinner,function = "mmc0";
>>>>> + allwinner,drive =
>>>>> <SUN4I_PINCTRL_30_MA>;
>>>>> + allwinner,pull =
>>>>> <SUN4I_PINCTRL_NO_PULL>;
>>>>> + };
>>>>> +
>>>>> + mmc0_cd_pin: mmc0_cd_pin@0 {
>>>>> + allwinner,pins = "PF6";
>>>>> + allwinner,function = "gpio_in";
>>>>> + allwinner,drive =
>>>>> <SUN4I_PINCTRL_10_MA>;
>>>>> + allwinner,pull =
>>>>> <SUN4I_PINCTRL_PULL_UP>;
>>>>> + };
>>>>
>>>>
>>>> This should be in the board DTS, unless this is the reference design,
>>>> in which case you should name the label like
>>>> "mmc0_cd_pin_reference_design".
>>>>
>>>
>>> The datasheet mentions SDC0_DET function on PF6
>
>
> Hmm, not in my version, I've "Allwinner_H3_Datasheet_V1.0.pdf" and there
> PF6 only has generic input / output functionality.
v1.1 has an "SDC0_DET" function for PF6 in the GPIO mux functions table.
>>> so I thought this is
>>>
>>> sort of fixed to this pin now. All designs I've seen use this pin.
>>
>>
>> Why is it set as a gpio then if it is a separate function?
>
>
> I guess because we do not support this in the mmc driver yet. Also on
> older devices the mmc controller has build-in card-detection features
> (using the data lines in that case) but we've never supported this since
> none of the boards sofar have been using it.
Unfortunately the H3 datasheet does not include a section for MMC.
I've requested the datasheet be publicly released with MMC and PRCM
sections included:
https://github.com/allwinner-zh/documents/issues/9
ChenYu
> For now we can just treat PF6 as a gpio, until someone figures out how
> to do this inside the mmc driver.
>
> Regards,
>
> Hans
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web