RE: [PATCH 1/2] ASoC: da7213: Delete an error message for a failed memory allocation in da7213_fw_to_pdata()

2017-11-24 Thread Adam Thomson
On 24 November 2017 11:33, Julia Lawall wrote: > On Fri, 24 Nov 2017, Adam Thomson wrote: > > > On 23 November 2017 17:33, SF Markus Elfring wrote: > > > > > From: Markus Elfring > > > Date: Thu, 23 Nov 2017 17:56:54 +0100 > > > > > > Omit an extra message for a

RE: [PATCH 1/2] ASoC: da7213: Delete an error message for a failed memory allocation in da7213_fw_to_pdata()

2017-11-24 Thread Adam Thomson
On 24 November 2017 11:33, Julia Lawall wrote: > On Fri, 24 Nov 2017, Adam Thomson wrote: > > > On 23 November 2017 17:33, SF Markus Elfring wrote: > > > > > From: Markus Elfring > > > Date: Thu, 23 Nov 2017 17:56:54 +0100 > > > > > > Omit an extra message for a memory allocation failure in

RE: [PATCH 1/2] ASoC: da7213: Delete an error message for a failed memory allocation in da7213_fw_to_pdata()

2017-11-24 Thread Julia Lawall
On Fri, 24 Nov 2017, Adam Thomson wrote: > On 23 November 2017 17:33, SF Markus Elfring wrote: > > > From: Markus Elfring > > Date: Thu, 23 Nov 2017 17:56:54 +0100 > > > > Omit an extra message for a memory allocation failure in this function. > > I'm not sure

RE: [PATCH 1/2] ASoC: da7213: Delete an error message for a failed memory allocation in da7213_fw_to_pdata()

2017-11-24 Thread Julia Lawall
On Fri, 24 Nov 2017, Adam Thomson wrote: > On 23 November 2017 17:33, SF Markus Elfring wrote: > > > From: Markus Elfring > > Date: Thu, 23 Nov 2017 17:56:54 +0100 > > > > Omit an extra message for a memory allocation failure in this function. > > I'm not sure this is an extra message. We

RE: [PATCH 1/2] ASoC: da7213: Delete an error message for a failed memory allocation in da7213_fw_to_pdata()

2017-11-24 Thread Adam Thomson
On 23 November 2017 17:33, SF Markus Elfring wrote: > From: Markus Elfring > Date: Thu, 23 Nov 2017 17:56:54 +0100 > > Omit an extra message for a memory allocation failure in this function. I'm not sure this is an extra message. We don't return -ENOMEM as a

RE: [PATCH 1/2] ASoC: da7213: Delete an error message for a failed memory allocation in da7213_fw_to_pdata()

2017-11-24 Thread Adam Thomson
On 23 November 2017 17:33, SF Markus Elfring wrote: > From: Markus Elfring > Date: Thu, 23 Nov 2017 17:56:54 +0100 > > Omit an extra message for a memory allocation failure in this function. I'm not sure this is an extra message. We don't return -ENOMEM as a result of this operation and probe