Hi Simon, > -----Original Message----- > From: Simon Glass <s...@chromium.org> > Sent: 10 March 2019 21:51 > To: Ibai Erkiaga Elorza <ib...@xilinx.com> > Cc: U-Boot Mailing List <u-boot@lists.denx.de>; Patrick Delaunay > <patrick.delau...@st.com>; Andy Shevchenko > <andriy.shevche...@linux.intel.com>; Bin Meng <bmeng...@gmail.com>; > Patrice Chotard <patrice.chot...@st.com> > Subject: Re: [U-Boot][PATCH] dm: check OF_LIVE is enabled > > Hi Ibai, > > On Tue, 26 Feb 2019 at 02:26, Ibai Erkiaga <ibai.erkiaga-elo...@xilinx.com> > wrote: > > > > Livetree implemented functions does not have conditional compilation > > so check if CONFIG_IS_ENABLED prior using those functions. > > > > The issue does not report any error in a normal build as the toolchain > > optimize the code. Using -O0 triggers the error so the patch is > > intended to fix issues on a ongoing effor to build U-Boot with -O0. > > > > Signed-off-by: Ibai Erkiaga <ibai.erkiaga-elo...@xilinx.com> > > --- > > > > drivers/core/ofnode.c | 60 > > +++++++++++++++++++++--------------------- > > drivers/serial/serial-uclass.c | 2 +- > > 2 files changed, 31 insertions(+), 31 deletions(-) > > > > This is supposed to work by using of_live_active(), which is called from > ofnode_is_np(). Instead of changing all this code, is it possible to update > of_live_active() somehow? >
I've been trying to figure out it but I think there is no way just changing of_live_active as the compiler does not discard branch statements with nested fixed return values. I was able to make it work just changing ofnode_is_np using pre-processor macro to set as false when OF_LIVE is not enabled, but not sure if it the right approach. I will take a deeper look to the entire OF code and maybe suggest a code refactoring. > Regards, > Simon Regards Ibai _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
