Re: [U-Boot] [PATCH v4 0/3] Move Stratix 10 SDRAM driver to DM

2019-05-06 Thread Marek Vasut 
On 5/6/19 3:50 AM, Ley Foon Tan wrote:
> On Fri, May 3, 2019 at 11:13 PM Marek Vasut  wrote:
>>
>> On 5/3/19 8:27 AM, Ley Foon Tan wrote:
>>> Compile ALTERA_SDRAM driver in SPL only and move Stratix 10 SDRAM driver to 
>>> DM.
>>>
>>> v3 -> v4:
>>> - Add CONFIG_SPL_ALTERA_SDRAM to config_whitelist.txt in patch [1/3] and 
>>> remove it in patch [3/3]
>>> - Remove _remove().
>>> - Update commit message in patch [2/3]
>>>
>>> v2->v3:
>>> ---
>>> - Compile ALTERA_SDRAM driver in SPL only
>>> - Separate dts change to new patch
>>> - Change to use #if CONFIG_IS_ENABLED(ALTERA_SDRAM)
>>> - Add _remove()
>>> - Remove #ifdef CONFIG_SPL_BUILD checking in sdram_s10.c
>>>
>>> History:
>>> 
>>> [v1]: https://patchwork.ozlabs.org/patch/1066765/
>>> [v2]: https://patchwork.ozlabs.org/patch/1089957/
>>> [v3]: https://patchwork.ozlabs.org/cover/1093091/
>>>
>>> Ley Foon Tan (3):
>>>   ddr: altera: Compile ALTERA SDRAM in SPL only
>>>   arm: dts: Stratix10: Add SDRAM node
>>>   arm: socfpga: Move Stratix 10 SDRAM driver to DM
>>>
>>>  Makefile  |   2 +-
>>>  arch/arm/dts/socfpga_stratix10.dtsi   |   9 +
>>>  arch/arm/mach-socfpga/Kconfig |   4 +-
>>>  arch/arm/mach-socfpga/spl_s10.c   |  16 +-
>>>  configs/socfpga_stratix10_defconfig   |   1 +
>>>  drivers/Makefile  |   2 +-
>>>  drivers/ddr/altera/Kconfig|  11 +-
>>>  drivers/ddr/altera/Makefile   |   2 +-
>>>  drivers/ddr/altera/sdram_s10.c| 243 --
>>>  .../mach => drivers/ddr/altera}/sdram_s10.h   |   4 -
>>>  include/configs/socfpga_stratix10_socdk.h |   5 -
>>>  11 files changed, 200 insertions(+), 99 deletions(-)
>>>  rename {arch/arm/mach-socfpga/include/mach => 
>>> drivers/ddr/altera}/sdram_s10.h (97%)
>>>
>> I wanted to pick them, but they don't apply to u-boot/master ?
>  I will rebase them to u-boot/master and resend again.

Thanks!

-- 
Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v4 0/3] Move Stratix 10 SDRAM driver to DM

2019-05-05 Thread Ley Foon Tan 
On Fri, May 3, 2019 at 11:13 PM Marek Vasut  wrote:
>
> On 5/3/19 8:27 AM, Ley Foon Tan wrote:
> > Compile ALTERA_SDRAM driver in SPL only and move Stratix 10 SDRAM driver to 
> > DM.
> >
> > v3 -> v4:
> > - Add CONFIG_SPL_ALTERA_SDRAM to config_whitelist.txt in patch [1/3] and 
> > remove it in patch [3/3]
> > - Remove _remove().
> > - Update commit message in patch [2/3]
> >
> > v2->v3:
> > ---
> > - Compile ALTERA_SDRAM driver in SPL only
> > - Separate dts change to new patch
> > - Change to use #if CONFIG_IS_ENABLED(ALTERA_SDRAM)
> > - Add _remove()
> > - Remove #ifdef CONFIG_SPL_BUILD checking in sdram_s10.c
> >
> > History:
> > 
> > [v1]: https://patchwork.ozlabs.org/patch/1066765/
> > [v2]: https://patchwork.ozlabs.org/patch/1089957/
> > [v3]: https://patchwork.ozlabs.org/cover/1093091/
> >
> > Ley Foon Tan (3):
> >   ddr: altera: Compile ALTERA SDRAM in SPL only
> >   arm: dts: Stratix10: Add SDRAM node
> >   arm: socfpga: Move Stratix 10 SDRAM driver to DM
> >
> >  Makefile  |   2 +-
> >  arch/arm/dts/socfpga_stratix10.dtsi   |   9 +
> >  arch/arm/mach-socfpga/Kconfig |   4 +-
> >  arch/arm/mach-socfpga/spl_s10.c   |  16 +-
> >  configs/socfpga_stratix10_defconfig   |   1 +
> >  drivers/Makefile  |   2 +-
> >  drivers/ddr/altera/Kconfig|  11 +-
> >  drivers/ddr/altera/Makefile   |   2 +-
> >  drivers/ddr/altera/sdram_s10.c| 243 --
> >  .../mach => drivers/ddr/altera}/sdram_s10.h   |   4 -
> >  include/configs/socfpga_stratix10_socdk.h |   5 -
> >  11 files changed, 200 insertions(+), 99 deletions(-)
> >  rename {arch/arm/mach-socfpga/include/mach => 
> > drivers/ddr/altera}/sdram_s10.h (97%)
> >
> I wanted to pick them, but they don't apply to u-boot/master ?
 I will rebase them to u-boot/master and resend again.

