Hi Laurent, Thanks for the feedback. > -----Original Message----- > From: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > Sent: Saturday, September 7, 2024 1:05 AM > Subject: Re: [PATCH] media: platform: rzg2l-cru: rzg2l-video: Fix the comment in > rzg2l_cru_start_streaming_vq() > > Hi Biju, > > Thank you for the patch. > > On Thu, Sep 05, 2024 at 12:25:05PM +0100, Biju Das wrote: > > Replace "buffer." -> "buffer", for consistency with rest of the > > comment blocks in rzg2l_cru_start_streaming_vq(). > > I usually go the other way around, sentences should end with a period. > You can go either way as it's your driver. Ok, I will send next version with sentences end with a period. Cheers, Biju > > > Reported-by: Pavel Machek <pavel@xxxxxxx> > > Closes: https://lore.kernel.org/all/ZtWNFuw70nkB37EK@xxxxxxxxxx/ > > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx> > > --- > > This patch have merge dependency on [1] [1] > > https://lore.kernel.org/all/20240826110740.271212-1-biju.das.jz@xxxxxx > > esas.com/ > > --- > > drivers/media/platform/renesas/rzg2l-cru/rzg2l-video.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/media/platform/renesas/rzg2l-cru/rzg2l-video.c > > b/drivers/media/platform/renesas/rzg2l-cru/rzg2l-video.c > > index bbf4674f888d..d17e3eac4177 100644 > > --- a/drivers/media/platform/renesas/rzg2l-cru/rzg2l-video.c > > +++ b/drivers/media/platform/renesas/rzg2l-cru/rzg2l-video.c > > @@ -648,7 +648,7 @@ static int rzg2l_cru_start_streaming_vq(struct vb2_queue *vq, unsigned int count > > goto assert_aresetn; > > } > > > > - /* Allocate scratch buffer. */ > > + /* Allocate scratch buffer */ > > cru->scratch = dma_alloc_coherent(cru->dev, cru->format.sizeimage, > > &cru->scratch_phys, GFP_KERNEL); > > if (!cru->scratch) { > > -- > Regards, > > Laurent Pinchart
