From patchwork Fri Sep 10 16:04:44 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?b?SsOpcsO0bWUgUG91aWxsZXI=?= X-Patchwork-Id: 509080 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-16.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_CR_TRAILER, INCLUDES_PATCH, MAILING_LIST_MULTI,MSGID_FROM_MTA_HEADER,SPF_HELO_NONE,SPF_PASS, USER_AGENT_GIT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 49E43C433F5 for ; Fri, 10 Sep 2021 16:08:19 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 32AEC611F2 for ; Fri, 10 Sep 2021 16:08:19 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233564AbhIJQJY (ORCPT ); Fri, 10 Sep 2021 12:09:24 -0400 Received: from mail-mw2nam10on2056.outbound.protection.outlook.com ([40.107.94.56]:59105 "EHLO NAM10-MW2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S230259AbhIJQIV (ORCPT ); Fri, 10 Sep 2021 12:08:21 -0400 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=jj+KvVsjyXBTfPTVZ6cWqFxrAixWuhnC2j65TxnDpDB/MT7c6NNM68BUhAhL7tIvxm7Gz6NXiC2FeZlUyRLTWq1tdwoCr4NZtGWz0oEz8HsrRMlU4sLhKBIhkdd/40Zxe2WtZdss0KW/0llCT/oUgY36I95k38lgUmYpkMX18jIyNvpKm1BJtyB3n5pOUCKHvMp6rOmNjoN8Wu1CDw1rC3b3AM/a2p+k4EBVN+sGAFXyMYnVJH+BJY0vGDR4E2GjIihjRmq6EgxB7mDYW9x3rm5OEuxLiUSgOoCqyUuJSeYErIKpzokdWWYo4SiYz8zvTPoFm7fkxzlovw6kEfRyeA== 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; bh=irK1qzipT3QHIWJf6rEuMgUX9uURfiAS1g+yt6uFeLc=; b=TqbLDIG6rzYx5XHAanvjuc3rPZX2b1pdwgjI+OqB0f48v7mLN2WqLVn84AQh0sEFvQe3pNiWXrG6eNE1WyCrhEam+NYNEI5krej265i7NV389ao1XEGkBWIky/tzZ9u1TcsL/MafNy9NSWmwBdHfbMDdk2ZP/NtCY+MHPTZLS9XHgqXT6pkVSRAqabdR9LMqRKqNEsqxlehIc7tf2SwL35wESSK5dPunaWOsq4gnk87X3CK9N6Wi3ZOH20YlCgnofBzi6FLFSeRzcFbAxdsefrNErZXwv5sx8tyqmMy0ZNwcFYrPFdoPKJrvOwPwIBfx94d7v75ii2Frjj4YF2qF6A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=silabs.com; dmarc=pass action=none header.from=silabs.com; dkim=pass header.d=silabs.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=silabs.onmicrosoft.com; s=selector2-silabs-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=irK1qzipT3QHIWJf6rEuMgUX9uURfiAS1g+yt6uFeLc=; b=TjTrz5CVjtnBhSe1q3IYAHYq7WmoQdbeUdUnlel0x3yAKvQLAoxdgoXcKowCiATvMj9nEZDp1ZAWbtejTMSFSYRiXD+cGzmN/uTWRnrYxfnn1h9K8Fi+5A72AubpjTDb+jND4ZEKjar2wJrI5+jqyAMUHoapdHHMNc/xdYtubFI= Authentication-Results: driverdev.osuosl.org; dkim=none (message not signed) header.d=none; driverdev.osuosl.org; dmarc=none action=none header.from=silabs.com; Received: from SN6PR11MB2718.namprd11.prod.outlook.com (2603:10b6:805:63::18) by SN6PR11MB3118.namprd11.prod.outlook.com (2603:10b6:805:dc::32) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4500.14; Fri, 10 Sep 2021 16:05:54 +0000 Received: from SN6PR11MB2718.namprd11.prod.outlook.com ([fe80::7050:a0a:415:2ccd]) by SN6PR11MB2718.namprd11.prod.outlook.com ([fe80::7050:a0a:415:2ccd%7]) with mapi id 15.20.4500.017; Fri, 10 Sep 2021 16:05:54 +0000 From: Jerome Pouiller To: devel@driverdev.osuosl.org, linux-wireless@vger.kernel.org Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Greg Kroah-Hartman , Kalle Valo , "David S . Miller" , =?utf-8?b?SsOpcsO0bWUgUG91aWxsZXI=?= Subject: [PATCH 11/31] staging: wfx: relax the PDS existence constraint Date: Fri, 10 Sep 2021 18:04:44 +0200 Message-Id: <20210910160504.1794332-12-Jerome.Pouiller@silabs.com> X-Mailer: git-send-email 2.33.0 In-Reply-To: <20210910160504.1794332-1-Jerome.Pouiller@silabs.com> References: <20210910160504.1794332-1-Jerome.Pouiller@silabs.com> X-ClientProxiedBy: SN4PR0601CA0006.namprd06.prod.outlook.com (2603:10b6:803:2f::16) To SN6PR11MB2718.namprd11.prod.outlook.com (2603:10b6:805:63::18) MIME-Version: 1.0 Received: from pc-42.silabs.com (2a01:e34:ecb5:66a0:9876:e1d7:65be:d294) by SN4PR0601CA0006.namprd06.prod.outlook.com (2603:10b6:803:2f::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4500.15 via Frontend Transport; Fri, 10 Sep 2021 16:05:52 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 2579bc4a-ef9d-4a39-7791-08d97474ddca X-MS-TrafficTypeDiagnostic: SN6PR11MB3118: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:2150; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 6qJLRuUDGoLqK8fkPW3jrNPnyHfBD6B82R3LDP5T8oEbY5lYfzYx1vV6ske0gMP/3cblWG9iV+gbG4oUFiTnh5sWkm7uD55sb7248sQUEa6ijKKp6ePOWeGLgF+ISGy9GE+aQzN8dWaRKGhO/5VFirYztXGYK02QSC7b80JiDAvc4foIU1V7J01FLI1E4Ew536yfCzhlHxaiHA3bZZyOTRvwJ0jg6JpTMOI16mIYXBSd1DHdpsfbI78wXa7Y9AWUet5MGpc/sX+xhle7zkqOPrQfC3G7RDAuhnBSzBso6aeXTQ3NXQTFc5PnShD4CxK2VpWVI12ZgfbxsAdgLMiqYnnoUZTJi+dfhmbTawMR2DinWJ2rd4h9kDGjyzxreMaWm0yDbIBqCW8GuJVs5g1ItW55H8SWCP0QkETz3BhPT9pMAlgIFrbeDFyEV+wSK3Q42ig/VG1Z2UBEy3TpyRt07oooapv3JhCMrgGiBCvL62zBm6KAuZnRoyZTBU2UjzEbfxyraQ0UT+x9nRrVb6/f5Hd20J5rbAKnnkIekLSwwVrBIshCvvpMuXZ74AmMmj4VkKNiuilCErPCVkhAr4HshhTZaNTmpEadzodUlHN7yn3PGsFSxq4cUeiU8l2xPxtN+6SUl09Oolw9JJxvShogHw== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:SN6PR11MB2718.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(136003)(366004)(376002)(346002)(39850400004)(396003)(4326008)(66476007)(107886003)(36756003)(186003)(316002)(6486002)(66556008)(8676002)(6666004)(86362001)(2906002)(54906003)(38100700002)(83380400001)(8936002)(66574015)(5660300002)(66946007)(1076003)(2616005)(52116002)(7696005)(478600001); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?mEmYKLiqtGWx3mbDJVi83yCZa?= =?utf-8?q?auEvsjviCAJTB1qaS1Vnt8sanYxBvoJVHDtwQpB4fwD8fGYmw2IYDsMc?= =?utf-8?q?broruqOKG5Qb2VrZgsBENKRO+wTSSoJHQnEvL4u200tuFeVVZyjal7yl?= =?utf-8?q?7IpGvZ/gayG8nKJxQD+b24fbCplfUUMKoMSNJDSVc5n+dxys2JHNGyga?= =?utf-8?q?yV04H/+3DH0idz7oO5FKmhX1qcDYFcYUqE94r4cAubAGa3N4pnMKULpx?= =?utf-8?q?+SOOl7/4n1uqFnyRXknNdAAqe8w83uD2bjXd/pqQN8xDfvYemFE3rRMo?= =?utf-8?q?2a19KIag9gYZcDP9nFX9uEeRY0x5REcH/BvHqMr+2L/khiBbpRBTcf/A?= =?utf-8?q?HN7b6+X0IITei16lm9ykh1tT8i+TYeGRJQbjGLv94TiU5jlG80s2QbKU?= =?utf-8?q?T3eIFRHPTylwXIO2rHeQ0TDNZUEVFRzfv5DvXN8LSsoYLm1jfac/D3/a?= =?utf-8?q?FMEkZJ42Y184H0HDmOBVG0561KYZSrX9lmEIPd3IoabcdokVNhYyFldZ?= =?utf-8?q?pOyyvnuaDztXAwAqKbr/31VmnwR/CAQkxnd//hdLWz5E+2PTXnq7fp/a?= =?utf-8?q?P7omtCtedh2G7eHFgYpXZ6zHSDexS5Ggc+ah7XOJkw8ftuhqMq2EF7DV?= =?utf-8?q?nK3uC0gBWydaGaP1wc/++mnr2BmhiipLGloB/7pFJCnUNkO+XHPZqbvV?= =?utf-8?q?LF2I5DwX4Jw0rCmH3b9N6vbbu/Hz35xVOt0N4Hu3izM7sFX3FTWWwBSE?= =?utf-8?q?T9DVIfOQArvvXpxJENd/lNqF+BztVEaXrOCBSfy92Hq3VUw4zMmT2fsX?= =?utf-8?q?+Jc1aNiv3DWduSZOZ8uc9gATK6jIWGNXLSC+KwtsB+61I0v4TF2vutrW?= =?utf-8?q?YseLGV1IBThKkOSm4Uk8kbG4wqSBHlrGZGST2QKDn2hIRIVieb+dcQG6?= =?utf-8?q?Bj42NmRRCMPazufORYdDkA8fDoMwtieucDJTbAj0T/a0aqGFfij0dFMh?= =?utf-8?q?rxkCSl9XD+NIMkCCIYfdZkMRN2wotlsKNBWwayTL9IoxUcpb4Pl3PR4q?= =?utf-8?q?Wpf4+C55g0FKNSOFdJoP4/gWyWbhenie4gGFTmlkWnKUhX+LsqW9YT6r?= =?utf-8?q?qpgVNfnZXUXiN1Dj3YNXsg2Xtz/YRDK17FveTgtA2k+qf1wm8sg26z1a?= =?utf-8?q?zOw9933ZT+79RyBPf4pBNfLBfRQc+6fwA4VA6ZHvx8gKonYm5ta8la+s?= =?utf-8?q?CwhKuFxkPAcoC2fAHwehKiN50Giq1BS1zc7m94lJUpkc+sXJanGCtNeH?= =?utf-8?q?lyY3kuVHEfxw2GVpWWUe6b1C6TByFE3lXN4Y9oJazt2J5q029ySonp5f?= =?utf-8?q?c9BYrNDz22iPgeNytqCXg848N0pQ3/RARXdp2KlnR8kAQUhASbBUfkFR?= =?utf-8?q?xyR/gpqD+615xR3Pjz9Jz8PlpQN8JNp1qRkUq5R?= X-OriginatorOrg: silabs.com X-MS-Exchange-CrossTenant-Network-Message-Id: 2579bc4a-ef9d-4a39-7791-08d97474ddca X-MS-Exchange-CrossTenant-AuthSource: SN6PR11MB2718.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Sep 2021 16:05:53.9035 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 54dbd822-5231-4b20-944d-6f4abcd541fb X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: KnuyreqUmevajPS8eNekTSpUaOH4nw0FFROJ0NuOmYdW+isAuJwmod4nC3BBRAdSliPn0zGFfrDCWqZuf296HA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN6PR11MB3118 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org From: Jérôme Pouiller The PDS file contains antenna parameters. The file is specific to each hardware design. Normally, the board designer should add a line in the of_device_id table with his own antenna parameters. Until, now the absence of PDS file is a hard fatal error. However, during the development, in most of the cases, an empty PDS file is sufficient to start WiFi communication. This patch keep an error, but allow the user to play with the device. Signed-off-by: Jérôme Pouiller --- drivers/staging/wfx/main.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/wfx/main.c b/drivers/staging/wfx/main.c index 0a9d02d1af2f..b790d8573de6 100644 --- a/drivers/staging/wfx/main.c +++ b/drivers/staging/wfx/main.c @@ -220,7 +220,7 @@ static int wfx_send_pdata_pds(struct wfx_dev *wdev) ret = request_firmware(&pds, wdev->pdata.file_pds, wdev->dev); if (ret) { - dev_err(wdev->dev, "can't load PDS file %s\n", + dev_err(wdev->dev, "can't load antenna parameters (PDS file %s). The device may be unstable.\n", wdev->pdata.file_pds); goto err1; } @@ -395,9 +395,7 @@ int wfx_probe(struct wfx_dev *wdev) dev_dbg(wdev->dev, "sending configuration file %s\n", wdev->pdata.file_pds); - err = wfx_send_pdata_pds(wdev); - if (err < 0) - goto err0; + wfx_send_pdata_pds(wdev); wdev->poll_irq = false; err = wdev->hwbus_ops->irq_subscribe(wdev->hwbus_priv);