Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Subject: Re: [PATCH net-next v3 01/47] dt-bindings: phy: Add Lynx 10G phy
 binding
From:   Sean Anderson <sean.anderson@seco.com>
To:     Rob Herring <robh@kernel.org>
Cc:     "David S . Miller" <davem@davemloft.net>,
        Jakub Kicinski <kuba@kernel.org>,
        Madalin Bucur <madalin.bucur@nxp.com>,
        netdev <netdev@vger.kernel.org>, Paolo Abeni <pabeni@redhat.com>,
        Eric Dumazet <edumazet@google.com>,
        linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
        Russell King <linux@armlinux.org.uk>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Kishon Vijay Abraham I <kishon@ti.com>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Vinod Koul <vkoul@kernel.org>, devicetree@vger.kernel.org,
        "open list:GENERIC PHY FRAMEWORK" <linux-phy@lists.infradead.org>
References: <20220715215954.1449214-1-sean.anderson@seco.com>
 <20220715215954.1449214-2-sean.anderson@seco.com>
 <20220720221704.GA4049520-robh@kernel.org>
 <d4d6d881-ca3e-b77f-cee0-70e2518bef69@seco.com>
 <CAL_JsqJqU4-F52NLWDWK=Qy0ECRrYL9fmJD4CLd=J1KzCBgU7Q@mail.gmail.com>
 <6240dce3-3b68-2df4-768e-ca82bcea518f@seco.com>
Message-ID: <3a255b33-ae5b-0e2e-d1d0-0ac9548c5837@seco.com>
Date:   Tue, 26 Jul 2022 11:44:56 -0400
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.10.0
In-Reply-To: <6240dce3-3b68-2df4-768e-ca82bcea518f@seco.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?NHpkblM3QnpiWVV1QXQzU3ozUXNxMU1ER2U4ZDdsR09xazlGRjI3S1krTXFw?=
 =?utf-8?B?TENFM0JINDBJY1ZUeHhINDkwZjJGVEFjMVpJSCtsUTRXdm84dkxFZE1xd1hD?=
 =?utf-8?B?SS8zQ3lyUnFKVHVaTzBOOW1JOXhUb2hLY2ZJNFNNZlhPamxZb1B5UXA3dTJJ?=
 =?utf-8?B?M3htOUMzSXdENjNERnFXalo5c1lSM3BqaEtXZXRnVWF0U3A5b3VyTWNJdVNG?=
 =?utf-8?B?MW1DSEd1U3M0Z29PNnRDZG12UUs1RUFOSTQwckZuMVhoUE5IY1dYZm00OW5h?=
 =?utf-8?B?d1pzZ3BLTHJNRHlKSEJod3RQOERRU3kxUFNnZ3J3REM5TmFJQldoK3dxUkJt?=
 =?utf-8?B?OWRQY0FQL0lFa3VlLzcxckRMT2psZUhISWZOMS9McTMxQmRkaFJiSmhYQk1E?=
 =?utf-8?B?OWQvamVEbm8zSURnWGpOazhwTDU0OGw5NXZ5SU1TRkpORmlDY2tUL3hFemRh?=
 =?utf-8?B?MHBVaSt4RHpaQ0VHOWdNcHJ6RnVOOGp1djE5ckltNzFpRGpoVnNiQXNZNGdv?=
 =?utf-8?B?SlNKNlB5dkNhczllamRUWHBFTnhkY2pwWXlpeVEvVU9JR1MrTXhvcE9ETXpj?=
 =?utf-8?B?SWtFT1R6NHhPc3FtdGx6aE5pMjJOVno4bnIxNlNMUVpjRndHMzk2YzFqTDY5?=
 =?utf-8?B?SmhObE9KbVJ1SHhUdktIS2xEckpBUFI1bkNkcU9WT3NEQzJCSkRWQTZVZmJu?=
 =?utf-8?B?MWkvZUN3ejJJalg2VEI2SnJQNTBVTG84K3NxSFRCZkw0K2xSb3JrbDN0aVZh?=
 =?utf-8?B?SFJQSEZnN09KUDV3MjJmWWRlTHdORFZQaHVKK2c2c2pTZ0NJSjBBNFM2L2xz?=
 =?utf-8?B?MFk3K3h0S2IzYzBPMm5tbmhrUGFqeDIyZFQzVTdCb3BkOEgycmd5VzBtYnZH?=
 =?utf-8?B?UGZ0elVsSE4zdG4zZGZZOTU4bnIzNmMzT0M4TVVsdjgyU2dlbjR3YWhUbTZh?=
 =?utf-8?B?NVlnZ2dGZlc2c05Vd1cwdE80ZWdVaWZkWGVLTmt2YzZ0Kzd5dW4rMW14S1FW?=
 =?utf-8?B?d2UzZUJmNkpTMnZaSzZPSXh2V1IrR0xySU5rS2RxTmRMbDZrWnNtZ3BDRGhz?=
 =?utf-8?B?aTA5bXZleEd2WVdBellqajFoWEVjM0Z0eGhUZEpiTjNDQjU5eXh1VzV5eWly?=
 =?utf-8?B?NURKdDE1WVJZamVCTEtYSmpyS0xWZG1oRjFPajV3dlMvOFJHZWJHNE96OFdw?=
 =?utf-8?B?bGM1Skt0ZEhzc1c0T3YzanBFYWJleFlFWkc4VjQ5V2NNaDd1RnhIUEtjYlcv?=
 =?utf-8?B?TFlMRWVmc3FuNFVGRi8xenJCYTJoODBNWDRtZklubWlXbm83eFozak80ZkxL?=
 =?utf-8?B?cmE4N1lWTFF2Q0pVVUU3a3RocVFkb0tHeWp0Q21oai9tSmNQS2tEZ01Tbloy?=
 =?utf-8?B?Tjg1TzNCLzZIaVhnUzJ1M2RsYlRtclRYckdoaFJ4NG91SGVSU3l1emdmMzFG?=
 =?utf-8?B?QkJJTG5CQkpOTkcvQks3QlNhUXBqS2xLbEcwbVFTdmVkSnoydWxhMTQ5NkhO?=
 =?utf-8?B?bUNUNjdDQ0ZObU5qYStEVU5aNEd5K05JbGU2ZEkyREFwcmZEZnNqZHQwenRh?=
 =?utf-8?B?cHp5dFFjOWJZL1Y1TW14U1B1akY2V0JkMElEbzNtUERDSVZzbE85NVRqNkZY?=
 =?utf-8?B?YUVUb2c4Z2pWT3I3L0N0blN0bjE4S1IyY2I5cElURlVPRjJmUjJuRkJWR1NI?=
 =?utf-8?B?U0UxbjVUZDhYZE1SaE01R3I3V2JQREdhWU5reGpwUVlPcmtBSVdxOXkwazZ6?=
 =?utf-8?B?aWI2KzZzWi9sR2xZS3NWQW9EelNvdU5GR3lRcmtGeWZaWER5dnVmeXFYdUxl?=
 =?utf-8?B?TXI3cHgrdk51RVFvNndvQzRhMzJSUnVpdVBoYTJJaE5sWlFrdXpxbFJYejA4?=
 =?utf-8?B?cHFQNlBxdXkvdTFINithWG5FaFFkNFpESHlkcCtDT0N0UjZIbEttSjZ3K1FJ?=
 =?utf-8?B?STVBdXY0M1AxbTY1V1pmYU5Gd05PVnIwYTdtUDJPTWMvMXVHbExjYklLaFBi?=
 =?utf-8?B?bm1ZN1Rid0kvUTM4ZGhyakV1Rm81YnFWd0grMGRUbHQ0dlNhQ1IzWGdDUzFN?=
 =?utf-8?B?MXhUYjZBazNhZnNsSG0xbEpRUGhCUk51d3Z5UGcvNkdTRjVzb1RLa1A1d3ZV?=
 =?utf-8?B?OXQ5eGpCQmpmcWwzTXdqbmcyMXA5eGNobFZNc1oxWmhqN2RsY0FBRGtiK0pI?=
 =?utf-8?B?ZGc9PQ==?=
X-OriginatorOrg: seco.com
X-MS-Exchange-CrossTenant-Network-Message-Id: ee50f105-c484-4c91-b997-08da6f1dced1
X-MS-Exchange-CrossTenant-AuthSource: DB7PR03MB4972.eurprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Jul 2022 15:45:04.2025
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: bebe97c3-6438-442e-ade3-ff17aa50e733
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: HcihQmZE2kJUUvroKF+9jxjHhzIIUF66gwi2Xxsdtf10JNArMBYPinl4tNa2eYsVoo53Y0u9tLFkHasww+62nw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBBPR03MB6987
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Rob,

On 7/21/22 7:35 PM, Sean Anderson wrote:
> What about
> 
> 	phys = <&serdes1_lane1>;
> 
> and then under the serdes node do something like
> 
> 	serdes1: phy@foo {
> 		...
> 
> 		serdes1_lane1 {
> 			first-lane = <1>;
> 
> 			sgmii {
> 				fsl,pccr = <0x8>;
> 				fsl,idx = <2>;
> 				fsl,cfg = <1>;
> 				fsl,proto = "sgmii";
> 				// or PHY_TYPE_SGMII
> 			};
> 
> 			qsgmii {
> 				...
> 			};
> 
> 			xfi {
> 				...
> 			};
> 		};
> 	};
> 
> and this way you could have something like a fsl,reserved property to
> deal with not-yet-supported lanes. And this could be added piecemeal by
> board configs.

Does this sound good? I would like to start working on v4 of this series,
and reworking the binding will be a big part of that. Am I heading in the
right direction? This seems to be a more common approach (e.g. mediatek,tphy).

--Sean