Received: by 2002:a05:6a10:d5a5:0:0:0:0 with SMTP id gn37csp285635pxb; Thu, 30 Sep 2021 06:08:37 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxXsHy9TSI57xluBsfyhjZVXfz4ObW1d2kzBwdM5Nsmv0y7byVBmr33ec+/3ZQD78/2Zy7m X-Received: by 2002:aa7:94a2:0:b0:447:ce69:7601 with SMTP id a2-20020aa794a2000000b00447ce697601mr4364489pfl.46.1633007317208; Thu, 30 Sep 2021 06:08:37 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1633007317; cv=none; d=google.com; s=arc-20160816; b=iy6bEqAlPMU7ngptfYQOUw32sFeC/nAK06RKvK/9eH57466Rygw4ftYwMuVGPZEOm8 l0ZKabpvRL10YwOMbaAyVNH+VC35JEMWrHOYSARxHBPMeQn9vMBHxyL8g4Xe/rfsofOh vhlj+IGMu2Gx1HNCzCBQyaNzq60aVz/0ZcX4JR4RdZWeOxHWGlDWoQwNbLTuDYSfGjkg G+KdBGnLAQ/vfViKXIay3AGTrkM9xCmziYyEn0fH9LjPj8fowGRvRWAAnzzCIoKeExO9 XxiF7RdeUYFhbsalhjU1vpfe8xnHQ/bi/GqVJK5mBJVqcwOuwMc3jn3AwWI9kYWB2Pur htDg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:user-agent:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :dkim-signature; bh=nWzXTg3QEaqkpCMGyZoW3dqBPp5HEQnQOA/Z2UqaDnI=; b=HIgpOfxiO6mLJmfmUqVatC7IYkj95OmMT8qwiM2r6W7c4f7gMD8/0Qket2SIKgkk5/ K5XzlegMN9pcq9wP9Twdu1PaJ4nKwTteDYbrr9MF3mEoi5hzi0Yjeifzy2ao4rByj7eR lzltG4UYVIbZTE7EcLaaZTvsMaGDJ1ylmRKYTc8hpZ6Wh+3p2iiPllpPAdwkFayZjRA0 sXHntiMB7AG2Q+URHWRRWdR/GprX7AAA5npW996H/tVCnosMGKxyo0FLCji8RVskdiFl bAOvH2PTCZF6CckEeyrpMjHxrTnMurYHLSJiMVLlJtE7A4sFrsCrLiaSX9PYFrFQF6oq 8DgA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=Jc3KonpS; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 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 vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id g12si5077306pjj.16.2021.09.30.06.08.17; Thu, 30 Sep 2021 06:08:37 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=Jc3KonpS; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1350653AbhI3MHo (ORCPT + 99 others); Thu, 30 Sep 2021 08:07:44 -0400 Received: from mail.kernel.org ([198.145.29.99]:49910 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1350490AbhI3MHn (ORCPT ); Thu, 30 Sep 2021 08:07:43 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 70345619E9; Thu, 30 Sep 2021 12:06:00 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1633003561; bh=WUXfDUc//JyjnG/jAFLzYhzrcKKxnKio8LRuz/Sa4SE=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Jc3KonpSQ9wIu8cbERb6KVjpU6LDGCPs80PSGa/jbjpkFYlKB8EBessNIIDdMPlZb mdOf3r6qDaOiloFsel5NQFH3syLotorSV9/I78onpGZBBaHVJCl9L4vPLz/wf/0B0X MigPNDhw6MAbsNwoOFmlyBA64VHiQxxhEOmmTpR5yvZGfkzKdSt8hwBIBchkgo6p2D Ud6vVQ8s/VLgdsAI936Z9lJVThvdHNORnb1q9dT20/+MoEYv+s1WYysiDGDspTmQt8 M1VHpHnhCPa64p8SOr36dNqfZ9XR4xSJnqmJoXTc+l6XPyxypm90eVSrFcEYoWvhG/ 202AMiuiTRmJQ== Date: Thu, 30 Sep 2021 13:05:10 +0100 From: Mark Brown To: Chunyan Zhang Cc: Liam Girdwood , Rob Herring , DTML , Baolin Wang , Orson Zhai , Chunyan Zhang , LKML , Lee Jones Subject: Re: [PATCH 1/2] regulator: Add Unisoc's SC2730 regulator driver Message-ID: <20210930120510.GW4199@sirena.org.uk> References: <20210928073609.198975-1-zhang.lyra@gmail.com> <20210928113057.GJ4199@sirena.org.uk> <20210929122912.GO4199@sirena.org.uk> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="mz5eWKkhz39mMVQV" Content-Disposition: inline In-Reply-To: X-Cookie: 98% lean. User-Agent: Mutt/1.10.1 (2018-07-13) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --mz5eWKkhz39mMVQV Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Thu, Sep 30, 2021 at 11:12:28AM +0800, Chunyan Zhang wrote: > On different platforms, sc27xx_spi consists of different sub-devices > with different compatible string, to avoid adding mfd_cells to > sc27xx_spi driver each time we add a new platform support, we changed > to use devm_of_platform_populate() which can automatically register > sun-devices listed in devicetree. > The above is my understand, please correct me if I'm missing something > @Lee Jones Right, so my pushback here is that it's not clear to me that avoiding adding compatibles to the MFD is a strong enough goal to justify creating ABI - it's hard to see the upsides. This is particularly the case here where the subdevice is really a random collection of devices that's particularly likely to change between variants and more likely that other OSs would want something different so it's even harder to achieve reuse. --mz5eWKkhz39mMVQV Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmFVp/UACgkQJNaLcl1U h9A0NQf+MVIV20djUxEYTyRsu5XcsUBXOCjUH8C6aiUvI1f+91mdMR5EjZGsWxOP Qxind1nfGXf7+41sboK/15s4xWsWpFDHq/4wBUkHdgiqoywBuYisngyieNGhVNeB 3FSJSdW0aNHNNNAjOSG0YXQRxog5XSBRSQ3s2614QFVXGFHh0DQoin2ud2rgPeZ0 H7A6BV2E0LR+w3YNs/twWw7j4aM/wkrVTCc9OfPNdX5gNC1SoK6KC6ApfZYUyT0G k2jcq3qnQyGuadQPkpgo9X3tW7enbWzyGAL3aEN/jbaugSPsiS3pJsPizR00n5Jz VIb/B5YtJxgs+XIw4I6aELxDkaDdhw== =OZXI -----END PGP SIGNATURE----- --mz5eWKkhz39mMVQV--