public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Felix Brack <fb@ltec.ch>
To: Simon Glass <sjg@chromium.org>,
	U-Boot Mailing List <u-boot@lists.denx.de>
Cc: "Tom Rini" <trini@konsulko.com>, "Marek Vasut" <marex@denx.de>,
	"Marcel Ziswiler" <marcel.ziswiler@toradex.com>,
	"Martyn Welch" <martyn.welch@collabora.com>,
	"Svyatoslav Ryhel" <clamor95@gmail.com>,
	"Adam Ford" <aford173@gmail.com>,
	"Alison Wang" <alison.wang@nxp.com>,
	"Allen Martin" <amartin@nvidia.com>,
	"Ariel D'Alessandro" <ariel.dalessandro@collabora.com>,
	"Chee Hong Ang" <chee.hong.ang@intel.com>,
	"Chin-Liang See" <clsee@altera.com>,
	"Dave Gerlach" <d-gerlach@ti.com>,
	"Dinh Nguyen" <dinguyen@kernel.org>,
	"Fabio Estevam" <festevam@denx.de>,
	"Green Wan" <green.wan@sifive.com>,
	"Heiko Thiery" <heiko.thiery@gmail.com>,
	"Ian Ray" <ian.ray@ge.com>,
	"Jaehoon Chung" <jh80.chung@samsung.com>,
	"Jan Kiszka" <jan.kiszka@siemens.com>,
	"Javier Martínez Canillas" <javier@dowhile0.org>,
	"Leo Yu-Chi Liang" <ycliang@andestech.com>,
	"Michal Simek" <michal.simek@amd.com>,
	"Mike Looijmans" <mike.looijmans@topic.nl>,
	"Nicolas Ferre" <nicolas.ferre@microchip.com>,
	"Oleksandr Suvorov" <oleksandr.suvorov@foundries.io>,
	"Oliver Graute" <oliver.graute@kococonnector.com>,
	"Palmer Dabbelt" <palmer@dabbelt.com>,
	"Parthiban Nallathambi" <parthiban@linumiz.com>,
	"Paul Walmsley" <paul.walmsley@sifive.com>,
	Peter.Chubb@data61.csiro.au, "Philip Oberfichtner" <pro@denx.de>,
	"Quentin Schulz" <quentin.schulz@theobroma-systems.com>,
	"Raffaele RECALCATI" <raffaele.recalcati@bticino.it>,
	"Roger Quadros" <rogerq@kernel.org>, "Stefan Roese" <sr@denx.de>,
	"Teresa Remmet" <t.remmet@phytec.de>,
	"Tianrui Wei" <tianrui-wei@outlook.com>,
	"Tim Harvey" <tharvey@gateworks.com>,
	"Uri Mashiach" <uri.mashiach@compulab.co.il>,
	"Yanhong Wang" <yanhong.wang@starfivetech.com>,
	kernel@puri.sm, uboot-stm32@st-md-mailman.stormreply.com
Subject: Re: [PATCH v3 01/38] spl: Use CONFIG_SPL... instead of CONFIG_..._SPL_...
Date: Mon, 25 Sep 2023 16:55:52 +0200	[thread overview]
Message-ID: <2a727cd4-bbda-409e-8d0e-1aafba182873@ltec.ch> (raw)
In-Reply-To: <20230924192536.1812799-2-sjg@chromium.org>

Hi Simon,

On 24.09.23 21:24, Simon Glass wrote:
> We like to put the SPL first so it is clear that it relates to SPL. Rename
> various malloc-related options which have crept in, to stick to this
> convention.
> 
> Signed-off-by: Simon Glass <sjg@chromium.org>
> Reviewed-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> Reviewed-by: Martyn Welch <martyn.welch@collabora.com>
> Reviewed-by: Svyatoslav Ryhel <clamor95@gmail.com>
> ---
> 
> (no changes since v1)
> .....

I just tried to apply the patch to master for testing and got the
following result:

error: patch failed: configs/am335x_baltos_defconfig:18
error: configs/am335x_baltos_defconfig: patch does not apply
error: patch failed: configs/am335x_igep003x_defconfig:20
error: configs/am335x_igep003x_defconfig: patch does not apply
error: patch failed: configs/dh_imx6_defconfig:39
error: configs/dh_imx6_defconfig: patch does not apply
error: patch failed: configs/starfive_visionfive2_defconfig:47
error: configs/starfive_visionfive2_defconfig: patch does not apply

