* [PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode
@ 2021-04-05 14:44 ` Aswath Govindraju
2021-04-05 22:18 ` Jaehoon Chung
` (2 more replies)
0 siblings, 3 replies; 16+ messages in thread
From: Aswath Govindraju @ 2021-04-05 14:44 UTC (permalink / raw)
To: u-boot
From: Faiz Abbas <faiz_abbas@ti.com>
Enable HS400 speed mode by writing to HOST_CONTROL2 register.
Signed-off-by: Faiz Abbas <faiz_abbas@ti.com>
Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>
---
drivers/mmc/sdhci.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c
index d9ab6a0a839e..eea4701d8af5 100644
--- a/drivers/mmc/sdhci.c
+++ b/drivers/mmc/sdhci.c
@@ -507,6 +507,9 @@ void sdhci_set_uhs_timing(struct sdhci_host *host)
case MMC_HS_200:
reg |= SDHCI_CTRL_UHS_SDR104;
break;
+ case MMC_HS_400:
+ reg |= SDHCI_CTRL_HS400;
+ break;
default:
reg |= SDHCI_CTRL_UHS_SDR12;
}
--
2.17.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode
2021-04-05 14:44 ` [PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode Aswath Govindraju
@ 2021-04-05 22:18 ` Jaehoon Chung
2021-04-06 5:20 ` Aswath Govindraju
2021-04-06 22:22 ` Jaehoon Chung
2021-07-29 5:14 ` Lokesh Vutla
2 siblings, 1 reply; 16+ messages in thread
From: Jaehoon Chung @ 2021-04-05 22:18 UTC (permalink / raw)
To: u-boot
Hi,
On 4/5/21 11:44 PM, Aswath Govindraju wrote:
> From: Faiz Abbas <faiz_abbas@ti.com>
>
> Enable HS400 speed mode by writing to HOST_CONTROL2 register.
I didn't find HS400 bit at HOST_CONTROL2 register. (I have checked SD Specific v4.20)
If I missed something, let me know, plz.
In include/sdhci.h, it mentioned to "Non-standard".
Best Regards,
Jaehoon Chung
>
> Signed-off-by: Faiz Abbas <faiz_abbas@ti.com>
> Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>
> ---
> drivers/mmc/sdhci.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c
> index d9ab6a0a839e..eea4701d8af5 100644
> --- a/drivers/mmc/sdhci.c
> +++ b/drivers/mmc/sdhci.c
> @@ -507,6 +507,9 @@ void sdhci_set_uhs_timing(struct sdhci_host *host)
> case MMC_HS_200:
> reg |= SDHCI_CTRL_UHS_SDR104;
> break;
> + case MMC_HS_400:
> + reg |= SDHCI_CTRL_HS400;
> + break;
> default:
> reg |= SDHCI_CTRL_UHS_SDR12;
> }
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode
2021-04-05 22:18 ` Jaehoon Chung
@ 2021-04-06 5:20 ` Aswath Govindraju
2021-04-06 8:26 ` Aswath Govindraju
0 siblings, 1 reply; 16+ messages in thread
From: Aswath Govindraju @ 2021-04-06 5:20 UTC (permalink / raw)
To: u-boot
Hi Jaehoon,
On 06/04/21 3:48 am, Jaehoon Chung wrote:
> Hi,
>
> On 4/5/21 11:44 PM, Aswath Govindraju wrote:
>> From: Faiz Abbas <faiz_abbas@ti.com>
>>
>> Enable HS400 speed mode by writing to HOST_CONTROL2 register.
>
> I didn't find HS400 bit at HOST_CONTROL2 register. (I have checked SD Specific v4.20)
> If I missed something, let me know, plz.
>
> In include/sdhci.h, it mentioned to "Non-standard".
>
Thank you for pointing this out.
Yes, this is not a part of SD specification and in the specification
this field is marked as reserved. I have added this, as this bit mask
has been defined in include/sdhci.h. Sorry about this. I'll post a
respin taking this into a consideration.
Thanks,
Aswath
> Best Regards,
> Jaehoon Chung
>
>>
>> Signed-off-by: Faiz Abbas <faiz_abbas@ti.com>
>> Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>
>> ---
>> drivers/mmc/sdhci.c | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c
>> index d9ab6a0a839e..eea4701d8af5 100644
>> --- a/drivers/mmc/sdhci.c
>> +++ b/drivers/mmc/sdhci.c
>> @@ -507,6 +507,9 @@ void sdhci_set_uhs_timing(struct sdhci_host *host)
>> case MMC_HS_200:
>> reg |= SDHCI_CTRL_UHS_SDR104;
>> break;
>> + case MMC_HS_400:
>> + reg |= SDHCI_CTRL_HS400;
>> + break;
>> default:
>> reg |= SDHCI_CTRL_UHS_SDR12;
>> }
>>
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode
2021-04-06 5:20 ` Aswath Govindraju
@ 2021-04-06 8:26 ` Aswath Govindraju
2021-04-06 9:14 ` Jaehoon Chung
0 siblings, 1 reply; 16+ messages in thread
From: Aswath Govindraju @ 2021-04-06 8:26 UTC (permalink / raw)
To: u-boot
Hi Jaehoon,
On 06/04/21 10:50 am, Aswath Govindraju wrote:
> Hi Jaehoon,
>
> On 06/04/21 3:48 am, Jaehoon Chung wrote:
>> Hi,
>>
>> On 4/5/21 11:44 PM, Aswath Govindraju wrote:
>>> From: Faiz Abbas <faiz_abbas@ti.com>
>>>
>>> Enable HS400 speed mode by writing to HOST_CONTROL2 register.
>>
>> I didn't find HS400 bit at HOST_CONTROL2 register. (I have checked SD Specific v4.20)
>> If I missed something, let me know, plz.
>>
>> In include/sdhci.h, it mentioned to "Non-standard".
>>
>
> Thank you for pointing this out.
>
> Yes, this is not a part of SD specification and in the specification
> this field is marked as reserved. I have added this, as this bit mask
> has been defined in include/sdhci.h. Sorry about this. I'll post a
> respin taking this into a consideration.
>
On the other hand, the same has been accepted in kernel i.e. setting
SDHCI_CTRL_HS400 bit in HOST_CONTROL2 register, drivers/mmc/host/sdhci.c
in sdhci_set_uhs_signaling(). I think although this bit has not been
mentioned in the specification, it seems it is used by many controllers
for HS400 mode. So, as this is not specific to a controller can setting
this, be allowed in the common sdhci_set_uhs_timing() function ?
Thanks,
Aswath
> Thanks,
> Aswath
>
>> Best Regards,
>> Jaehoon Chung
>>
>>>
>>> Signed-off-by: Faiz Abbas <faiz_abbas@ti.com>
>>> Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>
>>> ---
>>> drivers/mmc/sdhci.c | 3 +++
>>> 1 file changed, 3 insertions(+)
>>>
>>> diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c
>>> index d9ab6a0a839e..eea4701d8af5 100644
>>> --- a/drivers/mmc/sdhci.c
>>> +++ b/drivers/mmc/sdhci.c
>>> @@ -507,6 +507,9 @@ void sdhci_set_uhs_timing(struct sdhci_host *host)
>>> case MMC_HS_200:
>>> reg |= SDHCI_CTRL_UHS_SDR104;
>>> break;
>>> + case MMC_HS_400:
>>> + reg |= SDHCI_CTRL_HS400;
>>> + break;
>>> default:
>>> reg |= SDHCI_CTRL_UHS_SDR12;
>>> }
>>>
>>
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode
2021-04-06 8:26 ` Aswath Govindraju
@ 2021-04-06 9:14 ` Jaehoon Chung
0 siblings, 0 replies; 16+ messages in thread
From: Jaehoon Chung @ 2021-04-06 9:14 UTC (permalink / raw)
To: u-boot
Hi Aswath,
On 4/6/21 5:26 PM, Aswath Govindraju wrote:
> Hi Jaehoon,
>
> On 06/04/21 10:50 am, Aswath Govindraju wrote:
>> Hi Jaehoon,
>>
>> On 06/04/21 3:48 am, Jaehoon Chung wrote:
>>> Hi,
>>>
>>> On 4/5/21 11:44 PM, Aswath Govindraju wrote:
>>>> From: Faiz Abbas <faiz_abbas@ti.com>
>>>>
>>>> Enable HS400 speed mode by writing to HOST_CONTROL2 register.
>>>
>>> I didn't find HS400 bit at HOST_CONTROL2 register. (I have checked SD Specific v4.20)
>>> If I missed something, let me know, plz.
>>>
>>> In include/sdhci.h, it mentioned to "Non-standard".
>>>
>>
>> Thank you for pointing this out.
>>
>> Yes, this is not a part of SD specification and in the specification
>> this field is marked as reserved. I have added this, as this bit mask
>> has been defined in include/sdhci.h. Sorry about this. I'll post a
>> respin taking this into a consideration.
>>
>
> On the other hand, the same has been accepted in kernel i.e. setting
> SDHCI_CTRL_HS400 bit in HOST_CONTROL2 register, drivers/mmc/host/sdhci.c
> in sdhci_set_uhs_signaling(). I think although this bit has not been
> mentioned in the specification, it seems it is used by many controllers
> for HS400 mode. So, as this is not specific to a controller can setting
> this, be allowed in the common sdhci_set_uhs_timing() function ?
Frankly, i think that it can be used. Just I wondered that i missed something from Specification. :)
Thanks for sharing information. I didn't check on kernel side.
Best Regards,
Jaehoon Chung
>
> Thanks,
> Aswath
>
>> Thanks,
>> Aswath
>>
>>> Best Regards,
>>> Jaehoon Chung
>>>
>>>>
>>>> Signed-off-by: Faiz Abbas <faiz_abbas@ti.com>
>>>> Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>
>>>> ---
>>>> drivers/mmc/sdhci.c | 3 +++
>>>> 1 file changed, 3 insertions(+)
>>>>
>>>> diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c
>>>> index d9ab6a0a839e..eea4701d8af5 100644
>>>> --- a/drivers/mmc/sdhci.c
>>>> +++ b/drivers/mmc/sdhci.c
>>>> @@ -507,6 +507,9 @@ void sdhci_set_uhs_timing(struct sdhci_host *host)
>>>> case MMC_HS_200:
>>>> reg |= SDHCI_CTRL_UHS_SDR104;
>>>> break;
>>>> + case MMC_HS_400:
>>>> + reg |= SDHCI_CTRL_HS400;
>>>> + break;
>>>> default:
>>>> reg |= SDHCI_CTRL_UHS_SDR12;
>>>> }
>>>>
>>>
>>
>
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode
2021-04-05 14:44 ` [PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode Aswath Govindraju
2021-04-05 22:18 ` Jaehoon Chung
@ 2021-04-06 22:22 ` Jaehoon Chung
2021-05-10 13:48 ` Aswath Govindraju
2021-07-29 5:14 ` Lokesh Vutla
2 siblings, 1 reply; 16+ messages in thread
From: Jaehoon Chung @ 2021-04-06 22:22 UTC (permalink / raw)
To: u-boot
On 4/5/21 11:44 PM, Aswath Govindraju wrote:
> From: Faiz Abbas <faiz_abbas@ti.com>
>
> Enable HS400 speed mode by writing to HOST_CONTROL2 register.
>
> Signed-off-by: Faiz Abbas <faiz_abbas@ti.com>
> Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>
Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
Best Regards,
Jaehoon Chung
> ---
> drivers/mmc/sdhci.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c
> index d9ab6a0a839e..eea4701d8af5 100644
> --- a/drivers/mmc/sdhci.c
> +++ b/drivers/mmc/sdhci.c
> @@ -507,6 +507,9 @@ void sdhci_set_uhs_timing(struct sdhci_host *host)
> case MMC_HS_200:
> reg |= SDHCI_CTRL_UHS_SDR104;
> break;
> + case MMC_HS_400:
> + reg |= SDHCI_CTRL_HS400;
> + break;
> default:
> reg |= SDHCI_CTRL_UHS_SDR12;
> }
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode
2021-04-06 22:22 ` Jaehoon Chung
@ 2021-05-10 13:48 ` Aswath Govindraju
2021-06-09 15:26 ` Aswath Govindraju
0 siblings, 1 reply; 16+ messages in thread
From: Aswath Govindraju @ 2021-05-10 13:48 UTC (permalink / raw)
To: u-boot
Hi Peng,
On 07/04/21 3:52 am, Jaehoon Chung wrote:
> On 4/5/21 11:44 PM, Aswath Govindraju wrote:
>> From: Faiz Abbas <faiz_abbas@ti.com>
>>
>> Enable HS400 speed mode by writing to HOST_CONTROL2 register.
>>
>> Signed-off-by: Faiz Abbas <faiz_abbas@ti.com>
>> Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>
>
> Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
>
Can you please pick this patch if there are no comments.
Thanks,
Aswath
> Best Regards,
> Jaehoon Chung
>
>> ---
>> drivers/mmc/sdhci.c | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c
>> index d9ab6a0a839e..eea4701d8af5 100644
>> --- a/drivers/mmc/sdhci.c
>> +++ b/drivers/mmc/sdhci.c
>> @@ -507,6 +507,9 @@ void sdhci_set_uhs_timing(struct sdhci_host *host)
>> case MMC_HS_200:
>> reg |= SDHCI_CTRL_UHS_SDR104;
>> break;
>> + case MMC_HS_400:
>> + reg |= SDHCI_CTRL_HS400;
>> + break;
>> default:
>> reg |= SDHCI_CTRL_UHS_SDR12;
>> }
>>
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode
2021-05-10 13:48 ` Aswath Govindraju
@ 2021-06-09 15:26 ` Aswath Govindraju
2021-07-19 6:48 ` Aswath Govindraju
0 siblings, 1 reply; 16+ messages in thread
From: Aswath Govindraju @ 2021-06-09 15:26 UTC (permalink / raw)
To: Peng Fan
Cc: Jaehoon Chung, u-boot, Praneeth Bajjuri, Lokesh Vutla,
Kishon Vijay Abraham I, Vignesh Raghavendra, Faiz Abbas
Hi Peng,
On 10/05/21 7:18 pm, Aswath Govindraju wrote:
> Hi Peng,
>
> On 07/04/21 3:52 am, Jaehoon Chung wrote:
>> On 4/5/21 11:44 PM, Aswath Govindraju wrote:
>>> From: Faiz Abbas <faiz_abbas@ti.com>
>>>
>>> Enable HS400 speed mode by writing to HOST_CONTROL2 register.
>>>
>>> Signed-off-by: Faiz Abbas <faiz_abbas@ti.com>
>>> Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>
>>
>> Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
>>
>
> Can you please pick this patch if there are no comments.
>
May I know if this okay to be merged ?
Thanks,
Aswath
> Thanks,
> Aswath
>
>> Best Regards,
>> Jaehoon Chung
>>
>>> ---
>>> drivers/mmc/sdhci.c | 3 +++
>>> 1 file changed, 3 insertions(+)
>>>
>>> diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c
>>> index d9ab6a0a839e..eea4701d8af5 100644
>>> --- a/drivers/mmc/sdhci.c
>>> +++ b/drivers/mmc/sdhci.c
>>> @@ -507,6 +507,9 @@ void sdhci_set_uhs_timing(struct sdhci_host *host)
>>> case MMC_HS_200:
>>> reg |= SDHCI_CTRL_UHS_SDR104;
>>> break;
>>> + case MMC_HS_400:
>>> + reg |= SDHCI_CTRL_HS400;
>>> + break;
>>> default:
>>> reg |= SDHCI_CTRL_UHS_SDR12;
>>> }
>>>
>>
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode
2021-06-09 15:26 ` Aswath Govindraju
@ 2021-07-19 6:48 ` Aswath Govindraju
2021-07-19 22:15 ` Jaehoon Chung
0 siblings, 1 reply; 16+ messages in thread
From: Aswath Govindraju @ 2021-07-19 6:48 UTC (permalink / raw)
To: Peng Fan
Cc: Jaehoon Chung, u-boot, Praneeth Bajjuri, Lokesh Vutla,
Kishon Vijay Abraham I, Vignesh Raghavendra, Faiz Abbas
Hi Peng,
On 09/06/21 8:56 pm, Aswath Govindraju wrote:
> Hi Peng,
>
> On 10/05/21 7:18 pm, Aswath Govindraju wrote:
>> Hi Peng,
>>
>> On 07/04/21 3:52 am, Jaehoon Chung wrote:
>>> On 4/5/21 11:44 PM, Aswath Govindraju wrote:
>>>> From: Faiz Abbas <faiz_abbas@ti.com>
>>>>
>>>> Enable HS400 speed mode by writing to HOST_CONTROL2 register.
>>>>
>>>> Signed-off-by: Faiz Abbas <faiz_abbas@ti.com>
>>>> Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>
>>>
>>> Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
>>>
>>
>> Can you please pick this patch if there are no comments.
>>
>
> May I know if this okay to be merged ?
>
A gentle reminder on this patch. This patch has other dependencies that
and are pending merge[1][2].
[1] - https://patchwork.ozlabs.org/project/uboot/list/?series=247000
[2] - https://patchwork.ozlabs.org/project/uboot/list/?series=245579
Thanks,
Aswath
> Thanks,
> Aswath
>
>> Thanks,
>> Aswath
>>
>>> Best Regards,
>>> Jaehoon Chung
>>>
>>>> ---
>>>> drivers/mmc/sdhci.c | 3 +++
>>>> 1 file changed, 3 insertions(+)
>>>>
>>>> diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c
>>>> index d9ab6a0a839e..eea4701d8af5 100644
>>>> --- a/drivers/mmc/sdhci.c
>>>> +++ b/drivers/mmc/sdhci.c
>>>> @@ -507,6 +507,9 @@ void sdhci_set_uhs_timing(struct sdhci_host *host)
>>>> case MMC_HS_200:
>>>> reg |= SDHCI_CTRL_UHS_SDR104;
>>>> break;
>>>> + case MMC_HS_400:
>>>> + reg |= SDHCI_CTRL_HS400;
>>>> + break;
>>>> default:
>>>> reg |= SDHCI_CTRL_UHS_SDR12;
>>>> }
>>>>
>>>
>>
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode
2021-07-19 6:48 ` Aswath Govindraju
@ 2021-07-19 22:15 ` Jaehoon Chung
2021-07-20 4:10 ` Lokesh Vutla
0 siblings, 1 reply; 16+ messages in thread
From: Jaehoon Chung @ 2021-07-19 22:15 UTC (permalink / raw)
To: Aswath Govindraju, Peng Fan
Cc: u-boot, Praneeth Bajjuri, Lokesh Vutla, Kishon Vijay Abraham I,
Vignesh Raghavendra, Faiz Abbas
Hi Aswath,
On 7/19/21 3:48 PM, Aswath Govindraju wrote:
> Hi Peng,
>
> On 09/06/21 8:56 pm, Aswath Govindraju wrote:
>> Hi Peng,
>>
>> On 10/05/21 7:18 pm, Aswath Govindraju wrote:
>>> Hi Peng,
>>>
>>> On 07/04/21 3:52 am, Jaehoon Chung wrote:
>>>> On 4/5/21 11:44 PM, Aswath Govindraju wrote:
>>>>> From: Faiz Abbas <faiz_abbas@ti.com>
>>>>>
>>>>> Enable HS400 speed mode by writing to HOST_CONTROL2 register.
>>>>>
>>>>> Signed-off-by: Faiz Abbas <faiz_abbas@ti.com>
>>>>> Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>
>>>>
>>>> Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
>>>>
>>>
>>> Can you please pick this patch if there are no comments.
>>>
>>
>> May I know if this okay to be merged ?
>>
>
> A gentle reminder on this patch. This patch has other dependencies that
> and are pending merge[1][2].
I don't have the permission to merge on u-boot-mmc git.
I have the permission to merge on just u-boot-pmic..But I don't know whether it's the best way to apply this into my u-boot-pmic or not. :)
I had been already reviewed about this patch..
So if want to pick this, I think there is no problem about applied together with below patches.
Acked-by: Jaehoon Chung <jh80.chung@samsung.com>
Best Regards,
Jaehoon Chung
>
> [1] - https://protect2.fireeye.com/v1/url?k=bf905767-e00b6fae-bf91dc28-0cc47a336fae-6a3fa5da3333a309&q=1&e=ac3de2ef-41d1-4e60-952a-16e2b302cfdd&u=https%3A%2F%2Fpatchwork.ozlabs.org%2Fproject%2Fuboot%2Flist%2F%3Fseries%3D247000
> [2] - https://protect2.fireeye.com/v1/url?k=c93699cd-96ada104-c9371282-0cc47a336fae-de6d85845da27eb0&q=1&e=ac3de2ef-41d1-4e60-952a-16e2b302cfdd&u=https%3A%2F%2Fpatchwork.ozlabs.org%2Fproject%2Fuboot%2Flist%2F%3Fseries%3D245579
>
> Thanks,
> Aswath
>
>> Thanks,
>> Aswath
>>
>>> Thanks,
>>> Aswath
>>>
>>>> Best Regards,
>>>> Jaehoon Chung
>>>>
>>>>> ---
>>>>> drivers/mmc/sdhci.c | 3 +++
>>>>> 1 file changed, 3 insertions(+)
>>>>>
>>>>> diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c
>>>>> index d9ab6a0a839e..eea4701d8af5 100644
>>>>> --- a/drivers/mmc/sdhci.c
>>>>> +++ b/drivers/mmc/sdhci.c
>>>>> @@ -507,6 +507,9 @@ void sdhci_set_uhs_timing(struct sdhci_host *host)
>>>>> case MMC_HS_200:
>>>>> reg |= SDHCI_CTRL_UHS_SDR104;
>>>>> break;
>>>>> + case MMC_HS_400:
>>>>> + reg |= SDHCI_CTRL_HS400;
>>>>> + break;
>>>>> default:
>>>>> reg |= SDHCI_CTRL_UHS_SDR12;
>>>>> }
>>>>>
>>>>
>>>
>>
>
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode
2021-07-19 22:15 ` Jaehoon Chung
@ 2021-07-20 4:10 ` Lokesh Vutla
2021-07-20 5:21 ` Jaehoon Chung
2021-07-20 12:16 ` Tom Rini
0 siblings, 2 replies; 16+ messages in thread
From: Lokesh Vutla @ 2021-07-20 4:10 UTC (permalink / raw)
To: Jaehoon Chung, Aswath Govindraju, Peng Fan
Cc: u-boot, Praneeth Bajjuri, Kishon Vijay Abraham I,
Vignesh Raghavendra, Faiz Abbas, Tom Rini
+Tom
On 20/07/21 3:45 am, Jaehoon Chung wrote:
> Hi Aswath,
>
> On 7/19/21 3:48 PM, Aswath Govindraju wrote:
>> Hi Peng,
>>
>> On 09/06/21 8:56 pm, Aswath Govindraju wrote:
>>> Hi Peng,
>>>
>>> On 10/05/21 7:18 pm, Aswath Govindraju wrote:
>>>> Hi Peng,
>>>>
>>>> On 07/04/21 3:52 am, Jaehoon Chung wrote:
>>>>> On 4/5/21 11:44 PM, Aswath Govindraju wrote:
>>>>>> From: Faiz Abbas <faiz_abbas@ti.com>
>>>>>>
>>>>>> Enable HS400 speed mode by writing to HOST_CONTROL2 register.
>>>>>>
>>>>>> Signed-off-by: Faiz Abbas <faiz_abbas@ti.com>
>>>>>> Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>
>>>>>
>>>>> Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
>>>>>
>>>>
>>>> Can you please pick this patch if there are no comments.
>>>>
>>>
>>> May I know if this okay to be merged ?
>>>
>>
>> A gentle reminder on this patch. This patch has other dependencies that
>> and are pending merge[1][2].
>
> I don't have the permission to merge on u-boot-mmc git.
> I have the permission to merge on just u-boot-pmic..But I don't know whether it's the best way to apply this into my u-boot-pmic or not. :)
>
> I had been already reviewed about this patch..
> So if want to pick this, I think there is no problem about applied together with below patches.
>
> Acked-by: Jaehoon Chung <jh80.chung@samsung.com>
Tom,
Is it okay if I pick this patch as per above suggestion?
Thanks and regards,
Lokesh
>
> Best Regards,
> Jaehoon Chung
>
>>
>> [1] - https://protect2.fireeye.com/v1/url?k=bf905767-e00b6fae-bf91dc28-0cc47a336fae-6a3fa5da3333a309&q=1&e=ac3de2ef-41d1-4e60-952a-16e2b302cfdd&u=https%3A%2F%2Fpatchwork.ozlabs.org%2Fproject%2Fuboot%2Flist%2F%3Fseries%3D247000
>> [2] - https://protect2.fireeye.com/v1/url?k=c93699cd-96ada104-c9371282-0cc47a336fae-de6d85845da27eb0&q=1&e=ac3de2ef-41d1-4e60-952a-16e2b302cfdd&u=https%3A%2F%2Fpatchwork.ozlabs.org%2Fproject%2Fuboot%2Flist%2F%3Fseries%3D245579
>>
>> Thanks,
>> Aswath
>>
>>> Thanks,
>>> Aswath
>>>
>>>> Thanks,
>>>> Aswath
>>>>
>>>>> Best Regards,
>>>>> Jaehoon Chung
>>>>>
>>>>>> ---
>>>>>> drivers/mmc/sdhci.c | 3 +++
>>>>>> 1 file changed, 3 insertions(+)
>>>>>>
>>>>>> diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c
>>>>>> index d9ab6a0a839e..eea4701d8af5 100644
>>>>>> --- a/drivers/mmc/sdhci.c
>>>>>> +++ b/drivers/mmc/sdhci.c
>>>>>> @@ -507,6 +507,9 @@ void sdhci_set_uhs_timing(struct sdhci_host *host)
>>>>>> case MMC_HS_200:
>>>>>> reg |= SDHCI_CTRL_UHS_SDR104;
>>>>>> break;
>>>>>> + case MMC_HS_400:
>>>>>> + reg |= SDHCI_CTRL_HS400;
>>>>>> + break;
>>>>>> default:
>>>>>> reg |= SDHCI_CTRL_UHS_SDR12;
>>>>>> }
>>>>>>
>>>>>
>>>>
>>>
>>
>>
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode
2021-07-20 4:10 ` Lokesh Vutla
@ 2021-07-20 5:21 ` Jaehoon Chung
2021-07-20 5:23 ` Lokesh Vutla
2021-07-20 12:16 ` Tom Rini
1 sibling, 1 reply; 16+ messages in thread
From: Jaehoon Chung @ 2021-07-20 5:21 UTC (permalink / raw)
To: Lokesh Vutla, Aswath Govindraju, Peng Fan
Cc: u-boot, Praneeth Bajjuri, Kishon Vijay Abraham I,
Vignesh Raghavendra, Faiz Abbas, Tom Rini
Hi Lokesh,
On 7/20/21 1:10 PM, Lokesh Vutla wrote:
> +Tom
>
> On 20/07/21 3:45 am, Jaehoon Chung wrote:
>> Hi Aswath,
>>
>> On 7/19/21 3:48 PM, Aswath Govindraju wrote:
>>> Hi Peng,
>>>
>>> On 09/06/21 8:56 pm, Aswath Govindraju wrote:
>>>> Hi Peng,
>>>>
>>>> On 10/05/21 7:18 pm, Aswath Govindraju wrote:
>>>>> Hi Peng,
>>>>>
>>>>> On 07/04/21 3:52 am, Jaehoon Chung wrote:
>>>>>> On 4/5/21 11:44 PM, Aswath Govindraju wrote:
>>>>>>> From: Faiz Abbas <faiz_abbas@ti.com>
>>>>>>>
>>>>>>> Enable HS400 speed mode by writing to HOST_CONTROL2 register.
>>>>>>>
>>>>>>> Signed-off-by: Faiz Abbas <faiz_abbas@ti.com>
>>>>>>> Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>
>>>>>>
>>>>>> Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
>>>>>>
>>>>>
>>>>> Can you please pick this patch if there are no comments.
>>>>>
>>>>
>>>> May I know if this okay to be merged ?
>>>>
>>>
>>> A gentle reminder on this patch. This patch has other dependencies that
>>> and are pending merge[1][2].
>>
>> I don't have the permission to merge on u-boot-mmc git.
>> I have the permission to merge on just u-boot-pmic..But I don't know whether it's the best way to apply this into my u-boot-pmic or not. :)
>>
>> I had been already reviewed about this patch..
>> So if want to pick this, I think there is no problem about applied together with below patches.
>>
>> Acked-by: Jaehoon Chung <jh80.chung@samsung.com>
>
> Tom,
> Is it okay if I pick this patch as per above suggestion?
Isn't it enough my Acked-by tag?
Best Regards,
Jaehoon Chung
>
> Thanks and regards,
> Lokesh
>
>>
>> Best Regards,
>> Jaehoon Chung
>>
>>>
>>> [1] - https://protect2.fireeye.com/v1/url?k=bf905767-e00b6fae-bf91dc28-0cc47a336fae-6a3fa5da3333a309&q=1&e=ac3de2ef-41d1-4e60-952a-16e2b302cfdd&u=https%3A%2F%2Fpatchwork.ozlabs.org%2Fproject%2Fuboot%2Flist%2F%3Fseries%3D247000
>>> [2] - https://protect2.fireeye.com/v1/url?k=c93699cd-96ada104-c9371282-0cc47a336fae-de6d85845da27eb0&q=1&e=ac3de2ef-41d1-4e60-952a-16e2b302cfdd&u=https%3A%2F%2Fpatchwork.ozlabs.org%2Fproject%2Fuboot%2Flist%2F%3Fseries%3D245579
>>>
>>> Thanks,
>>> Aswath
>>>
>>>> Thanks,
>>>> Aswath
>>>>
>>>>> Thanks,
>>>>> Aswath
>>>>>
>>>>>> Best Regards,
>>>>>> Jaehoon Chung
>>>>>>
>>>>>>> ---
>>>>>>> drivers/mmc/sdhci.c | 3 +++
>>>>>>> 1 file changed, 3 insertions(+)
>>>>>>>
>>>>>>> diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c
>>>>>>> index d9ab6a0a839e..eea4701d8af5 100644
>>>>>>> --- a/drivers/mmc/sdhci.c
>>>>>>> +++ b/drivers/mmc/sdhci.c
>>>>>>> @@ -507,6 +507,9 @@ void sdhci_set_uhs_timing(struct sdhci_host *host)
>>>>>>> case MMC_HS_200:
>>>>>>> reg |= SDHCI_CTRL_UHS_SDR104;
>>>>>>> break;
>>>>>>> + case MMC_HS_400:
>>>>>>> + reg |= SDHCI_CTRL_HS400;
>>>>>>> + break;
>>>>>>> default:
>>>>>>> reg |= SDHCI_CTRL_UHS_SDR12;
>>>>>>> }
>>>>>>>
>>>>>>
>>>>>
>>>>
>>>
>>>
>>
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode
2021-07-20 5:21 ` Jaehoon Chung
@ 2021-07-20 5:23 ` Lokesh Vutla
2021-07-20 5:26 ` Jaehoon Chung
0 siblings, 1 reply; 16+ messages in thread
From: Lokesh Vutla @ 2021-07-20 5:23 UTC (permalink / raw)
To: Jaehoon Chung, Aswath Govindraju, Peng Fan
Cc: u-boot, Praneeth Bajjuri, Kishon Vijay Abraham I,
Vignesh Raghavendra, Faiz Abbas, Tom Rini
On 20/07/21 10:51 am, Jaehoon Chung wrote:
> Hi Lokesh,
>
> On 7/20/21 1:10 PM, Lokesh Vutla wrote:
>> +Tom
>>
>> On 20/07/21 3:45 am, Jaehoon Chung wrote:
>>> Hi Aswath,
>>>
>>> On 7/19/21 3:48 PM, Aswath Govindraju wrote:
>>>> Hi Peng,
>>>>
>>>> On 09/06/21 8:56 pm, Aswath Govindraju wrote:
>>>>> Hi Peng,
>>>>>
>>>>> On 10/05/21 7:18 pm, Aswath Govindraju wrote:
>>>>>> Hi Peng,
>>>>>>
>>>>>> On 07/04/21 3:52 am, Jaehoon Chung wrote:
>>>>>>> On 4/5/21 11:44 PM, Aswath Govindraju wrote:
>>>>>>>> From: Faiz Abbas <faiz_abbas@ti.com>
>>>>>>>>
>>>>>>>> Enable HS400 speed mode by writing to HOST_CONTROL2 register.
>>>>>>>>
>>>>>>>> Signed-off-by: Faiz Abbas <faiz_abbas@ti.com>
>>>>>>>> Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>
>>>>>>>
>>>>>>> Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
>>>>>>>
>>>>>>
>>>>>> Can you please pick this patch if there are no comments.
>>>>>>
>>>>>
>>>>> May I know if this okay to be merged ?
>>>>>
>>>>
>>>> A gentle reminder on this patch. This patch has other dependencies that
>>>> and are pending merge[1][2].
>>>
>>> I don't have the permission to merge on u-boot-mmc git.
>>> I have the permission to merge on just u-boot-pmic..But I don't know whether it's the best way to apply this into my u-boot-pmic or not. :)
>>>
>>> I had been already reviewed about this patch..
>>> So if want to pick this, I think there is no problem about applied together with below patches.
>>>
>>> Acked-by: Jaehoon Chung <jh80.chung@samsung.com>
>>
>> Tom,
>> Is it okay if I pick this patch as per above suggestion?
>
> Isn't it enough my Acked-by tag?
Sure it is :). I just want to make sure Tom doesn't flag it in the PR.
Thanks and regards,
Lokesh
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode
2021-07-20 5:23 ` Lokesh Vutla
@ 2021-07-20 5:26 ` Jaehoon Chung
0 siblings, 0 replies; 16+ messages in thread
From: Jaehoon Chung @ 2021-07-20 5:26 UTC (permalink / raw)
To: Lokesh Vutla, Aswath Govindraju, Peng Fan
Cc: u-boot, Praneeth Bajjuri, Kishon Vijay Abraham I,
Vignesh Raghavendra, Faiz Abbas, Tom Rini
On 7/20/21 2:23 PM, Lokesh Vutla wrote:
>
>
> On 20/07/21 10:51 am, Jaehoon Chung wrote:
>> Hi Lokesh,
>>
>> On 7/20/21 1:10 PM, Lokesh Vutla wrote:
>>> +Tom
>>>
>>> On 20/07/21 3:45 am, Jaehoon Chung wrote:
>>>> Hi Aswath,
>>>>
>>>> On 7/19/21 3:48 PM, Aswath Govindraju wrote:
>>>>> Hi Peng,
>>>>>
>>>>> On 09/06/21 8:56 pm, Aswath Govindraju wrote:
>>>>>> Hi Peng,
>>>>>>
>>>>>> On 10/05/21 7:18 pm, Aswath Govindraju wrote:
>>>>>>> Hi Peng,
>>>>>>>
>>>>>>> On 07/04/21 3:52 am, Jaehoon Chung wrote:
>>>>>>>> On 4/5/21 11:44 PM, Aswath Govindraju wrote:
>>>>>>>>> From: Faiz Abbas <faiz_abbas@ti.com>
>>>>>>>>>
>>>>>>>>> Enable HS400 speed mode by writing to HOST_CONTROL2 register.
>>>>>>>>>
>>>>>>>>> Signed-off-by: Faiz Abbas <faiz_abbas@ti.com>
>>>>>>>>> Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>
>>>>>>>>
>>>>>>>> Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
>>>>>>>>
>>>>>>>
>>>>>>> Can you please pick this patch if there are no comments.
>>>>>>>
>>>>>>
>>>>>> May I know if this okay to be merged ?
>>>>>>
>>>>>
>>>>> A gentle reminder on this patch. This patch has other dependencies that
>>>>> and are pending merge[1][2].
>>>>
>>>> I don't have the permission to merge on u-boot-mmc git.
>>>> I have the permission to merge on just u-boot-pmic..But I don't know whether it's the best way to apply this into my u-boot-pmic or not. :)
>>>>
>>>> I had been already reviewed about this patch..
>>>> So if want to pick this, I think there is no problem about applied together with below patches.
>>>>
>>>> Acked-by: Jaehoon Chung <jh80.chung@samsung.com>
>>>
>>> Tom,
>>> Is it okay if I pick this patch as per above suggestion?
>>
>> Isn't it enough my Acked-by tag?
>
> Sure it is :). I just want to make sure Tom doesn't flag it in the PR.
Thanks for explanation. :)
Best Regards,
Jaehoon Chung
>
> Thanks and regards,
> Lokesh
>
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode
2021-07-20 4:10 ` Lokesh Vutla
2021-07-20 5:21 ` Jaehoon Chung
@ 2021-07-20 12:16 ` Tom Rini
1 sibling, 0 replies; 16+ messages in thread
From: Tom Rini @ 2021-07-20 12:16 UTC (permalink / raw)
To: Lokesh Vutla
Cc: Jaehoon Chung, Aswath Govindraju, Peng Fan, u-boot,
Praneeth Bajjuri, Kishon Vijay Abraham I, Vignesh Raghavendra,
Faiz Abbas
[-- Attachment #1: Type: text/plain, Size: 1614 bytes --]
On Tue, Jul 20, 2021 at 09:40:22AM +0530, Lokesh Vutla wrote:
> +Tom
>
> On 20/07/21 3:45 am, Jaehoon Chung wrote:
> > Hi Aswath,
> >
> > On 7/19/21 3:48 PM, Aswath Govindraju wrote:
> >> Hi Peng,
> >>
> >> On 09/06/21 8:56 pm, Aswath Govindraju wrote:
> >>> Hi Peng,
> >>>
> >>> On 10/05/21 7:18 pm, Aswath Govindraju wrote:
> >>>> Hi Peng,
> >>>>
> >>>> On 07/04/21 3:52 am, Jaehoon Chung wrote:
> >>>>> On 4/5/21 11:44 PM, Aswath Govindraju wrote:
> >>>>>> From: Faiz Abbas <faiz_abbas@ti.com>
> >>>>>>
> >>>>>> Enable HS400 speed mode by writing to HOST_CONTROL2 register.
> >>>>>>
> >>>>>> Signed-off-by: Faiz Abbas <faiz_abbas@ti.com>
> >>>>>> Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>
> >>>>>
> >>>>> Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
> >>>>>
> >>>>
> >>>> Can you please pick this patch if there are no comments.
> >>>>
> >>>
> >>> May I know if this okay to be merged ?
> >>>
> >>
> >> A gentle reminder on this patch. This patch has other dependencies that
> >> and are pending merge[1][2].
> >
> > I don't have the permission to merge on u-boot-mmc git.
> > I have the permission to merge on just u-boot-pmic..But I don't know whether it's the best way to apply this into my u-boot-pmic or not. :)
> >
> > I had been already reviewed about this patch..
> > So if want to pick this, I think there is no problem about applied together with below patches.
> >
> > Acked-by: Jaehoon Chung <jh80.chung@samsung.com>
>
> Tom,
> Is it okay if I pick this patch as per above suggestion?
Yes that's fine, thanks.
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode
2021-04-05 14:44 ` [PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode Aswath Govindraju
2021-04-05 22:18 ` Jaehoon Chung
2021-04-06 22:22 ` Jaehoon Chung
@ 2021-07-29 5:14 ` Lokesh Vutla
2 siblings, 0 replies; 16+ messages in thread
From: Lokesh Vutla @ 2021-07-29 5:14 UTC (permalink / raw)
To: Aswath Govindraju, u-boot; +Cc: Lokesh Vutla
On Mon, 5 Apr 2021 20:14:28 +0530, Aswath Govindraju wrote:
> Enable HS400 speed mode by writing to HOST_CONTROL2 register.
Applied to https://source.denx.de/u-boot/custodians/u-boot-ti.git for-rc, thanks!
[1/1] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/bda47bef7c
--
Thanks and Regards,
Lokesh
^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2021-07-29 5:15 UTC | newest]
Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <CGME20210405144459epcas1p3d6df22765afb126a2b2037c5364b0b2e@epcas1p3.samsung.com>
2021-04-05 14:44 ` [PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode Aswath Govindraju
2021-04-05 22:18 ` Jaehoon Chung
2021-04-06 5:20 ` Aswath Govindraju
2021-04-06 8:26 ` Aswath Govindraju
2021-04-06 9:14 ` Jaehoon Chung
2021-04-06 22:22 ` Jaehoon Chung
2021-05-10 13:48 ` Aswath Govindraju
2021-06-09 15:26 ` Aswath Govindraju
2021-07-19 6:48 ` Aswath Govindraju
2021-07-19 22:15 ` Jaehoon Chung
2021-07-20 4:10 ` Lokesh Vutla
2021-07-20 5:21 ` Jaehoon Chung
2021-07-20 5:23 ` Lokesh Vutla
2021-07-20 5:26 ` Jaehoon Chung
2021-07-20 12:16 ` Tom Rini
2021-07-29 5:14 ` Lokesh Vutla
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.