Received: by 2002:a05:7412:d8a:b0:e2:908c:2ebd with SMTP id b10csp988461rdg; Fri, 13 Oct 2023 07:12:09 -0700 (PDT) X-Google-Smtp-Source: AGHT+IGTI6BIcCj+bAeqYozPKeCVStBH/apfLPfb0Z9+8wXHf+jq2pqETL3iYJVTGG4+7ny+EZq/ X-Received: by 2002:a17:903:32c1:b0:1ca:701:b0c8 with SMTP id i1-20020a17090332c100b001ca0701b0c8mr1503684plr.8.1697206328947; Fri, 13 Oct 2023 07:12:08 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1697206328; cv=none; d=google.com; s=arc-20160816; b=fM26LL8I43NfF6QZFrHBX9DFNNXKg7GWf+9uNQHLPbP2bjbsD3MldBI6lzY58r4oTG kyZy6ATS+4l98w40wFI6+RMdtX4mBRI2+llqv662mqR5A6T2vKO5JNTs3mGAyGdOKYHM AthmEovm+wjWibMevjSEf08Sz+YgU2XwibZ4WimmEs3aJjCdzXrsfkNU4Y+Kf9FgrHM9 UzSek+Tv/7Vp8L/OQ4FF758uE+Nrn10f4ntbHqX8D1aiMsxjMMpksqYnVYoHg2PGtsdv Yo8V1mGTFzxsWvV/zQxb56X+k0J5T8112YLPIw53cS5edWRYbCxoOmMGO6WfjmRndGWf HJlg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=3crenYOZz8vPxxiStao8xcBzWq+omg0z20ByfKHGQNY=; fh=/VZ+MKEMOpLn9o3kZPcAQjLXFwewI/pBZEhihq5m0JQ=; b=1CWZro/RDSLclbSf6Y1ToeXUeqYDaphx6jecUn5/0wJEf0foz7inTl2tNLncfdwWyv lAcAeQZbewa0j4nlPsGk25Ixq8Z3QXhU9ZJlvjLVOdLQcuzfwQe+kMLnnuqX5veHPmCm SqrrouXHkditdlqBmryX9A8TOj/1Mf6FfyQk/ngK98wF0cEe7mzivpXyjFuDshsic7DW BT4ifW983pX3zWrGNw8HWfX/GFJdkwxWqmsiP44MfxbkrMC4m9z/pxzXtnX7CxzEmWCJ lQHY2i7+tgYgDCQy/fYkLWtFlNyYAJcFbA+TQuSWgbcxftUSyYyxoiRE9iRdeM9Nbv70 21xA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=BDgw2I9R; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:8 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from fry.vger.email (fry.vger.email. [2620:137:e000::3:8]) by mx.google.com with ESMTPS id b7-20020a170903228700b001c746b986e2si5007139plh.346.2023.10.13.07.12.07 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 13 Oct 2023 07:12:08 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:8 as permitted sender) client-ip=2620:137:e000::3:8; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=BDgw2I9R; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:8 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by fry.vger.email (Postfix) with ESMTP id E8D1680A054C; Fri, 13 Oct 2023 07:11:15 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at fry.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232024AbjJMOLD (ORCPT + 99 others); Fri, 13 Oct 2023 10:11:03 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58320 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231194AbjJMOLB (ORCPT ); Fri, 13 Oct 2023 10:11:01 -0400 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8F07595 for ; Fri, 13 Oct 2023 07:11:00 -0700 (PDT) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 76931C433C7; Fri, 13 Oct 2023 14:10:59 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1697206260; bh=yblgfOK1RNan7Z74qeOHcSvb7Lln2WHm9wFm6gXbhf8=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=BDgw2I9RUVi8FmfXMdDwfGWWNWixrlRokLda/7jno+fJ0koZRbGXp4dIOEsmtjntk 6fTddidzhz/mBuaPzrgogyNNdQ4S/lmQAX+MOzr6brBSVnw8KeMcPv8anPgzJdIlKd 4trW7twlDDnTL7za8haPzW56eNNAUr/sVEvMqRvex2XCQQRRqBNKLpBFiZJEMthMLz 0RKpWnMpUztZPjibvq/g53a3cHwBso92RdxE1Hu11O9jXTAoWHQ+EnBQkRbto/DNjS qQ8ECY/K6f3DxAk2jfce54Obn1QWtZMLUuYoodqBF+OM0hH1f5lS2YGjeCbn5ELlkc kJow/8QLddtPw== Date: Fri, 13 Oct 2023 15:10:56 +0100 From: Mark Brown To: wangweidong.a@awinic.com Cc: lgirdwood@gmail.com, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, perex@perex.cz, tiwai@suse.com, herve.codina@bootlin.com, shumingf@realtek.com, rf@opensource.cirrus.com, arnd@arndb.de, 13916275206@139.com, ryans.lee@analog.com, linus.walleij@linaro.org, ckeepax@opensource.cirrus.com, fido_max@inbox.ru, sebastian.reichel@collabora.com, colin.i.king@gmail.com, liweilei@awinic.com, trix@redhat.com, dan.carpenter@linaro.org, alsa-devel@alsa-project.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH V1 3/3] ASoC: codecs: Add aw88399 amplifier driver Message-ID: References: <20231013104220.279953-1-wangweidong.a@awinic.com> <20231013104220.279953-4-wangweidong.a@awinic.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="ralK2eSCf+ixFXP0" Content-Disposition: inline In-Reply-To: <20231013104220.279953-4-wangweidong.a@awinic.com> X-Cookie: Save energy: Drive a smaller shell. X-Spam-Status: No, score=-1.2 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on fry.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 (fry.vger.email [0.0.0.0]); Fri, 13 Oct 2023 07:11:16 -0700 (PDT) --ralK2eSCf+ixFXP0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline 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. > +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? > +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? --ralK2eSCf+ixFXP0 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmUpT+8ACgkQJNaLcl1U h9CaAggAgc0J/p2bTSZNLjaIAKHFr9ldNYbl+3tMxNJwBnPfL3p07scPxSt/foY8 rLrtsYpfxKahAjbKlN1R9JtZVYuMyUfO33ed2Ww093sWB513nZMd31yCzh1+sY1g DlRT6ezLSe2XTkGTt2mFyzJcqLZOIrIzfOndism4/Esuk872ziqlaMTjq/2GBLVb WNY4D5bUcRxlFl1jphCDCSsJYCy9lpZMLaHPWrlbJh0tq1k5r7WvJ/rG0vOhGQTN GT5x1j0CifyeA5x5TjI/wWuJhAVRjHxc4NYaI8806i34cybrI8JUIf98JYyD/M7+ 7xDFdzxO/GXPZzSRctMol3uf+grzhQ== =22g1 -----END PGP SIGNATURE----- --ralK2eSCf+ixFXP0--