On Mon, 22 May 2017 08:20:45 +0200, Julia Lawall wrote:
On Mon, 22 May 2017, Bhumika Goyal wrote:
On Sun, May 21, 2017 at 12:39 PM, Takashi Iwai tiwai@suse.de wrote:
On Wed, 17 May 2017 14:33:45 +0200, Bhumika Goyal wrote:
On Wed, May 17, 2017 at 5:50 PM, Takashi Iwai tiwai@suse.de wrote:
On Wed, 17 May 2017 13:38:56 +0200, Bhumika Goyal wrote:
Declare snd_kcontrol_new structures as const as they are only passed an argument to the function snd_ctl_new1. This argument is of type const, so snd_kcontrol_new structures having this property can be made const. Done using Coccinelle:
@r disable optional_qualifier@ identifier x; position p; @@ static struct snd_kcontrol_new x@p={...};
@ok@ identifier r.x; position p; @@ snd_ctl_new1(&x@p,...)
@bad@ position p != {r.p,ok.p}; identifier r.x; @@ x@p
@depends on !bad disable optional_qualifier@ identifier r.x; @@ +const struct snd_kcontrol_new x;
Signed-off-by: Bhumika Goyal bhumirks@gmail.com
As I already asked, could you send as a patchset if you have multiple similar patches?
Sure. Will combine the already sent ones and some new ones into a patchset and send again.
Well, since there haven't been new submissions and the patches are trivial, I applied the existing four patches. If you have newer ones, please submit as a patchset.
Hey,
Thanks for applying the patches. I have a doubt that since some of my coming patches are for the sound subsystem containing similar changes and you will be applying those, so can I combine and put all the changes in a single patch or sending a patchset would be a correct way.
I think he means that when you have more sound patches, just do them all at once.
Yes, that's my point.
You've sent each patch individually, and this made me difficult to pick up, as I had to wonder "is this really all?" (then 10 minutes later another patch arrived :)
At best, submit all the relevant patches via git-send-email with the proper cover letter in a single thread.
thanks,
Takashi