Re: [alsa-devel] [PATCH 0/1] Submitting MAX98088 codec patch
On Thu, Jun 09, 2011 at 05:50:16PM +0900, Jin Park wrote:
The max98088 codec driver had unnecessary duplicated codes on DAI operations. For example, max98088_dai1_hw_params and max98088_dai2_hw_params, they are almost same, except only register offsets. So I have combined DAI operations for DAI1 and DAI2. Please review my patch.
Note that there's no need to have a cover letter for a single patch - any useful information should be in the patch itself.
On 09/06/11 11:47, Mark Brown wrote:
On Thu, Jun 09, 2011 at 05:50:16PM +0900, Jin Park wrote:
The max98088 codec driver had unnecessary duplicated codes on DAI operations. For example, max98088_dai1_hw_params and max98088_dai2_hw_params, they are almost same, except only register offsets. So I have combined DAI operations for DAI1 and DAI2. Please review my patch.
Note that there's no need to have a cover letter for a single patch - any useful information should be in the patch itself. _______________________________________________
Acked-by: Liam Girdwood lrg@ti.com
participants (2)
-
Liam Girdwood -
Mark Brown