Re: [PATCH v2 00/10] staging: ks7010: Fine-tuning for a SDIO card driver

2016-08-14 Thread Wolfram Sang
On Thu, Aug 11, 2016 at 07:17:12AM +0200, SF Markus Elfring wrote: > From: Markus Elfring > > Further update suggestions were taken into account Which ones? Please describe them to ease review. > after a patch was applied from static source code analysis. I

Re: [PATCH v2 00/10] staging: ks7010: Fine-tuning for a SDIO card driver

2016-08-14 Thread Wolfram Sang
On Thu, Aug 11, 2016 at 07:17:12AM +0200, SF Markus Elfring wrote: > From: Markus Elfring > > Further update suggestions were taken into account Which ones? Please describe them to ease review. > after a patch was applied from static source code analysis. I added some Acked- and Reviewed-by

Re: [PATCH v2 00/10] staging: ks7010: Fine-tuning for a SDIO card driver

2016-08-14 Thread SF Markus Elfring
> I added some Acked- and Reviewed-by tags last time. I noticed this of course. > Did the patches change Yes. - The amount of source code which I touched in this software module is different for the second series. > or why didn't you add them? I imagined that your acknowledgements to the

Re: [PATCH v2 00/10] staging: ks7010: Fine-tuning for a SDIO card driver

2016-08-14 Thread SF Markus Elfring
> I added some Acked- and Reviewed-by tags last time. I noticed this of course. > Did the patches change Yes. - The amount of source code which I touched in this software module is different for the second series. > or why didn't you add them? I imagined that your acknowledgements to the

[PATCH v2 00/10] staging: ks7010: Fine-tuning for a SDIO card driver

2016-08-10 Thread SF Markus Elfring
From: Markus Elfring Further update suggestions were taken into account after a patch was applied from static source code analysis. Markus Elfring (10): Delete unnecessary checks before the function call "kfree" Delete unnecessary assignments for buffer

[PATCH v2 00/10] staging: ks7010: Fine-tuning for a SDIO card driver

2016-08-10 Thread SF Markus Elfring
From: Markus Elfring Further update suggestions were taken into account after a patch was applied from static source code analysis. Markus Elfring (10): Delete unnecessary checks before the function call "kfree" Delete unnecessary assignments for buffer variables Return directly after a