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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 22859ECAAA1 for ; Mon, 12 Sep 2022 12:00:48 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Content-ID:In-Reply-To: References:Message-ID:Date:Subject:CC:To:From:Reply-To:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=V2mZdImIHK/hkFcXVCnp/Z/Hr91pfjtCJJE1hbKNT/c=; b=U8IQdd0RSVZIJf lWWPnxiewkO5R3B0bE3jOjFgAjmn6ZYeUR7+wUEy4vgGsPmuB/yWYWOmnUGBn1HzxU8eSI3QKPwU5 I+KcJb/HZY8tI6KGldzPb7KDu8kbOJYtXXs1FSi63vIrkLrbVGI2/Y+8oYHfg4wFPAGOprF7bwQ45 3CXtAUFyBzPaG6LZtx0CTfHb7ZSMm55ATOPuOoXdGSdwN9ggXU+KEWaKPDwfYJ3PzSZAH5L8UzB6w ZgjlEOFHCGSUitZaeh9ZJoTHUDC6vdU4IwEIBpJ0ocY2fNRf/p+DO3DviOyUT5f88/rdiHfjYKj94 jeZnq6tPFAwmKqcf0Pcw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oXi63-009XVm-1I; Mon, 12 Sep 2022 11:59:27 +0000 Received: from mail-am6eur05on2138.outbound.protection.outlook.com ([40.107.22.138] helo=EUR05-AM6-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oXi5y-009XRS-KP for linux-arm-kernel@lists.infradead.org; Mon, 12 Sep 2022 11:59:24 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=P05kkyVZPIabfNqTSBJyAauE9uunqDwWzMK7fGlnnZhUv2Miu72ZzbA60L1i4pDBUA50MXfZ1+CQM9nz7Cq7X+gsBiwTilHHgiifGKWmcOjdu9xMYLaAbSUiMvoHvIpXDxfblE6uu+HzeioBN0gi+mJ9VzItSWi7RimrJmZ8OtIVoUpk4J3UIT8OzSHrLVv9x81JFDYkaJHJG4PbEdUDE/fvi5bpA3req57g8Jiki2NLTwprBwInSlu0OAZqrXk8imiy1SYwsUk31sUuxWlgPCPSfZp+k5BWlvFHb4JpF7C7J0VCOgXv/jX2OAkYGB6WkOH5dJ7inR4SqqkdfA/h7w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=oaljpYSNJxGg0xM0WfQKxeO+ZAbtdWl7lek1ejdnU8Q=; b=jT0xMl2YkEltEPCoDZcvab8SqlyLKgkTxqcYo8M1WphOBTHWdCcmfV2jV+WRUuS5MJN4KqosEvDPFiIKrvyOKbWft+STeEV86Y04iIb8dcGv9/X0nh6Wz/js5sv2Nz3lk6RZk5XUPY37rDo8PS66BYWkLnsgjm8Dz6AbJuoalO0ldJDAADZj0XMFA/K+/pLMaajsI8GATKGKg7MoBjZNmqqSTc8/f/5Vcm0nML2/wXwJ8qDhcdi2np1f6aAyQxLU/quFlR0XXBVW60U+mG0EoX0CCw1WV6s5Lc4B4txRZ3ROtBoOFDbLcCJu/zBn4ZlHQ8GqL6/GgmUVfKOwRppp6Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=bang-olufsen.dk; dmarc=pass action=none header.from=bang-olufsen.dk; dkim=pass header.d=bang-olufsen.dk; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bang-olufsen.dk; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=oaljpYSNJxGg0xM0WfQKxeO+ZAbtdWl7lek1ejdnU8Q=; b=f5unecRkrP3gu8eaBgz6DScBuxZyAigtIA+TSCeXdztIQthSs1y3wap+rFUAAvyukeTjFLovRzoH3gn4ZVPi+dUX0SCNAJa1vyK4hGZYZBDZF/4KYDm43+Xq2ekH9SY/p5EUns8h6yvi3LjdPwOlq9C21UFUtOu88ZqOX1KCkpo= Received: from AM6PR03MB3943.eurprd03.prod.outlook.com (2603:10a6:20b:26::24) by DBAPR03MB6485.eurprd03.prod.outlook.com (2603:10a6:10:197::5) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5612.22; Mon, 12 Sep 2022 11:59:17 +0000 Received: from AM6PR03MB3943.eurprd03.prod.outlook.com ([fe80::f17f:3a97:3cfa:3cbb]) by AM6PR03MB3943.eurprd03.prod.outlook.com ([fe80::f17f:3a97:3cfa:3cbb%5]) with mapi id 15.20.5612.022; Mon, 12 Sep 2022 11:59:17 +0000 From: =?utf-8?B?QWx2aW4gxaBpcHJhZ2E=?= To: Russell King CC: Arend van Spriel , Franky Lin , Hante Meuleman , Alyssa Rosenzweig , "asahi@lists.linux.dev" , "brcm80211-dev-list.pdl@broadcom.com" , "David S. Miller" , "devicetree@vger.kernel.org" , Eric Dumazet , Hector Martin , Jakub Kicinski , Kalle Valo , Krzysztof Kozlowski , "linux-arm-kernel@lists.infradead.org" , "linux-wireless@vger.kernel.org" , "netdev@vger.kernel.org" , Paolo Abeni , Rafa__ Mi__ecki , Rob Herring , "SHA-cyfmac-dev-list@infineon.com" , Sven Peter , van Spriel Subject: Re: [PATCH wireless-next v2 01/12] dt-bindings: net: bcm4329-fmac: Add Apple properties & chips Thread-Topic: [PATCH wireless-next v2 01/12] dt-bindings: net: bcm4329-fmac: Add Apple properties & chips Thread-Index: AQHYxp8VRhIzhoN1vkifFhGA/p+3uA== Date: Mon, 12 Sep 2022 11:59:17 +0000 Message-ID: <20220912115911.e7dlm2xugfq57mei@bang-olufsen.dk> References: In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=bang-olufsen.dk; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: AM6PR03MB3943:EE_|DBAPR03MB6485:EE_ x-ms-office365-filtering-correlation-id: 92e17d87-83d2-44ee-42ae-08da94b63816 x-ms-exchange-atpmessageproperties: SA|SL x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 4R+UtDYlDwXWQqHDMpm+PMv4nJlVxgAJOQkwxThQz9sCNxRqHlju4oXDgLPcAEtu4m3hu0r716027An6Q/+ELjKu0yYPb3RRY55Pf9ul2QvwWBSVHbWM/mPPeUB8GVjNfyQdZhbQ3GsHN6vCyCYBZC9sJK1EUwhHPmTvnKBu5RShN1rJU2xynBYHGjKZjIi/LEs7MerD5G4+tmdKyQf6fBRLHjrhiWxjb3KMvL0fnwxUa8jW2iV6N3lEoVUpYQQEORqdH9NZBfLFQElVgAOHMWx+8DjuaoeyD2GeLFO9DXTqxx9SHJw3YJDLKhiitIEcIoPzFjbK89bSkFUNR7zADJHa32WmlQ2e1yoZ6AEysmio8CFIv4PWb3LUHG4IiKY+T7bDK8oSdO2EjPWHY0kbvKacHadFm0ooyuDl6O5GV203Y7xds42XRHgWNwabmNbMBAHIg6W5dX3MGMS85RJT2e/rHigMYcmFDlUA4+PHK34mG4wBakdXgyYQcHr34oT6r7zfEYqaXw372XsqFmtscfnXrFEE62vbDJRAhaj8q1wTusG8QfzUvJEwHdnz4B7qxI+QqIeHx0aJQ5xCSFLBv0ZNxqbvVYkbjHhaW5xDBULaLFnIirUSLQOUfgZc4v5P0mWzm6J7TSCh5vAHU+B/0AycgLfCOni4U18vW7PaNUivY7CgX8DU7o35D0NXPnYyXng/AZvAi1JufPk61/O5z8vLIl2Wjfx5+FuMftJeOD0kM3hJBqUotBihykv8gaT6LqkUAMnGNnhx+UbPb+PkIa9Xpl/n5XN9/WHhwpt5lt8nU00S2+dbAvGXVtSohxQ7iVkKv7QJcd2RqgyIqEWsyg== x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM6PR03MB3943.eurprd03.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230016)(4636009)(396003)(366004)(136003)(346002)(39850400004)(376002)(47660400002)(122000001)(38100700002)(6512007)(38070700005)(26005)(83380400001)(85202003)(2616005)(186003)(1076003)(6506007)(6486002)(4326008)(54906003)(966005)(478600001)(316002)(86362001)(64756008)(66476007)(66946007)(76116006)(66556008)(71200400001)(8676002)(41300700001)(91956017)(7416002)(2906002)(66446008)(85182001)(5660300002)(8936002)(8976002)(36756003)(46800400005);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?VVVFK3g4aTBJNmFVVzNoVTc0MDk3c2R0dU9lV1U3S2tzcWJUK0VJQ2l1WVk0?= =?utf-8?B?aGQ5R3FrZ3hlVW5mQk1McHJtQW5lbXFsR29pRWszYjFpNFlQYkV5ck5OTUpN?= =?utf-8?B?UnFhc281K2tDM1UvbUc1ekRGL2Uxa1BRRlJiWThkVURHMk5maUFaaDN5YmhX?= =?utf-8?B?RWxQSlRVNlY3M01MMVdNK1BlWHB5Z1BrdlRuNnZwb1hKVFYyWUVJT2JqSWtV?= =?utf-8?B?Q216dGkvL2ptQ1MxVjdOZVZHNEJRUTljWC82YitmaGxMM2lXS2ZhYytWbDIz?= =?utf-8?B?Ymw1cElmQkRqZWlUK1l2Sy9XbmVRRE81cGNPMmRycTFudGFLc1J3cXhtZ3A3?= =?utf-8?B?SHlTSW9NTHZPeFRUYithNnpzbUE3ZEl1NFB2UlEvV21nY3hxSzFpaVArME51?= =?utf-8?B?MjNHY1ltVm5CVGJla1hrQlR1dDYwcDZZOWRvVnNFVFlLYWdDM0tBWUxRZitN?= =?utf-8?B?SThTNEloYTEvckpoenZ6MlhtQVJUemdYY1B3ZForelRmU2VDNDJ2b0JnMnor?= =?utf-8?B?eExnU2tBSmI4dkY4RlNKay9aRmYyYStPRGVPak1PSXdMQjV5TVJ4ZlVuakhw?= =?utf-8?B?WDloQmhFK3Z5dXBOV096Q1dlRDY1N0tPOE9ucWJWTWcyMmtybnJEQXpCMWpQ?= =?utf-8?B?WkFTVlZSUS9aSldDK0NHWStjMDBiOFJzUDB1eGcrMlRQTzFxY2xWeGIvYkdB?= =?utf-8?B?OEF4TEl1ZCtScW9RU2twNTZNL3NreVNtUjJBRVVkdTZUcktZT3p0aTRGM2tI?= =?utf-8?B?YXRrOVlTenp0dmpXS1FDMFpNMlF0WVpTYTlZMzBHZUMyeHdiTmpMRUUxRWJo?= =?utf-8?B?a01kT0p4RVV0SVlOUXB3MnRtOHAwT2d5S1FtMXJTVE56NUdxOE9PckdCMmxJ?= =?utf-8?B?cnp5OXh0TzJ6UXhXU0pHcmVPdzVUM3NlVE1WcmR5eENnb3VuWFpaMFhSdHUz?= =?utf-8?B?Ry9tZGZqTUUwc1kwM0lPM2ZDZ1craTBhdGVZSHBGTTZVcW1qeE9mLzhTY05U?= =?utf-8?B?dXVXUHdmU1VnTyt1QTlnajA3SWRiTnJlMnJwaXlqaTEwb1hsWXlOMTZ1UHA2?= =?utf-8?B?S1Uva0Irb0pQb1pDeDg0akF1eUo2TVBJVlhsc3Z4bmhoNmZ4eEcwazJWeGF6?= =?utf-8?B?QXNjQlpxbTVlQlg3Wk96ajZRWSthUjQySDNTNndWZTlCWnBDWHJvdDFMOXNH?= =?utf-8?B?elpXVVJ0VUV3dFRSLzVoblBycXZtZzU5ay9CdHRlVXNCaE1naGV5UXlrem8y?= =?utf-8?B?QkxGcUxWb0ViS0oxdURpZ29MSGEzN2hEK2hnS0NiT09YeDZHTUEyTHNXN3lt?= =?utf-8?B?NXBZTWZIdTJaQVpFR0dIT1JZRm9lT3ZqejVHVXN6NjdIZUF0SnUyWjdNTGp3?= =?utf-8?B?Q1dqT2tLaGxmMW9tU3pKQkdlNUNreTFRTmVNcG9XVVFiVER3d2pmOC9zbkt1?= =?utf-8?B?VWV2alFWZjdqUENNdUVwWUlmcWFUOHNackxsSmltbnNVTkRxblpwZ1FVZVJ2?= =?utf-8?B?ZHpQcGEzcmtyV2JxdG5LbFl0UFpocFR4M1BHa1puNXZMb2lpTEJVakp2SlNk?= =?utf-8?B?SHdMMHMwSEI1b2hzV1N0SkZ2U2ZCdjNaZnNlR2lHM095bm96KzU5Y3FIYTRm?= =?utf-8?B?RHBNMlp1aHJDM2dSRFYyMUFQUjMramx5QWdYYVptN0xsUkxKQUw5MEM5MmJ3?= =?utf-8?B?U3BEZTVXb3hid295eHA5UHhRcnNsejByRUQ4NHo2SXFzNWFpNTRkWHB2T3pP?= =?utf-8?B?Mmd3QXhTUm1PZCtlQVVlRVJWaEVDek9HRXlxR1hUTUY3bTUzMEY0cGw1aW1s?= =?utf-8?B?cXRlZDhSVGxCWWdSaG40dWF6R3h2bTJ3TUROZEhjZlZmdlNNdEphY051NGZT?= =?utf-8?B?dVprZWlSeXZCcFNpYTlzV1RsaHRodzNpSTFMcnE1MUlJaExPNURlb0NnaGZ3?= =?utf-8?B?cmkvaHAwN3l1ZjZOaW1xeWlCMnlrQi9WSWlLSnBNWDFqUmdQR2kwTXNkWUo0?= =?utf-8?B?Y3pyWm0zeUFkY0pRbGt2Q3QyR0ZPZFRDNFhRS3JwV1hxZklienk2MThsNU5t?= =?utf-8?B?dHdKcEdtRGhMdytsMWVtMVljYWEvTXBTZFp1Z09qY2ZLYjYrdUU2WmM4WEIv?= =?utf-8?B?WDRYMVhTUjdlRzVQaTl2c0xIRThnQXMyUzRmRjJKNGFQd3Yva0xxdDBlU0hF?= =?utf-8?B?V1E9PQ==?= Content-ID: <0E831374D71B9A48B2DE80A1BFA3BAB2@eurprd03.prod.outlook.com> MIME-Version: 1.0 X-OriginatorOrg: bang-olufsen.dk X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: AM6PR03MB3943.eurprd03.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 92e17d87-83d2-44ee-42ae-08da94b63816 X-MS-Exchange-CrossTenant-originalarrivaltime: 12 Sep 2022 11:59:17.2131 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 210d08b8-83f7-470a-bc96-381193ca14a1 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: nFfnSn6rmnSlm1CqxQ0JYoLD61Du7yfgjLD+bbcV+IuPuz0zVGEFfHH3OCtdaVHbLxcre2rvYIssSqkIKe+Rpw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBAPR03MB6485 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220912_045923_242008_25F7E27B X-CRM114-Status: GOOD ( 31.92 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, Sep 12, 2022 at 10:52:41AM +0100, Russell King wrote: > From: Hector Martin > > This binding is currently used for SDIO devices, but these chips are > also used as PCIe devices on DT platforms and may be represented in the > DT. Re-use the existing binding and add chip compatibles used by Apple > T2 and M1 platforms (the T2 ones are not known to be used in DT > platforms, but we might as well document them). > > Then, add properties required for firmware selection and calibration on > M1 machines. > > Reviewed-by: Linus Walleij > Signed-off-by: Hector Martin > Reviewed-by: Mark Kettenis > Reviewed-by: Rob Herring > Signed-off-by: Russell King (Oracle) > --- > .../net/wireless/brcm,bcm4329-fmac.yaml | 39 +++++++++++++++++-- > 1 file changed, 35 insertions(+), 4 deletions(-) > > diff --git a/Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.yaml b/Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.yaml > index 53b4153d9bfc..fec1cc9b9a08 100644 > --- a/Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.yaml > +++ b/Documentation/devicetree/bindings/net/wireless/brcm,bcm4329-fmac.yaml > @@ -4,7 +4,7 @@ > $id: http://devicetree.org/schemas/net/wireless/brcm,bcm4329-fmac.yaml# > $schema: http://devicetree.org/meta-schemas/core.yaml# > > -title: Broadcom BCM4329 family fullmac wireless SDIO devices > +title: Broadcom BCM4329 family fullmac wireless SDIO/PCIE devices > > maintainers: > - Arend van Spriel > @@ -41,11 +41,17 @@ title: Broadcom BCM4329 family fullmac wireless SDIO devices > - cypress,cyw4373-fmac > - cypress,cyw43012-fmac > - const: brcm,bcm4329-fmac > - - const: brcm,bcm4329-fmac > + - enum: > + - brcm,bcm4329-fmac > + - pci14e4,43dc # BCM4355 > + - pci14e4,4464 # BCM4364 > + - pci14e4,4488 # BCM4377 > + - pci14e4,4425 # BCM4378 > + - pci14e4,4433 # BCM4387 > > reg: > - description: SDIO function number for the device, for most cases > - this will be 1. > + description: SDIO function number for the device (for most cases > + this will be 1) or PCI device identifier. > > interrupts: > maxItems: 1 > @@ -85,6 +91,31 @@ title: Broadcom BCM4329 family fullmac wireless SDIO devices > takes precedence. > type: boolean > > + brcm,cal-blob: > + $ref: /schemas/types.yaml#/definitions/uint8-array > + description: A per-device calibration blob for the Wi-Fi radio. This > + should be filled in by the bootloader from platform configuration > + data, if necessary, and will be uploaded to the device if present. Is this a leftover from a previous revision of the patchset? Because as far as I can tell, the CLM blob is (still) being loaded via firmware, and no additional parsing has been added for this particular OF property. Should it be dropped? The rest looks quite OK. Kind regards, Alvin > + > + brcm,board-type: > + $ref: /schemas/types.yaml#/definitions/string > + description: Overrides the board type, which is normally the compatible of > + the root node. This can be used to decouple the overall system board or > + device name from the board type for WiFi purposes, which is used to > + construct firmware and NVRAM configuration filenames, allowing for > + multiple devices that share the same module or characteristics for the > + WiFi subsystem to share the same firmware/NVRAM files. On Apple platforms, > + this should be the Apple module-instance codename prefixed by "apple,", > + e.g. "apple,honshu". nit: s/WiFi/Wi-Fi/ > + > + apple,antenna-sku: > + $ref: /schemas/types.yaml#/definitions/string > + description: Antenna SKU used to identify a specific antenna configuration > + on Apple platforms. This is use to build firmware filenames, to allow > + platforms with different antenna configs to have different firmware and/or > + NVRAM. This would normally be filled in by the bootloader from platform > + configuration data. > + > required: > - compatible > - reg > -- > 2.30.2 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel