Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Aditya Garg <gargaditya08@live.com>
To:     Hector Martin <marcan@marcan.st>
CC:     Arend van Spriel <arend.vanspriel@broadcom.com>,
        Ian Lin <ian.lin@infineon.com>,
        "alep@cypress.com" <alep@cypress.com>,
        "brcm80211-dev-list@broadcom.com" <brcm80211-dev-list@broadcom.com>,
        "brcm80211-dev-list@cypress.com" <brcm80211-dev-list@cypress.com>,
        "franky.lin@broadcom.com" <franky.lin@broadcom.com>,
        "hante.meuleman@broadcom.com" <hante.meuleman@broadcom.com>,
        "kvalo@kernel.org" <kvalo@kernel.org>,
        "Double.Lo@infineon.com" <Double.Lo@infineon.com>,
        "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
        LKML <linux-kernel@vger.kernel.org>,
        Asahi Linux <asahi@lists.linux.dev>
Subject: Re: [PATCH v2 2/4] brcmfmac: Support 89459 pcie
Thread-Topic: [PATCH v2 2/4] brcmfmac: Support 89459 pcie
Thread-Index: AQHZHo718NMBa+5HNU60gzVi1Spvqa6LJ2UAgAAA0gCAAAkygA==
Date:   Mon, 2 Jan 2023 14:34:17 +0000
Message-ID: <BA494267-7B5A-4042-BD33-44F62837C8DB@live.com>
References: <20220922104140.11889-1-ian.lin@infineon.com>
 <20220922104140.11889-3-ian.lin@infineon.com>
 <3eb9cc12-db75-8042-4106-fa6521c684a7@marcan.st>
 <6D6035D7-894B-44F6-B35A-3EC9C57029B5@live.com>
 <91c8fc96-083d-74b7-3015-e76f1ab59cec@broadcom.com>
 <6e204026-d102-6d81-f58b-f4340e7d07ac@marcan.st>
