| Message ID | 20191126135025.mligekonmv2u6dcl@kili.mountain (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | spi: pic32: Add a comment in pic32_spi_dma_prep() | expand |
Hi Dan, On 26/11/2019 15.50, Dan Carpenter wrote: > This code triggers a static checker warning about missed error codes. > It's slightly tricky that we fall back to PIO so let's silence the > checker and add a comment for anyone who is confused. > > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> > --- > I won't feel offended at all if people don't think it's worth applying > this patch. > > drivers/spi/spi-pic32.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/spi/spi-pic32.c b/drivers/spi/spi-pic32.c > index 156961b4ca86..835380ab8bf2 100644 > --- a/drivers/spi/spi-pic32.c > +++ b/drivers/spi/spi-pic32.c > @@ -633,8 +633,10 @@ static int pic32_spi_dma_prep(struct pic32_spi *pic32s, struct device *dev) > goto out_err; > } > > - if (pic32_spi_dma_config(pic32s, DMA_SLAVE_BUSWIDTH_1_BYTE)) > + if (pic32_spi_dma_config(pic32s, DMA_SLAVE_BUSWIDTH_1_BYTE)) { > + ret = 0; /* Fall back to PIO if DMA fails. */ The 'ret' is initialized to 0 in the function to handle this. After looking at the code again I see that pic32_spi_dma_config() does error prints for failures. A comment might be enough here if it is even needed. > goto out_err; > + } > > /* DMA chnls allocated and prepared */ > set_bit(PIC32F_DMA_PREP, &pic32s->flags); > - Péter Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
On Tue, Nov 26, 2019 at 03:55:25PM +0200, Peter Ujfalusi wrote: > Hi Dan, > > On 26/11/2019 15.50, Dan Carpenter wrote: > > This code triggers a static checker warning about missed error codes. > > It's slightly tricky that we fall back to PIO so let's silence the > > checker and add a comment for anyone who is confused. > > > > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> > > --- > > I won't feel offended at all if people don't think it's worth applying > > this patch. > > > > drivers/spi/spi-pic32.c | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/spi/spi-pic32.c b/drivers/spi/spi-pic32.c > > index 156961b4ca86..835380ab8bf2 100644 > > --- a/drivers/spi/spi-pic32.c > > +++ b/drivers/spi/spi-pic32.c > > @@ -633,8 +633,10 @@ static int pic32_spi_dma_prep(struct pic32_spi *pic32s, struct device *dev) > > goto out_err; > > } > > > > - if (pic32_spi_dma_config(pic32s, DMA_SLAVE_BUSWIDTH_1_BYTE)) > > + if (pic32_spi_dma_config(pic32s, DMA_SLAVE_BUSWIDTH_1_BYTE)) { > > + ret = 0; /* Fall back to PIO if DMA fails. */ > > The 'ret' is initialized to 0 in the function to handle this. > After looking at the code again I see that pic32_spi_dma_config() does > error prints for failures. > > A comment might be enough here if it is even needed. Yeah. The "ret = 0;" is just to silence static checkers. It doesn't change the object code (except line numbers). I'm not emotionally invested in this patch so if it's too much then that's fine. regards, dan carpenter
diff --git a/drivers/spi/spi-pic32.c b/drivers/spi/spi-pic32.c index 156961b4ca86..835380ab8bf2 100644 --- a/drivers/spi/spi-pic32.c +++ b/drivers/spi/spi-pic32.c @@ -633,8 +633,10 @@ static int pic32_spi_dma_prep(struct pic32_spi *pic32s, struct device *dev) goto out_err; } - if (pic32_spi_dma_config(pic32s, DMA_SLAVE_BUSWIDTH_1_BYTE)) + if (pic32_spi_dma_config(pic32s, DMA_SLAVE_BUSWIDTH_1_BYTE)) { + ret = 0; /* Fall back to PIO if DMA fails. */ goto out_err; + } /* DMA chnls allocated and prepared */ set_bit(PIC32F_DMA_PREP, &pic32s->flags);
This code triggers a static checker warning about missed error codes. It's slightly tricky that we fall back to PIO so let's silence the checker and add a comment for anyone who is confused. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> --- I won't feel offended at all if people don't think it's worth applying this patch. drivers/spi/spi-pic32.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)