diff mbox

[v2,1/3] Document: dw_mmc-k3: add document of hi6220 mmc

Message ID 1430985401-12202-2-git-send-email-zhangfei.gao@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Zhangfei Gao May 7, 2015, 7:56 a.m. UTC
Add bindings for hi6220 mmc support

Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org>
---
 .../devicetree/bindings/mmc/k3-dw-mshc.txt         | 28 ++++++++++++++++++++++
 1 file changed, 28 insertions(+)

Comments

Jaehoon Chung May 12, 2015, 7:36 a.m. UTC | #1
Hi, Zhangfei.

On 05/07/2015 04:56 PM, Zhangfei Gao wrote:
> Add bindings for hi6220 mmc support
> 
> Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org>
> ---
>  .../devicetree/bindings/mmc/k3-dw-mshc.txt         | 28 ++++++++++++++++++++++
>  1 file changed, 28 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/mmc/k3-dw-mshc.txt b/Documentation/devicetree/bindings/mmc/k3-dw-mshc.txt
> index 3b35449..df37058 100644
> --- a/Documentation/devicetree/bindings/mmc/k3-dw-mshc.txt
> +++ b/Documentation/devicetree/bindings/mmc/k3-dw-mshc.txt
> @@ -13,6 +13,10 @@ Required Properties:
>  
>  * compatible: should be one of the following.
>    - "hisilicon,hi4511-dw-mshc": for controllers with hi4511 specific extensions.
> +  - "hisilicon,hi6220-dw-mshc": for controllers with hi6220 specific extensions.
> +
> +Optional Properties:
> +- hisilicon,peripheral-syscon: phandle of syscon used to control peripheral.
>  
I'm not sure..
but i think this patch could be added just the compatible and optional properties descriptions.
If SoC will be added, it need not to add every example, doesn't it?

how about removing the below example? This is just my opinion. :)
If there is a special reason that add example...let me know, plz.

Best Regards,
Jaehoon Chung

>  Example:
>  
> @@ -42,3 +46,27 @@ Example:
>  		cap-mmc-highspeed;
>  		cap-sd-highspeed;
>  	};
> +
> +	/* for Hi6220 */
> +
> +	dwmmc_1: dwmmc1@f723e000 {
> +		compatible = "hisilicon,hi6220-dw-mshc";
> +		num-slots = <0x1>;
> +		bus-width = <0x4>;
> +		disable-wp;
> +		cap-sd-highspeed;
> +		sd-uhs-sdr12;
> +		sd-uhs-sdr25;
> +		card-detect-delay = <200>;
> +		hisilicon,peripheral-syscon = <&ao_ctrl>;
> +		reg = <0x0 0xf723e000 0x0 0x1000>;
> +		interrupts = <0x0 0x49 0x4>;
> +		clocks = <&clock_sys HI6220_MMC1_CIUCLK>, <&clock_sys HI6220_MMC1_CLK>;
> +		clock-names = "ciu", "biu";
> +		cd-gpios = <&gpio1 0 1>;
> +		pinctrl-names = "default", "idle";
> +		pinctrl-0 = <&sd_pmx_func &sd_clk_cfg_func &sd_cfg_func>;
> +		pinctrl-1 = <&sd_pmx_idle &sd_clk_cfg_idle &sd_cfg_idle>;
> +		vqmmc-supply = <&ldo7>;
> +		vmmc-supply = <&ldo10>;
> +	};
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Zhangfei Gao May 12, 2015, 7:59 a.m. UTC | #2
On 05/12/2015 03:36 PM, Jaehoon Chung wrote:
> Hi, Zhangfei.
>
> On 05/07/2015 04:56 PM, Zhangfei Gao wrote:
>> Add bindings for hi6220 mmc support
>>
>> Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org>
>> ---
>>   .../devicetree/bindings/mmc/k3-dw-mshc.txt         | 28 ++++++++++++++++++++++
>>   1 file changed, 28 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/mmc/k3-dw-mshc.txt b/Documentation/devicetree/bindings/mmc/k3-dw-mshc.txt
>> index 3b35449..df37058 100644
>> --- a/Documentation/devicetree/bindings/mmc/k3-dw-mshc.txt
>> +++ b/Documentation/devicetree/bindings/mmc/k3-dw-mshc.txt
>> @@ -13,6 +13,10 @@ Required Properties:
>>
>>   * compatible: should be one of the following.
>>     - "hisilicon,hi4511-dw-mshc": for controllers with hi4511 specific extensions.
>> +  - "hisilicon,hi6220-dw-mshc": for controllers with hi6220 specific extensions.
>> +
>> +Optional Properties:
>> +- hisilicon,peripheral-syscon: phandle of syscon used to control peripheral.
>>
> I'm not sure..
> but i think this patch could be added just the compatible and optional properties descriptions.
> If SoC will be added, it need not to add every example, doesn't it?
>
> how about removing the below example? This is just my opinion. :)
> If there is a special reason that add example...let me know, plz.

