[PATCH 4/4] dtc/libfdt: sparse fixes

David Gibson david at gibson.dropbear.id.au
Thu Nov 15 14:53:05 EST 2012


On Wed, Nov 14, 2012 at 06:46:30PM -0600, Kim Phillips wrote:
> On Wed, 14 Nov 2012 21:23:03 +1100
> David Gibson <david at gibson.dropbear.id.au> wrote:
> 
> > Hrm.  The patch comment shows a lot of sparse warnings, but the patch
> > looks like the basic conversion of the code from uint32_t to
> > annontated fdt32_t types.
> 
> not sure what you're getting at: isn't this to be expected?  Anyway,
> I'm going to guess you mean to re-word the commit message.

Ah, I see what you're getting at, and yes.  The current message
suggests this is fixing real endian problems found by sparse, rather
than being almost a mechanical replacement of uint32_t with fdt32_t.
If there are real bug fixes, I'd prefer to see that broken out into a
different patch.

> >  I also never saw a 3/4.  Has something gone
> > wrong with the patch sending?
> 
> I sent 3/4 in reply-to your comments on v1 of the patch, and the
> rest without the reply-to reference, so 3/4 might show up out of
> order in your inbox, depending on how it's configured.  I'll send
> the entire series with the same reference - jdl's comments to v2 -
> instead next time.

Ok.

-- 
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


More information about the devicetree-discuss mailing list