Re: [PATCH 1/2] staging: bcm2835-audio: Replace call to vchi_msg_queue with vchi_queue_kernel_message

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

 



On Wed, 2017-02-01 at 18:04 +0100, Stefan Wahren wrote:
> Am 01.02.2017 um 15:31 schrieb Michael Zoran:
> > The function vchi_msg_queue was made static in vc04_services
> > and replaced with vchi_queue_kernel_message.
> > 
> > Change the call to vchi_msg_queue to vchi_queue_kernel_message
> > 
> > Signed-off-by: Michael Zoran <mzoran@xxxxxxxxxxxx>
> > ---
> >  drivers/staging/bcm2835-audio/bcm2835-vchiq.c | 15 +++------------
> >  1 file changed, 3 insertions(+), 12 deletions(-)
> > 
> > diff --git a/drivers/staging/bcm2835-audio/bcm2835-vchiq.c
> > b/drivers/staging/bcm2835-audio/bcm2835-vchiq.c
> > index e8fd9c79bcfc..7c952bae4e52 100644
> > --- a/drivers/staging/bcm2835-audio/bcm2835-vchiq.c
> > +++ b/drivers/staging/bcm2835-audio/bcm2835-vchiq.c
> > @@ -81,23 +81,14 @@ static int bcm2835_audio_write_worker(struct
> > bcm2835_alsa_stream *alsa_stream,
> >  
> >  // Routine to send a message across a service
> >  
> > -static ssize_t
> > -bcm2835_vchi_msg_queue_callback(void *context, void *dest,
> > -				size_t offset, size_t maxsize)
> > -{
> > -	memcpy(dest, context + offset, maxsize);
> > -	return maxsize;
> > -}
> > -
> >  static int
> >  bcm2835_vchi_msg_queue(VCHI_SERVICE_HANDLE_T handle,
> >  		       void *data,
> >  		       unsigned int size)
> >  {
> > -	return vchi_msg_queue(handle,
> > -			      bcm2835_vchi_msg_queue_callback,
> > -			      data,
> > -			      size);
> > +	return vchi_queue_kernel_message(handle,
> > +					 data,
> > +					 size);
> >  }
> >  
> 
> How about calling vchi_queue_kernel_message directly in the driver
> and
> get the rid of bcm2835_vchiq_msg_queue, too?

Yes, that would be better.  Maybe that would be another patch.

I didn't know if the build break was a everything comes to a grinding
halt emergency or if this was just a warning.  I was also having
network issues at the time and it was a bad time for me.  So I just
made the safest quickest change possible.
_______________________________________________
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