From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:52119) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aIUDy-0007ns-HV for qemu-devel@nongnu.org; Sun, 10 Jan 2016 23:36:59 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aIUDx-0004Q4-Iv for qemu-devel@nongnu.org; Sun, 10 Jan 2016 23:36:58 -0500 Received: from ozlabs.org ([2401:3900:2:1::2]:52086) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aIUDx-0004PB-5j for qemu-devel@nongnu.org; Sun, 10 Jan 2016 23:36:57 -0500 Date: Mon, 11 Jan 2016 13:45:54 +1100 From: David Gibson Message-ID: <20160111024554.GD22925@voom.redhat.com> References: <1452093205-30167-1-git-send-email-eric.auger@linaro.org> <1452093205-30167-8-git-send-email-eric.auger@linaro.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="bajzpZikUji1w+G9" Content-Disposition: inline In-Reply-To: <1452093205-30167-8-git-send-email-eric.auger@linaro.org> Subject: Re: [Qemu-devel] [PATCH v2 7/7] hw/arm/sysbus-fdt: remove qemu_fdt_setprop returned value check List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eric Auger Cc: b.reynal@virtualopensystems.com, peter.maydell@linaro.org, thuth@redhat.com, eric.auger@st.com, patches@linaro.org, crosthwaitepeter@gmail.com, qemu-devel@nongnu.org, alex.williamson@redhat.com, qemu-arm@nongnu.org, suravee.suthikulpanit@amd.com, pbonzini@redhat.com, thomas.lendacky@amd.com, alex.bennee@linaro.org, christoffer.dall@linaro.org --bajzpZikUji1w+G9 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Jan 06, 2016 at 03:13:25PM +0000, Eric Auger wrote: > qemu_fdt_setprop self-exists in case of error hence no need to check > the returned value. >=20 > Signed-off-by: Eric Auger This change is fine, but in general I'm disinclined to invest too much in the qemu interfaces for manipulating flattened trees. I think our device tree manipulation in qemu is now complicated enough that we should move towards using an unflattened (i.e. pointer based) DT representation inside qemu, which is generally more suitable for complex manipulation. That would then get flattened into a blob for the guest in a single pass at reset time. --=20 David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson --bajzpZikUji1w+G9 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJWkxdiAAoJEGw4ysog2bOSE28P/1v3FkntvmqnDNNZ2yS0GiKJ QYB9oP1BP5ZvuzTGcjPP41tSTMCbUy5FqblZz97bIHbxNdIIpcf+YyChruNkRz2D gqrJ8Efa4E7A7rcNElzPSqXckv8oV20Iq45MfnMfa0Hw5suYY+VDK26nI/X+khcH xH1a65Kr9q9Tpe3sPI5g9FgACRhDgR1xA8KBik95sCRvnLWFElASTxv9Ci9HXHrb ixg322mLn7v/1hoZHzJcuI1ZZOfm/Yhg13zKLct9VlO6Kssz8yV9HdCeJ/YcWm3D rbt3clJRjIOkkWcOl9sS98ZE2a/o5G9uLWVJz7BjOv6TeiFLEAv1DOHirZNZu8lD 5ZHiIzV2k42jWKBNAezfc58hjbgOtZCYuESH1QJ9goo/i/etz1H2hL8ZAIHwVPHA 6AZaKxt7PIgp2mV/QBlRmFwuB+RbZAcAzaXWedPXearEpxDzpXeaIJkOdsrJdun7 DmqYkU9b8aN3kR2HbrgKP4pGx6c5sSBckiwLsMEbrPwXxGkkZ2gWzQbsXzTqfG5R hd/y+dQsnvBRgRvo0pJdjbypV49kjsOC+VXIArmY46AvnhrUjzacEWj19UvFXBsL A0G8gilgvnENFKE8Pwn4sDgdPEzlr8wcY5ZmftqoGAaTIdZ7YuRS/IochJsSCxWo 26vNk+U2sMv3iC7sjsWG =OXVK -----END PGP SIGNATURE----- --bajzpZikUji1w+G9--