mbox series

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

Message ID 1556864834-31023-1-git-send-email-ley.foon.tan@intel.com
Headers show
Series Move Stratix 10 SDRAM driver to DM | expand

Message

Ley Foon Tan May 3, 2019, 6:27 a.m. UTC
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%)

Comments

Marek Vasut May 3, 2019, 3:13 p.m. UTC | #1
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 ?
Ley Foon Tan May 6, 2019, 1:50 a.m. UTC | #2
On Fri, May 3, 2019 at 11:13 PM Marek Vasut <marex@denx.de> 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
Marek Vasut May 6, 2019, 10:43 a.m. UTC | #3
On 5/6/19 3:50 AM, Ley Foon Tan wrote:
> On Fri, May 3, 2019 at 11:13 PM Marek Vasut <marex@denx.de> 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!