[U-Boot] [PATCH 2/4] dm: do not check the existence of uclass operation

Simon Glass sjg at chromium.org
Fri Oct 10 04:41:43 CEST 2014


On 28 September 2014 09:15, Simon Glass <sjg at chromium.org> wrote:
> On 28 September 2014 07:52, Masahiro Yamada <yamada.m at jp.panasonic.com> wrote:
>> The function uclass_add() checks uc_drv->ops as follows:
>>
>>         if (uc_drv->ops) {
>>                 dm_warn("No ops for uclass id %d\n", id);
>>                 return -EINVAL;
>>         }
>>
>> It seems odd because it warns "No ops" when uc_drv->ops has
>> non-NULL pointer.  (Looks opposite.)
>>
>> Anyway, most of UCLASS_DRIVER entries have no .ops member.
>> This check makes no sense.
>>
>> Signed-off-by: Masahiro Yamada <yamada.m at jp.panasonic.com>
>
> Acked-by: Simon Glass <sjg at chromium.org>

Applied to u-boot-dm/next, thanks!


More information about the U-Boot mailing list