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
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux