[1/1] rpmsg: virtio_rpmsg_bus: Fix randomly placed semi-colon

Message ID 20160720092935.5902-1-lee.jones@linaro.org
State New
Headers show

Commit Message

Lee Jones July 20, 2016, 9:29 a.m.
It should never have been there in the first place.

Signed-off-by: Lee Jones <lee.jones@linaro.org>

---
 drivers/rpmsg/virtio_rpmsg_bus.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.9.0

Comments

Bjorn Andersson Aug. 16, 2016, 4:26 p.m. | #1
On Tue 16 Aug 05:52 PDT 2016, Lee Jones wrote:

> On Wed, 20 Jul 2016, Lee Jones wrote:

> 

> > It should never have been there in the first place.

> > 

> > Signed-off-by: Lee Jones <lee.jones@linaro.org>

> > ---

> >  drivers/rpmsg/virtio_rpmsg_bus.c | 2 +-

> >  1 file changed, 1 insertion(+), 1 deletion(-)

> 

> I'm assuming this was missed.  Would you like me to send it again, or

> do you have the original patch still in your inbox?

> 


It was applied end of last week, but forgot to respond to your email,
sorry about that.

Thanks,
Bjorn

> > diff --git a/drivers/rpmsg/virtio_rpmsg_bus.c b/drivers/rpmsg/virtio_rpmsg_bus.c

> > index fe03b2a..9823178 100644

> > --- a/drivers/rpmsg/virtio_rpmsg_bus.c

> > +++ b/drivers/rpmsg/virtio_rpmsg_bus.c

> > @@ -865,7 +865,7 @@ static void rpmsg_recv_done(struct virtqueue *rvq)

> >  		msgs_received++;

> >  

> >  		msg = virtqueue_get_buf(rvq, &len);

> > -	};

> > +	}

> >  

> >  	dev_dbg(dev, "Received %u messages\n", msgs_received);

> >  

> 

> -- 

> Lee Jones

> Linaro STMicroelectronics Landing Team Lead

> Linaro.org ??? Open source software for ARM SoCs

> Follow Linaro: Facebook | Twitter | Blog
Lee Jones Aug. 17, 2016, 7:46 a.m. | #2
On Tue, 16 Aug 2016, Bjorn Andersson wrote:

> On Tue 16 Aug 05:52 PDT 2016, Lee Jones wrote:

> 

> > On Wed, 20 Jul 2016, Lee Jones wrote:

> > 

> > > It should never have been there in the first place.

> > > 

> > > Signed-off-by: Lee Jones <lee.jones@linaro.org>

> > > ---

> > >  drivers/rpmsg/virtio_rpmsg_bus.c | 2 +-

> > >  1 file changed, 1 insertion(+), 1 deletion(-)

> > 

> > I'm assuming this was missed.  Would you like me to send it again, or

> > do you have the original patch still in your inbox?

> > 

> 

> It was applied end of last week, but forgot to respond to your email,

> sorry about that.


Ah, great, thanks.

> > > diff --git a/drivers/rpmsg/virtio_rpmsg_bus.c b/drivers/rpmsg/virtio_rpmsg_bus.c

> > > index fe03b2a..9823178 100644

> > > --- a/drivers/rpmsg/virtio_rpmsg_bus.c

> > > +++ b/drivers/rpmsg/virtio_rpmsg_bus.c

> > > @@ -865,7 +865,7 @@ static void rpmsg_recv_done(struct virtqueue *rvq)

> > >  		msgs_received++;

> > >  

> > >  		msg = virtqueue_get_buf(rvq, &len);

> > > -	};

> > > +	}

> > >  

> > >  	dev_dbg(dev, "Received %u messages\n", msgs_received);

> > >  

> > 


-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

Patch hide | download patch | download mbox

diff --git a/drivers/rpmsg/virtio_rpmsg_bus.c b/drivers/rpmsg/virtio_rpmsg_bus.c
index fe03b2a..9823178 100644
--- a/drivers/rpmsg/virtio_rpmsg_bus.c
+++ b/drivers/rpmsg/virtio_rpmsg_bus.c
@@ -865,7 +865,7 @@  static void rpmsg_recv_done(struct virtqueue *rvq)
 		msgs_received++;
 
 		msg = virtqueue_get_buf(rvq, &len);
-	};
+	}
 
 	dev_dbg(dev, "Received %u messages\n", msgs_received);