Frederic Barrat <fbar...@linux.vnet.ibm.com> writes: > Le 12/04/2017 à 09:52, Andrew Donnellan a écrit : >> On 08/04/17 00:11, Christophe Lombard wrote: >>> +static u32 get_phb_index(struct device_node *np) >>> { >>> u32 phb_index; >>> >>> if (of_property_read_u32(np, "ibm,phb-index", &phb_index)) >>> - return 0; >>> + return -ENODEV; >> >> Function is unsigned. > > [Christophe is off till the end of the week, so I'm following up] > > Michael: what's the easiest for you at this point? Shall I send a new > version of the 7th patch with all changes consolidated (tab error + doc > + Andrew's remark above)?
An incremental patch would have been easiest, but that's OK I've taken your remix. cheers