Message ID | 20210405144428.12159-1-a-govindraju@ti.com |
---|---|
State | Accepted |
Commit | bda47bef7c5aea2521fccdffb0020d9354194be9 |
Delegated to: | Lokesh Vutla |
Headers | show |
Series | mmc: sdhci: Write to HOST_CONTROL2 register for HS400 speed mode | expand |
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; > } >
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; >> } >> >
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; >>> } >>> >> >
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; >>>> } >>>> >>> >> > >
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; > } >
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; >> } >> >
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; >>> } >>> >> >
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; >>>> } >>>> >>> >> >
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; >>>>> } >>>>> >>>> >>> >> > >
+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; >>>>>> } >>>>>> >>>>> >>>> >>> >> >> >
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; >>>>>>> } >>>>>>> >>>>>> >>>>> >>>> >>> >>> >> >
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
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 > >
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.
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
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; }