Messages in this thread | | | Date | Sat, 23 Aug 2014 09:54:04 -0500 | From | Mark Brown <> | Subject | Re: [PATCH v2 1/2] pmbus: add regulator support |
| |
On Sat, Aug 23, 2014 at 07:00:44AM -0700, Guenter Roeck wrote: > On 08/22/2014 05:31 PM, atull wrote:
> >of_get_regulator_init_data() will only have an error if it cannot alloc > >the regulator_init_data struct. That's why I did -ENOMEM. If there > >is no platform data and nothing about the regulator in the device tree, we > >should end up with a zeroed out regulator_init_data.
> Yes, but if OF is not defined it will return NULL as well. Unless I am > missing something, that means that the code will now fail if there is > no platform init data and OF is not configured.
Indeed, and that's the more interesting case than running out of memory. [unhandled content-type:application/pgp-signature] | |