Re: [U-Boot] [v3] armv8: ls1012a: enable/disable eSDHC1 through hwconfig for new RDB

2017-12-07 Thread Y.b. Lu
Sent out v4 patch-set. Please help to review. Thanks a lot :) > -Original Message- > From: York Sun > Sent: 2017年12月8日 1:34 > To: Y.b. Lu ; u-boot@lists.denx.de > Subject: Re: [v3] armv8: ls1012a: enable/disable eSDHC1 through hwconfig for > new RDB > > On 12/07/2017

Re: [U-Boot] [v3] armv8: ls1012a: enable/disable eSDHC1 through hwconfig for new RDB

2017-12-07 Thread York Sun
On 12/07/2017 12:10 AM, Y.b. Lu wrote: > >> >>> printf("Error reading i2c boot information!\n"); >>> - return 0; /* Don't want to hang() on this error */ >>> + return 0; >> >> What does I2C failure mean here? Returning 0 will cause the code keep going >> in >>

Re: [U-Boot] [v3] armv8: ls1012a: enable/disable eSDHC1 through hwconfig for new RDB

2017-12-07 Thread Y.b. Lu
Hi York, > -Original Message- > From: York Sun > Sent: 2017年12月7日 3:08 > To: Y.b. Lu ; u-boot@lists.denx.de > Subject: Re: [v3] armv8: ls1012a: enable/disable eSDHC1 through hwconfig for > new RDB > > On 12/06/2017 02:19 AM, Yangbo Lu wrote: > > For LS1012ARDB RevD and

Re: [U-Boot] [v3] armv8: ls1012a: enable/disable eSDHC1 through hwconfig for new RDB

2017-12-06 Thread York Sun
On 12/06/2017 02:19 AM, Yangbo Lu wrote: > For LS1012ARDB RevD and later versions, the I2C reading for DIP > switch setting had been no longer reliable since the board was > reworked. This patch is to add hwconfig support to enable/disable I think this message is not accurate. How about saying

[U-Boot] [v3] armv8: ls1012a: enable/disable eSDHC1 through hwconfig for new RDB

2017-12-06 Thread Yangbo Lu
For LS1012ARDB RevD and later versions, the I2C reading for DIP switch setting had been no longer reliable since the board was reworked. This patch is to add hwconfig support to enable/disable eSDHC1 manually for these boards. Also let kernel decide status of esdhc0. Signed-off-by: Yangbo Lu