Re: [U-Boot] MMC_DAVINCI not compliant with CONFIG_BLK

2018-08-08 Thread Adam Ford 
On Mon, Aug 6, 2018 at 5:22 PM Peter Howard  wrote:
>
> Sorry a clarification - my main focus is the LCDK board - it's that one which 
> has the 4K overrun.
>
I wanted the rest of the people on this thread to know the status.

Both Peter and I did separate drivers.  His was more complete than
mine, so I merged a few of my items into his driver.  There are 3
patches waiting for him to review.  If it looks good, I am hoping
either of us can post it to the mailing list. This allows both DM_MMC
and CONFIG_BLK in U-Boot.  SPL still uses the older manual format
unless the board has DM enabled in SPL.  That should make the Davinci
MMC boards compliant with CONFIG_BLK and I'm hoping the MMC maintainer
and Tom will approve it for 2019.09 release once Peter has a chance to
review and test on his hardware. I know the 2019.09 release is coming
quickly.

adam

> -Original Message-
> From: Peter Howard
> Sent: Tuesday, 7 August 2018 8:22 AM
> To: 'Adam Ford' ; U-Boot Mailing List 
> 
> Cc: David Lechner ; Stefano Babic ; 
> Heiko Schocher ; Christoph Rüdisser 
> ; Manfred Rudigier 
> 
> Subject: RE: MMC_DAVINCI not compliant with CONFIG_BLK
>
> Hi Adam,
>
> My normal email is broken at the moment sorry for inflicting Outlook on you ☹
>
>
> I've been working on this for a few weeks and at this stage I have a u-boot 
> that builds.  The only problem is that introducing the DM into the SPL build 
> (for NAND) currently overruns the SRAM area by 4K or so.
>
> If you want to have a look I'm happy to send you my changes so-far.
>
> PJH
>
> -Original Message-
> From: Adam Ford 
> Sent: Monday, 6 August 2018 10:11 PM
> To: U-Boot Mailing List 
> Cc: Peter Howard ; David Lechner ; 
> Stefano Babic ; Heiko Schocher ; Christoph 
> Rüdisser ; Manfred Rudigier 
> 
> Subject: MMC_DAVINCI not compliant with CONFIG_BLK
>
> From what I gathered over the weekend, the MMC_DAVINCI hasn't migrated to 
> DM_MMC nor does CONFIG_BLK successfully build on ARCH_DAVINCI boards that 
> enable MMC_DAVINCI.
>
> I started work to attempt to make it DM_MMC and OF_CONTROL compliant using 
> the da850evm.
>
> Before I go too far, I just want to make sure nobody else is doing this work. 
>  I don't want to duplicate an effort or waste time if it's been done or is in 
> the works.
>
> If someone is interesting in testing the patch when I am done, it would be 
> very helpful, because I only have the da850evm.  If someone has a patch and 
> wants me to test it, I would also be willing to try it.
>
> adam
>
> __
> This email has been scanned by the Symantec Email Security.cloud service.
> For more information please visit http://www.symanteccloud.com 
> __
>
> __
> This email has been scanned by the Symantec Email Security.cloud service.
> For more information please visit http://www.symanteccloud.com
> __
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] MMC_DAVINCI not compliant with CONFIG_BLK

2018-08-06 Thread Peter Howard 
Hi Adam,

My normal email is broken at the moment sorry for inflicting Outlook on you ☹


I've been working on this for a few weeks and at this stage I have a u-boot 
that builds.  The only problem is that introducing the DM into the SPL build 
(for NAND) currently overruns the SRAM area by 4K or so.  

If you want to have a look I'm happy to send you my changes so-far.

PJH

-Original Message-
From: Adam Ford  
Sent: Monday, 6 August 2018 10:11 PM
To: U-Boot Mailing List 
Cc: Peter Howard ; David Lechner ; 
Stefano Babic ; Heiko Schocher ; Christoph 
Rüdisser ; Manfred Rudigier 

Subject: MMC_DAVINCI not compliant with CONFIG_BLK

From what I gathered over the weekend, the MMC_DAVINCI hasn't migrated to 
DM_MMC nor does CONFIG_BLK successfully build on ARCH_DAVINCI boards that 
enable MMC_DAVINCI.

I started work to attempt to make it DM_MMC and OF_CONTROL compliant using the 
da850evm.

Before I go too far, I just want to make sure nobody else is doing this work.  
I don't want to duplicate an effort or waste time if it's been done or is in 
the works.

If someone is interesting in testing the patch when I am done, it would be very 
helpful, because I only have the da850evm.  If someone has a patch and wants me 
to test it, I would also be willing to try it.

adam

__
This email has been scanned by the Symantec Email Security.cloud service.
For more information please visit http://www.symanteccloud.com 
__

