On 02/19/2019 08:38 AM, Ibai Erkiaga Elorza wrote:
Hi Alexander,
-----Original Message-----
From: Alexander Graf <ag...@suse.de>
Sent: 18 February 2019 13:50
To: Ibai Erkiaga Elorza <ib...@xilinx.com>; u-boot@lists.denx.de
Cc: 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: [PATCH] arm: fix hvc call
On 18.02.19 14:10, Ibai Erkiaga wrote:
HVC call makes use of 6 arguments rather than 7 in the same way as SMC
calls. The 7th argument is optional for both HVC and SMC but is
implemented as 16-bit parameter and register R7 or W7.
The patch description is lacking a bit more context. Why not change SMC to also
include x7?
Definitively both approaches might be valid. The aim of the patch is just avoid
compilation error due to an invalid asm code in the HVC call so that's why I'm
removing the 7th argument. The Client ID (7th argument) could be used either
for SMC or HVC but I don't have a testcase for it and seems that nobody is
using up to date.
All of the above should be mentioned in the commit message, so that
someone stumbling over the patch in 2 years will understand its rationale.
Alex
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
