[PATCH 2/3] spi: s3c64xx: remove a compilation warning with an assignment
Sachin Kamat
spk.linux at gmail.com
Tue Jun 10 02:45:03 PDT 2014
Hi Naveen,
On Tue, Jun 10, 2014 at 2:30 PM, Naveen Krishna Chatradhi
<ch.naveen at samsung.com> wrote:
> This patch returns an integer error value instead of the
> pointer.
>
> "warning: return makes integer from pointer without a cast"
>
> Signed-off-by: Naveen Krishna Chatradhi <ch.naveen at samsung.com>
> Cc: Javier Martinez Canillas <javier.martinez at collabora.co.uk>
> Cc: Doug Anderson <dianders at chromium.org>
> ---
> drivers/spi/spi-s3c64xx.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/spi/spi-s3c64xx.c b/drivers/spi/spi-s3c64xx.c
> index 0c6013f..4594dde 100644
> --- a/drivers/spi/spi-s3c64xx.c
> +++ b/drivers/spi/spi-s3c64xx.c
> @@ -817,7 +817,7 @@ static int s3c64xx_spi_setup(struct spi_device *spi)
>
> if (!spi->dev.of_node) {
> dev_err(&spi->dev, "device node not found\n");
> - return ERR_PTR(-EINVAL);
> + return -EINVAL;
Isn't this warning introduced by the previous patch (patch1/3)?
If so then this patch should be squashed into that.
--
Regards,
Sachin.
More information about the linux-arm-kernel
mailing list