In-Reply-To: <6e204026-d102-6d81-f58b-f4340e7d07ac@marcan.st>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-messagesentrepresentingtype: 1
x-tmn:  [5XsElTdK+5CFBK2m8CM/133hQGklwSZJaGCJSUNo9jWtfr0eGsISymOfuSkbof0q]
x-ms-publictraffictype: Email
x-ms-traffictypediagnostic: BM1PR01MB0931:EE_|MA0PR01MB6609:EE_
x-ms-office365-filtering-correlation-id: 7972c0e9-7187-49db-a12e-08daecce6ded
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: r/xWC5hW0xzD/UiB32JkMPPO57QsVQ47gDAWBDKeHahVJb4VzZUE+AQwCvx266QMiwCTOCYSwHiQFubv6PZiFnSLjMlfE2uGyrm9DXt1dade+q8IkrpYaEFjBS4uc6hoWU/ZyoeYoCjnhEhn5AW0bM4DOxm46jBg68ywsBnWBh713KwU6XOuimYqE/866UeFcVN/Z7G5KreVFfuD+fM2KmJaHxeqgnMLRJHVty7zEYtnCfyejHjwA5zU3c1rJCx6xgZEr6THjhezsT44ISjsB0Ccj/WPuI134UDxnvrBByjjxq9yvhO4+1XvpEKdc7/rc2EEsld/a/mPehbGssPa4NbLABR56BpFsEuvxjGVz0kVJvLAy1cKL6O25C/OW2j+jNsf+HYTLd6O+2EXNRnJL6w4adz/vgHnr7pyPexFJDaaKKKlvwreeBwP1TmbS1RkAMcmGy4/wi7dPGScXCrQsojSM+CeKia5T86uCHjCP0fl5B1S38nrH/DzNbp4qJ4zDq339CAgbJBbvmNIgH8DChNnmMsCbPSN5RuCrdrzGcaM+WIB65o3K3O+I0LLzS04a5IGX5yhYaUcRPw+sR6PuAziSn6cJvACZztx6/CJrEPUeofQvNoEmcWFs5zggZ3frwVDnOOIPYRlZQ8yqaCD0g==
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?bRCqQaOlkf/T0vG7yhYtStVTqpFSujxIOB5O/gp3ipz0fAeWb0/vku7pRefB?=
 =?us-ascii?Q?rtOU5TofyoF6ck26wrJ+gIaShgPDxjCDxy4IMLLBgdqrR0bw7OPAgbFzfWTr?=
 =?us-ascii?Q?/2U7yu7e4+5I7+JSGYVAwljwgpZITGblGoRjhBH2s5qTLpeR3gAJ3f/T5XeW?=
 =?us-ascii?Q?kHdyBwItnrI/zrqpEDVhZMVvwMIGfFZjPgeiU7PJM6yqBtAsypDOR/AxHGKl?=
 =?us-ascii?Q?G7ABxmoz8uKDUstaBSknyxOvMv9nMBe1YtAc1zJAbfng8/D/3Q0SolnJGR4x?=
 =?us-ascii?Q?vrP29ydkD7CVqlUW4SIhk1GT+r5hdygDXFmJ6ieEKkmVrKX/GoOmU3ogLl18?=
 =?us-ascii?Q?9ddwSpYVXstqVtBoidCqrjgw5yOwUElogN/4blYxgfcp4w0gL0Se2a/fU00j?=
 =?us-ascii?Q?ii37idvyhpAzuA4U8XqYw5rffd1yupWzbNeom7OuccHBiRpdKPv085pdJOgA?=
 =?us-ascii?Q?Wud3234mNuxHqFlXQiPW/2m07VUFkpE3YrYVd4vqIEz73CBv3OiUBClxETyp?=
 =?us-ascii?Q?I92BeO78FxORkdyWJPaK4FvJZP1/jMzQtv64NXvVCi5xyqtkChufv/2FU6sL?=
 =?us-ascii?Q?Lhv6V9BvNaJFoIiXLlNHLGd0HJUGs+Mk5ndQwH8YnEod2qXAuuMecgt9ZLV9?=
 =?us-ascii?Q?XXKcDc5kC+9fAMRQrHW8TH5hB7rjAtyyOnMliZXk62UGZQUxkezaRHipBptx?=
 =?us-ascii?Q?zkx/IGMK5GIdE4mWHyiIy6XD7JqkW8EcDLvAPqquNOnVweH52nkZ80/hzniU?=
 =?us-ascii?Q?SzxclDmrJqPWlj+QQl5o6YsVK0LqbzJbg9Jpn6iCmNVyPj76KMXOyYsD04oM?=
 =?us-ascii?Q?Q/0VT6zqiJ+UJAIwMlfWhD+IfvaQonXoev9meKRp6TG3wUXzEZWZ7YbsND/g?=
 =?us-ascii?Q?69l7r3krpsO77Dp6fJl3HzcRIzgAV9MgN6Ix9473YNcuoJ2sJaUWl1pnppvx?=
 =?us-ascii?Q?s2E+rXdY6Rj30gXQeupmPBbtp5etVamFwFntHyrje9594ZvHQuULLk4+MHtp?=
 =?us-ascii?Q?7CcqdoklwB0zUunlyRnVqJ7KzI0X6Cm6CbOz6Dh2umJQzMFBwknEBp9peIk7?=
 =?us-ascii?Q?9SaeqMhzuUM7U9eAst3npmmWnpcVShoThpEGrOJom6YePCm5s/6s5fZyVRod?=
 =?us-ascii?Q?3R5pcVkdpMCiJnJrz1B9QWjJPz8gYrkHymMsJinOCtvaQHro4VlPQ5YdjIgP?=
 =?us-ascii?Q?xgJUAFcgvnwzj4DfNI5fFXgSlqwFE1UpGq59b6/hE+tI//gi4jlc6sXe3Fio?=
 =?us-ascii?Q?xTB0Rl2orAJEUodYrc9knv6huvOgNyyCSOIwNZlVWsGkl31U6AF3rsTV4t6e?=
 =?us-ascii?Q?X2g/MuXYVm671vAhMCOIcpuA3jbRPjHeGnIVQz+4vPecvQ=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <C5222CA80F583B4DBEC100A09E97CD7F@INDPRD01.PROD.OUTLOOK.COM>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: sct-15-20-4755-11-msonline-outlook-42ed3.templateTenant
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: BM1PR01MB0931.INDPRD01.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000
X-MS-Exchange-CrossTenant-Network-Message-Id: 7972c0e9-7187-49db-a12e-08daecce6ded
X-MS-Exchange-CrossTenant-originalarrivaltime: 02 Jan 2023 14:34:17.7429
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa
X-MS-Exchange-CrossTenant-rms-persistedconsumerorg: 00000000-0000-0000-0000-000000000000
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MA0PR01MB6609
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_ENVFROM_END_DIGIT,
        FREEMAIL_FROM,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-wireless.vger.kernel.org>
X-Mailing-List: linux-wireless@vger.kernel.org


> I think he's proposing we special-case Apple chips into their own
> firmware table just to avoid colliding with non-Apple firmware usage,
> which is honestly kind of tempting as the safe option if nobody from the
> Broadcom/Cypress side is willing to clarify what, exactly, is the
> relationship between these chips and what their respective revision
> numbers are so we can *correctly* represent them and avoid further
> confusion and problems down the line.

Exactly

>=20
> You might be able to help with that ;)

I am sending the patch.

>=20
> - Hector