Received: by 2002:ab2:3350:0:b0:1f4:6588:b3a7 with SMTP id o16csp1993067lqe; Tue, 9 Apr 2024 06:53:06 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCUGf/oU3xy5ziGulMQaZF60Z/qIcNwoVp2PlHiEf393kiIOeMEDOfgB4uosS0CJ358ZE/9HIP25EcFNwZWjONpG+8/lPLPnkQ5LpznTmQ== X-Google-Smtp-Source: AGHT+IGrrabYY/04p24rUKNHGLP8CKni75klyhvYMQE/8Om7GwPv6MNoSR/ZNLoyW6emtVDisD4E X-Received: by 2002:ac8:5808:0:b0:434:d893:f230 with SMTP id g8-20020ac85808000000b00434d893f230mr1874893qtg.56.1712670785867; Tue, 09 Apr 2024 06:53:05 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1712670785; cv=pass; d=google.com; s=arc-20160816; b=Ilmz0Qylytaj7r8gHG3BNlByeOoVIdwEJRLqehWIIIJv2vVtb6C7rFmWPGhfQ6dcSA GLGY06lEMRJYsWAgI3Dj901KLObJU3j06Pmsw/e2FaPhmuIHt60A4y7whmsL7C4Br8YY bx9VTN89R21J20ufl6Lz1VBXhVQRni50EemJzouHvp3S+L290Ej9hUBaFyej8SQpGV4U jyhObztX9nLsffuZezEsU9Y72pCoI5X7q2PpiGVKrEZC/4vuKTovh6nyv2hYW2L7qh6F 3F3Qx1u0aW0ZbPfUfm++X3S28B7HUfvaxhkas3qXaOh0JY04zbjSRSjdHjv+/mtRqRWu ZtoQ== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:organization:references :in-reply-to:message-id:subject:cc:to:from:date:dkim-signature; bh=m5UDulmenCwJXHuGMpELuxhDA4GFyujVsdxEaX2Dr8g=; fh=ws8XzRtCsVHnhfJ66tqrW2bOLhv1Im7sd9Gph8ipVRk=; b=s2wMCmbWusSl+iaV+JA1hoAtMQ/6oDaMThcemSAPJRqjpKXYSJ9XOSF6RuVnLJ3eLS nTa4Eg9Zz4QZS2lSvx/pj3vApfNNA8S8SqA9bBexwx3mKbxiF0b+/6AJmm+F51RBnIZx 2PLoRRLqeDAiOZjGc1jaJJf1IagK5VuZahklHB346Apdv9ANuk4qMzNPleA0admm0oHN LnfyKWSG9ltIHslK16kw+m+tu4FuIkI0NRo6aX9i9DGBhaInTTnbClTcPnR1FjdCUre6 TeLBALeMI8qxK7Cy8WB6UtM9k52QOBRaKLsJgo9igru7ODfryVX8PErao0amufDojJz1 lqDg==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@bootlin.com header.s=gm1 header.b=Iy4NTzVF; arc=pass (i=1 spf=pass spfdomain=bootlin.com dkim=pass dkdomain=bootlin.com dmarc=pass fromdomain=bootlin.com); spf=pass (google.com: domain of linux-kernel+bounces-137018-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-137018-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=bootlin.com Return-Path: Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [2604:1380:45d1:ec00::1]) by mx.google.com with ESMTPS id s41-20020a05622a1aa900b00434a9c2887fsi3882514qtc.560.2024.04.09.06.53.05 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 09 Apr 2024 06:53:05 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-137018-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1; Authentication-Results: mx.google.com; dkim=pass header.i=@bootlin.com header.s=gm1 header.b=Iy4NTzVF; arc=pass (i=1 spf=pass spfdomain=bootlin.com dkim=pass dkdomain=bootlin.com dmarc=pass fromdomain=bootlin.com); spf=pass (google.com: domain of linux-kernel+bounces-137018-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-137018-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=bootlin.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id 75FFC1C20B7A for ; Tue, 9 Apr 2024 13:53:05 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id CE7FB12FB1D; Tue, 9 Apr 2024 13:43:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="Iy4NTzVF" Received: from relay1-d.mail.gandi.net (relay1-d.mail.gandi.net [217.70.183.193]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id AED6112F39A; Tue, 9 Apr 2024 13:43:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.70.183.193 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1712670232; cv=none; b=GiiNmgGvcrhGEFua5Impk+4NViFnxkD9JkqKaHHsxmf+9uWT41CXdg9yOr9vC27oyQFjLGc8CxqGgAnX0578A+SadefZdLj9KUAiu4AnB1+yq2ArrPOiag9MRp4E5sw51cM5pz1dJ/bgQz+zGPQ27TT1Yh38ID4zSGNxlVITlp0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1712670232; c=relaxed/simple; bh=y/xaSl/jhXl08yoalUQ5vH76MCEpZVtsR/1x4w43g8k=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=d6C78SBbRtpG7KbDxKkgD8DzeyqBFfgqlTVvCwWmRACM+WabTR6+024ZCVtvceOVSEOxGbURpvl8p4XaYmtpaXIcPTAumig3dXcUMi3RtMjjw6KwGhxur/CXLSstNNjZG5Chw5p52SqvzcVj3ebpNrXukEMgWyqk2Qve76nByUE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=Iy4NTzVF; arc=none smtp.client-ip=217.70.183.193 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Received: by mail.gandi.net (Postfix) with ESMTPSA id 3041F240008; Tue, 9 Apr 2024 13:43:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1712670228; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=m5UDulmenCwJXHuGMpELuxhDA4GFyujVsdxEaX2Dr8g=; b=Iy4NTzVFWYIISKvZb3P1zAUEKQXE6VFTgx/iZnMms4cEObuZ97PE1okjL+Zx6GUOhH358F 2TUZr25TtQN/aJaJVeGL0chkGZ/cllV0WOROlZwrnz63KbJg+ZWwW1XRb/T5jK5+dLIdsD O6x426KxtkdX3k+CnUx7Fzlrl714fiPVBxr5l5EutPwJJSKtFdwl3I+E/IUjaelg4j9R9B v4tvJls4ZOgiyxBiOvO7UHwoBygwhpUxWCetIkKY0XowlW2uLs5g0zh17FV5u96wtMk1nh hPIClv4uh+Ulc8a1PD8GzVB7Svcg0wZuAFHM0GL3R6rVBVu0As4CvUsd53TT4Q== Date: Tue, 9 Apr 2024 15:43:45 +0200 From: Kory Maincent To: Rob Herring Cc: "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Jonathan Corbet , Luis Chamberlain , Russ Weight , Greg Kroah-Hartman , "Rafael J. Wysocki" , Krzysztof Kozlowski , Conor Dooley , Oleksij Rempel , Mark Brown , Frank Rowand , Andrew Lunn , Heiner Kallweit , Russell King , Thomas Petazzoni , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, devicetree@vger.kernel.org, Dent Project Subject: Re: [PATCH net-next v6 14/17] dt-bindings: net: pse-pd: Add bindings for PD692x0 PSE controller Message-ID: <20240409154345.7a2a73a5@kmaincent-XPS-13-7390> In-Reply-To: <20240402132834.GB3744978-robh@kernel.org> References: <20240326-feature_poe-v6-0-c1011b6ea1cb@bootlin.com> <20240326-feature_poe-v6-14-c1011b6ea1cb@bootlin.com> <20240402132834.GB3744978-robh@kernel.org> Organization: bootlin X-Mailer: Claws Mail 4.0.0 (GTK+ 3.24.33; x86_64-pc-linux-gnu) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-GND-Sasl: kory.maincent@bootlin.com On Tue, 2 Apr 2024 08:28:34 -0500 Rob Herring wrote: > > + patternProperties: > > + "^manager@0[0-9]|1[0-2]$": =20 >=20 > Unit-addresses are typically in hex. >=20 > Is 'manager' something specific to this device or should be common? Specific to this device. =20 > > + $ref: /schemas/graph.yaml#/properties/ports =20 >=20 > This is not using the graph binding. Furthermore, I don't want to see=20 > new cases of 'port' node names which are not graph nodes. We have it=20 > already with ethernet switches, but 'ethernet-port' is preferred over=20 > 'port'. Ok I will remove the ref then. =20 > Why is this one 'managers' and the other device binding 'channels'? Here each manager can have up to 8 ports. The ports in tps23881 are called channels in the datasheet but I can use the port naming for both if you prefer. > > + description: > > + PD69208T4/PD69204T4/PD69208M PSE manager exposing 4 or 8 phy= sical > > + ports. > > + > > + properties: > > + reg: > > + description: > > + Incremental index of the PSE manager starting from 0, ra= nging > > + from lowest to highest chip select, up to 12. > > + maxItems: 1 > > + > > + patternProperties: > > + '^port@[0-7]$': > > + type: object > > + required: > > + - reg =20 >=20 > Any property you want is allowed in this node. You are missing=20 > 'additionalProperties'. Indeed. Regards, --=20 K=C3=B6ry Maincent, Bootlin Embedded Linux and kernel engineering https://bootlin.com