[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