[U-Boot] [ANN] U-Boot v2017.11-rc4 released

[U-Boot] [ANN] U-Boot v2017.11-rc4 released

[Tom Rini]

Hey all,

It's release day and v2017.11-rc4 is out.  It's a week until release
day.  Please let me know if you know of any regressions, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20171106/983761ce/attachment.sig>

[U-Boot] [ANN] U-Boot v2017.11-rc4 released

[Jorge Ramirez]

On 11/07/2017 12:28 AM, Tom Rini wrote:
> Hey all,
>
> It's release day and v2017.11-rc4 is out.  It's a week until release
> day.  Please let me know if you know of any regressions, thanks!

Hi Tom,

emmc accesses are broken for dragonboard410c
I posted the fix [1] last week. will you merge it?

thanks
jorge

[1]PATCH] mmc: sdhci: don't clear SDHCI_INT_STATUS register during 
CMD_INHIBIT

Fixes emmc initialization regression on the db410c platform.

Clearing this register while SDHCI_PRESENT_STATE reports
SDHCI_CMD_INHIBIT leads to undefined behaviour on the db410c.

When commit 7dde50 was merged (mmc: sdhci: Wait for SDHCI_INT_DATA_END
when transferring), SDHCI transfers transitioned to wait for bit
SDHCI_INT_DATA_END before flagging transfers done.

Without this patch, the db410 platform fails to initialize its eMMC
due to all of its transfers timing out (SDHCI_INT_DATA_END is never
raised after all the blocks have been transferred).

Signed-off-by: Jorge Ramirez-Ortiz<jorge.ramirez-ortiz@linaro.org>
---
  drivers/mmc/sdhci.c | 3 ++-
  1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c
