[PATCH v5 02/13] ASoC: Intel: catpt: Define DSP operations

Rojewski, Cezary cezary.rojewski at intel.com
Mon Sep 21 12:54:02 CEST 2020


On 2020-09-18 3:52 PM, Andy Shevchenko wrote:
> On Thu, Sep 17, 2020 at 03:29:36PM +0000, Rojewski, Cezary wrote:
>> On 2020-09-16 5:44 PM, Andy Shevchenko wrote:
>>> On Tue, Sep 15, 2020 at 06:29:33PM +0200, Cezary Rojewski wrote:
> 
> ...
> 
>>>> +	dmac->regs = cdev->lpe_ba +
>>>> +		     cdev->spec->host_dma_offset[CATPT_DMA_DEVID];
>>>
>>> One line?
>>>
>> It's exactly 81 after one-lining - that's why I kept it as is.
> 
> And 81 is okay when there is better readability.
> 
> I'm wondering why you don't go 100...
> 
> In any case it's not a reason to resend.
>

As I'll be sending v7, this nitpick will be addressed there too.

Thanks,
Czarek



More information about the Alsa-devel mailing list