__
This email has been scanned by the Symantec Email Security.cloud service.
For more information please visit http://www.symanteccloud.com
__
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] MMC_DAVINCI not compliant with CONFIG_BLK

2018-08-06 Thread Peter Howard 
Sorry a clarification - my main focus is the LCDK board - it's that one which 
has the 4K overrun.

-Original Message-
From: Peter Howard 
Sent: Tuesday, 7 August 2018 8:22 AM
To: 'Adam Ford' ; U-Boot Mailing List 
Cc: David Lechner ; Stefano Babic ; Heiko 
Schocher ; Christoph Rüdisser 
; Manfred Rudigier 

Subject: RE: MMC_DAVINCI not compliant with CONFIG_BLK

Hi Adam,

My normal email is broken at the moment sorry for inflicting Outlook on you ☹


I've been working on this for a few weeks and at this stage I have a u-boot 
that builds.  The only problem is that introducing the DM into the SPL build 
(for NAND) currently overruns the SRAM area by 4K or so.  

If you want to have a look I'm happy to send you my changes so-far.

PJH

-Original Message-
From: Adam Ford  
Sent: Monday, 6 August 2018 10:11 PM
To: U-Boot Mailing List 
Cc: Peter Howard ; David Lechner ; 
Stefano Babic ; Heiko Schocher ; Christoph 
Rüdisser ; Manfred Rudigier 

Subject: MMC_DAVINCI not compliant with CONFIG_BLK

From what I gathered over the weekend, the MMC_DAVINCI hasn't migrated to 
DM_MMC nor does CONFIG_BLK successfully build on ARCH_DAVINCI boards that 
enable MMC_DAVINCI.

I started work to attempt to make it DM_MMC and OF_CONTROL compliant using the 
da850evm.

Before I go too far, I just want to make sure nobody else is doing this work.  
I don't want to duplicate an effort or waste time if it's been done or is in 
the works.

If someone is interesting in testing the patch when I am done, it would be very 
helpful, because I only have the da850evm.  If someone has a patch and wants me 
to test it, I would also be willing to try it.

adam

__
This email has been scanned by the Symantec Email Security.cloud service.
For more information please visit http://www.symanteccloud.com 
__

__
This email has been scanned by the Symantec Email Security.cloud service.
For more information please visit http://www.symanteccloud.com
__
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] MMC_DAVINCI not compliant with CONFIG_BLK

2018-08-06 Thread Peter Howard 


>From: Adam Ford  
>Sent: Tuesday, 7 August 2018 9:12 AM
>To: Peter Howard 
>Cc: U-Boot Mailing List ; David Lechner 
>; Stefano Babic ; Heiko Schocher 
>; Christoph Rüdisser ; 
>Manfred Rudigier 
>Subject: Re: MMC_DAVINCI not compliant with CONFIG_BLK
>
>
>> On Mon, Aug 6, 2018, 5:22 PM Peter Howard  wrote:
>> Sorry a clarification - my main focus is the LCDK board - it's that one 
>> which has the 4K overrun.
>> 
>> -Original Message-
>> From: Peter Howard 
>> Sent: Tuesday, 7 August 2018 8:22 AM
>> To: 'Adam Ford' ; U-Boot Mailing List 
>> 
>> Cc: David Lechner ; Stefano Babic 
>> ; Heiko Schocher ; Christoph 
>> Rüdisser ; Manfred Rudigier  
>> 
>> Subject: RE: MMC_DAVINCI not compliant with CONFIG_BLK
>>
>> Hi Adam,
>>
>> My normal email is broken at the moment sorry for inflicting Outlook on you ☹
>>
>>
>> I've been working on this for a few weeks and at this stage I have a u-boot 
>> that builds.  The only problem is that introducing the DM into the SPL build 
>> (for NAND) currently overruns the SRAM area by 4K or so.  
>>
>
> Is DM required in SPL?
>
> There are a variety of drivers with #ifdef around various parts of the code.

That's a question I'm not sure about.  However . . . MMC is enabled in the SPL. 
 So, even if DM isn't mandated, you have to either use it or maintain both the 
old and new form of the driver.  Or disable MMC.  The only other config I could 
find which definitely combines MMC in the SPL with the DM is 
configs/stm32mp15_basic_defconfig - and it uses the DM form (as far as I can 
see  - I could be wrong).

>
>
>> If you want to have a look I'm happy to send you my changes so-far.
>>
>> PJH
>>
>> -Original Message-
>> From: Adam Ford  
>> Sent: Monday, 6 August 2018 10:11 PM
>> To: U-Boot Mailing List 
>> Cc: Peter Howard ; David Lechner 
>> ; Stefano Babic ; Heiko 
>> Schocher ; Christoph Rüdisser 
>> ;  Manfred Rudigier 
>> 
>> Subject: MMC_DAVINCI not compliant with CONFIG_BLK
>>
>> From what I gathered over the weekend, the MMC_DAVINCI hasn't migrated to 
>> DM_MMC nor does CONFIG_BLK successfully build on ARCH_DAVINCI boards that 
>> enable MMC_DAVINCI.
>>
>> I started work to attempt to make it DM_MMC and OF_CONTROL compliant using 
>> the da850evm.
>>
>> Before I go too far, I just want to make sure nobody else is doing this 
>> work.  I don't want to duplicate an effort or waste time if it's been done 
>> or is in the works.
>>
>> If someone is interesting in testing the patch when I am done, it would be 
>> very helpful, because I only have the da850evm.  If someone has a patch and 
>> wants me to test it, I would also be willing to try it.
>>
>> adam




__
This email has been scanned by the Symantec Email Security.cloud service.
For more information please visit http://www.symanteccloud.com
__
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] MMC_DAVINCI not compliant with CONFIG_BLK

2018-08-06 Thread Adam Ford 
On Mon, Aug 6, 2018, 6:30 PM Peter Howard  wrote:

>
>
> >From: Adam Ford 
> >Sent: Tuesday, 7 August 2018 9:12 AM
> >To: Peter Howard 
> >Cc: U-Boot Mailing List ; David Lechner <
> da...@lechnology.com>; Stefano Babic ; Heiko Schocher <
> h...@denx.de>; Christoph Rüdisser ;
> Manfred Rudigier 
> >Subject: Re: MMC_DAVINCI not compliant with CONFIG_BLK
> >
> >
> >> On Mon, Aug 6, 2018, 5:22 PM Peter Howard 
> wrote:
> >> Sorry a clarification - my main focus is the LCDK board - it's that one
> which has the 4K overrun.
> >>
> >> -Original Message-
> >> From: Peter Howard
> >> Sent: Tuesday, 7 August 2018 8:22 AM
> >> To: 'Adam Ford' ; U-Boot Mailing List
> 
> >> Cc: David Lechner ; Stefano Babic  sba...@denx.de>; Heiko Schocher ; Christoph Rüdisser
> ; Manfred Rudigier   manfred.rudig...@omicronenergy.com>
> >> Subject: RE: MMC_DAVINCI not compliant with CONFIG_BLK
> >>
> >> Hi Adam,
> >>
> >> My normal email is broken at the moment sorry for inflicting Outlook on
> you ☹
> >>
> >>
> >> I've been working on this for a few weeks and at this stage I have a
> u-boot that builds.  The only problem is that introducing the DM into the
> SPL build (for NAND) currently overruns the SRAM area by 4K or so.
> >>
> >
> > Is DM required in SPL?
> >
> > There are a variety of drivers with #ifdef around various parts of the
> code.
>
> That's a question I'm not sure about.  However . . . MMC is enabled in the
> SPL.  So, even if DM isn't mandated, you have to either use it or maintain
> both the old and new form of the driver.  Or disable MMC.  The only other
> config I could find which definitely combines MMC in the SPL with the DM is
> configs/stm32mp15_basic_defconfig - and it uses the DM form (as far as I
> can see  - I could be wrong).
>
>
> I was looking at the i.mx family as well as the omap have mmc driver and
> it looks like there's evidence that they're pointing the DM version of
> stuff to the older version when DM is enabled.
>

Adam

> >
> >
> >> If you want to have a look I'm happy to send you my changes so-far.
> >>
> >> PJH
> >>
> >> -Original Message-
> >> From: Adam Ford 
> >> Sent: Monday, 6 August 2018 10:11 PM
> >> To: U-Boot Mailing List 
> >> Cc: Peter Howard ; David Lechner  da...@lechnology.com>; Stefano Babic ; Heiko
> Schocher ; Christoph Rüdisser  christoph.ruedis...@omicronenergy.com>;  Manfred Rudigier  manfred.rudig...@omicronenergy.com>
> >> Subject: MMC_DAVINCI not compliant with CONFIG_BLK
> >>
> >> From what I gathered over the weekend, the MMC_DAVINCI hasn't migrated
> to DM_MMC nor does CONFIG_BLK successfully build on ARCH_DAVINCI boards
> that enable MMC_DAVINCI.
> >>
> >> I started work to attempt to make it DM_MMC and OF_CONTROL compliant
> using the da850evm.
> >>
> >> Before I go too far, I just want to make sure nobody else is doing this
> work.  I don't want to duplicate an effort or waste time if it's been done
> or is in the works.
> >>
> >> If someone is interesting in testing the patch when I am done, it would
> be very helpful, because I only have the da850evm.  If someone has a patch
> and wants me to test it, I would also be willing to try it.
> >>
> >> adam
>
>
>
>
> __
> This email has been scanned by the Symantec Email Security.cloud service.
> For more information please visit http://www.symanteccloud.com
> __
>
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] MMC_DAVINCI not compliant with CONFIG_BLK