Thanks.

Regards
Ley Foon
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v4 0/3] Move Stratix 10 SDRAM driver to DM

2019-05-03 Thread Marek Vasut 
On 5/3/19 8:27 AM, Ley Foon Tan wrote:
> Compile ALTERA_SDRAM driver in SPL only and move Stratix 10 SDRAM driver to 
> DM.
> 
> v3 -> v4:
> - Add CONFIG_SPL_ALTERA_SDRAM to config_whitelist.txt in patch [1/3] and 
> remove it in patch [3/3]
> - Remove _remove().
> - Update commit message in patch [2/3]
> 
> v2->v3:
> ---
> - Compile ALTERA_SDRAM driver in SPL only
> - Separate dts change to new patch
> - Change to use #if CONFIG_IS_ENABLED(ALTERA_SDRAM)
> - Add _remove()
> - Remove #ifdef CONFIG_SPL_BUILD checking in sdram_s10.c
> 
> History:
> 
> [v1]: https://patchwork.ozlabs.org/patch/1066765/
> [v2]: https://patchwork.ozlabs.org/patch/1089957/
> [v3]: https://patchwork.ozlabs.org/cover/1093091/
> 
> Ley Foon Tan (3):
>   ddr: altera: Compile ALTERA SDRAM in SPL only
>   arm: dts: Stratix10: Add SDRAM node
>   arm: socfpga: Move Stratix 10 SDRAM driver to DM
> 
>  Makefile  |   2 +-
>  arch/arm/dts/socfpga_stratix10.dtsi   |   9 +
>  arch/arm/mach-socfpga/Kconfig |   4 +-
>  arch/arm/mach-socfpga/spl_s10.c   |  16 +-
>  configs/socfpga_stratix10_defconfig   |   1 +
>  drivers/Makefile  |   2 +-
>  drivers/ddr/altera/Kconfig|  11 +-
>  drivers/ddr/altera/Makefile   |   2 +-
>  drivers/ddr/altera/sdram_s10.c| 243 --
>  .../mach => drivers/ddr/altera}/sdram_s10.h   |   4 -
>  include/configs/socfpga_stratix10_socdk.h |   5 -
>  11 files changed, 200 insertions(+), 99 deletions(-)
>  rename {arch/arm/mach-socfpga/include/mach => 
> drivers/ddr/altera}/sdram_s10.h (97%)
> 
I wanted to pick them, but they don't apply to u-boot/master ?

-- 
Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH v4 0/3] Move Stratix 10 SDRAM driver to DM

2019-05-03 Thread Ley Foon Tan 
Compile ALTERA_SDRAM driver in SPL only and move Stratix 10 SDRAM driver to DM.

v3 -> v4:
- Add CONFIG_SPL_ALTERA_SDRAM to config_whitelist.txt in patch [1/3] and remove 
it in patch [3/3]
- Remove _remove().
- Update commit message in patch [2/3]

v2->v3:
---
- Compile ALTERA_SDRAM driver in SPL only
- Separate dts change to new patch
- Change to use #if CONFIG_IS_ENABLED(ALTERA_SDRAM)
- Add _remove()
- Remove #ifdef CONFIG_SPL_BUILD checking in sdram_s10.c

History:

[v1]: https://patchwork.ozlabs.org/patch/1066765/
[v2]: https://patchwork.ozlabs.org/patch/1089957/
[v3]: https://patchwork.ozlabs.org/cover/1093091/

Ley Foon Tan (3):
  ddr: altera: Compile ALTERA SDRAM in SPL only
  arm: dts: Stratix10: Add SDRAM node
  arm: socfpga: Move Stratix 10 SDRAM driver to DM

 Makefile  |   2 +-
 arch/arm/dts/socfpga_stratix10.dtsi   |   9 +
 arch/arm/mach-socfpga/Kconfig |   4 +-
 arch/arm/mach-socfpga/spl_s10.c   |  16 +-
 configs/socfpga_stratix10_defconfig   |   1 +
 drivers/Makefile  |   2 +-
 drivers/ddr/altera/Kconfig|  11 +-
 drivers/ddr/altera/Makefile   |   2 +-
 drivers/ddr/altera/sdram_s10.c| 243 --
 .../mach => drivers/ddr/altera}/sdram_s10.h   |   4 -
 include/configs/socfpga_stratix10_socdk.h |   5 -
 11 files changed, 200 insertions(+), 99 deletions(-)
 rename {arch/arm/mach-socfpga/include/mach => drivers/ddr/altera}/sdram_s10.h 
(97%)

-- 
2.19.0

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot