On 07/12/2012 11:32 PM, Alex Courbot wrote: > On 07/12/2012 11:27 PM, Simon Glass wrote ... >> From my understanding mixing strings and numbers in a property is >> frowned on though. > > But doesn't it make sense in the current case? The power sequence is > basically a program that is run by an interpreter. From this > perspective, it makes more sense to me to have it as a binary field > rather than a hierarchy of nodes and properties that will be harder to > parse and will make error detection more complicated. I don't really see > any practical benefit from turning the steps into sub-nodes, but then > again I am not so familiar with the DT. Mixing strings and integers in a property isn't "allowed" (by convention of DT bindings reviewers - as you noticed, dtc will happily compile it); there are practical issues with attempting to do so, such as causing the integer values to be unaligned, and thus causing the current integer parsing code to fail, etc. -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
