From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f99.google.com (mail-pj1-f99.google.com [209.85.216.99]) (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 B017B43FD0D for ; Mon, 10 Aug 2026 19:22:00 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.99 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786389723; cv=none; b=Hf5jZG7aLZT+/JZLhnG1QMZ47UZB6EDvmSIE+Os426rmIYR6pjMJRMxDKFOtmZUiu6OEBOD1Pn9MsCbOOEp7tKIuMPoYV02ZYjnrw9R6OUeXTFvl9nUJCoQSOTcgIFe/X+JNnWhOD3eSdktXYULbt+GAoa5X83gDNqqsE8DRGek= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786389723; c=relaxed/simple; bh=byp/DGoBmK0QSy7inRGYoZ+tVgEDH9/VqoouG77P4Yw=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=JSl6mSCBlIWOyM1WQFJwey/6/691jBwLgn2oFoh3iFgmmjo7xIx7jVx43tg0o5IvkT76y+3CoQmgmjHilHPU3014n0SvsICSYxd81nPpC866XlSQDKwgNkM+q72V8HhZMmWvjbqrbXqRrWtGxs8CQ2R7H+NYo1VcR8hZG3cW+DU= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=broadcom.com; spf=fail smtp.mailfrom=broadcom.com; dkim=pass (1024-bit key) header.d=broadcom.com header.i=@broadcom.com header.b=bq0JnlIt; arc=none smtp.client-ip=209.85.216.99 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=broadcom.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=broadcom.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=broadcom.com header.i=@broadcom.com header.b="bq0JnlIt" Received: by mail-pj1-f99.google.com with SMTP id 98e67ed59e1d1-3811f512167so3205080a91.3 for ; Mon, 10 Aug 2026 12:22:00 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786389720; x=1786994520; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:dkim-signature:x-gm-gg:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to:content-type; bh=6dsTj9bVR22kgCOopy9QSe5LsU/US12529STCamM+18=; b=ZZ+7sehvWvNEanlJTBXkBrQlxFXELvpJvAYaWLZ4q9o6qQmP2A5lPtXSWeWE73ls0v xGAAvGlyR3kMl6/vhPnnHYqB42e2hatJw5axlillDbfgiTGG7+8G0jyDreK3yuQwYa7o 6POGxBKg36YAY/CLDOINikQ+ui3VPTqGBq4DXdPQWXQ7rL2lpRNVsd+oSSnX95oWtXBL kBrpFFXfXqqc+IsrzoIzQ3gC5U+E3u6ilHQ782saAQ0fQofHagd8LSmaap1/MoJ3idGf t2l8wp/E2HvVKBSTggaiJU60kbGuGFx5Q8cFs8VVt/lBSC5DaU5e7Kc56Sz1UwtVQ19x 5uiw== X-Forwarded-Encrypted: i=1; AHgh+RqBAZhwIXznE5ghpE9zDsJnV2Atbt7sWNV4sG5GybKlN1snSG2NTCU8dJ3MtoDjI2qdFaf1bGNokEYA/lE=@vger.kernel.org X-Gm-Message-State: AOJu0YxjWZq9Q2xQoytMz4lNvULY5gAR24orFjn7wkL3v6Dde64efiH7 uGnqHslCdlldIHfZJxBTC37hXgfRq3P2TPGG86tZ0uEEXGI3f2aF77LY47DL5IaOYfHK0LcN3cW +DXpMh9VdIkj6frBcCs1tME2vj8R7oui+dGx7wCe1shH+am/QOEvhkm0ZRAixEldise1c1ieDZG XgmDRFQY1Z31GMT3sNhTdmmQaThvs7b38DjDvAUBhVkuAxRdmLchMDcggJOfvr5nK5C/Th/ZaYo hU3C7V/67DiNpqaesOl5X7K7w== X-Gm-Gg: AR+sD12AG07oW1OW5+q7O4j7vTdZr6gA7sQAwNF/88RDymx+L7JLLew3N/7OrKgeaff WbaLu8FAHu4o/hP3SOyE6ZJwUSZcXW4UyTYdE2m5nQPw72lqWvyTbxkNVsZ8/ZvyGpbZ8M4Wfp7 M+DxuuyV1nM2ZpLXdnr/l3AgRBJNwTb3QnqEJtRabFgzDQnN2bEf40mHarLCmZJbdQzoKJ9iR3P QaFFtG5u5W6cb5RArsVf2vbpC/LaY6C2Frz0Kd1MvNwtfCwLm0kqJkX4TrO7btlJvwnszYW1vrc tOM2rqX4wXy5BGwk9QF05/DW/z1gEJungBVGiko5pwy/G1n/1NpO/jA8Hx8Fy45cHAyYy1BkVtz TtaD3BM/C5v1xxaeGOVSR6yMNMfdMn2l9tHb8pHSbn+iVUbTGTf4a1P/6aR6T7jJ0tbcJuTesj6 PonEjyexmW9UpUIrTSHLGDWC+KPEP/6Cr8PXWcH+eLFg== X-Received: by 2002:a17:90b:4c11:b0:38e:8021:2ea9 with SMTP id 98e67ed59e1d1-392ccab0c92mr3638391a91.19.1786389719664; Mon, 10 Aug 2026 12:21:59 -0700 (PDT) Received: from smtp-us-east1-p01-i01-si01.dlp.protect.broadcom.com (address-144-49-247-120.dlp.protect.broadcom.com. [144.49.247.120]) by smtp-relay.gmail.com with ESMTPS id 98e67ed59e1d1-392d5206928sm244863a91.10.2026.08.10.12.21.59 for (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Mon, 10 Aug 2026 12:21:59 -0700 (PDT) X-Relaying-Domain: broadcom.com X-CFilter-Loop: Reflected Received: by mail-qt1-f200.google.com with SMTP id d75a77b69052e-5283df62d68so27438971cf.0 for ; Mon, 10 Aug 2026 12:21:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; t=1786389718; x=1786994518; darn=vger.kernel.org; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:from:to:cc:subject:date:message-id:reply-to :content-type; bh=6dsTj9bVR22kgCOopy9QSe5LsU/US12529STCamM+18=; b=bq0JnlIt0S3HmE9MjcRDZNLHAOGz9EUChaV/k/4cprgeuAjLtZeSxxit2GF7i0vQ7r B6YtqKe5mz45ItUIxswo1jXZzV9HVKIksuaEbc0jlT/oU//ZSse9vk+0GE7jYr0b2onA 4us/y4o+Z2nI6Dduk0XH0nRVMS8vWIpbzDOfo= X-Forwarded-Encrypted: i=1; AHgh+RomW6UvFZqgPkpJ+3b64jViqYkhBGolq68UzM2tVj8bhaaxlcanLUyYgrbPdcRNMwdrwidVs0odTfkjCGA=@vger.kernel.org X-Received: by 2002:ac8:7f41:0:b0:519:ef86:127d with SMTP id d75a77b69052e-52d4bb91b9fmr47670211cf.18.1786389718298; Mon, 10 Aug 2026 12:21:58 -0700 (PDT) X-Received: by 2002:ac8:7f41:0:b0:519:ef86:127d with SMTP id d75a77b69052e-52d4bb91b9fmr47669641cf.18.1786389717837; Mon, 10 Aug 2026 12:21:57 -0700 (PDT) Received: from [192.168.178.26] (f215227.upc-f.chello.nl. [80.56.215.227]) by smtp.gmail.com with ESMTPSA id d75a77b69052e-52d1659c1d8sm79841321cf.16.2026.08.10.12.21.55 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 10 Aug 2026 12:21:57 -0700 (PDT) Message-ID: Date: Mon, 10 Aug 2026 21:21:56 +0200 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] dt-bindings: net: wireless: brcm,bcm4329-fmac: add nvmem-consumer reference To: Rob Herring Cc: Rosen Penev , devicetree@vger.kernel.org, Johannes Berg , Krzysztof Kozlowski , Conor Dooley , linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org References: <20260727225311.266596-1-rosenp@gmail.com> <20260802083905.3819981-7-arend.vanspriel@broadcom.com> <20260810173040.GA2218158-robh@kernel.org> Content-Language: en-US From: Arend van Spriel In-Reply-To: <20260810173040.GA2218158-robh@kernel.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-DetectorID-Processed: b00c1d49-9d2e-4205-b15f-d015386d3d5e On 10/08/2026 19:30, Rob Herring wrote: > On Sun, Aug 02, 2026 at 10:39:04AM +0200, Arend van Spriel wrote: >> On Mon, 27 Jul 2026 15:53:11 -0700, Rosen Penev wrote: >>> The brcm,bcm4329-fmac binding has unevaluatedProperties set to false >>> but does not allow nvmem-cells or nvmem-cell-names, which are used by >>> DTS files (e.g. bcm4709-linksys-ea9200.dts) to supply MAC addresses. >>> >>> The driver already supports nvmem-based MAC address retrieval via >>> of_get_mac_address(), so add the nvmem-consumer schema reference to >>> document and allow these properties. >>> --- >>> .../devicetree/bindings/net/wireless/brcm,bcm4329-fmac.yaml | 1 + >> >> The problem is real but I think the fix belongs in network-class.yaml rather >> than the brcmfmac binding specifically. ethernet-controller.yaml already >> defines nvmem-cells and nvmem-cell-names for wired NICs, but the wireless >> binding hierarchy (wireless-controller.yaml -> network-class.yaml) is missing >> them. Fixing it in network-class.yaml would cover all network nodes at once >> instead of requiring each wireless driver to add the same reference >> individually. > > It is wrong because nvmem-consumer.yaml does not define how many entries > and what they are. IOW, the valid names in nvmem-cell-names must be > defined. > > However, expecting all network class devices to use the same > nvmem-cells/nvmem-cell-names will probably not work long term. Some > device may use it, but not use "mac-address" entry. Unfortunately, > there's not a way to override a default in json-schema. So we'll > probably have to just define this in every user rather than centrally. > > So this should just add the definitions to the brcm,bcm4329-fmac > binding: > > nvmem-cells: > maxItems: 1 > > nvmem-cell-names: > const: mac-address Hi, Rob I try to stay away from devicetree bindings so I am not well-aware of the limitations to express things. Just applying my aim-for-reuse mantra ;-) Thanks for your guidance. Regards, Arend