Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <f621882b-f962-15f4-ac74-4e93edc5fde6@intel.com>
Date:   Mon, 7 Mar 2022 17:19:52 +0100
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Firefox/91.0 Thunderbird/91.6.1
Subject: Re: [PATCH v2] ASoC: Intel: catpt: use asoc_substream_to_rtd()
Content-Language: en-US
To:     Lianjie Zhang <zhanglianjie@uniontech.com>,
        Jaroslav Kysela <perex@perex.cz>
CC:     Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>,
        Liam Girdwood <liam.r.girdwood@linux.intel.com>,
        Jie Yang <yang.jie@linux.intel.com>,
        "Mark Brown" <broonie@kernel.org>, Takashi Iwai <tiwai@suse.com>,
        <alsa-devel@alsa-project.org>, <linux-kernel@vger.kernel.org>
References: <20220307151939.32870-1-zhanglianjie@uniontech.com>
From:   Cezary Rojewski <cezary.rojewski@intel.com>
In-Reply-To: <20220307151939.32870-1-zhanglianjie@uniontech.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?WXh2czN6U1pFZmtiRzRMZEZRRHlmbFJlR0lEWnhOU2VqSGRsVkx0a2ozLzFk?=
 =?utf-8?B?dFBFSWRxQTVPVit0MEsyUGdnSzVpeXB2emgreUtVblJkalRBcExHQW9yVGto?=
 =?utf-8?B?b3hLbzl6UUJqNGRkV3QrNkk3RzlOaEFLY2tESmNmb0xwRlNGRnlFOWVZdGhk?=
 =?utf-8?B?ZGp0UkhjZk5vdzZ6eVh2UWd6SEQ5N082S292elpLSTR0LzhnaktIZktEZm1U?=
 =?utf-8?B?aCtzZ28rRDc4RWxIc0kvOFRzUGdzSDU5eWROdEFMM25ob1hxRHZ1VTk3ZGlt?=
 =?utf-8?B?SFRxbnBPT2VWQzRUVHZEUjlwdjAxeGxNSko5N1MrTUVCR2Znc2I2UzVoUUFZ?=
 =?utf-8?B?b21SSWxVS2paUFJUQ3RqNUNaR2tNSHVWdUdqWDBHMGY0ZCthN3A1YzFaVGtI?=
 =?utf-8?B?M2Y4eTZoY1VTM0xYNldqbnd1SnJ0MWZyQkQyQzFNUFFySldNd01uV1JnNWRB?=
 =?utf-8?B?RzNORjgxUzZJTFh3Rzh3bGJjTFBoalZlYysrMHF6ZHU1Q0NNZTYyVDltbk5R?=
 =?utf-8?B?OFF3WSt1aGJSYXRYNGhlazBiRnFRdXp3ZTcwaWZxMjltd21KcnV5NmhRczVj?=
 =?utf-8?B?UW4zeE5YU05PZFByN05MczNqZGpGMHlDS2VOQnZEMElYcGFPQzkvSFY1QkxZ?=
 =?utf-8?B?WFhjdWp3WVJlK1diNGJnY2JuTlIvUWVsQzc3bHltK3M1K2QrOW9vVE5Ud0J0?=
 =?utf-8?B?MDJOd3BVdUFWVitvenlmeXpDcUVBRFo3YVZoRDdrOHdDaUEyQWJSWXVvK0E2?=
 =?utf-8?B?OGh4amJUUm1SU000anNHV2RnWU5ZaW1iNmVGK1I5Z1JaWklLTTIxeTg1NlBr?=
 =?utf-8?B?VjNyKzlxbE9ycW9ROHA1MmRuL0kwRnp0VEhkeDJMbUFlUHJtZkJDMWt6cWRQ?=
 =?utf-8?B?OTNwWWt0R3A5TFVLb25ETzI4VHRHOSs3elp3NmxlSXNBK29STG9zcWZiakN2?=
 =?utf-8?B?MldHN2RwUDNMVGVhK1daaXhxMVEzcHdkcldzbnZOYldveG0rWnpMcEF6QktT?=
 =?utf-8?B?YTdtNldvMGZQM2JFQUpNSXNoYjZRTHZid2RYWHorSDk2N0JWQk1kc2NlY05M?=
 =?utf-8?B?aHJkL0tPN1RwaCtBeXlwN0hMbUhxMzBDWTRXb2dvZzZ5ZDJ4T2FoMlZkWTNW?=
 =?utf-8?B?TXlSRkVUa0RkWXJ4eHpIRWI1NitJMmdLUlZxL1dIbjRFeTdGVXRGMWJxRlZP?=
 =?utf-8?B?eDJwU0VUZ3ZzcVhvbW9Uc2RFU1l0Q09PYWR3Nm8zSnExTDE1cUdGMng2Wi9u?=
 =?utf-8?B?ZjhXbGJ3VjZtYzZFc1lmM2ptZDd1S2hNRXc5UlB6M1NNK0RIaDRwbE04ckFv?=
 =?utf-8?B?MWxKc2VHenpvVXdPUkxzc2llbGJ6QjI4aGQ0Y1NWRVRWUE93bkorQjhONUdS?=
 =?utf-8?B?b0JDMzRjYThGcXgxdEFaMzI3M09IVU5hZmJ5M2NMc3ZFQUNBSjJiSXJrUThx?=
 =?utf-8?B?eWIrbTk3SkxlSFpzdjJzc1dtdkwzSEZ1cXBwUzJPMXVGVHpTcVhuZThVT0p6?=
 =?utf-8?B?UFNnTWlyM09XeEkxcWVlZEJSK0RzdG1RM2pzVFAydm1FdzFtcjdxSHMybHNL?=
 =?utf-8?B?VXA0RlpCbjRJUGZpbVNYeUZac3hGL2lPazNHS1B4TmkxZFN4SWpCbVprYUVU?=
 =?utf-8?B?NHRSWEprWldkMHpmdVNKLzNNRFVCdXZUK3ZqQ0RucUZOc29jUEc0eVlpcFJ1?=
 =?utf-8?B?WlZ1MTVQK2JNWkdrelU3TDBoVWlYLzZhZllIL1R6VnJad3ZDc1ZRZUkyYWNr?=
 =?utf-8?B?dEtVMGFVekkzcGlWZStCaVMyQWx6R0ZYVmlndVRKczhwZ0ZvTWdoV1F3a2lR?=
 =?utf-8?B?MEhxYnpEZU1nWDdLaGdVa2NIT1l2YWsxZW9UaGRETlFxOTJ6WVlHRkRISVVY?=
 =?utf-8?B?SkhCTm1UMVZ5RUk2VEVHQlY5YUdsSUdMZHFhckZJYlFyNThqU3EvOGx5T3gw?=
 =?utf-8?B?eDVWL1orUlN3NzNLeEpxTDZJTWRsQ2ZhRzNOeXIyZnovaDVrV3E0UlFRRlRQ?=
 =?utf-8?B?ckxSWUJtNHhPNUdCdkZ5dmF3Uzg0THg3QTlWRW1ZckRvSXp0cVhlV1RvUkJS?=
 =?utf-8?B?SXk0QzR4cFdDK3FxZklodytSZFk3Q2NmcjFVcEhOU09EbFpmSUY4eHhaaEpy?=
 =?utf-8?B?L3hSdnorcjVsZFB0OHRlbHFxZFBhc2tBOC9zV1hEZm5NSXJZSFhwRkpxV2gx?=
 =?utf-8?B?dm5lZ3ZGbkF6RnJnaCtFNnZQT3dBWkRGMlVvc3pLMUhwQmZIMXRtcWtSRzBG?=
 =?utf-8?Q?ROaWRj8sjuaSKTTXUnzRcTTClzl7PtcXEvki+jI1f4=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 72cc8f77-de17-47e7-fcee-08da005654ff
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1101MB2157.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Mar 2022 16:20:02.3658
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: X5OD37w5pxmzoqOjjkahWdHcU/6zZSJyAuQtg3MRNaaAaFPPR00Wv6IYbp7W8uDTvyHi2hjyVBkP2MlVqcUbX8LzIosCMtpu13VA6RuM6W4=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR11MB4135
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-7.6 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_HI,SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 2022-03-07 4:19 PM, Lianjie Zhang wrote:
> Now we can use asoc_substream_to_rtd() macro,
> let's use it.
> 
> Signed-off-by: Lianjie Zhang <zhanglianjie@uniontech.com>
> Reviewed-by: Cezary Rojewski <cezary.rojewski@intel.com>


Hello,

Please do not add reviewer tags until it's actually provided by them.
In regard to v2, thanks for addressing the title. Patch LGTM.


Regards,
Czarek