From: Jonathan Cameron <jic23@kernel.org>
To: Matheus Tavares <matheus.bernardino@usp.br>
Cc: Lars-Peter Clausen <lars@metafoo.de>,
Michael Hennerich <Michael.Hennerich@analog.com>,
Hartmut Knaack <knaack.h@gmx.de>,
Peter Meerwald-Stadler <pmeerw@pmeerw.net>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Rob Herring <robh+dt@kernel.org>,
Mark Rutland <mark.rutland@arm.com>,
linux-iio@vger.kernel.org, devel@driverdev.osuosl.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
Alexandru Ardelean <alexandru.ardelean@analog.com>,
kernel-usp@googlegroups.com, victorcolombo@gmail.com
Subject: Re: [PATCH v3 6/7] staging:iio:ad2s90: Add comment to device state mutex
Date: Sat, 1 Dec 2018 15:39:32 +0000 [thread overview]
Message-ID: <20181201153932.45e218f5@archlinux> (raw)
In-Reply-To: <20181124002312.6923-7-matheus.bernardino@usp.br>
On Fri, 23 Nov 2018 22:23:11 -0200
Matheus Tavares <matheus.bernardino@usp.br> wrote:
> From: Victor Colombo <victorcolombo@gmail.com>
>
> Fix the checkpatch.pl issue:
> "CHECK: struct mutex definition without comment".
>
> Signed-off-by: Victor Colombo <victorcolombo@gmail.com>
> Signed-off-by: Matheus Tavares <matheus.bernardino@usp.br>
Applied, thanks
Jonathan
> ---
> Changes in v3:
> - none
>
> Changes in v2:
> - Patch added in v2
>
> drivers/staging/iio/resolver/ad2s90.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/iio/resolver/ad2s90.c b/drivers/staging/iio/resolver/ad2s90.c
> index 678351dabe6b..a41f5cb10da5 100644
> --- a/drivers/staging/iio/resolver/ad2s90.c
> +++ b/drivers/staging/iio/resolver/ad2s90.c
> @@ -22,7 +22,7 @@
> #define AD2S90_MAX_SPI_FREQ_HZ 830000
>
> struct ad2s90_state {
> - struct mutex lock;
> + struct mutex lock; /* lock to protect rx buffer */
> struct spi_device *sdev;
> u8 rx[2] ____cacheline_aligned;
> };
WARNING: multiple messages have this Message-ID (diff)
From: Jonathan Cameron <jic23@kernel.org>
To: Matheus Tavares <matheus.bernardino@usp.br>
Cc: Mark Rutland <mark.rutland@arm.com>,
devel@driverdev.osuosl.org, Lars-Peter Clausen <lars@metafoo.de>,
Michael Hennerich <Michael.Hennerich@analog.com>,
devicetree@vger.kernel.org, linux-iio@vger.kernel.org,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
linux-kernel@vger.kernel.org, kernel-usp@googlegroups.com,
Rob Herring <robh+dt@kernel.org>,
Peter Meerwald-Stadler <pmeerw@pmeerw.net>,
Hartmut Knaack <knaack.h@gmx.de>,
Alexandru Ardelean <alexandru.ardelean@analog.com>,
victorcolombo@gmail.com
Subject: Re: [PATCH v3 6/7] staging:iio:ad2s90: Add comment to device state mutex
Date: Sat, 1 Dec 2018 15:39:32 +0000 [thread overview]
Message-ID: <20181201153932.45e218f5@archlinux> (raw)
In-Reply-To: <20181124002312.6923-7-matheus.bernardino@usp.br>
On Fri, 23 Nov 2018 22:23:11 -0200
Matheus Tavares <matheus.bernardino@usp.br> wrote:
> From: Victor Colombo <victorcolombo@gmail.com>
>
> Fix the checkpatch.pl issue:
> "CHECK: struct mutex definition without comment".
>
> Signed-off-by: Victor Colombo <victorcolombo@gmail.com>
> Signed-off-by: Matheus Tavares <matheus.bernardino@usp.br>
Applied, thanks
Jonathan
> ---
> Changes in v3:
> - none
>
> Changes in v2:
> - Patch added in v2
>
> drivers/staging/iio/resolver/ad2s90.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/iio/resolver/ad2s90.c b/drivers/staging/iio/resolver/ad2s90.c
> index 678351dabe6b..a41f5cb10da5 100644
> --- a/drivers/staging/iio/resolver/ad2s90.c
> +++ b/drivers/staging/iio/resolver/ad2s90.c
> @@ -22,7 +22,7 @@
> #define AD2S90_MAX_SPI_FREQ_HZ 830000
>
> struct ad2s90_state {
> - struct mutex lock;
> + struct mutex lock; /* lock to protect rx buffer */
> struct spi_device *sdev;
> u8 rx[2] ____cacheline_aligned;
> };
next prev parent reply other threads:[~2018-12-02 2:52 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-24 0:23 [PATCH v3 0/7] staging:iio:ad2s90: Add dt support and move out of staging Matheus Tavares
2018-11-24 0:23 ` Matheus Tavares
2018-11-24 0:23 ` [PATCH v3 1/7] staging:iio:ad2s90: Add device tree support Matheus Tavares
2018-12-01 15:34 ` Jonathan Cameron
2018-12-01 15:34 ` Jonathan Cameron
2018-11-24 0:23 ` [PATCH v3 2/7] staging:iio:ad2s90: Remove spi setup that should be done via dt Matheus Tavares
2018-11-24 0:23 ` Matheus Tavares
2018-12-01 15:34 ` Jonathan Cameron
2018-12-01 15:34 ` Jonathan Cameron
2018-11-24 0:23 ` [PATCH v3 3/7] staging:iio:ad2s90: Add max frequency check at probe Matheus Tavares
2018-12-01 15:37 ` Jonathan Cameron
2018-11-24 0:23 ` [PATCH v3 4/7] dt-bindings:iio:resolver: Add docs for ad2s90 Matheus Tavares
2018-11-27 1:52 ` Rob Herring
2018-12-01 15:37 ` Jonathan Cameron
2018-11-24 0:23 ` [PATCH v3 5/7] staging:iio:ad2s90: Replace license text w/ SPDX identifier Matheus Tavares
2018-12-01 15:38 ` Jonathan Cameron
2018-12-01 15:38 ` Jonathan Cameron
2018-11-24 0:23 ` [PATCH v3 6/7] staging:iio:ad2s90: Add comment to device state mutex Matheus Tavares
2018-11-24 0:23 ` Matheus Tavares
2018-12-01 15:39 ` Jonathan Cameron [this message]
2018-12-01 15:39 ` Jonathan Cameron
2018-11-24 0:23 ` [PATCH v3 7/7] staging:iio:ad2s90: Move out of staging Matheus Tavares
2018-11-25 9:56 ` Jonathan Cameron
2018-11-25 9:56 ` Jonathan Cameron
2018-11-26 7:34 ` Ardelean, Alexandru
2018-11-26 7:34 ` Ardelean, Alexandru
2018-11-26 7:34 ` Ardelean, Alexandru
2018-12-01 15:41 ` Jonathan Cameron
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20181201153932.45e218f5@archlinux \
--to=jic23@kernel.org \
--cc=Michael.Hennerich@analog.com \
--cc=alexandru.ardelean@analog.com \
--cc=devel@driverdev.osuosl.org \
--cc=devicetree@vger.kernel.org \
--cc=gregkh@linuxfoundation.org \
--cc=kernel-usp@googlegroups.com \
--cc=knaack.h@gmx.de \
--cc=lars@metafoo.de \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=matheus.bernardino@usp.br \
--cc=pmeerw@pmeerw.net \
--cc=robh+dt@kernel.org \
--cc=victorcolombo@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.