From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:60631) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yv3o6-0000F7-22 for qemu-devel@nongnu.org; Wed, 20 May 2015 09:13:10 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Yv3o1-0002ZO-Rk for qemu-devel@nongnu.org; Wed, 20 May 2015 09:13:10 -0400 Received: from mailapp01.imgtec.com ([195.59.15.196]:1369) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yv3o1-0002ZD-Lf for qemu-devel@nongnu.org; Wed, 20 May 2015 09:13:05 -0400 Message-ID: <555C87DE.8060908@imgtec.com> Date: Wed, 20 May 2015 14:10:54 +0100 From: Leon Alrae MIME-Version: 1.0 References: <1431085311-24617-1-git-send-email-leon.alrae@imgtec.com> <1431085311-24617-3-git-send-email-leon.alrae@imgtec.com> <555C41BF.3070900@imgtec.com> <555C4B06.6060608@imgtec.com> In-Reply-To: Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v3 2/2] semihosting: add --semihosting-config arg sub-argument List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Liviu Ionescu Cc: Peter Maydell , Christopher Covington , QEMU Developers , Matthew Fortune On 20/05/2015 12:12, Liviu Ionescu wrote: > yes, that's correct, my comment was related to the above quotes from your message, in the first part you mention "the path won't be passed to semihosting", which is not consistent with the actual behaviour, since the entire kernel path is passed unchanged. Ah, I see. Yes, that's a good point; I'm going to address this in v4. > > if you update the manual text, please rephrase to make the text easier to understand and remove this notice related to no-path kernel. I rephrased it a bit (but please remember that "easier to understand" is very subjective): "Allows the user to pass input arguments, and can be used multiple times to build up a list. The old-style -kernel/-append method of passing a command line is still supported for backward compatibility. If both the --semihosting-config arg and the -kernel/-append are specified, the former is passed to semihosting as it always takes precedence." Thanks, Leon