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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jan 03, 2017 at 01:46:09PM -0600, Jason Hrycay wrote:
> On Mon, Jan 02, 2017 at 02:54:37PM +0100, Johan Hovold wrote:
> > 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).
> > 
> 
> OK, makes sense. We ran into it while integrating the Motorola
> host-device stack into the latest version of greybus (we were
> previously forked from a version a year ago - wow time flies).

Cool. Yeah, things changed quite a bit over the last year. Just let us
know if you have any questions.

Johan
_______________________________________________
greybus-dev mailing list
greybus-dev@xxxxxxxxxxxxxxxx
https://lists.linaro.org/mailman/listinfo/greybus-dev




[Index of Archives]     [Asterisk App Development]     [PJ SIP]     [Gnu Gatekeeper]     [IETF Sipping]     [Info Cyrus]     [ALSA User]     [Fedora Linux Users]     [Linux SCTP]     [DCCP]     [Gimp]     [Yosemite News]     [Deep Creek Hot Springs]     [Yosemite Campsites]     [ISDN Cause Codes]     [Asterisk Books]

  Powered by Linux