Re: [greybus-dev] [PATCH] staging: greybus: add host device function pointer checks

On Tue, Dec 20, 2016 at 02:49:27PM -0600, Jason Hrycay wrote:

Add sanity checks for cport_quiesce and cport_clear before invoking the callbacks as these function pointers are not required during the host device registration. This follows the logic implemented elsewhere for various other function pointers.

Yeah, I allowed for some inconsistency here given that these callbacks are mandatory on our current platform.

No harm in checking this way though (well, at least as long as we remember to set the pointers).

Signed-off-by: Jason Hrycay jhrycay@gmail.com

Acked-by: Johan Hovold johan@kernel.org

Thanks, Johan