[PATCH phosphor-host-ipmid v4] Add get/set boot option ipmid command support with correct DBUS property handling.

Jeremy Kerr jk at ozlabs.org
Thu Jan 7 12:41:13 AEDT 2016


Hi all,

> Again, I'm not saying it won't work but I'm trying to understand why the
> complexity in these signatures, it doesn't seem like it's being used and
> pointless complexity doesn't help anyone.

Yes - I'd prefer to get these method signatures correct before other
code starts relying on it.

[the same philosophy applies to the contributions in general - we should
be aiming to get the patch as correct as possible, rather than trying to
fix up later]

Regards,


Jeremy


More information about the openbmc mailing list