* [PATCH] ASoC: samsung: i2s: Add clocks' macros descriptions [not found] <CGME20190924114843eucas1p2ea9c36289d3912e022ad7c9070fc00ee@eucas1p2.samsung.com> @ 2019-09-24 11:48 ` Marek Szyprowski 2019-09-24 11:53 ` Krzysztof Kozlowski 0 siblings, 1 reply; 3+ messages in thread From: Marek Szyprowski @ 2019-09-24 11:48 UTC (permalink / raw) To: devicetree, alsa-devel, linux-kernel, linux-samsung-soc Cc: Rob Herring, Mark Rutland, Krzysztof Kozlowski, Mark Brown, Sylwester Nawrocki, Liam Girdwood, Maciej Falkowski, Marek Szyprowski From: Maciej Falkowski <m.falkowski@samsung.com> To increase macro readability added descriptions to clocks macros. Signed-off-by: Maciej Falkowski <m.falkowski@samsung.com> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com> --- include/dt-bindings/sound/samsung-i2s.h | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/include/dt-bindings/sound/samsung-i2s.h b/include/dt-bindings/sound/samsung-i2s.h index 77545f14c379..fd752475c762 100644 --- a/include/dt-bindings/sound/samsung-i2s.h +++ b/include/dt-bindings/sound/samsung-i2s.h @@ -2,8 +2,13 @@ #ifndef _DT_BINDINGS_SAMSUNG_I2S_H #define _DT_BINDINGS_SAMSUNG_I2S_H -#define CLK_I2S_CDCLK 0 -#define CLK_I2S_RCLK_SRC 1 -#define CLK_I2S_RCLK_PSR 2 +/* the CDCLK (CODECLKO) gate clock */ +#define CLK_I2S_CDCLK 0 + +/* the RCLKSRC mux clock (corresponding to RCLKSRC bit in IISMOD register) */ +#define CLK_I2S_RCLK_SRC 1 + +/* the RCLK prescaler divider clock (corresponding to the IISPSR register) */ +#define CLK_I2S_RCLK_PSR 2 #endif /* _DT_BINDINGS_SAMSUNG_I2S_H */ -- 2.17.1 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] ASoC: samsung: i2s: Add clocks' macros descriptions 2019-09-24 11:48 ` [PATCH] ASoC: samsung: i2s: Add clocks' macros descriptions Marek Szyprowski @ 2019-09-24 11:53 ` Krzysztof Kozlowski 2019-09-24 17:53 ` Mark Brown 0 siblings, 1 reply; 3+ messages in thread From: Krzysztof Kozlowski @ 2019-09-24 11:53 UTC (permalink / raw) To: Marek Szyprowski Cc: devicetree, alsa-devel, linux-kernel, linux-samsung-soc, Rob Herring, Mark Rutland, Mark Brown, Sylwester Nawrocki, Liam Girdwood, Maciej Falkowski On Tue, Sep 24, 2019 at 01:48:38PM +0200, Marek Szyprowski wrote: > From: Maciej Falkowski <m.falkowski@samsung.com> > > To increase macro readability added descriptions > to clocks macros. > > Signed-off-by: Maciej Falkowski <m.falkowski@samsung.com> > Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com> > --- > include/dt-bindings/sound/samsung-i2s.h | 11 ++++++++--- > 1 file changed, 8 insertions(+), 3 deletions(-) > > diff --git a/include/dt-bindings/sound/samsung-i2s.h b/include/dt-bindings/sound/samsung-i2s.h > index 77545f14c379..fd752475c762 100644 > --- a/include/dt-bindings/sound/samsung-i2s.h > +++ b/include/dt-bindings/sound/samsung-i2s.h > @@ -2,8 +2,13 @@ > #ifndef _DT_BINDINGS_SAMSUNG_I2S_H > #define _DT_BINDINGS_SAMSUNG_I2S_H > > -#define CLK_I2S_CDCLK 0 > -#define CLK_I2S_RCLK_SRC 1 > -#define CLK_I2S_RCLK_PSR 2 > +/* the CDCLK (CODECLKO) gate clock */ > +#define CLK_I2S_CDCLK 0 I do not find it more readable because of removal of indent after define name. Also the description is not accurate - you documented, not increased readability. Best regards, Krzysztof > + > +/* the RCLKSRC mux clock (corresponding to RCLKSRC bit in IISMOD register) */ > +#define CLK_I2S_RCLK_SRC 1 > + > +/* the RCLK prescaler divider clock (corresponding to the IISPSR register) */ > +#define CLK_I2S_RCLK_PSR 2 > > #endif /* _DT_BINDINGS_SAMSUNG_I2S_H */ > -- > 2.17.1 > > > ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ASoC: samsung: i2s: Add clocks' macros descriptions 2019-09-24 11:53 ` Krzysztof Kozlowski @ 2019-09-24 17:53 ` Mark Brown 0 siblings, 0 replies; 3+ messages in thread From: Mark Brown @ 2019-09-24 17:53 UTC (permalink / raw) To: Krzysztof Kozlowski Cc: Marek Szyprowski, devicetree, alsa-devel, linux-kernel, linux-samsung-soc, Rob Herring, Mark Rutland, Sylwester Nawrocki, Liam Girdwood, Maciej Falkowski [-- Attachment #1: Type: text/plain, Size: 713 bytes --] On Tue, Sep 24, 2019 at 01:53:07PM +0200, Krzysztof Kozlowski wrote: > On Tue, Sep 24, 2019 at 01:48:38PM +0200, Marek Szyprowski wrote: > > +/* the CDCLK (CODECLKO) gate clock */ > > +#define CLK_I2S_CDCLK 0 > I do not find it more readable because of removal of indent > after define name. Also the description is not accurate - you > documented, not increased readability. Yeah, I agree on both points. How about putting everything on one line more like: #define CLK_I2S_CDCLK 0 /* the CDCLK (CODECLKO) gate clock */ instead? Please fix your mail client to word wrap within paragraphs at something substantially less than 80 columns. Doing this makes your messages much easier to read and reply to. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-09-24 17:53 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CGME20190924114843eucas1p2ea9c36289d3912e022ad7c9070fc00ee@eucas1p2.samsung.com>
2019-09-24 11:48 ` [PATCH] ASoC: samsung: i2s: Add clocks' macros descriptions Marek Szyprowski
2019-09-24 11:53 ` Krzysztof Kozlowski
2019-09-24 17:53 ` Mark Brown
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).