[PATCH v7 09/15] disk: Correct the conditions for SPL

[Tom Rini]

On Thu, Apr 14, 2022 at 05:21:52PM +0900, AKASHI Takahiro wrote:
> Hi Simon,
> 
> On Mon, Feb 28, 2022 at 07:16:51AM -0700, Simon Glass wrote:
> > These filesystems may have different settings for SPL and TPL. Use the
> > correct Makefile variable to handle this.
> > 
> > Signed-off-by: Simon Glass <sjg at chromium.org>
> > ---
> > 
> > Changes in v7:
> > - Add new patch to correct the conditions for SPL
> > 
> >  disk/Makefile | 10 +++++-----
> >  1 file changed, 5 insertions(+), 5 deletions(-)
> > 
> > diff --git a/disk/Makefile b/disk/Makefile
> > index 6ce5a687b3..83f5af6069 100644
> > --- a/disk/Makefile
> > +++ b/disk/Makefile
> > @@ -6,8 +6,8 @@
> >  #ccflags-y += -DET_DEBUG -DDEBUG
> >  
> >  obj-$(CONFIG_PARTITIONS)	+= part.o
> > -obj-$(CONFIG_$(SPL_)MAC_PARTITION)   += part_mac.o
> > -obj-$(CONFIG_$(SPL_)DOS_PARTITION)   += part_dos.o
> > -obj-$(CONFIG_$(SPL_)ISO_PARTITION)   += part_iso.o
> > -obj-$(CONFIG_$(SPL_)AMIGA_PARTITION) += part_amiga.o
> > -obj-$(CONFIG_$(SPL_)EFI_PARTITION)   += part_efi.o
> > +obj-$(CONFIG_$(SPL_TPL_)MAC_PARTITION)   += part_mac.o
> > +obj-$(CONFIG_$(SPL_TPL_)DOS_PARTITION)   += part_dos.o
> > +obj-$(CONFIG_$(SPL_TPL_)ISO_PARTITION)   += part_iso.o
> > +obj-$(CONFIG_$(SPL_TPL_)AMIGA_PARTITION) += part_amiga.o
> > +obj-$(CONFIG_$(SPL_TPL_)EFI_PARTITION)   += part_efi.o
> 
> While none of CONFIG_TPL_xxx_PARTITION is defined,
> how do those rules work for TPL?

The makefile logic means that if the options are added then they would
work as intended.  With the options not being in Kconfig they'll simply
never be evaluated to y.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20220414/0530a899/attachment.sig>