* [RESEND PATCH 0/2] splash: add more options/support @ 2022-10-12 11:38 Julien Masson 2022-10-12 11:38 ` [RESEND PATCH 1/2] splash: support raw image from MMC Julien Masson 2022-10-12 11:38 ` [RESEND PATCH 2/2] splash: get devpart from environment variable Julien Masson 0 siblings, 2 replies; 10+ messages in thread From: Julien Masson @ 2022-10-12 11:38 UTC (permalink / raw) To: u-boot; +Cc: Simon Glass This patch series add several options/support for splashscreen feature: - support raw image from MMC - get devpart from environment variable With these changes the user has now more options and can change default configurations through environment variables. Example: image located in splashscreen partition (MMC as raw) ``` splashsource=mmc_raw splashdevpart=0#splashscreen ``` Julien Masson (2): splash: support raw image from MMC splash: get devpart from environment variable common/splash.c | 6 ++++++ common/splash_source.c | 34 ++++++++++++++++++++++++++++++++++ 2 files changed, 40 insertions(+) -- 2.37.3 ^ permalink raw reply [flat|nested] 10+ messages in thread
* [RESEND PATCH 1/2] splash: support raw image from MMC 2022-10-12 11:38 [RESEND PATCH 0/2] splash: add more options/support Julien Masson @ 2022-10-12 11:38 ` Julien Masson 2022-10-12 12:59 ` Simon Glass 2022-10-12 11:38 ` [RESEND PATCH 2/2] splash: get devpart from environment variable Julien Masson 1 sibling, 1 reply; 10+ messages in thread From: Julien Masson @ 2022-10-12 11:38 UTC (permalink / raw) To: u-boot; +Cc: Simon Glass The user has now the choice to specify the splash location in the MMC as a raw storage. Signed-off-by: Julien Masson <jmasson@baylibre.com> --- common/splash.c | 6 ++++++ common/splash_source.c | 29 +++++++++++++++++++++++++++++ 2 files changed, 35 insertions(+) diff --git a/common/splash.c b/common/splash.c index 0e520cc103..5206e35f74 100644 --- a/common/splash.c +++ b/common/splash.c @@ -39,6 +39,12 @@ static struct splash_location default_splash_locations[] = { .flags = SPLASH_STORAGE_FS, .devpart = "0:1", }, + { + .name = "mmc_raw", + .storage = SPLASH_STORAGE_MMC, + .flags = SPLASH_STORAGE_RAW, + .devpart = "0:1", + }, { .name = "usb_fs", .storage = SPLASH_STORAGE_USB, diff --git a/common/splash_source.c b/common/splash_source.c index 87e55a54f8..b4bf6f1336 100644 --- a/common/splash_source.c +++ b/common/splash_source.c @@ -65,6 +65,33 @@ static int splash_nand_read_raw(u32 bmp_load_addr, int offset, size_t read_size) } #endif +#ifdef CONFIG_CMD_MMC +static int splash_mmc_read_raw(u32 bmp_load_addr, struct splash_location *location, + size_t read_size) +{ + struct disk_partition partition; + struct blk_desc *desc; + lbaint_t blkcnt; + int ret, n; + + ret = part_get_info_by_dev_and_name_or_num("mmc", location->devpart, &desc, + &partition, 1); + if (ret < 0) + return ret; + + blkcnt = DIV_ROUND_UP(read_size, partition.blksz); + n = blk_dread(desc, partition.start, blkcnt, (void *)(uintptr_t)bmp_load_addr); + + return (n == blkcnt) ? 0 : -EINVAL; +} +#else +static int splash_mmc_read_raw(u32 bmp_load_addr, int offset, size_t read_size) +{ + debug("%s: mmc support not available\n", __func__); + return -ENOSYS; +} +#endif + static int splash_storage_read_raw(struct splash_location *location, u32 bmp_load_addr, size_t read_size) { @@ -75,6 +102,8 @@ static int splash_storage_read_raw(struct splash_location *location, offset = location->offset; switch (location->storage) { + case SPLASH_STORAGE_MMC: + return splash_mmc_read_raw(bmp_load_addr, location, read_size); case SPLASH_STORAGE_NAND: return splash_nand_read_raw(bmp_load_addr, offset, read_size); case SPLASH_STORAGE_SF: -- 2.37.3 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [RESEND PATCH 1/2] splash: support raw image from MMC 2022-10-12 11:38 ` [RESEND PATCH 1/2] splash: support raw image from MMC Julien Masson @ 2022-10-12 12:59 ` Simon Glass 2022-10-13 16:08 ` Julien Masson 0 siblings, 1 reply; 10+ messages in thread From: Simon Glass @ 2022-10-12 12:59 UTC (permalink / raw) To: Julien Masson; +Cc: u-boot HI Julien, On Wed, 12 Oct 2022 at 05:38, Julien Masson <jmasson@baylibre.com> wrote: > > The user has now the choice to specify the splash location in the MMC > as a raw storage. > > Signed-off-by: Julien Masson <jmasson@baylibre.com> > --- > common/splash.c | 6 ++++++ > common/splash_source.c | 29 +++++++++++++++++++++++++++++ > 2 files changed, 35 insertions(+) > > diff --git a/common/splash.c b/common/splash.c > index 0e520cc103..5206e35f74 100644 > --- a/common/splash.c > +++ b/common/splash.c > @@ -39,6 +39,12 @@ static struct splash_location default_splash_locations[] = { > .flags = SPLASH_STORAGE_FS, > .devpart = "0:1", > }, > + { > + .name = "mmc_raw", > + .storage = SPLASH_STORAGE_MMC, > + .flags = SPLASH_STORAGE_RAW, > + .devpart = "0:1", > + }, > { > .name = "usb_fs", > .storage = SPLASH_STORAGE_USB, > diff --git a/common/splash_source.c b/common/splash_source.c > index 87e55a54f8..b4bf6f1336 100644 > --- a/common/splash_source.c > +++ b/common/splash_source.c > @@ -65,6 +65,33 @@ static int splash_nand_read_raw(u32 bmp_load_addr, int offset, size_t read_size) > } > #endif > > +#ifdef CONFIG_CMD_MMC > +static int splash_mmc_read_raw(u32 bmp_load_addr, struct splash_location *location, > + size_t read_size) > +{ > + struct disk_partition partition; > + struct blk_desc *desc; > + lbaint_t blkcnt; > + int ret, n; > + > + ret = part_get_info_by_dev_and_name_or_num("mmc", location->devpart, &desc, > + &partition, 1); > + if (ret < 0) > + return ret; > + > + blkcnt = DIV_ROUND_UP(read_size, partition.blksz); > + n = blk_dread(desc, partition.start, blkcnt, (void *)(uintptr_t)bmp_load_addr); > + > + return (n == blkcnt) ? 0 : -EINVAL; -EIO would be better > +} > +#else > +static int splash_mmc_read_raw(u32 bmp_load_addr, int offset, size_t read_size) > +{ > + debug("%s: mmc support not available\n", __func__); > + return -ENOSYS; Rather than the #ifdef you should be able to put these two lines in the first function. Does patman not warn you about this sort of thing? > +} > +#endif > + > static int splash_storage_read_raw(struct splash_location *location, > u32 bmp_load_addr, size_t read_size) > { > @@ -75,6 +102,8 @@ static int splash_storage_read_raw(struct splash_location *location, > > offset = location->offset; > switch (location->storage) { > + case SPLASH_STORAGE_MMC: > + return splash_mmc_read_raw(bmp_load_addr, location, read_size); > case SPLASH_STORAGE_NAND: > return splash_nand_read_raw(bmp_load_addr, offset, read_size); > case SPLASH_STORAGE_SF: > -- > 2.37.3 > Regards, SImon ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [RESEND PATCH 1/2] splash: support raw image from MMC 2022-10-12 12:59 ` Simon Glass @ 2022-10-13 16:08 ` Julien Masson 2022-10-14 15:55 ` Simon Glass 0 siblings, 1 reply; 10+ messages in thread From: Julien Masson @ 2022-10-13 16:08 UTC (permalink / raw) To: Simon Glass; +Cc: u-boot Hi Simon, On Thu 13 Oct 2022 at 17:51, Simon Glass <sjg@chromium.org> wrote: > HI Julien, > > On Wed, 12 Oct 2022 at 05:38, Julien Masson <jmasson@baylibre.com> wrote: >> >> The user has now the choice to specify the splash location in the MMC >> as a raw storage. >> >> Signed-off-by: Julien Masson <jmasson@baylibre.com> >> --- >> common/splash.c | 6 ++++++ >> common/splash_source.c | 29 +++++++++++++++++++++++++++++ >> 2 files changed, 35 insertions(+) >> >> diff --git a/common/splash.c b/common/splash.c >> index 0e520cc103..5206e35f74 100644 >> --- a/common/splash.c >> +++ b/common/splash.c >> @@ -39,6 +39,12 @@ static struct splash_location default_splash_locations[] = { >> .flags = SPLASH_STORAGE_FS, >> .devpart = "0:1", >> }, >> + { >> + .name = "mmc_raw", >> + .storage = SPLASH_STORAGE_MMC, >> + .flags = SPLASH_STORAGE_RAW, >> + .devpart = "0:1", >> + }, >> { >> .name = "usb_fs", >> .storage = SPLASH_STORAGE_USB, >> diff --git a/common/splash_source.c b/common/splash_source.c >> index 87e55a54f8..b4bf6f1336 100644 >> --- a/common/splash_source.c >> +++ b/common/splash_source.c >> @@ -65,6 +65,33 @@ static int splash_nand_read_raw(u32 bmp_load_addr, int offset, size_t read_size) >> } >> #endif >> >> +#ifdef CONFIG_CMD_MMC >> +static int splash_mmc_read_raw(u32 bmp_load_addr, struct splash_location *location, >> + size_t read_size) >> +{ >> + struct disk_partition partition; >> + struct blk_desc *desc; >> + lbaint_t blkcnt; >> + int ret, n; >> + >> + ret = part_get_info_by_dev_and_name_or_num("mmc", location->devpart, &desc, >> + &partition, 1); >> + if (ret < 0) >> + return ret; >> + >> + blkcnt = DIV_ROUND_UP(read_size, partition.blksz); >> + n = blk_dread(desc, partition.start, blkcnt, (void *)(uintptr_t)bmp_load_addr); >> + >> + return (n == blkcnt) ? 0 : -EINVAL; > > -EIO would be better Yes I will change this in V2. > >> +} >> +#else >> +static int splash_mmc_read_raw(u32 bmp_load_addr, int offset, size_t read_size) I just discovered that my the second argument type is wrong. I will fix that in V2, sorry. >> +{ >> + debug("%s: mmc support not available\n", __func__); >> + return -ENOSYS; > > Rather than the #ifdef you should be able to put these two lines in > the first function. Does patman not warn you about this sort of thing? Yes that is true I can place the #ifdef inside the function but I've "voluntary" done that to stay consistent with other functions: splash_(sf|nand).* No I did not use patman, I've run checkpatch first and git send-email: $ ./scripts/checkpatch.pl --strict --u-boot But even with patman I don't see the warning you are talking: $ ./tools/patman/patman -n Cleaned 2 patches 0 errors, 1 warnings, 0 checks for 0002-splash-support-raw-image-from-MMC.patch: common/splash_source.c:68: warning: Use 'if (IS_ENABLED(CONFIG...))' instead of '#if or #ifdef' where possible checkpatch.pl found 0 error(s), 1 warning(s), 0 checks(s) Alias 'splash' not found Alias 'splash' not found Not sending emails due to errors/warnings Dry run, so not doing much. But I would do this: ... What command do you use to have this warning ? Thanks > >> +} >> +#endif >> + >> static int splash_storage_read_raw(struct splash_location *location, >> u32 bmp_load_addr, size_t read_size) >> { >> @@ -75,6 +102,8 @@ static int splash_storage_read_raw(struct splash_location *location, >> >> offset = location->offset; >> switch (location->storage) { >> + case SPLASH_STORAGE_MMC: >> + return splash_mmc_read_raw(bmp_load_addr, location, read_size); >> case SPLASH_STORAGE_NAND: >> return splash_nand_read_raw(bmp_load_addr, offset, read_size); >> case SPLASH_STORAGE_SF: >> -- >> 2.37.3 >> > > Regards, > SImon -- Julien Masson ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [RESEND PATCH 1/2] splash: support raw image from MMC 2022-10-13 16:08 ` Julien Masson @ 2022-10-14 15:55 ` Simon Glass 0 siblings, 0 replies; 10+ messages in thread From: Simon Glass @ 2022-10-14 15:55 UTC (permalink / raw) To: Julien Masson; +Cc: u-boot Hi Julien, On Thu, 13 Oct 2022 at 10:08, Julien Masson <jmasson@baylibre.com> wrote: > > Hi Simon, > > On Thu 13 Oct 2022 at 17:51, Simon Glass <sjg@chromium.org> wrote: > > > HI Julien, > > > > On Wed, 12 Oct 2022 at 05:38, Julien Masson <jmasson@baylibre.com> wrote: > >> > >> The user has now the choice to specify the splash location in the MMC > >> as a raw storage. > >> > >> Signed-off-by: Julien Masson <jmasson@baylibre.com> > >> --- > >> common/splash.c | 6 ++++++ > >> common/splash_source.c | 29 +++++++++++++++++++++++++++++ > >> 2 files changed, 35 insertions(+) > >> > >> diff --git a/common/splash.c b/common/splash.c > >> index 0e520cc103..5206e35f74 100644 > >> --- a/common/splash.c > >> +++ b/common/splash.c > >> @@ -39,6 +39,12 @@ static struct splash_location default_splash_locations[] = { > >> .flags = SPLASH_STORAGE_FS, > >> .devpart = "0:1", > >> }, > >> + { > >> + .name = "mmc_raw", > >> + .storage = SPLASH_STORAGE_MMC, > >> + .flags = SPLASH_STORAGE_RAW, > >> + .devpart = "0:1", > >> + }, > >> { > >> .name = "usb_fs", > >> .storage = SPLASH_STORAGE_USB, > >> diff --git a/common/splash_source.c b/common/splash_source.c > >> index 87e55a54f8..b4bf6f1336 100644 > >> --- a/common/splash_source.c > >> +++ b/common/splash_source.c > >> @@ -65,6 +65,33 @@ static int splash_nand_read_raw(u32 bmp_load_addr, int offset, size_t read_size) > >> } > >> #endif > >> > >> +#ifdef CONFIG_CMD_MMC > >> +static int splash_mmc_read_raw(u32 bmp_load_addr, struct splash_location *location, > >> + size_t read_size) > >> +{ > >> + struct disk_partition partition; > >> + struct blk_desc *desc; > >> + lbaint_t blkcnt; > >> + int ret, n; > >> + > >> + ret = part_get_info_by_dev_and_name_or_num("mmc", location->devpart, &desc, > >> + &partition, 1); > >> + if (ret < 0) > >> + return ret; > >> + > >> + blkcnt = DIV_ROUND_UP(read_size, partition.blksz); > >> + n = blk_dread(desc, partition.start, blkcnt, (void *)(uintptr_t)bmp_load_addr); > >> + > >> + return (n == blkcnt) ? 0 : -EINVAL; > > > > -EIO would be better > > Yes I will change this in V2. > > > > >> +} > >> +#else > >> +static int splash_mmc_read_raw(u32 bmp_load_addr, int offset, size_t read_size) > > I just discovered that my the second argument type is wrong. > I will fix that in V2, sorry. > > >> +{ > >> + debug("%s: mmc support not available\n", __func__); > >> + return -ENOSYS; > > > > Rather than the #ifdef you should be able to put these two lines in > > the first function. Does patman not warn you about this sort of thing? > > Yes that is true I can place the #ifdef inside the function but I've > "voluntary" done that to stay consistent with other functions: > splash_(sf|nand).* > > No I did not use patman, I've run checkpatch first and git send-email: > $ ./scripts/checkpatch.pl --strict --u-boot > > But even with patman I don't see the warning you are talking: > $ ./tools/patman/patman -n > Cleaned 2 patches > 0 errors, 1 warnings, 0 checks for 0002-splash-support-raw-image-from-MMC.patch: > common/splash_source.c:68: warning: Use 'if (IS_ENABLED(CONFIG...))' instead of '#if or #ifdef' where possible ^ That is the warning > checkpatch.pl found 0 error(s), 1 warning(s), 0 checks(s) > Alias 'splash' not found > Alias 'splash' not found > Not sending emails due to errors/warnings > Dry run, so not doing much. But I would do this: > ... > > What command do you use to have this warning ? > > Thanks > > > > >> +} > >> +#endif > >> + > >> static int splash_storage_read_raw(struct splash_location *location, > >> u32 bmp_load_addr, size_t read_size) > >> { > >> @@ -75,6 +102,8 @@ static int splash_storage_read_raw(struct splash_location *location, > >> > >> offset = location->offset; > >> switch (location->storage) { > >> + case SPLASH_STORAGE_MMC: > >> + return splash_mmc_read_raw(bmp_load_addr, location, read_size); > >> case SPLASH_STORAGE_NAND: > >> return splash_nand_read_raw(bmp_load_addr, offset, read_size); > >> case SPLASH_STORAGE_SF: > >> -- > >> 2.37.3 > >> > > > > Regards, > > SImon > > -- > Julien Masson Regards, Simon ^ permalink raw reply [flat|nested] 10+ messages in thread
* [RESEND PATCH 2/2] splash: get devpart from environment variable 2022-10-12 11:38 [RESEND PATCH 0/2] splash: add more options/support Julien Masson 2022-10-12 11:38 ` [RESEND PATCH 1/2] splash: support raw image from MMC Julien Masson @ 2022-10-12 11:38 ` Julien Masson 2022-10-12 12:59 ` Simon Glass 1 sibling, 1 reply; 10+ messages in thread From: Julien Masson @ 2022-10-12 11:38 UTC (permalink / raw) To: u-boot; +Cc: Simon Glass By default several types of splash locations are supported and the user can select one of them through environment var (splashsource). However the devpart is still hardcoded and we cannot change it from the environment. This patch add the support of "splashdevpart" which allow the user to set the devpart though this environment variable. Example: image located in splashscreen partition (MMC as raw) ``` splashsource=mmc_raw splashdevpart=0#splashscreen ``` Signed-off-by: Julien Masson <jmasson@baylibre.com> --- common/splash_source.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/common/splash_source.c b/common/splash_source.c index b4bf6f1336..1f99f44f78 100644 --- a/common/splash_source.c +++ b/common/splash_source.c @@ -451,6 +451,7 @@ int splash_source_load(struct splash_location *locations, uint size) { struct splash_location *splash_location; char *env_splashimage_value; + char *env_splashdevpart_value; u32 bmp_load_addr; env_splashimage_value = env_get("splashimage"); @@ -467,6 +468,10 @@ int splash_source_load(struct splash_location *locations, uint size) if (!splash_location) return -EINVAL; + env_splashdevpart_value = env_get("splashdevpart"); + if (env_splashdevpart_value) + splash_location->devpart = env_splashdevpart_value; + if (splash_location->flags == SPLASH_STORAGE_RAW) return splash_load_raw(splash_location, bmp_load_addr); else if (splash_location->flags == SPLASH_STORAGE_FS) -- 2.37.3 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [RESEND PATCH 2/2] splash: get devpart from environment variable 2022-10-12 11:38 ` [RESEND PATCH 2/2] splash: get devpart from environment variable Julien Masson @ 2022-10-12 12:59 ` Simon Glass 2022-10-13 15:51 ` Julien Masson 0 siblings, 1 reply; 10+ messages in thread From: Simon Glass @ 2022-10-12 12:59 UTC (permalink / raw) To: Julien Masson; +Cc: u-boot Hi Julien, On Wed, 12 Oct 2022 at 05:38, Julien Masson <jmasson@baylibre.com> wrote: > > By default several types of splash locations are supported and the > user can select one of them through environment var (splashsource). > > However the devpart is still hardcoded and we cannot change it from > the environment. > > This patch add the support of "splashdevpart" which allow the user to > set the devpart though this environment variable. > > Example: image located in splashscreen partition (MMC as raw) > ``` > splashsource=mmc_raw > splashdevpart=0#splashscreen > ``` > > Signed-off-by: Julien Masson <jmasson@baylibre.com> > --- > common/splash_source.c | 5 +++++ > 1 file changed, 5 insertions(+) Reviewed-by: Simon Glass <sjg@chromium.org> > > diff --git a/common/splash_source.c b/common/splash_source.c > index b4bf6f1336..1f99f44f78 100644 > --- a/common/splash_source.c > +++ b/common/splash_source.c > @@ -451,6 +451,7 @@ int splash_source_load(struct splash_location *locations, uint size) > { > struct splash_location *splash_location; > char *env_splashimage_value; > + char *env_splashdevpart_value; How about just 'devpar' as it is shorter and easier to read? > u32 bmp_load_addr; > > env_splashimage_value = env_get("splashimage"); > @@ -467,6 +468,10 @@ int splash_source_load(struct splash_location *locations, uint size) > if (!splash_location) > return -EINVAL; > > + env_splashdevpart_value = env_get("splashdevpart"); > + if (env_splashdevpart_value) > + splash_location->devpart = env_splashdevpart_value; > + > if (splash_location->flags == SPLASH_STORAGE_RAW) > return splash_load_raw(splash_location, bmp_load_addr); > else if (splash_location->flags == SPLASH_STORAGE_FS) > -- > 2.37.3 > Regards, Simon ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [RESEND PATCH 2/2] splash: get devpart from environment variable 2022-10-12 12:59 ` Simon Glass @ 2022-10-13 15:51 ` Julien Masson 2022-10-14 15:55 ` Simon Glass 0 siblings, 1 reply; 10+ messages in thread From: Julien Masson @ 2022-10-13 15:51 UTC (permalink / raw) To: Simon Glass; +Cc: u-boot Hi Simon, Thanks for the review. On Thu 13 Oct 2022 at 17:44, Simon Glass <sjg@chromium.org> wrote: > Hi Julien, > > On Wed, 12 Oct 2022 at 05:38, Julien Masson <jmasson@baylibre.com> wrote: >> >> By default several types of splash locations are supported and the >> user can select one of them through environment var (splashsource). >> >> However the devpart is still hardcoded and we cannot change it from >> the environment. >> >> This patch add the support of "splashdevpart" which allow the user to >> set the devpart though this environment variable. >> >> Example: image located in splashscreen partition (MMC as raw) >> ``` >> splashsource=mmc_raw >> splashdevpart=0#splashscreen >> ``` >> >> Signed-off-by: Julien Masson <jmasson@baylibre.com> >> --- >> common/splash_source.c | 5 +++++ >> 1 file changed, 5 insertions(+) > > Reviewed-by: Simon Glass <sjg@chromium.org> >> >> diff --git a/common/splash_source.c b/common/splash_source.c >> index b4bf6f1336..1f99f44f78 100644 >> --- a/common/splash_source.c >> +++ b/common/splash_source.c >> @@ -451,6 +451,7 @@ int splash_source_load(struct splash_location *locations, uint size) >> { >> struct splash_location *splash_location; >> char *env_splashimage_value; >> + char *env_splashdevpart_value; > > How about just 'devpar' as it is shorter and easier to read? > Yes I initially follow the same "syntax" of splashimage var but I agree it can be shorter, what name would you prefer: - char *devpart - char *env_devpart - char *env_devpart_value ? >> u32 bmp_load_addr; >> >> env_splashimage_value = env_get("splashimage"); >> @@ -467,6 +468,10 @@ int splash_source_load(struct splash_location *locations, uint size) >> if (!splash_location) >> return -EINVAL; >> >> + env_splashdevpart_value = env_get("splashdevpart"); >> + if (env_splashdevpart_value) >> + splash_location->devpart = env_splashdevpart_value; >> + >> if (splash_location->flags == SPLASH_STORAGE_RAW) >> return splash_load_raw(splash_location, bmp_load_addr); >> else if (splash_location->flags == SPLASH_STORAGE_FS) >> -- >> 2.37.3 >> > > Regards, > Simon -- Julien Masson ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [RESEND PATCH 2/2] splash: get devpart from environment variable 2022-10-13 15:51 ` Julien Masson @ 2022-10-14 15:55 ` Simon Glass 2022-10-17 7:50 ` Julien Masson 0 siblings, 1 reply; 10+ messages in thread From: Simon Glass @ 2022-10-14 15:55 UTC (permalink / raw) To: Julien Masson; +Cc: u-boot Hi Julien, On Thu, 13 Oct 2022 at 09:51, Julien Masson <jmasson@baylibre.com> wrote: > > Hi Simon, > > Thanks for the review. > > On Thu 13 Oct 2022 at 17:44, Simon Glass <sjg@chromium.org> wrote: > > > Hi Julien, > > > > On Wed, 12 Oct 2022 at 05:38, Julien Masson <jmasson@baylibre.com> wrote: > >> > >> By default several types of splash locations are supported and the > >> user can select one of them through environment var (splashsource). > >> > >> However the devpart is still hardcoded and we cannot change it from > >> the environment. > >> > >> This patch add the support of "splashdevpart" which allow the user to > >> set the devpart though this environment variable. > >> > >> Example: image located in splashscreen partition (MMC as raw) > >> ``` > >> splashsource=mmc_raw > >> splashdevpart=0#splashscreen > >> ``` > >> > >> Signed-off-by: Julien Masson <jmasson@baylibre.com> > >> --- > >> common/splash_source.c | 5 +++++ > >> 1 file changed, 5 insertions(+) > > > > Reviewed-by: Simon Glass <sjg@chromium.org> > >> > >> diff --git a/common/splash_source.c b/common/splash_source.c > >> index b4bf6f1336..1f99f44f78 100644 > >> --- a/common/splash_source.c > >> +++ b/common/splash_source.c > >> @@ -451,6 +451,7 @@ int splash_source_load(struct splash_location *locations, uint size) > >> { > >> struct splash_location *splash_location; > >> char *env_splashimage_value; > >> + char *env_splashdevpart_value; > > > > How about just 'devpar' as it is shorter and easier to read? > > > > Yes I initially follow the same "syntax" of splashimage var but I agree > it can be shorter, what name would you prefer: > - char *devpart > - char *env_devpart > - char *env_devpart_value > ? devpart seems good to me > > >> u32 bmp_load_addr; > >> > >> env_splashimage_value = env_get("splashimage"); > >> @@ -467,6 +468,10 @@ int splash_source_load(struct splash_location *locations, uint size) > >> if (!splash_location) > >> return -EINVAL; > >> > >> + env_splashdevpart_value = env_get("splashdevpart"); > >> + if (env_splashdevpart_value) > >> + splash_location->devpart = env_splashdevpart_value; > >> + > >> if (splash_location->flags == SPLASH_STORAGE_RAW) > >> return splash_load_raw(splash_location, bmp_load_addr); > >> else if (splash_location->flags == SPLASH_STORAGE_FS) > >> -- > >> 2.37.3 > >> Regards, Simon ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [RESEND PATCH 2/2] splash: get devpart from environment variable 2022-10-14 15:55 ` Simon Glass @ 2022-10-17 7:50 ` Julien Masson 0 siblings, 0 replies; 10+ messages in thread From: Julien Masson @ 2022-10-17 7:50 UTC (permalink / raw) To: Simon Glass; +Cc: u-boot Hi Simon, On Mon 17 Oct 2022 at 09:49, Simon Glass <sjg@chromium.org> wrote: > Hi Julien, > > > On Thu, 13 Oct 2022 at 09:51, Julien Masson <jmasson@baylibre.com> wrote: >> >> Hi Simon, >> >> Thanks for the review. >> >> On Thu 13 Oct 2022 at 17:44, Simon Glass <sjg@chromium.org> wrote: >> >>> Hi Julien, >>> >>> On Wed, 12 Oct 2022 at 05:38, Julien Masson <jmasson@baylibre.com> wrote: >>>> >>>> By default several types of splash locations are supported and the >>>> user can select one of them through environment var (splashsource). >>>> >>>> However the devpart is still hardcoded and we cannot change it from >>>> the environment. >>>> >>>> This patch add the support of "splashdevpart" which allow the user to >>>> set the devpart though this environment variable. >>>> >>>> Example: image located in splashscreen partition (MMC as raw) >>>> ``` >>>> splashsource=mmc_raw >>>> splashdevpart=0#splashscreen >>>> ``` >>>> >>>> Signed-off-by: Julien Masson <jmasson@baylibre.com> >>>> --- >>>> common/splash_source.c | 5 +++++ >>>> 1 file changed, 5 insertions(+) >>> >>> Reviewed-by: Simon Glass <sjg@chromium.org> >>>> >>>> diff --git a/common/splash_source.c b/common/splash_source.c >>>> index b4bf6f1336..1f99f44f78 100644 >>>> --- a/common/splash_source.c >>>> +++ b/common/splash_source.c >>>> @@ -451,6 +451,7 @@ int splash_source_load(struct splash_location *locations, uint size) >>>> { >>>> struct splash_location *splash_location; >>>> char *env_splashimage_value; >>>> + char *env_splashdevpart_value; >>> >>> How about just 'devpar' as it is shorter and easier to read? >>> >> >> Yes I initially follow the same "syntax" of splashimage var but I agree >> it can be shorter, what name would you prefer: >> - char *devpart >> - char *env_devpart >> - char *env_devpart_value >> ? > > devpart seems good to me Got it thanks, I'll change that in v2. > > >> >>>> u32 bmp_load_addr; >>>> >>>> env_splashimage_value = env_get("splashimage"); >>>> @@ -467,6 +468,10 @@ int splash_source_load(struct splash_location *locations, uint size) >>>> if (!splash_location) >>>> return -EINVAL; >>>> >>>> + env_splashdevpart_value = env_get("splashdevpart"); >>>> + if (env_splashdevpart_value) >>>> + splash_location->devpart = env_splashdevpart_value; >>>> + >>>> if (splash_location->flags == SPLASH_STORAGE_RAW) >>>> return splash_load_raw(splash_location, bmp_load_addr); >>>> else if (splash_location->flags == SPLASH_STORAGE_FS) >>>> -- >>>> 2.37.3 >>>> > Regards, > Simon -- Julien Masson ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2022-10-17 7:50 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2022-10-12 11:38 [RESEND PATCH 0/2] splash: add more options/support Julien Masson 2022-10-12 11:38 ` [RESEND PATCH 1/2] splash: support raw image from MMC Julien Masson 2022-10-12 12:59 ` Simon Glass 2022-10-13 16:08 ` Julien Masson 2022-10-14 15:55 ` Simon Glass 2022-10-12 11:38 ` [RESEND PATCH 2/2] splash: get devpart from environment variable Julien Masson 2022-10-12 12:59 ` Simon Glass 2022-10-13 15:51 ` Julien Masson 2022-10-14 15:55 ` Simon Glass 2022-10-17 7:50 ` Julien Masson
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox