[Buildroot] [PATCH v3 5/7] boot/beaglev-ddrinit: new package

Yann E. MORIN yann.morin.1998 at free.fr
Sat May 8 07:37:18 UTC 2021


Drew, All,

On 2021-05-07 16:30 -0700, Drew Fustini spake thusly:
> On Fri, May 7, 2021, 15:04 Yann E. MORIN < [1]yann.morin.1998 at free.fr> wrote:
>   On 2021-05-04 22:51 +0200, Thomas Petazzoni spake thusly:
>   > This commit adds a package for the DDR initialization code used on the
>   > BeagleV platform.
> 
>   The upstream package name is beaglev-ddrlnit, yes, with an 'l' instead
>   of an 'i'. I've renamed the package to match upstream name, so that
>   users that look for it can actually find it.
> Would it be helpful if I get StarFive to fix the typo and rename it to ddrinit?
> I am thinking that would probably be good to have happen before other projects
> start pointing to it. Thomas was so fast with the buildroot patches I think it
> is the first to notice this.

It is not so important that the upstream name gets fixed; in fact quite
the opposite: we are now already referencing it. Granted, it's brand
new, and we can fix it quick before the release. But note that our
2021.05-rc1 is due this WE, for example; we can still fix this kind of
thing post rc1, of course, but we can't fix a release...

But as you mention, once other projects start referencing it, it will be
too late. For example, Khem Raj already has started such support:

    https://github.com/riscv/meta-riscv/blob/master/beaglev.yml
    https://github.com/YoeDistro/yoe-distro/blob/master/beaglev-starlight-jh7100-envsetup.sh

So even if it is still very early, I think it is already too late to
rename the repository...

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'



More information about the buildroot mailing list