For what I want to test (changes for PDU001 board) this doesn't matter
however. I'm just not sure: should I go ahead with testing or wait for
an updated patch?

regards Felix

  reply	other threads:[~2023-09-25 15:01 UTC|newest]

Thread overview: 50+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-09-24 19:24 [PATCH v3 00/38] spl: Preparation for Universal Payload Simon Glass
2023-09-24 19:24 ` [PATCH v3 01/38] spl: Use CONFIG_SPL... instead of CONFIG_..._SPL_ Simon Glass
2023-09-25 14:55   ` Felix Brack [this message]
2023-09-25 16:08     ` Simon Glass
2023-10-09 16:15   ` Felix Brack
2023-09-24 19:24 ` [PATCH v3 02/38] spl: Rename SYS_SPL_ARGS_ADDR to SPL_PAYLOAD_ARGS_ADDR Simon Glass
2023-09-24 19:24 ` [PATCH v3 03/38] spl: Avoid #ifdef with CONFIG_SPL_SYS_MALLOC Simon Glass
2023-09-24 19:24 ` [PATCH v3 04/38] spl: mx6: powerpc: Drop the condition on timer_init() Simon Glass
2023-09-26  6:45   ` Christophe Leroy
2023-09-27 14:19     ` Simon Glass
2023-09-24 19:24 ` [PATCH v3 05/38] spl: Drop #ifdefs for BOARD_INIT and watchdog Simon Glass
2023-09-24 19:24 ` [PATCH v3 06/38] spl: Avoid #ifdef with CONFIG_SPL_PAYLOAD_ARGS_ADDR Simon Glass
2023-09-24 19:24 ` [PATCH v3 07/38] spl: Drop the switch() statement for OS selection Simon Glass
2023-09-24 19:24 ` [PATCH v3 08/38] spl: Create proper symbols for enabling the malloc() pool Simon Glass
2023-09-25 19:46   ` Tom Rini
2023-09-24 19:24 ` [PATCH v3 09/38] spl: Enable CONFIG_SPL_SYS_MALLOC_F where needed Simon Glass
2023-09-24 19:24 ` [PATCH v3 10/38] tpl: Enable CONFIG_TPL_SYS_MALLOC_F " Simon Glass
2023-09-24 19:24 ` [PATCH v3 11/38] spl: Use SYS_MALLOC_F instead of SYS_MALLOC_F_LEN Simon Glass
2023-09-24 19:24 ` [PATCH v3 12/38] Tidy up uses of CONFIG_SYS_MALLOC_F_LEN Simon Glass
2023-09-24 19:24 ` [PATCH v3 13/38] doc: Clean up SYS_MALLOC_SIMPLE Simon Glass
2023-09-24 19:24 ` [PATCH v3 14/38] dm: core: Correct help in TPL_DM and VPL_DM Simon Glass
2023-09-24 19:25 ` [PATCH v3 15/38] spl: Avoid an #ifdef when printing gd->malloc_ptr Simon Glass
2023-09-24 19:25 ` [PATCH v3 16/38] spl: Remove #ifdefs with BOOTSTAGE Simon Glass
2023-09-25 19:56   ` Tom Rini
2023-09-24 19:25 ` [PATCH v3 17/38] spl: Rename spl_load_fit_image() to load_simple_fit() Simon Glass
2023-09-24 19:25 ` [PATCH v3 18/38] spl: Move the full FIT code to spl_fit.c Simon Glass
2023-09-24 19:25 ` [PATCH v3 19/38] spl: Use the correct FIT_..._PROP constants Simon Glass
2023-09-24 19:25 ` [PATCH v3 20/38] spl: Move bloblist writing until the image is known Simon Glass
2023-09-24 19:25 ` [PATCH v3 21/38] dm: core: Reverse the argument order in ofnode_copy_props() Simon Glass
2023-09-24 19:25 ` [PATCH v3 22/38] dm: core: Ensure we run flattree tests on ofnode Simon Glass
2023-09-24 19:25 ` [PATCH v3 23/38] dm: core: Tidy up comments in the ofnode tests Simon Glass
2023-09-24 19:25 ` [PATCH v3 24/38] dm: core: Add a function to create an empty tree Simon Glass
2023-09-24 19:25 ` [PATCH v3 25/38] dm: core: Add a way to copy a node Simon Glass
2023-09-24 19:25 ` [PATCH v3 26/38] dm: core: Add a way to delete " Simon Glass
2023-09-24 19:25 ` [PATCH v3 27/38] dm: core: Add a way to convert a devicetree to a dtb Simon Glass
2023-09-24 19:25 ` [PATCH v3 28/38] dm: core: Support writing a boolean Simon Glass
2023-09-24 19:25 ` [PATCH v3 29/38] dm: core: Support writing a 64-bit value Simon Glass
2023-09-24 19:25 ` [PATCH v3 30/38] dm: core: Add tests for oftree_path() Simon Glass
2023-09-24 19:25 ` [PATCH v3 31/38] sandbox: Move reading the RAM buffer into a better place Simon Glass
2023-09-24 19:25 ` [PATCH v3 32/38] sandbox: Init the EC properly even if no state file is available Simon Glass
2023-09-24 19:25 ` [PATCH v3 33/38] sandbox: Only read the state if we have a state file Simon Glass
2023-09-24 19:25 ` [PATCH v3 34/38] sandbox: Move the bloblist down a little in memory Simon Glass
2023-09-24 19:25 ` [PATCH v3 35/38] bloblist: Support initing from multiple places Simon Glass
2023-09-24 19:25 ` [PATCH v3 36/38] fdt: Allow the devicetree to come from a bloblist Simon Glass
2023-09-24 19:25 ` [PATCH v3 37/38] command: Include a required header in command.h Simon Glass
2023-09-24 19:25 ` [PATCH v3 38/38] pci: serial: Support reading PCI-register size with base Simon Glass
2023-09-25 20:17   ` Tom Rini
2023-09-26 11:37     ` Simon Glass
2023-09-26 14:05       ` Tom Rini
2023-09-25 20:36 ` [PATCH v3 00/38] spl: Preparation for Universal Payload Tom Rini

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=2a727cd4-bbda-409e-8d0e-1aafba182873@ltec.ch \
    --to=fb@ltec.ch \
    --cc=Peter.Chubb@data61.csiro.au \
    --cc=aford173@gmail.com \
    --cc=alison.wang@nxp.com \
    --cc=amartin@nvidia.com \
    --cc=ariel.dalessandro@collabora.com \
    --cc=chee.hong.ang@intel.com \
    --cc=clamor95@gmail.com \
    --cc=clsee@altera.com \
    --cc=d-gerlach@ti.com \
    --cc=dinguyen@kernel.org \
    --cc=festevam@denx.de \
    --cc=green.wan@sifive.com \
    --cc=heiko.thiery@gmail.com \
    --cc=ian.ray@ge.com \
    --cc=jan.kiszka@siemens.com \
    --cc=javier@dowhile0.org \
    --cc=jh80.chung@samsung.com \
    --cc=kernel@puri.sm \
    --cc=marcel.ziswiler@toradex.com \
    --cc=marex@denx.de \
    --cc=martyn.welch@collabora.com \
    --cc=michal.simek@amd.com \
    --cc=mike.looijmans@topic.nl \
    --cc=nicolas.ferre@microchip.com \
    --cc=oleksandr.suvorov@foundries.io \
    --cc=oliver.graute@kococonnector.com \
    --cc=palmer@dabbelt.com \
    --cc=parthiban@linumiz.com \
    --cc=paul.walmsley@sifive.com \
    --cc=pro@denx.de \
    --cc=quentin.schulz@theobroma-systems.com \
    --cc=raffaele.recalcati@bticino.it \
    --cc=rogerq@kernel.org \
    --cc=sjg@chromium.org \
    --cc=sr@denx.de \
    --cc=t.remmet@phytec.de \
    --cc=tharvey@gateworks.com \
    --cc=tianrui-wei@outlook.com \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    --cc=uboot-stm32@st-md-mailman.stormreply.com \
    --cc=uri.mashiach@compulab.co.il \
    --cc=yanhong.wang@starfivetech.com \
    --cc=ycliang@andestech.com \
    /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: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox