[PATCH] mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode

Jaehoon Chung jh80.chung at samsung.com
Tue Jul 20 07:21:11 CEST 2021


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 at ti.com>
>>>>>>>
>>>>>>> Enable HS400 speed mode by writing to HOST_CONTROL2 register.
>>>>>>>
>>>>>>> Signed-off-by: Faiz Abbas <faiz_abbas at ti.com>
>>>>>>> Signed-off-by: Aswath Govindraju <a-govindraju at ti.com>
>>>>>>
>>>>>> Reviewed-by: Jaehoon Chung <jh80.chung at 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 at 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;
>>>>>>>  	}
>>>>>>>
>>>>>>
>>>>>
>>>>
>>>
>>>
>>
> 



More information about the U-Boot mailing list