I am adding this example just for two reasons

1. There is still some components need to be upstreamed before we add to 
dts, like pinctrl and regulator, which still need some time.
So add here for quick reference after the component is enabled.
Had experience before that hip04_eth is merged but dts is delayed, 
thanks to the document, we can add it quickly.

2. sd has some special requirement of "hisilicon,peripheral-syscon", 
which used for controlling voltage


What do you think?

Thanks
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Jaehoon Chung May 13, 2015, 10:17 p.m. UTC | #3
On 05/12/2015 04:59 PM, zhangfei wrote:
> 
> 
> On 05/12/2015 03:36 PM, Jaehoon Chung wrote:
>> Hi, Zhangfei.
>>
>> On 05/07/2015 04:56 PM, Zhangfei Gao wrote:
>>> Add bindings for hi6220 mmc support
>>>
>>> Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org>
>>> ---
>>>   .../devicetree/bindings/mmc/k3-dw-mshc.txt         | 28 ++++++++++++++++++++++
>>>   1 file changed, 28 insertions(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/mmc/k3-dw-mshc.txt b/Documentation/devicetree/bindings/mmc/k3-dw-mshc.txt
>>> index 3b35449..df37058 100644
>>> --- a/Documentation/devicetree/bindings/mmc/k3-dw-mshc.txt
>>> +++ b/Documentation/devicetree/bindings/mmc/k3-dw-mshc.txt
>>> @@ -13,6 +13,10 @@ Required Properties:
>>>
>>>   * compatible: should be one of the following.
>>>     - "hisilicon,hi4511-dw-mshc": for controllers with hi4511 specific extensions.
>>> +  - "hisilicon,hi6220-dw-mshc": for controllers with hi6220 specific extensions.
>>> +
>>> +Optional Properties:
>>> +- hisilicon,peripheral-syscon: phandle of syscon used to control peripheral.
>>>
>> I'm not sure..
>> but i think this patch could be added just the compatible and optional properties descriptions.
>> If SoC will be added, it need not to add every example, doesn't it?
>>
>> how about removing the below example? This is just my opinion. :)
>> If there is a special reason that add example...let me know, plz.
> 
> I am adding this example just for two reasons
> 
> 1. There is still some components need to be upstreamed before we add to dts, like pinctrl and regulator, which still need some time.
> So add here for quick reference after the component is enabled.
> Had experience before that hip04_eth is merged but dts is delayed, thanks to the document, we can add it quickly.
> 
> 2. sd has some special requirement of "hisilicon,peripheral-syscon", which used for controlling voltage
> 
> 
> What do you think?

Thanks for kindly explanation! I understood your intention.

Best Regards,
Jaehoon Chung

> 
> Thanks
> -- 
> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/mmc/k3-dw-mshc.txt b/Documentation/devicetree/bindings/mmc/k3-dw-mshc.txt
index 3b35449..df37058 100644
--- a/Documentation/devicetree/bindings/mmc/k3-dw-mshc.txt
+++ b/Documentation/devicetree/bindings/mmc/k3-dw-mshc.txt
@@ -13,6 +13,10 @@  Required Properties:
 
 * compatible: should be one of the following.
   - "hisilicon,hi4511-dw-mshc": for controllers with hi4511 specific extensions.
+  - "hisilicon,hi6220-dw-mshc": for controllers with hi6220 specific extensions.
+
+Optional Properties:
+- hisilicon,peripheral-syscon: phandle of syscon used to control peripheral.
 
 Example:
 
@@ -42,3 +46,27 @@  Example:
 		cap-mmc-highspeed;
 		cap-sd-highspeed;
 	};
+
+	/* for Hi6220 */
+
+	dwmmc_1: dwmmc1@f723e000 {
+		compatible = "hisilicon,hi6220-dw-mshc";
+		num-slots = <0x1>;
+		bus-width = <0x4>;
+		disable-wp;
+		cap-sd-highspeed;
+		sd-uhs-sdr12;
+		sd-uhs-sdr25;
+		card-detect-delay = <200>;
+		hisilicon,peripheral-syscon = <&ao_ctrl>;
+		reg = <0x0 0xf723e000 0x0 0x1000>;
+		interrupts = <0x0 0x49 0x4>;
+		clocks = <&clock_sys HI6220_MMC1_CIUCLK>, <&clock_sys HI6220_MMC1_CLK>;
+		clock-names = "ciu", "biu";
+		cd-gpios = <&gpio1 0 1>;
+		pinctrl-names = "default", "idle";
+		pinctrl-0 = <&sd_pmx_func &sd_clk_cfg_func &sd_cfg_func>;
+		pinctrl-1 = <&sd_pmx_idle &sd_clk_cfg_idle &sd_cfg_idle>;
+		vqmmc-supply = <&ldo7>;
+		vmmc-supply = <&ldo10>;
+	};