contrib/dtc/checks.c: quiesce clang warning

[sbruno]

Summary

Silence clang warning about extraneous parenthsis

[imp]

This is OK, but can you check to see if a newer version of upstream dtc has this fix already? I'd rather import it that way than like this since it has the potential for complicating future imports.

[sbruno]

In D913#3, @imp wrote:

This is OK, but can you check to see if a newer version of upstream dtc has this fix already? I'd rather import it that way than like this since it has the potential for complicating future imports.

I checked the URLs in the source and can't seem to find this software online anymore. Did it move to gnu.org or something?

[sbruno]

In D913#4, @sbruno wrote:

In D913#3, @imp wrote:

This is OK, but can you check to see if a newer version of upstream dtc has this fix already? I'd rather import it that way than like this since it has the potential for complicating future imports.

I checked the URLs in the source and can't seem to find this software online anymore. Did it move to gnu.org or something?

oh is this it?
http://www.devicetree.org/Device_Tree_Compiler

[sbruno]

Verified that upstream code has the same issue. Still trying to figure out where the mailing list for this project actually is.

[imp]

From git://xenbits.xen.org/people/ianc/device-tree-rebasing

is where I've been grabbing from...

Oh, wait, that's for the .dts files.

[sbruno]

Hrm, I have no real clue where this code comes from ...

[sbruno]

https://git.kernel.org/cgit/utils/dtc/dtc.git/tree/checks.c#n628

Confirmed that this matches upstream. I don't know of anyone that is getting ready to update dtc though, should I just commit this?

[imp]

These are fine. They look correct and have been verified upstream. Committing will cause no issues for next update.