From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f45.google.com (mail-wm1-f45.google.com [209.85.128.45]) (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 38B6C3064AE for ; Wed, 17 Jun 2026 06:50:48 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.45 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781679049; cv=none; b=BJ4/0UBE8QoCI6gJSjrbkhPP+YJJIDs2qPR/0+Kb25LcR4+OPww8ACkd/f9MCM4sAYON02Fp1OqlqscwDp70RPoooJCNKTOftGDNeyxr83uBHYRLrScWX7+W2ibb4qyVuIFQZbLGI0jTHI3Uw4lTkkZpOvMqYAHlxvjvoNETCYs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781679049; c=relaxed/simple; bh=C7YFZCCu1dOeCsLg6DoF+9ZVtNHvPRq+h+Zo6dLBmB0=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=DuNjliq3HxMeytqYFNSxprO49CLFXscHSgPwTSFG1OC9WbXuTKQUFJGbYbuzMqxgtKPf+e29kVkyh/Qizq7bE7NwS4tjvn0Y9J60iyj7yUJ5jdEsFRbZ1cgOSFiNjSkH1Awl3AMqWDzarJKb/8SRDRGgchkM/9qL8AaLrSjWTXY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=WCqJCXi9; arc=none smtp.client-ip=209.85.128.45 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="WCqJCXi9" Received: by mail-wm1-f45.google.com with SMTP id 5b1f17b1804b1-490b4a8e28bso40238195e9.1 for ; Tue, 16 Jun 2026 23:50:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1781679047; x=1782283847; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=Trq1GUdrJefwFDKO73LEpHAox6MBJ2GUwy+Cih5DopY=; b=WCqJCXi9yibRSJ5D2QIBg96wVMVBsltG7hqEhWSiMMjpnepNe7JZcNqou8TvpStsC5 0O2VRVRDsZr8u6wYnVE9CMPpGHdot2XRhpzV5dySGs2ikkqUNBVPnz3FHJ7bZ1+V+TEP aihcJCNbrHAZozcBZVodaoqGw+qhMxEUK8KO5H5WWYnmMpFxSCOLQKtK/fGUyB/z8sxB ijoPo6PGn3OTd36WIVXzLOKNiutmb6rQoKwKI6s7cRcJ/20qJnOCoCpTeuT7D0VkSDkp 2SON5gb7viS29bcJumAjOzlr6/idKj4TDUPRF4JNb8d4dR/xoIkC+K73wDSZqiZ4ZhXu aoVg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1781679047; x=1782283847; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=Trq1GUdrJefwFDKO73LEpHAox6MBJ2GUwy+Cih5DopY=; b=ciOMaqnPVLrQCkSsfdkNH2JblxwVshWyI2cs+xrwOkelA+2Gm7swFuXoAtpvIoyRb5 bJNpfLXHz40+o05oTCf8bZHeJVWJF85+LqoVbv3JagIT/789MNlGKurtWLQriY06LfHc Sjgk3mds+h5wwKXSzNXXlX++omvjR1d8j5EzbzMCqvEwZYch+V9e6D9KnKlZXvy1sZxe vL0AMkV0C/ILLcFk4/kD5qbHhtUbKqsQR7qayAiB8Bvz/iDRQzbSJ8K3VXdf/oOHIPfP mb/ZpXaP+Oo/Ak0YCMPd+UqliU8OgMmCy1/3zDSq+nqsqzHxptCJ4nn7yuYUngG8vpOa O8Lg== X-Forwarded-Encrypted: i=1; AFNElJ8+qowhvRJCuavs76ktBTofNI25J1u7wBuNFb/BXqGjJyjuWRChgVU1BwSdtbXckPFPGEemVvWDqQP3lzE=@vger.kernel.org X-Gm-Message-State: AOJu0Yxvo8ksHWyn/SsV6JDnuzS2YD4s/rW+3TWaQujm1uMqB7UQ9uXi rzmQ6v9PQ5LikBHSt/N6O+Lbng9DXCMtDPsTTYLn7d0zDWtO63WD6+3v X-Gm-Gg: Acq92OED95pSev94w+6O4LvOpPbiiqB1IFlXMFRiTgWIW3WhLvplf/mvicZwLhc7Wwa 0PzUPY/O9mblFdXDA4Hl7xLXBTmHWCmD1Djkhu0PwdBNNW5/7wi8GJ6YjgcVAGMv4bsB+pFpgRb s5/2Napz3VHaLxRA4YqN1k8KPqLlMmxNrbAw0kpDvdCL9bMF5gJod255URtoymzEIENlWtBRdjd k0/L5IKMH2ziboiPDflFDFnlZuxaw8DgaZ5fklCrlecuAwKtWYiQDVDA/zz5gVB9Y5wGpL2tHJz lOaK96eGMfJXxMVRAxOTa8zQiAdhT0jKysqjaHHTy8ttKQPxIOPKwLIY/7P+3GivcBX6WS/4TYN XbSUnQiFM5gVVB/cNYP7oenjPw2g0vit8zxYCCnIKJwvC8t5w6EnyXzqC+afpDllO32mGiRPLpl WSEdaS3islYFbi0BVHN4m4Ad3AScxnsWS2FLbzz0veUSrq0XxvgfCVhz7GjGmdIsl96Evd/lApa 6fsvHR/OOaG3BQ= X-Received: by 2002:a05:600c:a111:b0:490:e5c1:b89d with SMTP id 5b1f17b1804b1-492333ffd28mr36380385e9.0.1781679046294; Tue, 16 Jun 2026 23:50:46 -0700 (PDT) Received: from ?IPV6:2001:9e8:f13a:1701:b918:4041:7c7d:eb1d? ([2001:9e8:f13a:1701:b918:4041:7c7d:eb1d]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-49233bf0881sm41430175e9.2.2026.06.16.23.50.45 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 16 Jun 2026 23:50:45 -0700 (PDT) Message-ID: <36668b89-77ef-406f-9edc-ed57a025754b@gmail.com> Date: Wed, 17 Jun 2026 08:50:44 +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 net-next v2 1/2] dt-bindings: net: pse-pd: add bindings for Realtek/Broadcom PSE MCU Content-Language: en-US To: Rob Herring Cc: Oleksij Rempel , Kory Maincent , Andrew Lunn , "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Krzysztof Kozlowski , Conor Dooley , netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Daniel Golle , =?UTF-8?Q?Bj=C3=B8rn_Mork?= References: <20260612132944.460646-1-jelonek.jonas@gmail.com> <20260612132944.460646-2-jelonek.jonas@gmail.com> <20260615212959.GA1679454-robh@kernel.org> From: Jonas Jelonek In-Reply-To: <20260615212959.GA1679454-robh@kernel.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Hi Rob, On 15.06.26 23:29, Rob Herring wrote: > On Fri, Jun 12, 2026 at 01:29:41PM +0000, Jonas Jelonek wrote: >> [...] >> >> +properties: >> + compatible: >> + enum: >> + - realtek,pse-mcu-rtk > The "rtk" feels redundant. In the full Realtek case yes. Do you have a suggestion to improve that? >> + - realtek,pse-mcu-bcm > "brcm" is the standard vendor prefix, so use that instead of "bcm". > Though who defined the protocol in this case? Realtek or Broadcom? In > the latter case, I'd argue that "brcm" should be the vendor prefix. I'll switch to brcm. As Daniel already mentioned, Realtek defines the firmware and thus the protocol, in both cases. >> + >> + reg: >> + maxItems: 1 >> + >> + power-supply: >> + description: Regulator supplying the PoE power rail. >> + >> + enable-gpios: >> + maxItems: 1 >> + >> + realtek,i2c-protocol: >> + $ref: /schemas/types.yaml#/definitions/string >> + enum: [ i2c, smbus ] >> + description: | >> + Wire framing the MCU firmware expects on the I2C bus. "smbus" means >> + reads carry a leading command byte (0x00) and a repeated start; "i2c" >> + means bare 12-byte writes and reads with no command prefix. Only >> + applies to the Realtek I2C attachment. > I tend to think this should be distinguished by the compatible string. > That would simplify the schema given it only applies to one of the > compatible strings. In theory this could also apply to the Broadcom dialect, however I didn't have a device with that variant on my desk. If you would go with separate compatibles, do you thought about something like "realtek,pse-mcu-rtk-i2c" and "realtek,pse-mcu-rtk-smbus" (given we stick with rtk) ? > Rob Best regards, Jonas