* [PATCH] iio:dummy: Fix coding style in Kconfig
@ 2018-02-20 13:28 Rodrigo Siqueira
2018-02-24 12:24 ` Jonathan Cameron
0 siblings, 1 reply; 5+ messages in thread
From: Rodrigo Siqueira @ 2018-02-20 13:28 UTC (permalink / raw)
To: Jonathan Cameron, Hartmut Knaack, Lars-Peter Clausen,
Peter Meerwald-Stadler
Cc: daniel.baluta, kernel-janitors, linux-iio, linux-kernel
This patch fixes the checkpatch.pl warning:
drivers/iio/dummy/Kconfig:21: WARNING: please write a paragraph that
describes the config symbol fully
drivers/iio/dummy/Kconfig:29: WARNING: please write a paragraph that
describes the config symbol fully
This patch expands the explanation about IIO_DUMMY_EVGEN by using the
code documentation found in iio/dummy/iio_dummy_evgen.c. In the same
way, the information related to IIO_SIMPLE_DUMMY_BUFFER was extracted
from file iio_simple_dummy_buffer.c. Finally, this patch apply the
coding-style for Kconfig files (Documentation/process/coding-style.rst)
Signed-off-by: Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com>
---
drivers/iio/dummy/Kconfig | 26 ++++++++++++++++----------
1 file changed, 16 insertions(+), 10 deletions(-)
diff --git a/drivers/iio/dummy/Kconfig b/drivers/iio/dummy/Kconfig
index 5a29fbd3c531..4a7127fb2979 100644
--- a/drivers/iio/dummy/Kconfig
+++ b/drivers/iio/dummy/Kconfig
@@ -9,20 +9,24 @@ config IIO_DUMMY_EVGEN
tristate
config IIO_SIMPLE_DUMMY
- tristate "An example driver with no hardware requirements"
- depends on IIO_SW_DEVICE
- help
- Driver intended mainly as documentation for how to write
- a driver. May also be useful for testing userspace code
- without hardware.
+ tristate "An example driver with no hardware requirements"
+ depends on IIO_SW_DEVICE
+ help
+ Driver intended mainly as documentation for how to write
+ a driver. May also be useful for testing userspace code
+ without hardware.
if IIO_SIMPLE_DUMMY
config IIO_SIMPLE_DUMMY_EVENTS
- bool "Event generation support"
- select IIO_DUMMY_EVGEN
- help
- Add some dummy events to the simple dummy driver.
+ bool "Event generation support"
+ select IIO_DUMMY_EVGEN
+ help
+ Add some dummy events to the simple dummy driver.
+
+ The purpose of this is to generate 'fake' event interrupts thus
+ allowing that driver's code to be as close as possible to that
+ a normal driver talking to hardware.
config IIO_SIMPLE_DUMMY_BUFFER
bool "Buffered capture support"
@@ -32,6 +36,8 @@ config IIO_SIMPLE_DUMMY_BUFFER
help
Add buffered data capture to the simple dummy driver.
+ Buffer handling elements of industrial I/O reference driver.
+ Uses the kfifo buffer.
endif # IIO_SIMPLE_DUMMY
endmenu
--
2.16.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] iio:dummy: Fix coding style in Kconfig
2018-02-20 13:28 [PATCH] iio:dummy: Fix coding style in Kconfig Rodrigo Siqueira
@ 2018-02-24 12:24 ` Jonathan Cameron
2018-02-24 12:36 ` Rodrigo Siqueira
0 siblings, 1 reply; 5+ messages in thread
From: Jonathan Cameron @ 2018-02-24 12:24 UTC (permalink / raw)
To: Rodrigo Siqueira
Cc: Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald-Stadler,
daniel.baluta, kernel-janitors, linux-iio, linux-kernel
On Tue, 20 Feb 2018 10:28:04 -0300
Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com> wrote:
> This patch fixes the checkpatch.pl warning:
>
> drivers/iio/dummy/Kconfig:21: WARNING: please write a paragraph that
> describes the config symbol fully
> drivers/iio/dummy/Kconfig:29: WARNING: please write a paragraph that
> describes the config symbol fully
>
> This patch expands the explanation about IIO_DUMMY_EVGEN by using the
> code documentation found in iio/dummy/iio_dummy_evgen.c. In the same
> way, the information related to IIO_SIMPLE_DUMMY_BUFFER was extracted
> from file iio_simple_dummy_buffer.c. Finally, this patch apply the
> coding-style for Kconfig files (Documentation/process/coding-style.rst)
>
> Signed-off-by: Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com>
Several different things going on in the one patch, please do
it as a short series with one thing per patch.
Certainly do the coding style separately from the additional documentation.
Thanks,
Jonathan
> ---
> drivers/iio/dummy/Kconfig | 26 ++++++++++++++++----------
> 1 file changed, 16 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/iio/dummy/Kconfig b/drivers/iio/dummy/Kconfig
> index 5a29fbd3c531..4a7127fb2979 100644
> --- a/drivers/iio/dummy/Kconfig
> +++ b/drivers/iio/dummy/Kconfig
> @@ -9,20 +9,24 @@ config IIO_DUMMY_EVGEN
> tristate
>
> config IIO_SIMPLE_DUMMY
> - tristate "An example driver with no hardware requirements"
> - depends on IIO_SW_DEVICE
> - help
> - Driver intended mainly as documentation for how to write
> - a driver. May also be useful for testing userspace code
> - without hardware.
> + tristate "An example driver with no hardware requirements"
> + depends on IIO_SW_DEVICE
> + help
> + Driver intended mainly as documentation for how to write
> + a driver. May also be useful for testing userspace code
> + without hardware.
>
> if IIO_SIMPLE_DUMMY
>
> config IIO_SIMPLE_DUMMY_EVENTS
> - bool "Event generation support"
> - select IIO_DUMMY_EVGEN
> - help
> - Add some dummy events to the simple dummy driver.
> + bool "Event generation support"
> + select IIO_DUMMY_EVGEN
> + help
> + Add some dummy events to the simple dummy driver.
> +
> + The purpose of this is to generate 'fake' event interrupts thus
> + allowing that driver's code to be as close as possible to that
> + a normal driver talking to hardware.
>
> config IIO_SIMPLE_DUMMY_BUFFER
> bool "Buffered capture support"
> @@ -32,6 +36,8 @@ config IIO_SIMPLE_DUMMY_BUFFER
> help
> Add buffered data capture to the simple dummy driver.
>
> + Buffer handling elements of industrial I/O reference driver.
> + Uses the kfifo buffer.
> endif # IIO_SIMPLE_DUMMY
>
> endmenu
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] iio:dummy: Fix coding style in Kconfig
2018-02-24 12:24 ` Jonathan Cameron
@ 2018-02-24 12:36 ` Rodrigo Siqueira
2018-02-24 13:44 ` Jonathan Cameron
0 siblings, 1 reply; 5+ messages in thread
From: Rodrigo Siqueira @ 2018-02-24 12:36 UTC (permalink / raw)
To: Jonathan Cameron
Cc: Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald-Stadler,
daniel.baluta, kernel-janitors, linux-iio, linux-kernel
On 02/24, Jonathan Cameron wrote:
> On Tue, 20 Feb 2018 10:28:04 -0300
> Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com> wrote:
>
> > This patch fixes the checkpatch.pl warning:
> >
> > drivers/iio/dummy/Kconfig:21: WARNING: please write a paragraph that
> > describes the config symbol fully
> > drivers/iio/dummy/Kconfig:29: WARNING: please write a paragraph that
> > describes the config symbol fully
> >
> > This patch expands the explanation about IIO_DUMMY_EVGEN by using the
> > code documentation found in iio/dummy/iio_dummy_evgen.c. In the same
> > way, the information related to IIO_SIMPLE_DUMMY_BUFFER was extracted
> > from file iio_simple_dummy_buffer.c. Finally, this patch apply the
> > coding-style for Kconfig files (Documentation/process/coding-style.rst)
> >
> > Signed-off-by: Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com>
>
> Several different things going on in the one patch, please do
> it as a short series with one thing per patch.
>
> Certainly do the coding style separately from the additional documentation.
>
> Thanks,
>
> Jonathan
So, should I split it into two different patches? One patch for the fix
and other for the extra documentation?
Finally, in the two warnings related to "please write a paragraph that
describes the config symbol fully" the extra documentation that fixes
it. Other fixes, are related to the coding-style.rst however checkpatch
does not complain about them.
>
> > ---
> > drivers/iio/dummy/Kconfig | 26 ++++++++++++++++----------
> > 1 file changed, 16 insertions(+), 10 deletions(-)
> >
> > diff --git a/drivers/iio/dummy/Kconfig b/drivers/iio/dummy/Kconfig
> > index 5a29fbd3c531..4a7127fb2979 100644
> > --- a/drivers/iio/dummy/Kconfig
> > +++ b/drivers/iio/dummy/Kconfig
> > @@ -9,20 +9,24 @@ config IIO_DUMMY_EVGEN
> > tristate
> >
> > config IIO_SIMPLE_DUMMY
> > - tristate "An example driver with no hardware requirements"
> > - depends on IIO_SW_DEVICE
> > - help
> > - Driver intended mainly as documentation for how to write
> > - a driver. May also be useful for testing userspace code
> > - without hardware.
> > + tristate "An example driver with no hardware requirements"
> > + depends on IIO_SW_DEVICE
> > + help
> > + Driver intended mainly as documentation for how to write
> > + a driver. May also be useful for testing userspace code
> > + without hardware.
> >
> > if IIO_SIMPLE_DUMMY
> >
> > config IIO_SIMPLE_DUMMY_EVENTS
> > - bool "Event generation support"
> > - select IIO_DUMMY_EVGEN
> > - help
> > - Add some dummy events to the simple dummy driver.
> > + bool "Event generation support"
> > + select IIO_DUMMY_EVGEN
> > + help
> > + Add some dummy events to the simple dummy driver.
> > +
> > + The purpose of this is to generate 'fake' event interrupts thus
> > + allowing that driver's code to be as close as possible to that
> > + a normal driver talking to hardware.
> >
> > config IIO_SIMPLE_DUMMY_BUFFER
> > bool "Buffered capture support"
> > @@ -32,6 +36,8 @@ config IIO_SIMPLE_DUMMY_BUFFER
> > help
> > Add buffered data capture to the simple dummy driver.
> >
> > + Buffer handling elements of industrial I/O reference driver.
> > + Uses the kfifo buffer.
> > endif # IIO_SIMPLE_DUMMY
> >
> > endmenu
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] iio:dummy: Fix coding style in Kconfig
2018-02-24 12:36 ` Rodrigo Siqueira
@ 2018-02-24 13:44 ` Jonathan Cameron
2018-02-24 13:53 ` Rodrigo Siqueira
0 siblings, 1 reply; 5+ messages in thread
From: Jonathan Cameron @ 2018-02-24 13:44 UTC (permalink / raw)
To: Rodrigo Siqueira
Cc: Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald-Stadler,
daniel.baluta, kernel-janitors, linux-iio, linux-kernel
On Sat, 24 Feb 2018 09:36:09 -0300
Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com> wrote:
> On 02/24, Jonathan Cameron wrote:
> > On Tue, 20 Feb 2018 10:28:04 -0300
> > Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com> wrote:
> >
> > > This patch fixes the checkpatch.pl warning:
> > >
> > > drivers/iio/dummy/Kconfig:21: WARNING: please write a paragraph that
> > > describes the config symbol fully
> > > drivers/iio/dummy/Kconfig:29: WARNING: please write a paragraph that
> > > describes the config symbol fully
> > >
> > > This patch expands the explanation about IIO_DUMMY_EVGEN by using the
> > > code documentation found in iio/dummy/iio_dummy_evgen.c. In the same
> > > way, the information related to IIO_SIMPLE_DUMMY_BUFFER was extracted
> > > from file iio_simple_dummy_buffer.c. Finally, this patch apply the
> > > coding-style for Kconfig files (Documentation/process/coding-style.rst)
> > >
> > > Signed-off-by: Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com>
> >
> > Several different things going on in the one patch, please do
> > it as a short series with one thing per patch.
> >
> > Certainly do the coding style separately from the additional documentation.
> >
> > Thanks,
> >
> > Jonathan
>
> So, should I split it into two different patches? One patch for the fix
> and other for the extra documentation?
Yes that would be perfect.
>
> Finally, in the two warnings related to "please write a paragraph that
> describes the config symbol fully" the extra documentation that fixes
> it. Other fixes, are related to the coding-style.rst however checkpatch
> does not complain about them.
Sure, checkpatch doesn't get everything.
Jonathan
>
> >
> > > ---
> > > drivers/iio/dummy/Kconfig | 26 ++++++++++++++++----------
> > > 1 file changed, 16 insertions(+), 10 deletions(-)
> > >
> > > diff --git a/drivers/iio/dummy/Kconfig b/drivers/iio/dummy/Kconfig
> > > index 5a29fbd3c531..4a7127fb2979 100644
> > > --- a/drivers/iio/dummy/Kconfig
> > > +++ b/drivers/iio/dummy/Kconfig
> > > @@ -9,20 +9,24 @@ config IIO_DUMMY_EVGEN
> > > tristate
> > >
> > > config IIO_SIMPLE_DUMMY
> > > - tristate "An example driver with no hardware requirements"
> > > - depends on IIO_SW_DEVICE
> > > - help
> > > - Driver intended mainly as documentation for how to write
> > > - a driver. May also be useful for testing userspace code
> > > - without hardware.
> > > + tristate "An example driver with no hardware requirements"
> > > + depends on IIO_SW_DEVICE
> > > + help
> > > + Driver intended mainly as documentation for how to write
> > > + a driver. May also be useful for testing userspace code
> > > + without hardware.
> > >
> > > if IIO_SIMPLE_DUMMY
> > >
> > > config IIO_SIMPLE_DUMMY_EVENTS
> > > - bool "Event generation support"
> > > - select IIO_DUMMY_EVGEN
> > > - help
> > > - Add some dummy events to the simple dummy driver.
> > > + bool "Event generation support"
> > > + select IIO_DUMMY_EVGEN
> > > + help
> > > + Add some dummy events to the simple dummy driver.
> > > +
> > > + The purpose of this is to generate 'fake' event interrupts thus
> > > + allowing that driver's code to be as close as possible to that
> > > + a normal driver talking to hardware.
> > >
> > > config IIO_SIMPLE_DUMMY_BUFFER
> > > bool "Buffered capture support"
> > > @@ -32,6 +36,8 @@ config IIO_SIMPLE_DUMMY_BUFFER
> > > help
> > > Add buffered data capture to the simple dummy driver.
> > >
> > > + Buffer handling elements of industrial I/O reference driver.
> > > + Uses the kfifo buffer.
> > > endif # IIO_SIMPLE_DUMMY
> > >
> > > endmenu
> >
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] iio:dummy: Fix coding style in Kconfig
2018-02-24 13:44 ` Jonathan Cameron
@ 2018-02-24 13:53 ` Rodrigo Siqueira
0 siblings, 0 replies; 5+ messages in thread
From: Rodrigo Siqueira @ 2018-02-24 13:53 UTC (permalink / raw)
To: Jonathan Cameron
Cc: Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald-Stadler,
daniel.baluta, kernel-janitors, linux-iio, linux-kernel
On 02/24, Jonathan Cameron wrote:
> On Sat, 24 Feb 2018 09:36:09 -0300
> Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com> wrote:
>
> > On 02/24, Jonathan Cameron wrote:
> > > On Tue, 20 Feb 2018 10:28:04 -0300
> > > Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com> wrote:
> > >
> > > > This patch fixes the checkpatch.pl warning:
> > > >
> > > > drivers/iio/dummy/Kconfig:21: WARNING: please write a paragraph that
> > > > describes the config symbol fully
> > > > drivers/iio/dummy/Kconfig:29: WARNING: please write a paragraph that
> > > > describes the config symbol fully
> > > >
> > > > This patch expands the explanation about IIO_DUMMY_EVGEN by using the
> > > > code documentation found in iio/dummy/iio_dummy_evgen.c. In the same
> > > > way, the information related to IIO_SIMPLE_DUMMY_BUFFER was extracted
> > > > from file iio_simple_dummy_buffer.c. Finally, this patch apply the
> > > > coding-style for Kconfig files (Documentation/process/coding-style.rst)
> > > >
> > > > Signed-off-by: Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com>
> > >
> > > Several different things going on in the one patch, please do
> > > it as a short series with one thing per patch.
> > >
> > > Certainly do the coding style separately from the additional documentation.
> > >
> > > Thanks,
> > >
> > > Jonathan
> >
> > So, should I split it into two different patches? One patch for the fix
> > and other for the extra documentation?
> Yes that would be perfect.
Ok, I will send a v2 with two patches.
Thanks
> >
> > Finally, in the two warnings related to "please write a paragraph that
> > describes the config symbol fully" the extra documentation that fixes
> > it. Other fixes, are related to the coding-style.rst however checkpatch
> > does not complain about them.
>
> Sure, checkpatch doesn't get everything.
>
> Jonathan
> >
> > >
> > > > ---
> > > > drivers/iio/dummy/Kconfig | 26 ++++++++++++++++----------
> > > > 1 file changed, 16 insertions(+), 10 deletions(-)
> > > >
> > > > diff --git a/drivers/iio/dummy/Kconfig b/drivers/iio/dummy/Kconfig
> > > > index 5a29fbd3c531..4a7127fb2979 100644
> > > > --- a/drivers/iio/dummy/Kconfig
> > > > +++ b/drivers/iio/dummy/Kconfig
> > > > @@ -9,20 +9,24 @@ config IIO_DUMMY_EVGEN
> > > > tristate
> > > >
> > > > config IIO_SIMPLE_DUMMY
> > > > - tristate "An example driver with no hardware requirements"
> > > > - depends on IIO_SW_DEVICE
> > > > - help
> > > > - Driver intended mainly as documentation for how to write
> > > > - a driver. May also be useful for testing userspace code
> > > > - without hardware.
> > > > + tristate "An example driver with no hardware requirements"
> > > > + depends on IIO_SW_DEVICE
> > > > + help
> > > > + Driver intended mainly as documentation for how to write
> > > > + a driver. May also be useful for testing userspace code
> > > > + without hardware.
> > > >
> > > > if IIO_SIMPLE_DUMMY
> > > >
> > > > config IIO_SIMPLE_DUMMY_EVENTS
> > > > - bool "Event generation support"
> > > > - select IIO_DUMMY_EVGEN
> > > > - help
> > > > - Add some dummy events to the simple dummy driver.
> > > > + bool "Event generation support"
> > > > + select IIO_DUMMY_EVGEN
> > > > + help
> > > > + Add some dummy events to the simple dummy driver.
> > > > +
> > > > + The purpose of this is to generate 'fake' event interrupts thus
> > > > + allowing that driver's code to be as close as possible to that
> > > > + a normal driver talking to hardware.
> > > >
> > > > config IIO_SIMPLE_DUMMY_BUFFER
> > > > bool "Buffered capture support"
> > > > @@ -32,6 +36,8 @@ config IIO_SIMPLE_DUMMY_BUFFER
> > > > help
> > > > Add buffered data capture to the simple dummy driver.
> > > >
> > > > + Buffer handling elements of industrial I/O reference driver.
> > > > + Uses the kfifo buffer.
> > > > endif # IIO_SIMPLE_DUMMY
> > > >
> > > > endmenu
> > >
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-02-24 13:54 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-02-20 13:28 [PATCH] iio:dummy: Fix coding style in Kconfig Rodrigo Siqueira
2018-02-24 12:24 ` Jonathan Cameron
2018-02-24 12:36 ` Rodrigo Siqueira
2018-02-24 13:44 ` Jonathan Cameron
2018-02-24 13:53 ` Rodrigo Siqueira
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).