2018-08-06 Thread Adam Ford 
On Mon, Aug 6, 2018, 5:22 PM Peter Howard  wrote:

> Sorry a clarification - my main focus is the LCDK board - it's that one
> which has the 4K overrun.
>
> -Original Message-
> From: Peter Howard
> Sent: Tuesday, 7 August 2018 8:22 AM
> To: 'Adam Ford' ; U-Boot Mailing List <
> u-boot@lists.denx.de>
> Cc: David Lechner ; Stefano Babic ;
> Heiko Schocher ; Christoph Rüdisser <
> christoph.ruedis...@omicronenergy.com>; Manfred Rudigier <
> manfred.rudig...@omicronenergy.com>
> Subject: RE: MMC_DAVINCI not compliant with CONFIG_BLK
>
> Hi Adam,
>
> My normal email is broken at the moment sorry for inflicting Outlook on
> you ☹
>
>
> I've been working on this for a few weeks and at this stage I have a
> u-boot that builds.  The only problem is that introducing the DM into the
> SPL build (for NAND) currently overruns the SRAM area by 4K or so.
>


Is DM required in SPL?

There are a variety of drivers with #ifdef around various parts of the code.



> If you want to have a look I'm happy to send you my changes so-far.
>
> PJH
>
> -Original Message-
> From: Adam Ford 
> Sent: Monday, 6 August 2018 10:11 PM
> To: U-Boot Mailing List 
> Cc: Peter Howard ; David Lechner ;
> Stefano Babic ; Heiko Schocher ; Christoph
> Rüdisser ; Manfred Rudigier <
> manfred.rudig...@omicronenergy.com>
> Subject: MMC_DAVINCI not compliant with CONFIG_BLK
>
> From what I gathered over the weekend, the MMC_DAVINCI hasn't migrated to
> DM_MMC nor does CONFIG_BLK successfully build on ARCH_DAVINCI boards that
> enable MMC_DAVINCI.
>
> I started work to attempt to make it DM_MMC and OF_CONTROL compliant using
> the da850evm.
>
> Before I go too far, I just want to make sure nobody else is doing this
> work.  I don't want to duplicate an effort or waste time if it's been done
> or is in the works.
>
> If someone is interesting in testing the patch when I am done, it would be
> very helpful, because I only have the da850evm.  If someone has a patch and
> wants me to test it, I would also be willing to try it.
>
> adam
>
> __
> This email has been scanned by the Symantec Email Security.cloud service.
> For more information please visit http://www.symanteccloud.com
> __
>
> __
> This email has been scanned by the Symantec Email Security.cloud service.
> For more information please visit http://www.symanteccloud.com
> __
>
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] MMC_DAVINCI not compliant with CONFIG_BLK

2018-08-06 Thread Adam Ford 
On Mon, Aug 6, 2018, 5:21 PM Peter Howard  wrote:

> Hi Adam,
>
> My normal email is broken at the moment sorry for inflicting Outlook on
> you ☹
>
>
> I've been working on this for a few weeks and at this stage I have a
> u-boot that builds.  The only problem is that introducing the DM into the
> SPL build (for NAND) currently overruns the SRAM area by 4K or so.
>
> If you want to have a look I'm happy to send you my changes so-far.
>


I would love to give them a try.

Adam

>
> PJH
>
> -Original Message-
> From: Adam Ford 
> Sent: Monday, 6 August 2018 10:11 PM
> To: U-Boot Mailing List 
> Cc: Peter Howard ; David Lechner ;
> Stefano Babic ; Heiko Schocher ; Christoph
> Rüdisser ; Manfred Rudigier <
> manfred.rudig...@omicronenergy.com>
> Subject: MMC_DAVINCI not compliant with CONFIG_BLK
>
> From what I gathered over the weekend, the MMC_DAVINCI hasn't migrated to
> DM_MMC nor does CONFIG_BLK successfully build on ARCH_DAVINCI boards that
> enable MMC_DAVINCI.
>
> I started work to attempt to make it DM_MMC and OF_CONTROL compliant using
> the da850evm.
>
> Before I go too far, I just want to make sure nobody else is doing this
> work.  I don't want to duplicate an effort or waste time if it's been done
> or is in the works.
>
> If someone is interesting in testing the patch when I am done, it would be
> very helpful, because I only have the da850evm.  If someone has a patch and
> wants me to test it, I would also be willing to try it.
>
> adam
>
> __
> This email has been scanned by the Symantec Email Security.cloud service.
> For more information please visit http://www.symanteccloud.com
> __
>
> __
> This email has been scanned by the Symantec Email Security.cloud service.
> For more information please visit http://www.symanteccloud.com
> __
>
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot