[PATCH] ASoC: fsl_asrc: protect macro argument

[PATCH] ASoC: fsl_asrc: protect macro argument

[Stefan Agner]

Protect macro argument with parentheses to avoid ambiguity.
This fixes a warning seen with clang:
  warning: logical not is only applied to the left hand side of this comparison

Signed-off-by: Stefan Agner <stefan@agner.ch>
---
 sound/soc/fsl/fsl_asrc.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/fsl/fsl_asrc.h b/sound/soc/fsl/fsl_asrc.h
index 0f163abe4ba3..ec33dab4b909 100644
--- a/sound/soc/fsl/fsl_asrc.h
+++ b/sound/soc/fsl/fsl_asrc.h
@@ -57,7 +57,7 @@
 #define REG_ASRDOC			0x74
 #define REG_ASRDI(i)			(REG_ASRDIA + (i << 3))
 #define REG_ASRDO(i)			(REG_ASRDOA + (i << 3))
-#define REG_ASRDx(x, i)			(x == IN ? REG_ASRDI(i) : REG_ASRDO(i))
+#define REG_ASRDx(x, i)			((x) == IN ? REG_ASRDI(i) : REG_ASRDO(i))
 
 #define REG_ASRIDRHA			0x80
 #define REG_ASRIDRLA			0x84
-- 
2.13.3

Re: [PATCH] ASoC: fsl_asrc: protect macro argument

[Nicolin Chen]

On Sun, Aug 06, 2017 at 11:36:57PM -0700, Stefan Agner wrote:
> Protect macro argument with parentheses to avoid ambiguity.
> This fixes a warning seen with clang:
>   warning: logical not is only applied to the left hand side of this comparison
> 
> Signed-off-by: Stefan Agner <stefan@agner.ch>

Acked-by: Nicolin Chen <nicoleotsuka@gmail.com>

Thanks

> ---
>  sound/soc/fsl/fsl_asrc.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/sound/soc/fsl/fsl_asrc.h b/sound/soc/fsl/fsl_asrc.h
> index 0f163abe4ba3..ec33dab4b909 100644
> --- a/sound/soc/fsl/fsl_asrc.h
> +++ b/sound/soc/fsl/fsl_asrc.h
> @@ -57,7 +57,7 @@
>  #define REG_ASRDOC			0x74
>  #define REG_ASRDI(i)			(REG_ASRDIA + (i << 3))
>  #define REG_ASRDO(i)			(REG_ASRDOA + (i << 3))
> -#define REG_ASRDx(x, i)			(x == IN ? REG_ASRDI(i) : REG_ASRDO(i))
> +#define REG_ASRDx(x, i)			((x) == IN ? REG_ASRDI(i) : REG_ASRDO(i))
>  
>  #define REG_ASRIDRHA			0x80
>  #define REG_ASRIDRLA			0x84
> -- 
> 2.13.3
> 

Re: [PATCH] ASoC: fsl_asrc: protect macro argument

[Stefan Agner]

On 2017-08-07 09:08, Nicolin Chen wrote:
> On Sun, Aug 06, 2017 at 11:36:57PM -0700, Stefan Agner wrote:
>> Protect macro argument with parentheses to avoid ambiguity.
>> This fixes a warning seen with clang:
>>   warning: logical not is only applied to the left hand side of this comparison
>>
>> Signed-off-by: Stefan Agner <stefan@agner.ch>
> 
> Acked-by: Nicolin Chen <nicoleotsuka@gmail.com>

Mark, you seem to have merged other patches for this driver, can you
merge this one too?

--
Stefan

> 
> Thanks
> 
>> ---
>>  sound/soc/fsl/fsl_asrc.h | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/sound/soc/fsl/fsl_asrc.h b/sound/soc/fsl/fsl_asrc.h
>> index 0f163abe4ba3..ec33dab4b909 100644
>> --- a/sound/soc/fsl/fsl_asrc.h
>> +++ b/sound/soc/fsl/fsl_asrc.h
>> @@ -57,7 +57,7 @@
>>  #define REG_ASRDOC			0x74
>>  #define REG_ASRDI(i)			(REG_ASRDIA + (i << 3))
>>  #define REG_ASRDO(i)			(REG_ASRDOA + (i << 3))
>> -#define REG_ASRDx(x, i)			(x == IN ? REG_ASRDI(i) : REG_ASRDO(i))
>> +#define REG_ASRDx(x, i)			((x) == IN ? REG_ASRDI(i) : REG_ASRDO(i))
>>
>>  #define REG_ASRIDRHA			0x80
>>  #define REG_ASRIDRLA			0x84
>> --
>> 2.13.3
>>

Re: [PATCH] ASoC: fsl_asrc: protect macro argument

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 987 bytes --]

On Wed, Dec 06, 2017 at 02:53:39PM +0100, Stefan Agner wrote:
> On 2017-08-07 09:08, Nicolin Chen wrote:
> > On Sun, Aug 06, 2017 at 11:36:57PM -0700, Stefan Agner wrote:

> Mark, you seem to have merged other patches for this driver, can you
> merge this one too?

Please don't send content free pings and please allow a reasonable time
for review.  People get busy, go on holiday, attend conferences and so 
on so unless there is some reason for urgency (like critical bug fixes)
please allow at least a couple of weeks for review.  If there have been
review comments then people may be waiting for those to be addressed.

Sending content free pings adds to the mail volume (if they are seen at
all) which is often the problem and since they can't be reviewed
directly if something has gone wrong you'll have to resend the patches
anyway, though there are some other maintainers who like them - if in
doubt look at how patches for the subsystem are normally handled.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: [PATCH] ASoC: fsl_asrc: protect macro argument

[Stefan Agner]

On 2017-12-06 15:11, Mark Brown wrote:
> On Wed, Dec 06, 2017 at 02:53:39PM +0100, Stefan Agner wrote:
>> On 2017-08-07 09:08, Nicolin Chen wrote:
>> > On Sun, Aug 06, 2017 at 11:36:57PM -0700, Stefan Agner wrote:
> 
>> Mark, you seem to have merged other patches for this driver, can you
>> merge this one too?
> 
> Please don't send content free pings and please allow a reasonable time
> for review.  People get busy, go on holiday, attend conferences and so 
> on so unless there is some reason for urgency (like critical bug fixes)
> please allow at least a couple of weeks for review.  If there have been
> review comments then people may be waiting for those to be addressed.

Well, it was not entirely content free since I added you/Liam to the
list of recipient which I missed in the initial patch. I probably should
have mentioned that.

I did not meant to bug people, just wanted to make sure the patch does
not falls through the cracks... After all, patch as well as the Ack
already more than 3 months old....

--
Stefan

Re: [PATCH] ASoC: fsl_asrc: protect macro argument

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 1199 bytes --]

On Wed, Dec 06, 2017 at 03:25:37PM +0100, Stefan Agner wrote:
> On 2017-12-06 15:11, Mark Brown wrote:

> > Please don't send content free pings and please allow a reasonable time
> > for review.  People get busy, go on holiday, attend conferences and so 
> > on so unless there is some reason for urgency (like critical bug fixes)
> > please allow at least a couple of weeks for review.  If there have been
> > review comments then people may be waiting for those to be addressed.

> Well, it was not entirely content free since I added you/Liam to the
> list of recipient which I missed in the initial patch. I probably should
> have mentioned that.

Then it's definitely just a content free ping since I obviously don't
have the patch if you didn't send it to me and as the mail you're
replying to said you'll need to send me a copy.

> I did not meant to bug people, just wanted to make sure the patch does
> not falls through the cracks... After all, patch as well as the Ack
> already more than 3 months old....

You need to send patches to the relevant maintainers as covered in
SubmittingPatches, if you don't send us patches we won't have them and
can't apply them.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: [PATCH] ASoC: fsl_asrc: protect macro argument

[Mark Brown]

[-- Attachment #1.1: Type: text/plain, Size: 1199 bytes --]

On Wed, Dec 06, 2017 at 03:25:37PM +0100, Stefan Agner wrote:
> On 2017-12-06 15:11, Mark Brown wrote:

> > Please don't send content free pings and please allow a reasonable time
> > for review.  People get busy, go on holiday, attend conferences and so 
> > on so unless there is some reason for urgency (like critical bug fixes)
> > please allow at least a couple of weeks for review.  If there have been
> > review comments then people may be waiting for those to be addressed.

> Well, it was not entirely content free since I added you/Liam to the
> list of recipient which I missed in the initial patch. I probably should
> have mentioned that.

Then it's definitely just a content free ping since I obviously don't
have the patch if you didn't send it to me and as the mail you're
replying to said you'll need to send me a copy.

> I did not meant to bug people, just wanted to make sure the patch does
> not falls through the cracks... After all, patch as well as the Ack
> already more than 3 months old....

You need to send patches to the relevant maintainers as covered in
SubmittingPatches, if you don't send us patches we won't have them and
can't apply them.

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]

Re: [PATCH] ASoC: fsl_asrc: protect macro argument

[Stefan Agner]

On 2017-12-06 15:39, Mark Brown wrote:
> On Wed, Dec 06, 2017 at 03:25:37PM +0100, Stefan Agner wrote:
>> On 2017-12-06 15:11, Mark Brown wrote:
> 
[...]
>> I did not meant to bug people, just wanted to make sure the patch does
>> not falls through the cracks... After all, patch as well as the Ack
>> already more than 3 months old....
> 
> You need to send patches to the relevant maintainers as covered in
> SubmittingPatches, if you don't send us patches we won't have them and
> can't apply them.

Yeah this is where it gets a bit fuzzy around the affected file, there
are maintainers mentioned in the MAINTAINER file but I just realized
today that it was typically you which merged patches for the file.

Will resend the patch and add you as recipient.

--
Stefan

Re: [PATCH] ASoC: fsl_asrc: protect macro argument

[Stefan Agner]

On 2017-12-06 15:39, Mark Brown wrote:
> On Wed, Dec 06, 2017 at 03:25:37PM +0100, Stefan Agner wrote:
>> On 2017-12-06 15:11, Mark Brown wrote:
> 
[...]
>> I did not meant to bug people, just wanted to make sure the patch does
>> not falls through the cracks... After all, patch as well as the Ack
>> already more than 3 months old....
> 
> You need to send patches to the relevant maintainers as covered in
> SubmittingPatches, if you don't send us patches we won't have them and
> can't apply them.

Yeah this is where it gets a bit fuzzy around the affected file, there
are maintainers mentioned in the MAINTAINER file but I just realized
today that it was typically you which merged patches for the file.

Will resend the patch and add you as recipient.

--
Stefan

Applied "ASoC: fsl_asrc: protect macro argument" to the asoc tree

[Mark Brown]

The patch

   ASoC: fsl_asrc: protect macro argument

has been applied to the asoc tree at

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git 

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.  

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark

>From d1b726a9018e3f684ce190a1cbe012cb64f363d8 Mon Sep 17 00:00:00 2001
From: Stefan Agner <stefan@agner.ch>
Date: Wed, 6 Dec 2017 16:17:27 +0100
Subject: [PATCH] ASoC: fsl_asrc: protect macro argument

Protect macro argument with parentheses to avoid ambiguity.
This fixes a warning seen with clang:
  warning: logical not is only applied to the left hand side of this comparison

Signed-off-by: Stefan Agner <stefan@agner.ch>
Acked-by: Nicolin Chen <nicoleotsuka@gmail.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
---
 sound/soc/fsl/fsl_asrc.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/fsl/fsl_asrc.h b/sound/soc/fsl/fsl_asrc.h
index 0f163abe4ba3..ec33dab4b909 100644
--- a/sound/soc/fsl/fsl_asrc.h
+++ b/sound/soc/fsl/fsl_asrc.h
@@ -57,7 +57,7 @@
 #define REG_ASRDOC			0x74
 #define REG_ASRDI(i)			(REG_ASRDIA + (i << 3))
 #define REG_ASRDO(i)			(REG_ASRDOA + (i << 3))
-#define REG_ASRDx(x, i)			(x == IN ? REG_ASRDI(i) : REG_ASRDO(i))
+#define REG_ASRDx(x, i)			((x) == IN ? REG_ASRDI(i) : REG_ASRDO(i))
 
 #define REG_ASRIDRHA			0x80
 #define REG_ASRIDRLA			0x84
-- 
2.15.0

Applied "ASoC: fsl_asrc: protect macro argument" to the asoc tree

[Mark Brown]

The patch

   ASoC: fsl_asrc: protect macro argument

has been applied to the asoc tree at

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git 

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.  

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark

>From d1b726a9018e3f684ce190a1cbe012cb64f363d8 Mon Sep 17 00:00:00 2001
From: Stefan Agner <stefan@agner.ch>
Date: Wed, 6 Dec 2017 16:17:27 +0100
Subject: [PATCH] ASoC: fsl_asrc: protect macro argument

Protect macro argument with parentheses to avoid ambiguity.
This fixes a warning seen with clang:
  warning: logical not is only applied to the left hand side of this comparison

Signed-off-by: Stefan Agner <stefan@agner.ch>
Acked-by: Nicolin Chen <nicoleotsuka@gmail.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
---
 sound/soc/fsl/fsl_asrc.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/fsl/fsl_asrc.h b/sound/soc/fsl/fsl_asrc.h
index 0f163abe4ba3..ec33dab4b909 100644
--- a/sound/soc/fsl/fsl_asrc.h
+++ b/sound/soc/fsl/fsl_asrc.h
@@ -57,7 +57,7 @@
 #define REG_ASRDOC			0x74
 #define REG_ASRDI(i)			(REG_ASRDIA + (i << 3))
 #define REG_ASRDO(i)			(REG_ASRDOA + (i << 3))
-#define REG_ASRDx(x, i)			(x == IN ? REG_ASRDI(i) : REG_ASRDO(i))
+#define REG_ASRDx(x, i)			((x) == IN ? REG_ASRDI(i) : REG_ASRDO(i))
 
 #define REG_ASRIDRHA			0x80
 #define REG_ASRIDRLA			0x84
-- 
2.15.0