Hi Tom > -----Original Message----- > From: Tom Rini <tr...@konsulko.com> > Sent: 15 March 2019 16:08 > To: Ibai Erkiaga Elorza <ib...@xilinx.com> > Cc: Alexander Graf <ag...@suse.de>; u-boot@lists.denx.de; Sumit Garg > <sumit.g...@linaro.org>; Heinrich Schuchardt <xypron.g...@gmx.de>; Albert > Aribaud <albert.u.b...@aribaud.net> > Subject: Re: [U-Boot][PATCH v2] arm: fix hvc call > > On Fri, Mar 15, 2019 at 12:13:11PM +0000, Ibai Erkiaga Elorza wrote: > > Hi Alex, > > > > > -----Original Message----- > > > From: Alexander Graf <ag...@suse.de> > > > Sent: 25 February 2019 13:09 > > > To: Ibai Erkiaga Elorza <ib...@xilinx.com> > > > Cc: u-boot@lists.denx.de; Sumit Garg <sumit.g...@linaro.org>; > > > Heinrich Schuchardt <xypron.g...@gmx.de>; Tom Rini > > > <tr...@konsulko.com>; Albert Aribaud <albert.u.b...@aribaud.net> > > > Subject: Re: [U-Boot][PATCH v2] arm: fix hvc call > > > > > > > > > > > > > Am 25.02.2019 um 02:11 schrieb Ibai Erkiaga <ibai.erkiaga- > > > elo...@xilinx.com>: > > > > > > > > HVC call makes use of 6 mandatory arguments rather than 7 in the > > > > same way as SMC calls. The 7th argument is optional (Client ID) > > > > for both HVC and SMC but is implemented as 16-bit parameter and > > > > register R7 or W7. The aim of this patch is just fix compilation > > > > error due to an invalid asm code in the HVC call so that's why the 7th > argument is removed. > > > > > > > > The issue does not report any error in a normal build as hvc_call > > > > is not used at all and is optimized by the compiler. 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> > > > > > > Reviewed-by: Alexander Graf <ag...@suse.de> > > > > Thanks for the review. Do I need to CC somebody else to the patch be > accepted? > > No, but I'm inclined currently to take for for the next release rather than > right > now, unless there's an important argument I'm missing? Which happens more > than I'd care to admit ;) > No rush, I'm pretty new contributor and just wondering if something else was missing after Alexander's Review.
> -- > Tom Ibai _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
