From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lj1-f179.google.com (mail-lj1-f179.google.com [209.85.208.179]) (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 18EBA172760 for ; Thu, 5 Sep 2024 18:19:48 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.179 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1725560390; cv=none; b=SvU5R3X7lphbcIfQPl0zDTWpcdSYCaJI3qMrPgqqCOYgcCHstRGFT3iQ25C/JOXE4RjVxLjXEnCI8Nbtk4Sc18Bd/J1Ybdf5NFF6TAZ1dnHpjLMIV0WHPc4q0YuL1XQA8/TxqM+rpw+tVFqQSVoQ40/1sKIH+qew6kJW4v01ITE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1725560390; c=relaxed/simple; bh=DEcRTH4Ksx3HqcRZk39EA6SSP0yZhl3amYb4gGAKjSw=; h=MIME-Version:References:In-Reply-To:From:Date:Message-ID:Subject: To:Cc:Content-Type; b=iEBDjF609zWu82XQ9rtqPhSOQPUGH2ypB8ysiVZSRhriVQ+Q0WjLpZskx1ECQKiQFhmA3dtzppCTtPvtssjU0DvJiBeLIHP8MVMaI6vUEHoYs9hG6lT/xlFBxOf5JRNRn4YAVWWPGWT9WOJzrQ8XKirrDVg3OwiK8ydiG1xJpwA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bgdev.pl; spf=none smtp.mailfrom=bgdev.pl; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b=C8Bh7oqr; arc=none smtp.client-ip=209.85.208.179 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bgdev.pl Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=bgdev.pl Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b="C8Bh7oqr" Received: by mail-lj1-f179.google.com with SMTP id 38308e7fff4ca-2f3e9fb6ee9so15174311fa.3 for ; Thu, 05 Sep 2024 11:19:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bgdev-pl.20230601.gappssmtp.com; s=20230601; t=1725560387; x=1726165187; darn=vger.kernel.org; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=WtLGB7A2P30eTe38aKz83I8MagUAeBQdY4kLOW9iG00=; b=C8Bh7oqrWIgMZ2zoZq5olIW051cgItRHSx32mybhIXDB737qIPuJ3gyn/E2dVGN/TO AZNGhMxITOUkQLLlmPNm7FW/x9MrbNeY8vYOuAzXYQCCRyVScgfcAdlbl868erkDgmuD CCvfdsPh8KeJtwHqn25cP2NSqyjKyptCWZ2RRsN7QWDKyOLV74sr5cxw0sfc+WuSvRun fH0L8WRw8wLPOJNiN96vKq3PcgFEU1CkMIZSVScFiTnJlBzK0o9qqO/IIfCG6KBVUYgj Qp62S6meXhu/LQgcOFAJz5wWvWbbGN+/7KeSCKPoHCYOxATR4BsxiJcDAAMnivqZv0nm mmQg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1725560387; x=1726165187; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=WtLGB7A2P30eTe38aKz83I8MagUAeBQdY4kLOW9iG00=; b=dBsJpb/dqmbBfU0xy0LIQjp0bvLbvdc5OhFJVJRwPAmLDbLwZ2s7/M7EIQBxPzE42O lFzd3Nefqp6jHKZjTBuUUg1j32/IcU35WnaO5u1C891pEwPk5kh2ANRjLlnfaKLf9VCV B8L9TKzge8F94oWJfHzWcM+sDjcdGQNr65ducwLTn90yL7bCYT5zt1yME2UBJqj3wdb8 lQjoiLyDhnf3WiD6JWD8ovup6Z15W4xgb3Pu2ARNvci9th93EBVFEAFj3IvjqEGBnDuU ORRl8R8sOVKeKyAHYGCjU5NRRVQXfRfkK9d3S0kBGXQUqwd2RZenu1WuuniITqqXzS7l jafw== X-Forwarded-Encrypted: i=1; AJvYcCV3CCEUGPIS1xQZgSRgLmUY7chdE1ZWWGdMc/e2UtXW4EBItrpcOwdkQ5N2B18ba5eapiJfeFHTmhFX@vger.kernel.org X-Gm-Message-State: AOJu0YyBy/OsMfcVIpsTUq8XLXNgmyCl+Bi0nXazFP/d/uYyPLNywqby kM0+AHMD53LgjorP16S+JaK7GhVuF/gJoX9DxhXcdTQFv0vXcrOw5uN7Tue/AIqwJcdAemYG+Jw iiRllcyETrwy8o7FnE/LcU6geIe7XTemObLb2Ig== X-Google-Smtp-Source: AGHT+IGO7FS+EJM8t1Z0R7LnnK3tpz34UapBrYrk0FGwJ34Y7Hp9Wg/QTQV1SbaJDUTpykBxZNv37DoXB5odh9EO2F4= X-Received: by 2002:a05:6512:104f:b0:536:54d6:e6d6 with SMTP id 2adb3069b0e04-53654d6e81bmr1544176e87.17.1725560386293; Thu, 05 Sep 2024 11:19:46 -0700 (PDT) Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 References: <20240814082301.8091-1-brgl@bgdev.pl> <87a5hcyite.fsf@kernel.org> <87y146ayrm.fsf@kernel.org> In-Reply-To: <87y146ayrm.fsf@kernel.org> From: Bartosz Golaszewski Date: Thu, 5 Sep 2024 20:19:34 +0200 Message-ID: Subject: Re: [PATCH net-next v2] dt-bindings: net: ath11k: document the inputs of the ath11k on WCN6855 To: Kalle Valo Cc: "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Jeff Johnson , linux-wireless@vger.kernel.org, netdev@vger.kernel.org, devicetree@vger.kernel.org, ath11k@lists.infradead.org, linux-kernel@vger.kernel.org, Bartosz Golaszewski Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Thu, Sep 5, 2024 at 5:47=E2=80=AFPM Kalle Valo wrote: > > Bartosz Golaszewski writes: > > >> > + - if: > >> > + properties: > >> > + compatible: > >> > + contains: > >> > + const: pci17cb,1103 > >> > + then: > >> > + required: > >> > + - vddrfacmn-supply > >> > + - vddaon-supply > >> > + - vddwlcx-supply > >> > + - vddwlmx-supply > >> > + - vddrfa0p8-supply > >> > + - vddrfa1p2-supply > >> > + - vddrfa1p8-supply > >> > + - vddpcie0p9-supply > >> > + - vddpcie1p8-supply > >> > >> Like we discussed before, shouldn't these supplies be optional as not > >> all modules need them? > >> > > > > The answer is still the same: the ATH11K inside a WCN6855 does - in > > fact - always need them. The fact that the X13s doesn't define them is > > bad representation of HW and I'm fixing it in a subsequent DTS patch. > > But, like we discussed earlier, M.2 boards don't need these so I think > this should be optional. > If they are truly dynamic, plug-and-play M.2 boards then they shouldn't need any description in device-tree. If they are M.2 sockets that use custom, vendor-specific pins (like what is the case on sc8280xp-crd and X13s) then the HW they carry needs to be described correctly. We've discussed that before. Bart