Stephen Rothwell wrote:

> Why not:
>       const u64 *iprop;
>       int len;
> 
>       iprop = of_get_property(fs, "extended-modes", &len);
>       if (iprop && (len == sizeof(u64))
>               qe_firmware_info.extended_modes = *iprop;

Actually, I like my way better, because then I don't have to define 'len', and 
I 
don't have to define two integer property variables one for u32 and one for 
u64. 
  Besides, now that I'm at version 6, I'd rather not post updates for minor 
stylistic changes.

-- 
Timur Tabi
Linux kernel developer at Freescale
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Reply via email to