From: Jagan Teki <jagan@amarulasolutions.com> To: Mark Kettenis <mark.kettenis@xs4all.nl> Cc: Tom Rini <trini@konsulko.com>, "open list:ARM/Rockchip SoC..." <linux-rockchip@lists.infradead.org>, "Matwey V . Kornilov" <matwey.kornilov@gmail.com>, U-Boot-Denx <u-boot@lists.denx.de> Subject: Re: [PATCH v3 2/5] Makefile: Rename idbloader.img with u-boot-spl-rockchip.bin Date: Tue, 22 Oct 2019 15:40:21 +0530 [thread overview] Message-ID: <CAMty3ZCrcExQ+KmvWR_0Rus6hLAXySMu2hoGpUHLEBAZaO4kEw@mail.gmail.com> (raw) In-Reply-To: <51fabf8ac7d59d16@bloch.sibelius.xs4all.nl> On Mon, Oct 21, 2019 at 1:43 PM Mark Kettenis <mark.kettenis@xs4all.nl> wrote: > > > From: Jagan Teki <jagan@amarulasolutions.com> > > Date: Mon, 21 Oct 2019 10:56:39 +0530 > > > > Hi Kever, > > > > On Fri, Oct 18, 2019 at 4:26 PM Kever Yang <kever.yang@rock-chips.com> wrote: > > > > > > Jagan, > > > > > > > > > On 2019/10/18 上午3:07, Jagan Teki wrote: > > > > idbloader.img name is specific to rockchip, > > > > > > This is specific for rockchip, like rksd, rkspi type in mkimage, since > > > it's clear for what it stands for, > > > > > > I think it can also used in U-Boot. > > > > > > > where it usually > > > > created using rockchip tools. > > > > > > > > > No, idbloader stands for the image with idb header packaged with two > > > stage loader, eg. TPL+SPL > > > > > > or ddr.bin+miniloader.bin, and maybe ddr.bin + SPL, TPL+miniloader.bin, > > > > > > not related to rockchip tools, it can also be created with U-Boot > > > mkimage tool. > > > > I understand what idbloader contains, it is like rockchip SPL. but the > > main concern here is the naming convention used in U-Boot. It would be > > a standard way of using naming conventions where the final output > > naming conventions should have u-boot support stages(TPL, SPL, U-Boot) > > and platform specific name. This way it is less confused what exactly > > it contains. > > > > Moreover most of the platform specific bin ended up using this type of > > conventions. > > > > - u-boot-spl-mtk.bin - Mediatek SPL bin > > - u-boot-mtk.bin - Mediatek U-Boot bin (would be final image) > > - u-boot-sunxi-with-spl.bin - Allwinner U-Boot with SPL > > - u-boot-x86-start16-tpl.bin > > No real consistency there though... Not sure what you pointed here, each bin listed above shows the platform and u-boot stage and prefix with u-boot- as well. of course all can't maintain the same but has mentioned details. > > > Similar naming conventions used for tegra, x86 etc. > > > > I'm completely aware of what you're trying to abbreviate > > idbloader.img, but having u-boot-spl-rockchip.bin is more readable and > > understand than idbloader. > > As was discussed before, idbloader.img is the name used in external > documentation. Yes, used in external documentation. when it comes to upstream it would follow as per existing platforms does atleast. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
WARNING: multiple messages have this Message-ID (diff)
From: Jagan Teki <jagan@amarulasolutions.com> To: u-boot@lists.denx.de Subject: [U-Boot] [PATCH v3 2/5] Makefile: Rename idbloader.img with u-boot-spl-rockchip.bin Date: Tue, 22 Oct 2019 15:40:21 +0530 [thread overview] Message-ID: <CAMty3ZCrcExQ+KmvWR_0Rus6hLAXySMu2hoGpUHLEBAZaO4kEw@mail.gmail.com> (raw) In-Reply-To: <51fabf8ac7d59d16@bloch.sibelius.xs4all.nl> On Mon, Oct 21, 2019 at 1:43 PM Mark Kettenis <mark.kettenis@xs4all.nl> wrote: > > > From: Jagan Teki <jagan@amarulasolutions.com> > > Date: Mon, 21 Oct 2019 10:56:39 +0530 > > > > Hi Kever, > > > > On Fri, Oct 18, 2019 at 4:26 PM Kever Yang <kever.yang@rock-chips.com> wrote: > > > > > > Jagan, > > > > > > > > > On 2019/10/18 上午3:07, Jagan Teki wrote: > > > > idbloader.img name is specific to rockchip, > > > > > > This is specific for rockchip, like rksd, rkspi type in mkimage, since > > > it's clear for what it stands for, > > > > > > I think it can also used in U-Boot. > > > > > > > where it usually > > > > created using rockchip tools. > > > > > > > > > No, idbloader stands for the image with idb header packaged with two > > > stage loader, eg. TPL+SPL > > > > > > or ddr.bin+miniloader.bin, and maybe ddr.bin + SPL, TPL+miniloader.bin, > > > > > > not related to rockchip tools, it can also be created with U-Boot > > > mkimage tool. > > > > I understand what idbloader contains, it is like rockchip SPL. but the > > main concern here is the naming convention used in U-Boot. It would be > > a standard way of using naming conventions where the final output > > naming conventions should have u-boot support stages(TPL, SPL, U-Boot) > > and platform specific name. This way it is less confused what exactly > > it contains. > > > > Moreover most of the platform specific bin ended up using this type of > > conventions. > > > > - u-boot-spl-mtk.bin - Mediatek SPL bin > > - u-boot-mtk.bin - Mediatek U-Boot bin (would be final image) > > - u-boot-sunxi-with-spl.bin - Allwinner U-Boot with SPL > > - u-boot-x86-start16-tpl.bin > > No real consistency there though... Not sure what you pointed here, each bin listed above shows the platform and u-boot stage and prefix with u-boot- as well. of course all can't maintain the same but has mentioned details. > > > Similar naming conventions used for tegra, x86 etc. > > > > I'm completely aware of what you're trying to abbreviate > > idbloader.img, but having u-boot-spl-rockchip.bin is more readable and > > understand than idbloader. > > As was discussed before, idbloader.img is the name used in external > documentation. Yes, used in external documentation. when it comes to upstream it would follow as per existing platforms does atleast.
next prev parent reply other threads:[~2019-10-22 10:10 UTC|newest] Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-10-17 19:07 [PATCH v3 0/5] rockchip: Add Binman support Jagan Teki 2019-10-17 19:07 ` [U-Boot] " Jagan Teki [not found] ` <20191017190710.29985-1-jagan-dyjBcgdgk7Pe9wHmmfpqLFaTQe2KTcn/@public.gmane.org> 2019-10-17 19:07 ` [PATCH v3 1/5] Makefile: Add rockchip image type Jagan Teki 2019-10-17 19:07 ` [U-Boot] " Jagan Teki 2019-11-10 14:16 ` Kever Yang 2019-11-10 14:16 ` [U-Boot] " Kever Yang 2019-10-17 19:07 ` [PATCH v3 2/5] Makefile: Rename idbloader.img with u-boot-spl-rockchip.bin Jagan Teki 2019-10-17 19:07 ` [U-Boot] " Jagan Teki 2019-10-18 10:56 ` Kever Yang 2019-10-18 10:56 ` [U-Boot] " Kever Yang 2019-10-21 5:26 ` Jagan Teki 2019-10-21 5:26 ` [U-Boot] " Jagan Teki 2019-10-21 8:13 ` Mark Kettenis 2019-10-21 8:13 ` [U-Boot] " Mark Kettenis 2019-10-22 10:10 ` Jagan Teki [this message] 2019-10-22 10:10 ` Jagan Teki 2019-11-15 6:34 ` Kever Yang 2019-11-15 6:34 ` [U-Boot] " Kever Yang 2019-10-17 19:07 ` [PATCH v3 3/5] arm: dts: rk3368: Add rk3368-u-boot.dtsi Jagan Teki 2019-10-17 19:07 ` [U-Boot] " Jagan Teki 2019-10-17 19:07 ` [PATCH v3 4/5] rockchip: Add Binman support Jagan Teki 2019-10-17 19:07 ` [U-Boot] " Jagan Teki [not found] ` <20191017190710.29985-5-jagan-dyjBcgdgk7Pe9wHmmfpqLFaTQe2KTcn/@public.gmane.org> 2019-10-21 8:16 ` Jagan Teki 2019-10-21 8:16 ` [U-Boot] " Jagan Teki 2019-10-17 19:07 ` [PATCH v3 5/5] doc: boards: Add rockchip documentation Jagan Teki 2019-10-17 19:07 ` [U-Boot] " Jagan Teki [not found] ` <20191017190710.29985-6-jagan-dyjBcgdgk7Pe9wHmmfpqLFaTQe2KTcn/@public.gmane.org> 2019-10-18 12:20 ` Robin Murphy 2019-10-18 12:20 ` [U-Boot] " Robin Murphy
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=CAMty3ZCrcExQ+KmvWR_0Rus6hLAXySMu2hoGpUHLEBAZaO4kEw@mail.gmail.com \ --to=jagan@amarulasolutions.com \ --cc=linux-rockchip@lists.infradead.org \ --cc=mark.kettenis@xs4all.nl \ --cc=matwey.kornilov@gmail.com \ --cc=trini@konsulko.com \ --cc=u-boot@lists.denx.de \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.