From: 정재훈 <jh80.chung@samsung.com>
To: "'Simon Glass'" <sjg@chromium.org>,
"'U-Boot Mailing List'" <u-boot@lists.denx.de>
Cc: "'Tom Rini'" <trini@konsulko.com>,
"'Andrew Scull'" <ascull@google.com>,
"'Heinrich Schuchardt'" <xypron.glpk@gmx.de>,
"'Marek Vasut'" <marex@denx.de>,
"'Mark Kettenis'" <kettenis@openbsd.org>,
"'Pavel Herrmann'" <morpheus.ibis@gmail.com>,
"'Peng Fan'" <peng.fan@nxp.com>, "'Stefan Roese'" <sr@denx.de>,
"'Ye Li'" <ye.li@nxp.com>
Subject: RE: [PATCH v2 21/45] dm: blk: mmc: Tidy up some Makefile rules for SPL
Date: Mon, 17 Oct 2022 14:50:51 +0900 [thread overview]
Message-ID: <112201d8e1ec$6a760300$3f620900$@samsung.com> (raw)
In-Reply-To: <20221013122927.636867-22-sjg@chromium.org>
> -----Original Message-----
> From: Simon Glass [mailto:sjg@chromium.org]
> Sent: Thursday, October 13, 2022 9:29 PM
> To: U-Boot Mailing List <u-boot@lists.denx.de>
> Cc: Tom Rini <trini@konsulko.com>; Simon Glass <sjg@chromium.org>; Andrew Scull
> <ascull@google.com>; Heinrich Schuchardt <xypron.glpk@gmx.de>; Jaehoon Chung
> <jh80.chung@samsung.com>; Marek Vasut <marex@denx.de>; Mark Kettenis <kettenis@openbsd.org>; Pavel
> Herrmann <morpheus.ibis@gmail.com>; Peng Fan <peng.fan@nxp.com>; Stefan Roese <sr@denx.de>; Ye Li
> <ye.li@nxp.com>
> Subject: [PATCH v2 21/45] dm: blk: mmc: Tidy up some Makefile rules for SPL
>
> Use the correct SPL_TPL_ variable so that these features can be enabled in
> TPL and VPL as needed.
>
> Disable it by default in TPL to avoid any code-size increase. No boards
> are actually using it since the Makefile rules don't allow including
> drivers/block/ with TPL_DM enabled. It can be manually enabled as needed.
>
> Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
Best Regards,
Jaehoon Chung
> ---
>
> Changes in v2:
> - Disable TPL_BLK by default
>
> drivers/Makefile | 2 +-
> drivers/block/Kconfig | 1 -
> drivers/block/Makefile | 2 +-
> drivers/mmc/Makefile | 2 +-
> 4 files changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/Makefile b/drivers/Makefile
> index 9d9f69a3c9a..a1700c819de 100644
> --- a/drivers/Makefile
> +++ b/drivers/Makefile
> @@ -1,5 +1,6 @@
> # SPDX-License-Identifier: GPL-2.0+
>
> +obj-$(CONFIG_$(SPL_TPL_)BLK) += block/
> obj-$(CONFIG_$(SPL_TPL_)BOOTCOUNT_LIMIT) += bootcount/
> obj-$(CONFIG_$(SPL_TPL_)BUTTON) += button/
> obj-$(CONFIG_$(SPL_TPL_)CACHE) += cache/
> @@ -60,7 +61,6 @@ obj-$(CONFIG_SPL_USB_HOST) += usb/host/
> obj-$(CONFIG_OMAP_USB_PHY) += usb/phy/
> obj-$(CONFIG_SPL_SATA) += ata/ scsi/
> obj-$(CONFIG_SPL_LEGACY_BLOCK) += block/
> -obj-$(CONFIG_SPL_BLK) += block/
> obj-$(CONFIG_SPL_THERMAL) += thermal/
>
> endif
> diff --git a/drivers/block/Kconfig b/drivers/block/Kconfig
> index 707e2bcd238..e95da48bdc0 100644
> --- a/drivers/block/Kconfig
> +++ b/drivers/block/Kconfig
> @@ -37,7 +37,6 @@ config SPL_BLK
> config TPL_BLK
> bool "Support block devices in TPL"
> depends on TPL_DM && BLK
> - default y
> help
> Enable support for block devices, such as SCSI, MMC and USB
> flash sticks. These provide a block-level interface which permits
> diff --git a/drivers/block/Makefile b/drivers/block/Makefile
> index f48d3e12140..19d9317c825 100644
> --- a/drivers/block/Makefile
> +++ b/drivers/block/Makefile
> @@ -3,7 +3,7 @@
> # (C) Copyright 2000-2007
> # Wolfgang Denk, DENX Software Engineering, wd@denx.de.
>
> -obj-$(CONFIG_$(SPL_)BLK) += blk-uclass.o
> +obj-$(CONFIG_$(SPL_TPL_)BLK) += blk-uclass.o
>
> ifndef CONFIG_$(SPL_)BLK
> obj-$(CONFIG_SPL_LEGACY_BLOCK) += blk_legacy.o
> diff --git a/drivers/mmc/Makefile b/drivers/mmc/Makefile
> index 5d5104bbf1d..3a664c2ebbb 100644
> --- a/drivers/mmc/Makefile
> +++ b/drivers/mmc/Makefile
> @@ -10,7 +10,7 @@ ifdef CONFIG_$(SPL_TPL_)DM_MMC
> obj-$(CONFIG_$(SPL_TPL_)BOOTSTD) += mmc_bootdev.o
> endif
>
> -obj-$(CONFIG_$(SPL_)MMC_WRITE) += mmc_write.o
> +obj-$(CONFIG_$(SPL_TPL_)MMC_WRITE) += mmc_write.o
> obj-$(CONFIG_MMC_PWRSEQ) += mmc-pwrseq.o
> obj-$(CONFIG_MMC_SDHCI_ADMA_HELPERS) += sdhci-adma.o
>
> --
> 2.38.0.rc1.362.ged0d419d3c-goog
next prev parent reply other threads:[~2022-10-17 5:51 UTC|newest]
Thread overview: 54+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-13 12:28 [PATCH v2 00/45] vbe: Implement the full firmware flow Simon Glass
2022-10-13 12:28 ` [PATCH v2 03/45] Rename CONFIG_SYS_TEXT_BASE to CONFIG_TEXT_BASE Simon Glass
2022-10-13 12:28 ` [PATCH v2 04/45] disk: Drop debug messages in part_efi Simon Glass
2022-10-13 15:59 ` Heinrich Schuchardt
2022-10-13 12:28 ` [PATCH v2 05/45] spl: Use binman suffix allow symbols of any SPL etype Simon Glass
2022-10-13 12:28 ` [PATCH v2 06/45] spl: Refactor controls for console output Simon Glass
2022-10-13 12:28 ` [PATCH v2 07/45] spl: Add a separate silence option for SPL Simon Glass
2022-10-13 12:28 ` [PATCH v2 08/45] binman: Allow obtaining a symbol value Simon Glass
2022-10-13 12:28 ` [PATCH v2 09/45] binman: Split out looking up a symbol into a function Simon Glass
2022-10-13 12:28 ` [PATCH v2 10/45] binman: Handle writing ELF symbols in the Entry class Simon Glass
2022-10-13 12:28 ` [PATCH v2 11/45] binman: Support writing symbols into ELF files Simon Glass
2022-10-13 12:28 ` [PATCH v2 12/45] test: Report skippped tests Simon Glass
2022-10-13 16:05 ` Heinrich Schuchardt
2022-10-13 12:28 ` [PATCH v2 13/45] test: Update tests to use the skip feature Simon Glass
2022-10-13 12:28 ` [PATCH v2 14/45] test: Support tests which can only be run manually Simon Glass
2022-10-13 16:14 ` Heinrich Schuchardt
2022-10-14 15:55 ` Simon Glass
2022-10-13 12:28 ` [PATCH v2 15/45] bootstd: Add a way to set up a bootflow Simon Glass
2022-10-13 12:28 ` [PATCH v2 16/45] disk: Rename block functions Simon Glass
2022-10-13 16:17 ` Heinrich Schuchardt
2022-10-13 12:28 ` [PATCH v2 17/45] disk: Rename block_dev to desc Simon Glass
2022-10-13 16:18 ` Heinrich Schuchardt
2022-10-13 12:29 ` [PATCH v2 18/45] dm: blk: Add udevice functions Simon Glass
2022-10-13 12:29 ` [PATCH v2 19/45] usb: Update the test to cover reading and writing Simon Glass
2022-10-13 12:29 ` [PATCH v2 20/45] bloblist: Drop debugging Simon Glass
2022-10-13 12:29 ` [PATCH v2 21/45] dm: blk: mmc: Tidy up some Makefile rules for SPL Simon Glass
2022-10-17 5:50 ` 정재훈 [this message]
2022-10-13 12:29 ` [PATCH v2 22/45] dm: mmc: Allow sandbox emulator to build without writes Simon Glass
2022-10-17 5:51 ` 정재훈
2022-10-13 12:29 ` [PATCH v2 23/45] sandbox: Drop message about writing sandbox state Simon Glass
2022-10-13 12:29 ` [PATCH v2 24/45] image: Move comment for fit_conf_find_compat() Simon Glass
2022-10-13 12:29 ` [PATCH v2 25/45] sandbox: Generalise SPL booting Simon Glass
2022-10-13 12:29 ` [PATCH v2 26/45] sandbox: Add a way to specify the sandbox executable Simon Glass
2022-10-13 12:29 ` [PATCH v2 27/45] image: Add the concept of a phase to FIT Simon Glass
2022-10-13 12:29 ` [PATCH v2 28/45] image: Allow loading a FIT image for a particular phase Simon Glass
2022-10-13 12:29 ` [PATCH v2 29/45] image: Correct strncpy() warning with image_set_name() Simon Glass
2022-10-13 16:23 ` Heinrich Schuchardt
2022-10-14 15:55 ` Simon Glass
2022-10-13 12:29 ` [PATCH v2 30/45] vbe: Rename vbe_fixup to vbe_request Simon Glass
2022-10-13 12:29 ` [PATCH v2 31/45] vbe: Use a warning for a failed requests Simon Glass
2022-10-13 12:29 ` [PATCH v2 32/45] spl: Allow multiple loaders of the same time Simon Glass
2022-10-13 12:29 ` [PATCH v2 33/45] sandbox: Support obtaining the next phase from an image Simon Glass
2022-10-13 12:29 ` [PATCH v2 34/45] vbe: Support selecting operations by SPL phase Simon Glass
2022-10-13 12:29 ` [PATCH v2 35/45] vbe: Support reading the next SPL phase via VBE Simon Glass
2022-10-13 12:29 ` [PATCH v2 36/45] vbe: Move OS implementation into a separate file Simon Glass
2022-10-13 12:29 ` [PATCH v2 37/45] vbe: Drop the U-Boot prefix from the version Simon Glass
2022-10-13 12:29 ` [PATCH v2 38/45] vbe: Add Kconfig options for VPL Simon Glass
2022-10-13 12:29 ` [PATCH v2 39/45] vbe: Add info about the VBE device to the fwupd node Simon Glass
2022-10-13 12:29 ` [PATCH v2 40/45] sandbox: Add an image for VPL Simon Glass
2022-10-13 12:29 ` [PATCH v2 41/45] vpl: Allow signature verification Simon Glass
2022-10-13 12:29 ` [PATCH v2 42/45] vbe: Use a manual test Simon Glass
2022-10-13 12:29 ` [PATCH v2 43/45] vbe: Record which phases loaded using VBE Simon Glass
2022-10-13 12:29 ` [PATCH v2 44/45] vbe: Add a command to show the VBE state Simon Glass
2022-10-13 12:29 ` [PATCH v2 45/45] vbe: Add a test for the VBE flow into U-Boot proper Simon Glass
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='112201d8e1ec$6a760300$3f620900$@samsung.com' \
--to=jh80.chung@samsung.com \
--cc=ascull@google.com \
--cc=kettenis@openbsd.org \
--cc=marex@denx.de \
--cc=morpheus.ibis@gmail.com \
--cc=peng.fan@nxp.com \
--cc=sjg@chromium.org \
--cc=sr@denx.de \
--cc=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
--cc=xypron.glpk@gmx.de \
--cc=ye.li@nxp.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