Received: by 2002:a05:7412:d8a:b0:e2:908c:2ebd with SMTP id b10csp2349734rdg; Mon, 16 Oct 2023 01:10:49 -0700 (PDT) X-Google-Smtp-Source: AGHT+IGjTwWV76xAVJYXOrJ8zrRx48U5Q1kPIgtC98O/d+t9CsUK4vzvOv/1vYdYGCyaoGjc9m6W X-Received: by 2002:a05:6808:d54:b0:3af:b467:88b4 with SMTP id w20-20020a0568080d5400b003afb46788b4mr39371552oik.35.1697443849407; Mon, 16 Oct 2023 01:10:49 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1697443849; cv=none; d=google.com; s=arc-20160816; b=XnMCI2TbfxUZ3AyOsFlkSXNktCjgm7BjIjE/1aCf2ggqivFWhyq8Eq77+nf2bQO6Rd Y2GUOy/IFBghPWvs0hHZxCvvbI4uNbQqOdkNIRJu/0zx9rfH/XxYcN3/KaPwKVYgfy0x FK06GhdvtP3FX7NburjT/0zScdQNUF3HEVWDnmbKjV//XynrPS+wDAFgBQmIeaJi8HDk XX4zebTTyfEBVENtBspwCZRdaYDMY9joO0841w7I6e+glEx9Uqwq9dGd0qbVRD6owu1+ z6c4RcYKEz0s728muJHr05zgXdZWj3fSi5eaiXD9tiuKUohNbuCOIUzTGgoJWW9dAp22 xkOg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from; bh=cB9ohWb5xpYmylFkZCUpDW/jCSnSoxzAPtuF6NvEufU=; fh=6B73KIKx0C02K0HFHMQGy21RtTS94RQNuOXnLPNCJio=; b=M/vifzSCpKaj3LqGFUoelddCU3CoNRy2miKRJ5G0fp3qVPqG9vHTnlyN1nPD6F+qZt uxTWeNW5z+p1a5Na+jo1dBatM1HW/JUhoCOT54q8CEaodu8uOrZY9POYBXcfUo6rCvBn +/PZxxkUSI5Ij4tmwUl3TpQwc1oIOIDJ6vUazOQvHY6KId7tDs+90e+TWcY2/4F1/Oi9 UdetdWq1tlEDO/i8lFhWsuzWYhFqOQLtx4PQ4n2XaUGb96vvnB2T9mT6FmY/zPql3J92 mMV3br/P5bFlV2SDDlplSbbUAGVNaECK7PksWk+FMx8i7wluQ37zvvuTDj+OojZ9FAOc Bh5Q== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:3 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from lipwig.vger.email (lipwig.vger.email. [2620:137:e000::3:3]) by mx.google.com with ESMTPS id v6-20020a63f206000000b005652bd27e80si9961775pgh.588.2023.10.16.01.10.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 16 Oct 2023 01:10:49 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:3 as permitted sender) client-ip=2620:137:e000::3:3; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:3 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by lipwig.vger.email (Postfix) with ESMTP id 190E3806B9E7; Mon, 16 Oct 2023 01:10:46 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at lipwig.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229501AbjJPIKj (ORCPT + 99 others); Mon, 16 Oct 2023 04:10:39 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47292 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231861AbjJPIKh (ORCPT ); Mon, 16 Oct 2023 04:10:37 -0400 Received: from out28-172.mail.aliyun.com (out28-172.mail.aliyun.com [115.124.28.172]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 28A9EEB; Mon, 16 Oct 2023 01:10:31 -0700 (PDT) X-Alimail-AntiSpam: AC=CONTINUE;BC=0.07441231|-1;CH=green;DM=|CONTINUE|false|;DS=CONTINUE|ham_system_inform|0.0174846-0.000237189-0.982278;FP=0|0|0|0|0|-1|-1|-1;HT=ay29a033018047209;MF=wangweidong.a@awinic.com;NM=1;PH=DS;RN=26;RT=26;SR=0;TI=SMTPD_---.V04yDun_1697443818; Received: from ubuntu-VirtualBox..(mailfrom:wangweidong.a@awinic.com fp:SMTPD_---.V04yDun_1697443818) by smtp.aliyun-inc.com; Mon, 16 Oct 2023 16:10:26 +0800 From: wangweidong.a@awinic.com To: broonie@kernel.org Cc: 13916275206@139.com, alsa-devel@alsa-project.org, arnd@arndb.de, ckeepax@opensource.cirrus.com, colin.i.king@gmail.com, conor+dt@kernel.org, dan.carpenter@linaro.org, devicetree@vger.kernel.org, fido_max@inbox.ru, herve.codina@bootlin.com, krzysztof.kozlowski+dt@linaro.org, lgirdwood@gmail.com, linus.walleij@linaro.org, linux-kernel@vger.kernel.org, liweilei@awinic.com, perex@perex.cz, rf@opensource.cirrus.com, robh+dt@kernel.org, ryans.lee@analog.com, sebastian.reichel@collabora.com, shumingf@realtek.com, tiwai@suse.com, trix@redhat.com, wangweidong.a@awinic.com, yijiangtao@awinic.com Subject: Re: [PATCH V1 3/3] ASoC: codecs: Add aw88399 amplifier driver Date: Mon, 16 Oct 2023 16:10:17 +0800 Message-ID: <20231016081017.46963-1-wangweidong.a@awinic.com> X-Mailer: git-send-email 2.41.0 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-0.7 required=5.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,UNPARSEABLE_RELAY autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lipwig.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (lipwig.vger.email [0.0.0.0]); Mon, 16 Oct 2023 01:10:46 -0700 (PDT) Thank you very much for your review On Fri, Oct 13, 2023 at 03:10:56 PM +0100, broonie@kernel.org wrote: > On Fri, Oct 13, 2023 at 06:42:20PM +0800, wangweidong.a@awinic.com wrote: > This looks good - some *very* minor comments below. >> +static const struct regmap_config aw88399_remap_config = { >> + .val_bits = 16, >> + .reg_bits = 8, >> + .max_register = AW88399_REG_MAX - 1, > I see this is already the case for the aw88261 driver but it is a bit > weird that _REG_MAX isn't the maximum register - it looks like it should > probably be _NUM_REG. Not the end of the world though. Thank you very much, I will modify it to ".max_register = AW88399_REG_MAX;" >> +static int aw_dev_dsp_update_container(struct aw_device *aw_dev, >> + unsigned char *data, unsigned int len, unsigned short base) >> +{ >> + int i, ret; >> + >> +#ifdef AW88399_DSP_I2C_WRITES >> + u32 tmp_len; > This looks like debug code which can hopefully be removed in favour of > the regmap implementation? Thank you very much, but this is not a debugging statement, This is designed to be compatible with different platforms that may not support continuous writing. >> +static int aw88399_codec_probe(struct snd_soc_component *component) >> +{ >> + struct snd_soc_dapm_context *dapm = snd_soc_component_get_dapm(component); >> + struct aw88399 *aw88399 = snd_soc_component_get_drvdata(component); >> + int ret; >> + >> + ret = aw88399_request_firmware_file(aw88399); >> + if (ret < 0) { >> + dev_err(aw88399->aw_pa->dev, "%s failed\n", __func__); >> + return ret; >> + } >> + >> + INIT_DELAYED_WORK(&aw88399->start_work, aw88399_startup_work); >> + >> + /* add widgets */ >> + ret = snd_soc_dapm_new_controls(dapm, aw88399_dapm_widgets, >> + ARRAY_SIZE(aw88399_dapm_widgets)); >> + if (ret < 0) >> + return ret; >> + >> + /* add route */ >> + ret = snd_soc_dapm_add_routes(dapm, aw88399_audio_map, >> + ARRAY_SIZE(aw88399_audio_map)); >> + if (ret < 0) >> + return ret; >> + >> + ret = snd_soc_add_component_controls(component, aw88399_controls, >> + ARRAY_SIZE(aw88399_controls)); > You should just be able to pass these arrays in the component strucutre? Thank you very much. I will modify it Best regards, Weidong Wang