From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Glass Subject: Re: [PATCH v2 3/6] Add fdtget utility to read property values from device tree Date: Mon, 19 Sep 2011 08:11:02 -0700 Message-ID: References: <20110908052547.GR30278@yookeroo.fritz.box> <20110909044945.GF21002@yookeroo.fritz.box> <20110912005357.GI9025@yookeroo.fritz.box> <20110916081804.GF9025@yookeroo.fritz.box> <20110919020440.GA15001@yookeroo.fritz.box> <20110919080557.GA29197@yookeroo.fritz.box> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <20110919080557.GA29197-787xzQ0H9iQXU02nzanrWNbf9cGiqdzd@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Sender: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org To: David Gibson Cc: Devicetree Discuss List-Id: devicetree@vger.kernel.org Hi David, On Mon, Sep 19, 2011 at 1:05 AM, David Gibson wrote: > On Sun, Sep 18, 2011 at 10:46:04PM -0700, Simon Glass wrote: >> Hi David, >> >> On Sun, Sep 18, 2011 at 7:04 PM, David Gibson >> wrote: >> > On Fri, Sep 16, 2011 at 09:25:10AM -0700, Simon Glass wrote: >> >> Hi David, >> > [snip] >> >> > One useful extension of what you have now might be to add a "raw >> >> > binary" format. =A0Not much use for humans, but could be handy for >> >> > scripts extracting blobs from the device tree. =A0In fact, using the >> >> > same interpretation of existing printf() descriptors, that would >> >> > actually be equivalent to 'c'. >> >> >> >> Yes I was thinking about how to do that as it might be useful. I think >> >> 'c' makes perfect sense. Will add to the todo. >> > >> > Actually, on second thoughts using 'c' for this doesn't quite work. >> > For the other format specifiers, you will presumably have some sort of >> > separator between each repeat (newline, I'm guessing). =A0True binary >> > output, obviously, doesn't want that. >> >> There is a space between each repeat and a newline after each property >> printed. It makes sense to suppress the space for binary output and I >> think an option to flick this switch either way might be in order, too >> (even something to select a new separator char?). Something else for >> the todo once this patch set is in perhaps. > > Yeah, fair enough. =A0I don't think skipping the separators should be > explicit one way or another, not a magic side-effect of using a 'c' > type. OK I think you are saying you want an explicit switch. > >> I suppose we need to be a bit careful not to build out the pier too >> far until we have some pedestrian traffic. > > A good point. > >> Did you see patch set 3? > > Uh, no I don't think so. OK I will resend to list. Regards Simon > > -- > David Gibson =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0| I'll have my music = baroque, and my code > david AT gibson.dropbear.id.au =A0| minimalist, thank you. =A0NOT _the_ _= other_ > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0| _way_ _a= round_! > http://www.ozlabs.org/~dgibson >