index 11d1f0c..f0c5aad 100644
--- a/drivers/mmc/sdhci.c
+++ b/drivers/mmc/sdhci.c
@@ -157,7 +157,6 @@ static int sdhci_send_command(struct mmc *mmc, struct mmc_cmd *cmd,
  	/* Timeout unit - ms */
  	static unsigned int cmd_timeout = SDHCI_CMD_DEFAULT_TIMEOUT;
  
-	sdhci_writel(host, SDHCI_INT_ALL_MASK, SDHCI_INT_STATUS);
  	mask = SDHCI_CMD_INHIBIT | SDHCI_DATA_INHIBIT;
  
  	/* We shouldn't wait for data inihibit for stop commands, even
@@ -181,6 +180,8 @@ static int sdhci_send_command(struct mmc *mmc, struct mmc_cmd *cmd,
  		udelay(1000);
  	}
  
+	sdhci_writel(host, SDHCI_INT_ALL_MASK, SDHCI_INT_STATUS);
+
  	mask = SDHCI_INT_RESPONSE;
  	if (!(cmd->resp_type & MMC_RSP_PRESENT))
  		flags = SDHCI_CMD_RESP_NONE;



>
>
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot

[U-Boot] [ANN] U-Boot v2017.11-rc4 released

[Tom Rini]

On Tue, Nov 07, 2017 at 08:13:02AM +0100, Jorge Ramirez wrote:
> On 11/07/2017 12:28 AM, Tom Rini wrote:
> >Hey all,
> >
> >It's release day and v2017.11-rc4 is out.  It's a week until release
> >day.  Please let me know if you know of any regressions, thanks!
> 
> Hi Tom,
> 
> emmc accesses are broken for dragonboard410c
> I posted the fix [1] last week. will you merge it?

[1] is: https://patchwork.ozlabs.org/patch/833376/

Jaehoon, are you OK with this patch?  Thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20171107/0e450324/attachment.sig>

[U-Boot] sdhci - review?

[Jorge Ramirez]

On 11/07/2017 11:48 PM, Tom Rini wrote:
> On Tue, Nov 07, 2017 at 08:13:02AM +0100, Jorge Ramirez wrote:
>> On 11/07/2017 12:28 AM, Tom Rini wrote:
>>> Hey all,
>>>
>>> It's release day and v2017.11-rc4 is out.  It's a week until release
>>> day.  Please let me know if you know of any regressions, thanks!
>> Hi Tom,
>>
>> emmc accesses are broken for dragonboard410c
>> I posted the fix [1] last week. will you merge it?

just changed the email subject

> [1] is: https://patchwork.ozlabs.org/patch/833376/
>
> Jaehoon, are you OK with this patch?  Thanks!
>

[U-Boot] [ANN] U-Boot v2017.11-rc4 released

[Jaehoon Chung]

Hi Tom,

On 2017년 11월 08일 07:48, Tom Rini wrote:
> On Tue, Nov 07, 2017 at 08:13:02AM +0100, Jorge Ramirez wrote:
>> On 11/07/2017 12:28 AM, Tom Rini wrote:
>>> Hey all,
>>>
>>> It's release day and v2017.11-rc4 is out.  It's a week until release
>>> day.  Please let me know if you know of any regressions, thanks!
>>
>> Hi Tom,
>>
>> emmc accesses are broken for dragonboard410c
>> I posted the fix [1] last week. will you merge it?
> 
> [1] is: https://patchwork.ozlabs.org/patch/833376/
> 
> Jaehoon, are you OK with this patch?  Thanks!

If you didn't apply this, i will pick after checking. 
And will send PR with other patches.

Thanks!

Best Regards,
Jaehoon Chung

> 
> 
> 
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot
> 

[U-Boot] sdhci - review?

[Jorge Ramirez]

On 11/10/2017 01:29 PM, Jorge Ramirez wrote:
> On 11/07/2017 11:48 PM, Tom Rini wrote:
>> On Tue, Nov 07, 2017 at 08:13:02AM +0100, Jorge Ramirez wrote:
>>> On 11/07/2017 12:28 AM, Tom Rini wrote:
>>>> Hey all,
>>>>
>>>> It's release day and v2017.11-rc4 is out.  It's a week until release
>>>> day.  Please let me know if you know of any regressions, thanks!
>>> Hi Tom,
>>>
>>> emmc accesses are broken for dragonboard410c
>>> I posted the fix [1] last week. will you merge it?
>
> just changed the email subject
>
>> [1] is: https://patchwork.ozlabs.org/patch/833376/
>>
>> Jaehoon, are you OK with this patch?  Thanks!
>>
>

a ping on this matter - as the db410c maintainer a quick reminder that 
the emmc support remains broken until this (above [1]) (or an 
alternative fix) is merged.

[U-Boot] sdhci - review?

[Jaehoon Chung]

On 11/20/2017 09:52 PM, Jorge Ramirez wrote:
> On 11/10/2017 01:29 PM, Jorge Ramirez wrote:
>> On 11/07/2017 11:48 PM, Tom Rini wrote:
>>> On Tue, Nov 07, 2017 at 08:13:02AM +0100, Jorge Ramirez wrote:
>>>> On 11/07/2017 12:28 AM, Tom Rini wrote:
>>>>> Hey all,
>>>>>
>>>>> It's release day and v2017.11-rc4 is out.  It's a week until release
>>>>> day.  Please let me know if you know of any regressions, thanks!
>>>> Hi Tom,
>>>>
>>>> emmc accesses are broken for dragonboard410c
>>>> I posted the fix [1] last week. will you merge it?
>>
>> just changed the email subject
>>
>>> [1] is: https://patchwork.ozlabs.org/patch/833376/
>>>
>>> Jaehoon, are you OK with this patch?  Thanks!
>>>
>>
> 
> a ping on this matter - as the db410c maintainer a quick reminder that the emmc support remains broken until this (above [1]) (or an alternative fix) is merged.

I sent the email to Tom about 1 weeks ago..if it doesn't apply yet, i will pick it..
Maybe i had replied with the my gmail account, something missed.

Sorry.

> 
> 
> 

[U-Boot] sdhci - review?

[Tom Rini]

On Wed, Nov 22, 2017 at 05:25:02PM +0900, Jaehoon Chung wrote:
> On 11/20/2017 09:52 PM, Jorge Ramirez wrote:
> > On 11/10/2017 01:29 PM, Jorge Ramirez wrote:
> >> On 11/07/2017 11:48 PM, Tom Rini wrote:
> >>> On Tue, Nov 07, 2017 at 08:13:02AM +0100, Jorge Ramirez wrote:
> >>>> On 11/07/2017 12:28 AM, Tom Rini wrote:
> >>>>> Hey all,
> >>>>>
> >>>>> It's release day and v2017.11-rc4 is out.  It's a week until release
> >>>>> day.  Please let me know if you know of any regressions, thanks!
> >>>> Hi Tom,
> >>>>
> >>>> emmc accesses are broken for dragonboard410c
> >>>> I posted the fix [1] last week. will you merge it?
> >>
> >> just changed the email subject
> >>
> >>> [1] is: https://patchwork.ozlabs.org/patch/833376/
> >>>
> >>> Jaehoon, are you OK with this patch?  Thanks!
> >>>
> >>
> > 
> > a ping on this matter - as the db410c maintainer a quick reminder that the emmc support remains broken until this (above [1]) (or an alternative fix) is merged.
> 
> I sent the email to Tom about 1 weeks ago..if it doesn't apply yet, i will pick it..
> Maybe i had replied with the my gmail account, something missed.

Yeah, I missed that, sorry.  Please put together a pull request of all
the various outstanding MMC changes ASAP, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20171122/16523ec4/attachment.sig>

[U-Boot] sdhci - review?

[Jaehoon Chung]

On 11/22/2017 09:27 PM, Tom Rini wrote:
> On Wed, Nov 22, 2017 at 05:25:02PM +0900, Jaehoon Chung wrote:
>> On 11/20/2017 09:52 PM, Jorge Ramirez wrote:
>>> On 11/10/2017 01:29 PM, Jorge Ramirez wrote:
>>>> On 11/07/2017 11:48 PM, Tom Rini wrote:
>>>>> On Tue, Nov 07, 2017 at 08:13:02AM +0100, Jorge Ramirez wrote:
>>>>>> On 11/07/2017 12:28 AM, Tom Rini wrote:
>>>>>>> Hey all,
>>>>>>>
>>>>>>> It's release day and v2017.11-rc4 is out.  It's a week until release
>>>>>>> day.  Please let me know if you know of any regressions, thanks!
>>>>>> Hi Tom,
>>>>>>
>>>>>> emmc accesses are broken for dragonboard410c
>>>>>> I posted the fix [1] last week. will you merge it?
>>>>
>>>> just changed the email subject
>>>>
>>>>> [1] is: https://patchwork.ozlabs.org/patch/833376/
>>>>>
>>>>> Jaehoon, are you OK with this patch?  Thanks!
>>>>>
>>>>
>>>
>>> a ping on this matter - as the db410c maintainer a quick reminder that the emmc support remains broken until this (above [1]) (or an alternative fix) is merged.
>>
>> I sent the email to Tom about 1 weeks ago..if it doesn't apply yet, i will pick it..
>> Maybe i had replied with the my gmail account, something missed.
> 
> Yeah, I missed that, sorry.  Please put together a pull request of all
> the various outstanding MMC changes ASAP, thanks!

Thanks for checking! I will send PR with all.!

Best Regards,
Jaehoon Chung

> 

[U-Boot] sdhci - review?

[Jorge Ramirez]

On 11/20/2017 01:52 PM, Jorge Ramirez wrote:
> On 11/10/2017 01:29 PM, Jorge Ramirez wrote:
>> On 11/07/2017 11:48 PM, Tom Rini wrote:
>>> On Tue, Nov 07, 2017 at 08:13:02AM +0100, Jorge Ramirez wrote:
>>>> On 11/07/2017 12:28 AM, Tom Rini wrote:
>>>>> Hey all,
>>>>>
>>>>> It's release day and v2017.11-rc4 is out.  It's a week until release
>>>>> day.  Please let me know if you know of any regressions, thanks!
>>>> Hi Tom,
>>>>
>>>> emmc accesses are broken for dragonboard410c
>>>> I posted the fix [1] last week. will you merge it?
>>
>> just changed the email subject
>>
>>> [1] is: https://patchwork.ozlabs.org/patch/833376/
>>>
>>> Jaehoon, are you OK with this patch?  Thanks!
>>>
>>
>
> a ping on this matter - as the db410c maintainer a quick reminder that 
> the emmc support remains broken until this (above [1]) (or an 
> alternative fix) is merged.

ping again :)

I can see the patch in 
http://git.denx.de/?p=u-boot/u-boot-mmc.git;a=commit;h=c7e47ab0aa98da76824b6fa166fb02cfaa7284ad 
so I am hoping it will be merged in this cycle?
This fix was submitted at the end of October to allow db410c users 
access to the emmc (fix a regression)

[U-Boot] sdhci - review?

[Jorge Ramirez]

On 01/09/2018 10:48 AM, Jorge Ramirez wrote:
> On 11/20/2017 01:52 PM, Jorge Ramirez wrote:
>> On 11/10/2017 01:29 PM, Jorge Ramirez wrote:
>>> On 11/07/2017 11:48 PM, Tom Rini wrote:
>>>> On Tue, Nov 07, 2017 at 08:13:02AM +0100, Jorge Ramirez wrote:
>>>>> On 11/07/2017 12:28 AM, Tom Rini wrote:
>>>>>> Hey all,
>>>>>>
>>>>>> It's release day and v2017.11-rc4 is out.  It's a week until release
>>>>>> day.  Please let me know if you know of any regressions, thanks!
>>>>> Hi Tom,
>>>>>
>>>>> emmc accesses are broken for dragonboard410c
>>>>> I posted the fix [1] last week. will you merge it?
>>>
>>> just changed the email subject
>>>
>>>> [1] is: https://patchwork.ozlabs.org/patch/833376/
>>>>
>>>> Jaehoon, are you OK with this patch?  Thanks!
>>>>
>>>
>>
>> a ping on this matter - as the db410c maintainer a quick reminder 
>> that the emmc support remains broken until this (above [1]) (or an 
>> alternative fix) is merged.
>
> ping again :)
>
> I can see the patch in 
> http://git.denx.de/?p=u-boot/u-boot-mmc.git;a=commit;h=c7e47ab0aa98da76824b6fa166fb02cfaa7284ad 
> so I am hoping it will be merged in this cycle?
> This fix was submitted at the end of October to allow db410c users 
> access to the emmc (fix a regression)

please ignore. I can see it in u-boot/next.

TIA

>
>
>
>
>
>
>
>
>