From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 53710C433EF for ; Mon, 18 Jul 2022 22:59:30 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236324AbiGRW73 (ORCPT ); Mon, 18 Jul 2022 18:59:29 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55166 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236132AbiGRW7V (ORCPT ); Mon, 18 Jul 2022 18:59:21 -0400 Received: from mail-pl1-x62e.google.com (mail-pl1-x62e.google.com [IPv6:2607:f8b0:4864:20::62e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A6AAE2AE13 for ; Mon, 18 Jul 2022 15:59:18 -0700 (PDT) Received: by mail-pl1-x62e.google.com with SMTP id j12so10377726plj.8 for ; Mon, 18 Jul 2022 15:59:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to; bh=yZE3AZU192cNuyR4pcOHjhgxZjP1cI+fyOQtlDvGmfE=; b=WFuLl1N7WrQzLWDJwdYc1Qv46FMtauNeNxhehkZJ2KXy3pI4+0OmFlut35Ig8uDqiS 2YMQlw+ThmKgRT3Bp0IAl1iELzOAqBGM9qAdvIZKXv+wlv/rPnHWJhSQw0YS5UaKY0kb aLeQvsICxtHNx5OM9VkBqPBb3606Fi9OqXx4Y= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to; bh=yZE3AZU192cNuyR4pcOHjhgxZjP1cI+fyOQtlDvGmfE=; b=Gw2lm/ki2qLrPGZrmJrEBZlnRXCKnJTgpI7XckZNL6IDWSCi0rJnbzQ9D0+XA4T7Ho aPfcfQXmI3Zen366l7K0u87aJ9oB8Zg0NNMmtVqb1QoqSfLj+g0E4B3h2DBEnuN8Y1fU DtH2q7bkkRECTO57sJnRm8f2+FpGTvvQBYLgXbdVjVgnuFddwy8rWh9wtTew0al020GF gizTBrJl8M9HvB6LqBEHqcm+wWXHEzag0PFMb6b0B+zsofd6Jta1AKqxSbOJSCOYpUFY GUASrfYERq7xNHoJLIQ5+szWJwID4HruVWysjlVU4+JzPb8i9g6FXxVkvL4nmYSU2a1Q hj2w== X-Gm-Message-State: AJIora//qjGTQNRvZ/rPZ9MFjXjy6Xe/QlUCjhfbBh/8BfCcNbjf3mP7 HKS+YXufF2qxUXlYk6C8Lpi7iaGtUDXt4hZVg3iC25J/WntnfOj/KJCKfySIAS7RwYBIbJ+jpz4 U7V6RPHwXCDbDM4ljxbW9TYvKz+5WfZl/5cxr8kqM2oaWwlSdQBTPD4QuvssccCLcrTWfIXhnNs vwM4Y8IIsN X-Google-Smtp-Source: AGRyM1tvRrd0/Gvr4TMEgCeF8PBAoX1ATEhVwRJQ6zuHbXDzCdfG5fnfHHVKYj7wm2Ip3vUfUZl/rg== X-Received: by 2002:a17:903:110c:b0:16c:cfae:e48c with SMTP id n12-20020a170903110c00b0016ccfaee48cmr17286702plh.12.1658185157727; Mon, 18 Jul 2022 15:59:17 -0700 (PDT) Received: from bcacpedev-irv-3.lvn.broadcom.net ([192.19.161.250]) by smtp.gmail.com with ESMTPSA id t1-20020a1709027fc100b0016a11b9aeb3sm9959725plb.224.2022.07.18.15.59.15 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Mon, 18 Jul 2022 15:59:16 -0700 (PDT) Subject: Re: [RFC PATCH 1/3] dt-bindings: arm64: bcmbca: Merge BCM4908 into BCMBCA To: Rob Herring Cc: =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= , Krzysztof Kozlowski , Linux ARM List , kursad.oney@broadcom.com, anand.gore@broadcom.com, dan.beygelman@broadcom.com, f.fainelli@gmail.com, Broadcom Kernel List , joel.peshkin@broadcom.com, Krzysztof Kozlowski , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20220712021144.7068-1-william.zhang@broadcom.com> <20220712021144.7068-2-william.zhang@broadcom.com> <94b0ab39-279d-d3c2-98a4-054c10ad041c@broadcom.com> <6efb1cfe-6129-276a-eeb3-44147304d211@broadcom.com> <85219d59e2906534409fc24ad2e5e4c9@milecki.pl> <147bc812-3971-9832-fb39-5545e280f562@broadcom.com> <20220718200210.GA3436421-robh@kernel.org> From: William Zhang Message-ID: Date: Mon, 18 Jul 2022 15:59:15 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: <20220718200210.GA3436421-robh@kernel.org> Content-Type: multipart/signed; protocol="application/pkcs7-signature"; micalg=sha-256; boundary="00000000000054a7c505e41c51dc" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --00000000000054a7c505e41c51dc Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit On 07/18/2022 01:02 PM, Rob Herring wrote: > On Wed, Jul 13, 2022 at 11:37:18AM -0700, William Zhang wrote: >> Hi Rafal, >> >> On 7/13/22 03:58, Rafał Miłecki wrote: >>> On 2022-07-13 12:50, Rafał Miłecki wrote: >>>> On 2022-07-13 02:57, William Zhang wrote: >>>>> On 7/12/22 11:18, Krzysztof Kozlowski wrote: >>>>>> On 12/07/2022 19:37, William Zhang wrote: >>>>>>>>> +      - description: BCM4908 Family based boards >>>>>>>>> +        items: >>>>>>>>> +          - enum: >>>>>>>>> +              # BCM4908 SoC based boards >>>>>>>>> +              - brcm,bcm94908 >>>>>>>>> +              - asus,gt-ac5300 >>>>>>>>> +              - netgear,raxe500 >>>>>>>>> +              # BCM4906 SoC based boards >>>>>>>>> +              - brcm,bcm94906 >>>>>>>>> +              - netgear,r8000p >>>>>>>>> +              - tplink,archer-c2300-v1 >>>>>>>>> +          - enum: >>>>>>>>> +              - brcm,bcm4908 >>>>>>>>> +              - brcm,bcm4906 >>>>>>>>> +              - brcm,bcm49408 >>>>>>>> >>>>>>>> This is wrong.  brcm,bcm94908 followed by brcm,bcm4906 does not look >>>>>>>> like valid list of compatibles. >>>>>>>> >>>>>>> For 4908 board variant, it will need to be followed by >>>>>>> 4908 chip. Sorry >>>>>>> for the basic question but is there any requirement to >>>>>>> enforce this kind >>>>>>> of rule?  I would assume dts writer know what he/she is >>>>>>> doing and select >>>>>>> the right combination. >>>>>> >>>>>> The entire point of DT schema is to validate DTS. >>>>>> Combination like above >>>>>> prevents that goal. >>>>>> >>>>>> Best regards, >>>>>> Krzysztof >>>>> Understand the DT schema purpose. But items property allows multiple >>>>> enums in the list which gives a lot of flexibility but make it hard to >>>>> validate. I am not familiar with DT schema, is there any directive to >>>>> specify one enum value depending on another so dts validation tool can >>>>> report error if combination is wrong? >>>>> >>>>> This is our preferred format of all bcmbca compatible string >>>>> especially when we could have more than 10 chip variants for the same >>>>> chip family and we really want to work on the chip family id.  We will >>>>> make sure they are in the right combination in our own patch and patch >>>>> from other contributors. Would this work? If not, I will probably have >>>>> to revert the change of 4908(maybe append brcm,bcmbca as this chip >>>>> belongs to the same bca group) and use "enum board variant", "const >>>>> main chip id", "brcm,bca" for all other chips as our secondary choice. >>>> >>>> I'm not sure why I didn't even receive 1/3 and half of discussion >>>> e-mails. >>>> >>>> You can't just put all strings into a single bag and allow mixing them >>>> in any combos. Please check how it's properly handled in the current >>>> existing binding: >>>> Documentation/devicetree/bindings/arm/bcm/brcm,bcm4908.yaml >>>> >>>> Above binding enforces that non-matching compatible strings are not used >>>> together. >>> >>> I just noticed you're actually removing brcm,bcm4908.yaml in the 2/3 so >>> you must be aware of that file. >>> >>> So you see a cleanly working binding in the brcm,bcm4908.yaml but >>> instead copying it you decided to wrote your own one from scratch. >>> Incorrectly. >>> >>> This smells of NIH (not invented here). Please just use that binding I >>> wrote and move if it needed. >> >> Not mean to discredit any of your work and I did copy over your binding and >> combine them into one SoC entry to the new bcmbca.yaml and add you as one of >> the maintainer to this file. As this change would certainly concern you, >> that's why I sent RFC first. As I explained in the cover letter, the >> purpose of the change is to reduce the number of compatible strings and keep >> one entry for one chip family due to possible large number of chip variants. >> But since there is no way to validate the combination, I will copy the >> existing 4908 bindings as they are now but I would propose to append "brcm, >> bcmbca" as it is part of bcmbca chip. And for the other chips, we would just >> use enum "board variant", const "main chip id", const "brcm,bca". Does that >> sound good to you? > > If you want fewer combinations of compatibles, adding a genericish > "brcm,bcmbca" is not going to help. Is there much value to adding it? > What can you do with that information (and nothing else) is the > question to ask. > Since we moved all the Broadcom Broadband BCA origin SoC to ARCH_BCMBCA arch, I think it makes sense to have that as the last part of the compatible string, not really for reducing the number. And for that matter, we have agreed to not change anything to Rafal dts definitions including the number of compat strings but just appending brcm,bca. So we are all good now. > Rob > --00000000000054a7c505e41c51dc Content-Type: application/pkcs7-signature; name="smime.p7s" Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="smime.p7s" Content-Description: S/MIME Cryptographic Signature MIIQcAYJKoZIhvcNAQcCoIIQYTCCEF0CAQExDzANBglghkgBZQMEAgEFADALBgkqhkiG9w0BBwGg gg3HMIIFDTCCA/WgAwIBAgIQeEqpED+lv77edQixNJMdADANBgkqhkiG9w0BAQsFADBMMSAwHgYD VQQLExdHbG9iYWxTaWduIFJvb3QgQ0EgLSBSMzETMBEGA1UEChMKR2xvYmFsU2lnbjETMBEGA1UE AxMKR2xvYmFsU2lnbjAeFw0yMDA5MTYwMDAwMDBaFw0yODA5MTYwMDAwMDBaMFsxCzAJBgNVBAYT AkJFMRkwFwYDVQQKExBHbG9iYWxTaWduIG52LXNhMTEwLwYDVQQDEyhHbG9iYWxTaWduIEdDQyBS MyBQZXJzb25hbFNpZ24gMiBDQSAyMDIwMIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8AMIIBCgKCAQEA vbCmXCcsbZ/a0fRIQMBxp4gJnnyeneFYpEtNydrZZ+GeKSMdHiDgXD1UnRSIudKo+moQ6YlCOu4t rVWO/EiXfYnK7zeop26ry1RpKtogB7/O115zultAz64ydQYLe+a1e/czkALg3sgTcOOcFZTXk38e aqsXsipoX1vsNurqPtnC27TWsA7pk4uKXscFjkeUE8JZu9BDKaswZygxBOPBQBwrA5+20Wxlk6k1 e6EKaaNaNZUy30q3ArEf30ZDpXyfCtiXnupjSK8WU2cK4qsEtj09JS4+mhi0CTCrCnXAzum3tgcH cHRg0prcSzzEUDQWoFxyuqwiwhHu3sPQNmFOMwIDAQABo4IB2jCCAdYwDgYDVR0PAQH/BAQDAgGG MGAGA1UdJQRZMFcGCCsGAQUFBwMCBggrBgEFBQcDBAYKKwYBBAGCNxQCAgYKKwYBBAGCNwoDBAYJ KwYBBAGCNxUGBgorBgEEAYI3CgMMBggrBgEFBQcDBwYIKwYBBQUHAxEwEgYDVR0TAQH/BAgwBgEB /wIBADAdBgNVHQ4EFgQUljPR5lgXWzR1ioFWZNW+SN6hj88wHwYDVR0jBBgwFoAUj/BLf6guRSSu TVD6Y5qL3uLdG7wwegYIKwYBBQUHAQEEbjBsMC0GCCsGAQUFBzABhiFodHRwOi8vb2NzcC5nbG9i YWxzaWduLmNvbS9yb290cjMwOwYIKwYBBQUHMAKGL2h0dHA6Ly9zZWN1cmUuZ2xvYmFsc2lnbi5j b20vY2FjZXJ0L3Jvb3QtcjMuY3J0MDYGA1UdHwQvMC0wK6ApoCeGJWh0dHA6Ly9jcmwuZ2xvYmFs c2lnbi5jb20vcm9vdC1yMy5jcmwwWgYDVR0gBFMwUTALBgkrBgEEAaAyASgwQgYKKwYBBAGgMgEo CjA0MDIGCCsGAQUFBwIBFiZodHRwczovL3d3dy5nbG9iYWxzaWduLmNvbS9yZXBvc2l0b3J5LzAN BgkqhkiG9w0BAQsFAAOCAQEAdAXk/XCnDeAOd9nNEUvWPxblOQ/5o/q6OIeTYvoEvUUi2qHUOtbf jBGdTptFsXXe4RgjVF9b6DuizgYfy+cILmvi5hfk3Iq8MAZsgtW+A/otQsJvK2wRatLE61RbzkX8 9/OXEZ1zT7t/q2RiJqzpvV8NChxIj+P7WTtepPm9AIj0Keue+gS2qvzAZAY34ZZeRHgA7g5O4TPJ /oTd+4rgiU++wLDlcZYd/slFkaT3xg4qWDepEMjT4T1qFOQIL+ijUArYS4owpPg9NISTKa1qqKWJ jFoyms0d0GwOniIIbBvhI2MJ7BSY9MYtWVT5jJO3tsVHwj4cp92CSFuGwunFMzCCA18wggJHoAMC AQICCwQAAAAAASFYUwiiMA0GCSqGSIb3DQEBCwUAMEwxIDAeBgNVBAsTF0dsb2JhbFNpZ24gUm9v dCBDQSAtIFIzMRMwEQYDVQQKEwpHbG9iYWxTaWduMRMwEQYDVQQDEwpHbG9iYWxTaWduMB4XDTA5 MDMxODEwMDAwMFoXDTI5MDMxODEwMDAwMFowTDEgMB4GA1UECxMXR2xvYmFsU2lnbiBSb290IENB IC0gUjMxEzARBgNVBAoTCkdsb2JhbFNpZ24xEzARBgNVBAMTCkdsb2JhbFNpZ24wggEiMA0GCSqG SIb3DQEBAQUAA4IBDwAwggEKAoIBAQDMJXaQeQZ4Ihb1wIO2hMoonv0FdhHFrYhy/EYCQ8eyip0E XyTLLkvhYIJG4VKrDIFHcGzdZNHr9SyjD4I9DCuul9e2FIYQebs7E4B3jAjhSdJqYi8fXvqWaN+J J5U4nwbXPsnLJlkNc96wyOkmDoMVxu9bi9IEYMpJpij2aTv2y8gokeWdimFXN6x0FNx04Druci8u nPvQu7/1PQDhBjPogiuuU6Y6FnOM3UEOIDrAtKeh6bJPkC4yYOlXy7kEkmho5TgmYHWyn3f/kRTv riBJ/K1AFUjRAjFhGV64l++td7dkmnq/X8ET75ti+w1s4FRpFqkD2m7pg5NxdsZphYIXAgMBAAGj QjBAMA4GA1UdDwEB/wQEAwIBBjAPBgNVHRMBAf8EBTADAQH/MB0GA1UdDgQWBBSP8Et/qC5FJK5N UPpjmove4t0bvDANBgkqhkiG9w0BAQsFAAOCAQEAS0DbwFCq/sgM7/eWVEVJu5YACUGssxOGhigH M8pr5nS5ugAtrqQK0/Xx8Q+Kv3NnSoPHRHt44K9ubG8DKY4zOUXDjuS5V2yq/BKW7FPGLeQkbLmU Y/vcU2hnVj6DuM81IcPJaP7O2sJTqsyQiunwXUaMld16WCgaLx3ezQA3QY/tRG3XUyiXfvNnBB4V 14qWtNPeTCekTBtzc3b0F5nCH3oO4y0IrQocLP88q1UOD5F+NuvDV0m+4S4tfGCLw0FREyOdzvcy a5QBqJnnLDMfOjsl0oZAzjsshnjJYS8Uuu7bVW/fhO4FCU29KNhyztNiUGUe65KXgzHZs7XKR1g/ XzCCBU8wggQ3oAMCAQICDDbx5fpN++xs1+5IgzANBgkqhkiG9w0BAQsFADBbMQswCQYDVQQGEwJC RTEZMBcGA1UEChMQR2xvYmFsU2lnbiBudi1zYTExMC8GA1UEAxMoR2xvYmFsU2lnbiBHQ0MgUjMg UGVyc29uYWxTaWduIDIgQ0EgMjAyMDAeFw0yMTAyMjIwODA1MjJaFw0yMjA5MDUwODEwMTZaMIGQ MQswCQYDVQQGEwJJTjESMBAGA1UECBMJS2FybmF0YWthMRIwEAYDVQQHEwlCYW5nYWxvcmUxFjAU BgNVBAoTDUJyb2FkY29tIEluYy4xFjAUBgNVBAMTDVdpbGxpYW0gWmhhbmcxKTAnBgkqhkiG9w0B CQEWGndpbGxpYW0uemhhbmdAYnJvYWRjb20uY29tMIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8AMIIB CgKCAQEA4fxIZbzNLvB+7yJE8mbojRaOoaK1uZy1/etc55NzisSJJfY36BAlb7LlMDsza2/BcjXh lSACuzeOyI8sy2pKHGt5SZCMHeHaxP8q4ZNR6EGz7+5Lopw6ies8fkDoZ/XFIHpfU2eKcIYrxI25 bTaYAPDA50BHTPDFzPNkWEIIQaSBBkk55bndnMmB/pPR/IhKjLefDIhIsiWLrvQstTiSf7iUCwMf TltlrAeBKRJ1M9O/DY5v7L1Yrs//7XIRg/d2ZPAOSGBQzFYjYTFWwNBiR1s1zP0m2y56DPbS5gwj fqAN/I4PJHIvTh3zUgHXNKadYoYRiPHXfaTWO9UhzysOpQIDAQABo4IB2zCCAdcwDgYDVR0PAQH/ BAQDAgWgMIGjBggrBgEFBQcBAQSBljCBkzBOBggrBgEFBQcwAoZCaHR0cDovL3NlY3VyZS5nbG9i YWxzaWduLmNvbS9jYWNlcnQvZ3NnY2NyM3BlcnNvbmFsc2lnbjJjYTIwMjAuY3J0MEEGCCsGAQUF BzABhjVodHRwOi8vb2NzcC5nbG9iYWxzaWduLmNvbS9nc2djY3IzcGVyc29uYWxzaWduMmNhMjAy MDBNBgNVHSAERjBEMEIGCisGAQQBoDIBKAowNDAyBggrBgEFBQcCARYmaHR0cHM6Ly93d3cuZ2xv YmFsc2lnbi5jb20vcmVwb3NpdG9yeS8wCQYDVR0TBAIwADBJBgNVHR8EQjBAMD6gPKA6hjhodHRw Oi8vY3JsLmdsb2JhbHNpZ24uY29tL2dzZ2NjcjNwZXJzb25hbHNpZ24yY2EyMDIwLmNybDAlBgNV HREEHjAcgRp3aWxsaWFtLnpoYW5nQGJyb2FkY29tLmNvbTATBgNVHSUEDDAKBggrBgEFBQcDBDAf BgNVHSMEGDAWgBSWM9HmWBdbNHWKgVZk1b5I3qGPzzAdBgNVHQ4EFgQUohM5GmNlGWe5wpzDxzIy +EgzbRswDQYJKoZIhvcNAQELBQADggEBACKu9JSQAYTlmC+JTniO/C/UcXGonATI/muBjWTxtkHc abZtz0uwzzrRrpV+mbHLGVFFeRbXSLvcEzqHp8VomXifEZlfsE9LajSehzaqhd+np+tmUPz1RlI/ ibZ7vW+1VF18lfoL+wHs2H0fsG6JfoqZldEWYXASXnUrs0iTLgXxvwaQj69cSMuzfFm1X5kWqWCP W0KkR8025J0L5L4yXfkSO6psD/k4VcTsMJHLN4RfMuaXIT6EM0cNO6h3GypyTuPf1N1X+F6WQPKb 1u+rvdML63P9fX7e7mwwGt5klRnf8aK2VU7mIdYCcrFHaKDTW3fkG6kIgrE1wWSgiZYL400xggJt MIICaQIBATBrMFsxCzAJBgNVBAYTAkJFMRkwFwYDVQQKExBHbG9iYWxTaWduIG52LXNhMTEwLwYD VQQDEyhHbG9iYWxTaWduIEdDQyBSMyBQZXJzb25hbFNpZ24gMiBDQSAyMDIwAgw28eX6TfvsbNfu SIMwDQYJYIZIAWUDBAIBBQCggdQwLwYJKoZIhvcNAQkEMSIEID0NDS37710qqajRGRknXMOVJ/Iu cXj5P39kn2wNi8q4MBgGCSqGSIb3DQEJAzELBgkqhkiG9w0BBwEwHAYJKoZIhvcNAQkFMQ8XDTIy MDcxODIyNTkxOFowaQYJKoZIhvcNAQkPMVwwWjALBglghkgBZQMEASowCwYJYIZIAWUDBAEWMAsG CWCGSAFlAwQBAjAKBggqhkiG9w0DBzALBgkqhkiG9w0BAQowCwYJKoZIhvcNAQEHMAsGCWCGSAFl AwQCATANBgkqhkiG9w0BAQEFAASCAQBFStGgNZbnxhjtpm1EyYJ43rX4a592+0gBW9Jze5703g5w QHSufkJPzwbafr/rvRXD0SJEaQgFjkFMkNlLM2DhcfcdbNSJTlj50NYhGRxS+deEDY3b4GnAGF0N No0HNwSayArgGgRAB7hBTWb129PI87t2rRNNYHW70Nus1eTadI5tiCrYn3msz1H0Th0V7yg7TFi7 0ftu+eHJZfSti6LsdXR1FzYVyOBu2Fe53umU/5VQrdR8UXu99tLoN8BbnHIWbML13gBCFXZklj4x Gtmyn3xnMPoIWz+eAKPYfF6snJvUIfP2HTzlsh8JLW8kisxvmxyPInVL20M69L58tBjT --00000000000054a7c505e41c51dc--