On Mon, Jan 31, 2022 at 06:03:58PM +0100, Marek Vasut wrote:
> On 1/31/22 17:58, Tommaso Merciai wrote:
> > Override env_get_location function at board level, previously dropped
> > down from arch/arm/mach-imx/imx8m/soc.c
> >
> > References:
> > - commit 98af80d3c969e69a1b8ce98bb20e5ad844022da2
> >
> > Signed-off-by: Tommaso Merciai <tommaso.merc...@amarulasolutions.com>
> > ---
> > board/phytec/phycore_imx8mp/phycore-imx8mp.c | 33 ++++++++++++++++++++
> > 1 file changed, 33 insertions(+)
> >
> > diff --git a/board/phytec/phycore_imx8mp/phycore-imx8mp.c
> > b/board/phytec/phycore_imx8mp/phycore-imx8mp.c
> > index a8f0821437..05926eefa3 100644
> > --- a/board/phytec/phycore_imx8mp/phycore-imx8mp.c
> > +++ b/board/phytec/phycore_imx8mp/phycore-imx8mp.c
> > @@ -11,9 +11,42 @@
> > #include <asm/mach-imx/boot_mode.h>
> > #include <env.h>
> > #include <miiphy.h>
> > +#include <env_internal.h>
> > DECLARE_GLOBAL_DATA_PTR;
> > +enum env_location env_get_location(enum env_operation op, int prio)
> > +{
>
> Why don't you just turn this into default __weak function and override it on
> board level when it is really needed to be overridden ?
Hi Marek,
env_get_location is already declared as __weak, check env/env.c. We
can't override it 2 times.
References:
-
https://developer.arm.com/documentation/dui0491/i/Compiler-specific-Features/--weak
--
Tommaso Merciai
Embedded Linux Developer
tommaso.merc...@amarulasolutions.com
__________________________________
Amarula Solutions SRL
Via Le Canevare 30, 31100 Treviso, Veneto, IT
T. +39 042 243 5310
i...@amarulasolutions.com
www.amarulasolutions.com
