On Sat, Oct 07, 2023 at 05:12:27PM -0600, Simon Glass wrote: > This code is normally compiled for sifive, but sandbox can also compile > it. We should not use UNIT_TEST as a synonym for SANDBOX, since it is > possible to disable UNIT_TEST for sandbox. > > Correct the condition. > > Signed-off-by: Simon Glass <[email protected]> > --- > > (no changes since v1) > > include/k210/pll.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/k210/pll.h b/include/k210/pll.h > index fd16a89cb203..6dd60b2eb4fc 100644 > --- a/include/k210/pll.h > +++ b/include/k210/pll.h > @@ -13,7 +13,7 @@ struct k210_pll_config { > u8 od; > }; > > -#ifdef CONFIG_UNIT_TEST > +#ifdef CONFIG_SANDBOX > TEST_STATIC int k210_pll_calc_config(u32 rate, u32 rate_in, > struct k210_pll_config *best); > #ifndef nop
Sean just explained that this is wrong. -- Tom
signature.asc
Description: PGP signature
