Stephen, > If you don't use the value of the len variable, then you can pass > NULL to of_get_property. Sure, but that's probably a separate patch, no ? > Not *phys_addr? That's what the existing code does. I'm guessing that it treats phys_addr as an array with one entry, not a single u32. Cheers, Jeremy