From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f46.google.com (mail-wr1-f46.google.com [209.85.221.46]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6679629B77C for ; Wed, 8 Oct 2025 02:18:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.46 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1759889933; cv=none; b=HGLc0Mdkyo9Oep0zicT69+svA+8NBCHb3JaOSqPxCTw8gJ3OVxHpQ7bpUqO2KXcEgTGmAr2BLjV4blz7EYD+PJkufOkD79WVq1yO95s3hCHKAtg2z4ZtO5dlY3dBvxL2YBm6idZQ4tYLqJ/9yjX+I4RhPXepZgQjxluvSkCYyvw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1759889933; c=relaxed/simple; bh=eLUKCid2mCzEr3f2IULaa6Mi8dzk04iOMQ1Cu+awDD0=; h=Mime-Version:Content-Type:Date:Message-Id:Cc:Subject:From:To: References:In-Reply-To; b=IQVTycfxv6JYg5dDkHuhASnE4u2S3n5qXLVyGKqSM6YRux4kht99d4P0wzfECW8Bifg1YtJFicCfvGrfM8t5KYipL7NktB7UVMy6TValRNdeQ1IYbfnp1K8hg1dXCRQR7lgtSv0TTHn47kotMBMB/4WjrBCG1G5PuZKuxzoPcFE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=ahaQ+NAg; arc=none smtp.client-ip=209.85.221.46 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="ahaQ+NAg" Received: by mail-wr1-f46.google.com with SMTP id ffacd0b85a97d-3c68ac7e18aso1070642f8f.2 for ; Tue, 07 Oct 2025 19:18:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1759889929; x=1760494729; darn=vger.kernel.org; h=in-reply-to:references:to:from:subject:cc:message-id:date :content-transfer-encoding:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=micbjQhFRi56rmBZKMQCddkVTulgFVdWxWszGqRTF54=; b=ahaQ+NAgrWEPQIWuYqcDIOvUoZ3mn+pgDminfwb0D23x9JG3hjU5U92zv2E3HqJMO6 gNawUbzwU2KVurUIPikMg9FOl2X/zB+5JZ0Qm+pQTMzwia3+/qHPF+/sJYrxsMj9KqHg GoiozcubMSwutgCST7fk3Fgh46HutkmdyGafcQJL76KUMdnL2bIzwpprswGJp1Bj7NAu AMIis8KZLI4JdFV6TBfdsfK+MuDnA0/wI6omoHxCAKvg9/t7Jnp2XCSBUx4AdDbCYeKf LXok8IgB0KIRL1Jr6znbw2hpYMT643pXKwmSLOn6d1VFMzZIHvJekxYTtk8bW8p+fG1m fK3Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1759889929; x=1760494729; h=in-reply-to:references:to:from:subject:cc:message-id:date :content-transfer-encoding:mime-version:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to; bh=micbjQhFRi56rmBZKMQCddkVTulgFVdWxWszGqRTF54=; b=ihLUGcjNheUMHi+5/XmwqYSENtAJzc+JbIqAp1F3SM0bVq7ezzjputJJ4e2ClhBSug 15YIb4kNzUpHpkklbTtCPXlm16hGycwCxXMB3xzoi3rVCv5taGZgaJhXxeAGBJzrq3pj Kof6gyxxcNa8YlOcUbxG9WqMD/H19xvAl+zc/AlWpN5bTmB4MrE7EpTWUKkoedy6yh1s cYtMZY3wfQP/seQKcBTXTWAawM7VaklwdCgw9JjYSo+vjKc1/ee8fUWOszBIVs+FRsJv SyXd7gfkaC4LXmT9hDzvStyWY2uNAj4VunGx5vr6IifVpNd7N6ATU/ko8XGNNOFjQ7kU vxHw== X-Forwarded-Encrypted: i=1; AJvYcCVM7z0NkAUXvkWPbHSRSisIq4B9TWrA39Vs9ovcp3HGDyxN64ilv/egBxF5Az968LVQVV3zkZ+AUSZ64wt4@vger.kernel.org X-Gm-Message-State: AOJu0Ywvurydpsto4iPXVmOQI4/nGvGO2t6BceiK47IUnIYNKK8c7OIe 3YXoFOTmsIV1Bn+yN2cXv0M6OjRSP6AlMfB6Slz2+pDi/SoKqmcubCfdoOtaUErcb9E= X-Gm-Gg: ASbGncv8MZ/tZF5QqC+qfgDKjikgJeu8gt8i3nskNF5Y/QYgG5/TT/BkaQfeFJeRvhI IemUPrXjk4GzVjOjANz4EDdwlmpeSTZKoe1kXXEDljwkmOZd/MjXN64Qw1Nj6KMVaRfZFtBFuts cR3xv8adhpxn4R7CloXw8WjtA7NDet/GCECb34Tj8fFgpICaOyS5hwjYbRWxP2L73HiIu7aukEe +Y2r8dsSb8d5ayshIOAfkVoWXrTqAbWs1JAsmAL7Uch1ozpE6qHH8Ybm6NjQW93z2ZfaXXAy3kh 9pzJUGaUeGB45LR+UpmJkutg9HCfl5btdlFj6zJ4eRAybP0moYiAq5FlK3jUgQAEvAisZOD6lKK ZeuqWvt4rujiTzbfYurDtWHqWM6uQqTSY2qAGd66i1SGBOtMgJAdokqw= X-Google-Smtp-Source: AGHT+IFPMFJByHAQ3JW76Qjmt7UP+fMZMpkYgj4Y9V9tn3CZdsWTV3HC+ojk6qaBDfa1ihXXI4WNNA== X-Received: by 2002:a05:6000:3105:b0:3fc:54ff:edb6 with SMTP id ffacd0b85a97d-4266e7dfdb3mr815912f8f.35.1759889928751; Tue, 07 Oct 2025 19:18:48 -0700 (PDT) Received: from localhost ([2a02:c7c:7259:a00:8c32:dd4d:57f2:8be7]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-46fa9c16990sm15668705e9.10.2025.10.07.19.18.47 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 07 Oct 2025 19:18:48 -0700 (PDT) Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Wed, 08 Oct 2025 03:18:47 +0100 Message-Id: Cc: "Srinivas Kandagatla" , "Liam Girdwood" , "Mark Brown" , "Rob Herring" , "Krzysztof Kozlowski" , "Conor Dooley" , "Stephen Boyd" , "Lee Jones" , "Jaroslav Kysela" , "Takashi Iwai" , , , , , "Dmitry Baryshkov" , "Srinivas Kandagatla" , Subject: Re: [PATCH v4 2/4] dt-bindings: mfd: qcom,spmi-pmic: add qcom,pm4125-codec compatible From: "Alexey Klimov" To: "Krzysztof Kozlowski" X-Mailer: aerc 0.20.0 References: <20250915-pm4125_audio_codec_v1-v4-0-b247b64eec52@linaro.org> <20250915-pm4125_audio_codec_v1-v4-2-b247b64eec52@linaro.org> <20250918-wonderful-deft-jackal-7d3bbc@kuoka> In-Reply-To: <20250918-wonderful-deft-jackal-7d3bbc@kuoka> On Thu Sep 18, 2025 at 3:03 AM BST, Krzysztof Kozlowski wrote: > On Mon, Sep 15, 2025 at 05:27:49PM +0100, Alexey Klimov wrote: >> Add qcom,pm4125-codec compatible to pattern properties in mfd >> qcom,spmi-pmic schema so the devicetree for this audio block of PMIC >> can be validated properly. >>=20 >> Signed-off-by: Alexey Klimov >> --- >> Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml | 6 ++++++ >> 1 file changed, 6 insertions(+) >>=20 >> diff --git a/Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml b= /Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml >> index 078a6886f8b1e9ceb2187e988ce7c9514ff6dc2c..776c51a66f6e7260b7e3e183= d693e3508cbc531e 100644 >> --- a/Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml >> +++ b/Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml >> @@ -137,6 +137,12 @@ patternProperties: >> =20 >> "^audio-codec@[0-9a-f]+$": >> type: object >> + oneOf: >> + - $ref: /schemas/sound/qcom,pm8916-wcd-analog-codec.yaml# >> + - properties: >> + compatible: >> + const: qcom,pm4125-codec > > > Not much improved. Same feedback applies. Around the time of sending this I thought to set separate follow-up patch that fixes the other part here -- pm8916-wcd-analog-codec. At this point, is it fine to send follow-up patch that does smth like this: + oneOf: + - $ref: /schemas/sound/qcom,pm8916-wcd-analog-codec.yaml# + - properties: + compatible: + - enaum: + - qcom,pm4125-codec + - qcom,pm8916-wcd-analog-codec ? (I didn't check how if it will compile or pass checks) Hope this is okay. Best regards, Alexey