All of lore.kernel.org
 help / color / mirror / Atom feed
From: Li Liu <john.liuli@huawei.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: Peter Crosthwaite <peter.crosthwaite@xilinx.com>,
	Alexander Graf <agraf@suse.de>,
	QEMU Developers <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] [PATCH 0/6] add dumpdts ability to convert dtb to dts
Date: Tue, 26 Aug 2014 10:04:12 +0800	[thread overview]
Message-ID: <53FBEB1C.40809@huawei.com> (raw)
In-Reply-To: <CAFEAcA_gyVrVeC4KHR4DnSvwLJyrnkFtfW0D+We1PSvv57eDog@mail.gmail.com>



On 2014/8/25 20:22, Peter Maydell wrote:
> On 25 August 2014 05:00, john.liuli <john.liuli@huawei.com> wrote:
>> From: Li Liu <john.liuli@huawei.com>
>>
>> This patchset let qemu can convert dtb file to dts for two demands:
>>
>> Some archtectures may generate the dtb file dynamically through
>> qemu device tree functions. So this let it's possiable to dump final
>> dtb to dts and save it as a reference.
>>
>> For novices to debugging the issues caused by wrong dtb parameters.
>> It will be easy to check the dts directly without copying the
>> dtb which may be generated by 'dumpdtb' to the PC and dtc or fdtdump
>> it.
>>
>> The outputed dts format is compatile with 'dtc -I dtb -O dts xxx.dtb'.
>> There's a new parameter 'dumpdts' which is similar to 'dumpdtb'. so try
>> it like '-machine dumpdts=/tmp/xxx.dts'.
> 
> Hi. Thanks for this patchset, but I'm afraid this doesn't
> seem to me like something that should be in QEMU.
> As you say, you can easily turn the dtb blob into a source file
> with dtc. That gets you a definitely-correct disassembly of the
> blob, and we don't need to maintain a possibly-buggy
> reimplementation of the dtb disassembler in QEMU.
> 
> thanks
> -- PMM
> 

That makes sense. It's mostly used for debugging.

Best regards
Li.

> 

  reply	other threads:[~2014-08-26  2:04 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-25  4:00 [Qemu-devel] [PATCH 0/6] add dumpdts ability to convert dtb to dts john.liuli
2014-08-25  4:00 ` [Qemu-devel] [PATCH 1/6] device_tree.c: Introduce a function to check multiple strings for dts john.liuli
2014-08-25  4:00 ` [Qemu-devel] [PATCH 2/6] device_tree.c: dump three kind data types of dts to a file desciptor john.liuli
2014-08-25  4:00 ` [Qemu-devel] [PATCH 3/6] device_tree.c: Format a blob in memory as dts stream and dump to a file john.liuli
2014-08-25  4:00 ` [Qemu-devel] [PATCH 4/6] device_tree.c: add the main function to analyse the parameter 'dumpdts' john.liuli
2014-08-25  4:00 ` [Qemu-devel] [PATCH 5/6] machine: add 'dumpdts' parameter to 'machine' optslist john.liuli
2014-08-25  4:00 ` [Qemu-devel] [PATCH 6/6] arm: Allow dumping generated dtb to dts file john.liuli
2014-08-25 12:22 ` [Qemu-devel] [PATCH 0/6] add dumpdts ability to convert dtb to dts Peter Maydell
2014-08-26  2:04   ` Li Liu [this message]
2014-08-26  2:41     ` Peter Crosthwaite

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=53FBEB1C.40809@huawei.com \
    --to=john.liuli@huawei.com \
    --cc=agraf@suse.de \
    --cc=peter.crosthwaite@xilinx.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.