From patchwork Sat Jan 6 02:25:22 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Suma Hegde X-Patchwork-Id: 13512643 Received: from NAM12-MW2-obe.outbound.protection.outlook.com (mail-mw2nam12on2065.outbound.protection.outlook.com [40.107.244.65]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 579D015AB for ; Sat, 6 Jan 2024 02:26:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="Ed5aEhud" ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=IsKWLvBx5eH0MtqmmzvzcZwUvj1Z53hAVPfcj/irPezCnTGzpDnj6b1R/Lz3v1FeRtUEcb2Na+OnvPCOInWFrHvd30g3l6UKdOf7y23lI9oM1YbdNnYDx0r9pFe1Ucx+u23rERmQKPkBIEDP5wPPWekkQgV66Rj3Fw6qD2PMcDaVX2o8sD+csmCeXRzwZ5BKEVS4ALdYvzktdREmtzWsTd0BkckQISt8xU3XQ8or0ZyxFTGfHGYb/leXoxwFVXmwy2yA6xsulgpbXhcjM6EzSCJmB96YFr+ZzY43E/Ys66WdQHrqkjgmVg9Ax5qLqNnAxM/JqQcUuMLLEzDISQ9ygg== 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=+WvWvYN9ua0WI4AGxLC8lIQig9Gw3hjXmkXGsvtyXr0=; b=BhtI7bwb3ihPsSvKxhUXxLCwPLL1iC/hSQto3/sfq9usjN6fdXFWqjEgGmn7oN4g+OmwlRnOdqopyWq/O9OAEvSvZBcmtT27z6QkLP1fQ2vLRyr1aPgd04oAS4HHwI6KhBaEUP3U2bOkUI9neWQAk1qZA212zL2o3X/6wMOL6aAKAlmrIQgg5el6+/4g3ScG4q5C8FA4SApc9I/GZJvRc8g8RebA6xQo43Ge0EdBXhnXrGQ0kOlIgay33ZlkYwrWxPCfsLZFLdRKLn0ItIJdi8xwa0610qcoe5GPa6GkE0l+E5gB3cifi/94lsOfAl4j6fpzudFSJeUtqBPPlMIAlg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=+WvWvYN9ua0WI4AGxLC8lIQig9Gw3hjXmkXGsvtyXr0=; b=Ed5aEhudkn0WFrC4YI35g/HWVZABWs9u4XwVEactdRcD4BlXxyVxUWjwwU3I5nMO4CDIbfbwtHdwc1P8CSPbkHGPhiIw1EuoxeFjJarZzvNWJkkp2PqHSIl9rHjd+dsSEbKeWpZTrTgaH1/J9uqHAw5irFSza2S7lbx2WBXLcLQ= Received: from PH8PR07CA0033.namprd07.prod.outlook.com (2603:10b6:510:2cf::20) by SA3PR12MB7903.namprd12.prod.outlook.com (2603:10b6:806:307::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.17; Sat, 6 Jan 2024 02:26:09 +0000 Received: from SN1PEPF0002BA52.namprd03.prod.outlook.com (2603:10b6:510:2cf:cafe::a4) by PH8PR07CA0033.outlook.office365.com (2603:10b6:510:2cf::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.17 via Frontend Transport; Sat, 6 Jan 2024 02:26:09 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by SN1PEPF0002BA52.mail.protection.outlook.com (10.167.242.75) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.7159.9 via Frontend Transport; Sat, 6 Jan 2024 02:26:08 +0000 Received: from amd.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.34; Fri, 5 Jan 2024 20:26:06 -0600 From: Suma Hegde To: CC: , , Suma Hegde , Naveen Krishna Chatradhi Subject: [PATCH v5 01/11] platform/x86/amd/hsmp: Move hsmp_test to probe Date: Sat, 6 Jan 2024 02:25:22 +0000 Message-ID: <20240106022532.1746932-1-suma.hegde@amd.com> X-Mailer: git-send-email 2.25.1 Precedence: bulk X-Mailing-List: platform-driver-x86@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-ClientProxiedBy: SATLEXMB04.amd.com (10.181.40.145) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SN1PEPF0002BA52:EE_|SA3PR12MB7903:EE_ X-MS-Office365-Filtering-Correlation-Id: 4feb963e-6d04-49ee-44c6-08dc0e5ed7cf X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: v6VnhDGsOwQYVM+aFeug5otmlfrQBgjUFRju5rwx7zqEu6YMFNB1nruuhHjsb1pK1o7rmng0O37wf9HCL5MN5CeyGVuPJGBZyg5Ms/lCKJm34GVUpOhUo9Kwbp2GObMN3wjKPYg+k/hIW8rWnjPLi8Idhf4y7/VArAy88u5BWDknk20jas8SKsf8fMbHhhycN/fsd9dDr3m+d51JziD2TZrM/hVlXyqQTvKoyw8QsNlkREEiiOvhv6AFjHJ1a+a96PU//7zRzwijoalas6V5+mu6PalKiSuBP1J1Rl6YPXNOabz66lRFhMsJiuzWGbgElT/wgYyito5PRWdDpYY86k/gxzAAQ/zzIyX7b/cxeQZjqmYCuYBAUeU5Gp2YBrEmM56oInpBPJASt8FXqVeI0RolmxCwt7EJHzYKVi4Jx4uGF6dpCFt5IiRe6BTYW++My2FggHh1gqiQGGElw4FnPjHOvrIZC55xGncofJLYvQiNHYUGzEIBAatFzlOUi7cmfqzpWv6k0bQz6vjN6ppvSzZip+2hbUeEJeEbV27vinikFFIPehGsyzFKerhRAAjjWmU0kPrStmk5KKO8Uz+kSJGbrATf/A8WLiPK+waBLp6HqeXQF20EFTuxtn3ZpC2tghvJAi+BJMy4gjqQehtCNEdIfDXh8OAE45f8P0/e/K2/6QJ6XvWuPGjmB0AmO3gJduBlJ1H/lsydn7OAtS/AEGKPUqLM2wkgS39Hpu7VE5gI9BflleNU0xTs9TJZEKCY7/ODwRMEkU8q2+qlqtoRKg== X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230031)(4636009)(396003)(346002)(376002)(39860400002)(136003)(230922051799003)(1800799012)(451199024)(186009)(64100799003)(82310400011)(40470700004)(46966006)(36840700001)(83380400001)(336012)(16526019)(26005)(426003)(66574015)(1076003)(6666004)(7696005)(2616005)(47076005)(5660300002)(316002)(44832011)(8676002)(41300700001)(2906002)(4326008)(70586007)(8936002)(54906003)(36860700001)(6916009)(478600001)(70206006)(82740400003)(86362001)(356005)(36756003)(81166007)(40480700001)(40460700003)(36900700001);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Jan 2024 02:26:08.9386 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 4feb963e-6d04-49ee-44c6-08dc0e5ed7cf X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: SN1PEPF0002BA52.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA3PR12MB7903 This is in advance to supporting ACPI based probe. In case of non-ACPI driver, hsmp_test() can be performed either in plat init() or in probe(). however, in case of ACPI probing, hsmp_test() cannot be called in init(), as the mailbox reg offsets and base addresses are read from ACPI table in the probe(). Hence, move hsmp_test() to probe as preparation for ACPI support. Also use hsmp_send_message() directly in hsmp_test() as the semaphore is already initialized in probe. Signed-off-by: Suma Hegde Signed-off-by: Naveen Krishna Chatradhi Reviewed-by: Hans de Goede Reviewed-by: Ilpo Järvinen --- Changes since v4: None Changes since v3: None Changes since v2: 1. Add "Reviewed-by: Ilpo Järvinen " Changes since v1: 1. Add "Reviewed-by: Hans de Goede " drivers/platform/x86/amd/hsmp.c | 30 +++++++++++------------------- 1 file changed, 11 insertions(+), 19 deletions(-) diff --git a/drivers/platform/x86/amd/hsmp.c b/drivers/platform/x86/amd/hsmp.c index b55d80e29139..3c17b488f4f8 100644 --- a/drivers/platform/x86/amd/hsmp.c +++ b/drivers/platform/x86/amd/hsmp.c @@ -244,12 +244,7 @@ EXPORT_SYMBOL_GPL(hsmp_send_message); static int hsmp_test(u16 sock_ind, u32 value) { struct hsmp_message msg = { 0 }; - struct amd_northbridge *nb; - int ret = -ENODEV; - - nb = node_to_amd_nb(sock_ind); - if (!nb || !nb->root) - return ret; + int ret; /* * Test the hsmp port by performing TEST command. The test message @@ -261,7 +256,7 @@ static int hsmp_test(u16 sock_ind, u32 value) msg.args[0] = value; msg.sock_ind = sock_ind; - ret = __hsmp_send_message(nb->root, &msg); + ret = hsmp_send_message(&msg); if (ret) return ret; @@ -504,6 +499,15 @@ static int hsmp_pltdrv_probe(struct platform_device *pdev) for (i = 0; i < plat_dev.num_sockets; i++) { sema_init(&plat_dev.sock[i].hsmp_sem, 1); plat_dev.sock[i].sock_ind = i; + + /* Test the hsmp interface on each socket */ + ret = hsmp_test(i, 0xDEADBEEF); + if (ret) { + pr_err("HSMP test message failed on Fam:%x model:%x\n", + boot_cpu_data.x86, boot_cpu_data.x86_model); + pr_err("Is HSMP disabled in BIOS ?\n"); + return ret; + } } plat_dev.hsmp_device.name = HSMP_CDEV_NAME; @@ -544,7 +548,6 @@ static struct platform_device *amd_hsmp_platdev; static int __init hsmp_plt_init(void) { int ret = -ENODEV; - int i; if (boot_cpu_data.x86_vendor != X86_VENDOR_AMD || boot_cpu_data.x86 < 0x19) { pr_err("HSMP is not supported on Family:%x model:%x\n", @@ -560,17 +563,6 @@ static int __init hsmp_plt_init(void) if (plat_dev.num_sockets == 0) return ret; - /* Test the hsmp interface on each socket */ - for (i = 0; i < plat_dev.num_sockets; i++) { - ret = hsmp_test(i, 0xDEADBEEF); - if (ret) { - pr_err("HSMP test message failed on Fam:%x model:%x\n", - boot_cpu_data.x86, boot_cpu_data.x86_model); - pr_err("Is HSMP disabled in BIOS ?\n"); - return ret; - } - } - ret = platform_driver_register(&amd_hsmp_driver); if (ret) return ret; From patchwork Sat Jan 6 02:25:23 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Suma Hegde X-Patchwork-Id: 13512644 Received: from NAM11-CO1-obe.outbound.protection.outlook.com (mail-co1nam11on2051.outbound.protection.outlook.com [40.107.220.51]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B85E2139C for ; Sat, 6 Jan 2024 02:26:14 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="Q1OV4rXl" ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=KsgZBOd3PqIAFonvEe/EqooFHg9lHyD3Ya+4RqmboECjZ6fuvu5h8SgqSmGC4CKovhDk26ta4W3wPTd8qGD2MOPGIC9yAvNuZs9hsJhc22QhMkDS5ahJMperYIB5wU9mOhOr9sWNC8kiFzO0OHTUhzt2/DCArqu/NCOdPsINJdVhnAVRaLVhFnF4sjhFD2lmt+2g53ARNb0tmWqRGgI5/pBZOgp8nZODNs87GOC9brs63nV4Ev3eRQmeG2a3SOggGtBxAGgc13AjG/ZYdX2fNZK4fcuWBnObptyGTgwnyBrWlXkuU2L+fC5G7ciTC4fHXKGz/ru/uxnJUn047hB1Mw== 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=bXu7g0SjVM2PXOG8/tikO/DFLGIL69ExD/xoLVXFpzs=; b=kgySnVIomsX5wJvYf64RpK6S20jrknFiZ11P4kOhRLAv0MpPkIbaD7+Lmi5uRU349o3npzFNYdVLHkFG5D0khgdncZ3vqLaZwXtVaw4nLGk+BHkGIqjBR4Xrpl+BWsQYmHJCtHXEgvYngMu1tvRaNN2MU5Y2K35d089WR5aZcXZBLHxvmUJh9kVQJTKIukaZCY5l7TZYkr+26rakJrx2cpqNY9JDVhVcg+wY7q7YZpLpb38ZzwUwyb8rAFw2K7JnHltVbO6oXfoqDu0WiPOSYx9XzVu/2yIj+APHWyeLXHW5Ek0bS3BrysZtYahKprzVdM53Arq/86ck17B0GL2teg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=bXu7g0SjVM2PXOG8/tikO/DFLGIL69ExD/xoLVXFpzs=; b=Q1OV4rXlaII9S95MnACdPXPkAasq0mG1ZJRBmcrFYTtpcYfcDYEISzE0JNlPrt5FY72Ekc6GtBP7cabL5ZMsuGxY5cBHq900qY9TSCNSQrNGbnQ07LZ93o7/G7kZBUmiAr7LUUlskwlDudgUkPf0iMEMJHfMeLM+YQCK/9bnrqw= Received: from PH0PR07CA0008.namprd07.prod.outlook.com (2603:10b6:510:5::13) by DS7PR12MB6288.namprd12.prod.outlook.com (2603:10b6:8:93::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7135.25; Sat, 6 Jan 2024 02:26:11 +0000 Received: from SN1PEPF0002BA4D.namprd03.prod.outlook.com (2603:10b6:510:5:cafe::b) by PH0PR07CA0008.outlook.office365.com (2603:10b6:510:5::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.16 via Frontend Transport; Sat, 6 Jan 2024 02:26:11 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by SN1PEPF0002BA4D.mail.protection.outlook.com (10.167.242.70) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.7159.9 via Frontend Transport; Sat, 6 Jan 2024 02:26:10 +0000 Received: from amd.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.34; Fri, 5 Jan 2024 20:26:08 -0600 From: Suma Hegde To: CC: , , Suma Hegde , Naveen Krishna Chatradhi Subject: [PATCH v5 02/11] platform/x86/amd/hsmp: Cache pci_dev in struct hsmp_socket Date: Sat, 6 Jan 2024 02:25:23 +0000 Message-ID: <20240106022532.1746932-2-suma.hegde@amd.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20240106022532.1746932-1-suma.hegde@amd.com> References: <20240106022532.1746932-1-suma.hegde@amd.com> Precedence: bulk X-Mailing-List: platform-driver-x86@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-ClientProxiedBy: SATLEXMB04.amd.com (10.181.40.145) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SN1PEPF0002BA4D:EE_|DS7PR12MB6288:EE_ X-MS-Office365-Filtering-Correlation-Id: 86db6224-ef13-4176-5397-08dc0e5ed8f5 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 4blS5inopZRy9vdmJEtqWaeE2urXlYUkAextC7+1rLrzg9EFJzAWf4OlaeLjiESjPxhZIRHpxjyxH7HrFP+LaIeOVj9lXY/wiJHQQJ2XJRj2ldDTN1uMhid0D6xNOmDTibUg7AqAzRSDiA+wBpLSLZTo/4nWtSErAUjJGEjJ2qIGY92vkUhH799Lo313lYaLkGrFVp6UVI/2MzGKnkQVVTVmw6FA99r6fkrqlYkuDTfPGTXAwwhCnZ2xihqkLUErU8pBLXFT83QtpXbQaT3bfo3MswLG7yU167elR8xuwq7qKbEice+ZD72FswsirmVm9CU4lHxnjv2P8pk8NDnORb5E9GA6h/kkXxewrFaYaWbrfZ5dlULt9vRdbZg9OZDv57PcSRTs/hnvoEI/hST7OgcWW50O4bbsKUnt9+eOp9ZxILEfRFs1w4y9yoXrUXhsyuTQkXVbLngMvV7EPj2sHvcTTwazGftkDDWeUWyUKaX/JBy4trDNsf05pcdqLU7lsSrTmq0untimiSucBK8pUe7WqpPAQmVhIup5jwK2pT0BGzvMEzSmQpTA1Pd8mx3knPVqNBehkdYGBaU9EW2qix8ybu2skZUIC+2CwYIjdw8TgqqL8RJSmiPQHAyxCBKVyiTR7v/EMrDr1jG5vrYlnIEJ73S2h0IgNrYgdMQcS5YtSgcYZlgUD1Yxb5nzT7TM+rbZYeOfaa+buOYLQYMH4phfRrJtWyHmoqt+eOM7ACaeXddNioztjvPmZ/XpIv0Q3WylgqRyPa+rKMAf5MCOZg== X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230031)(4636009)(376002)(346002)(136003)(39860400002)(396003)(230922051799003)(1800799012)(186009)(64100799003)(82310400011)(451199024)(40470700004)(46966006)(36840700001)(40480700001)(40460700003)(36756003)(44832011)(70586007)(7696005)(70206006)(86362001)(478600001)(66574015)(2906002)(16526019)(6666004)(81166007)(336012)(356005)(83380400001)(2616005)(1076003)(47076005)(82740400003)(6916009)(426003)(4326008)(5660300002)(41300700001)(8676002)(54906003)(8936002)(316002)(36860700001)(26005)(36900700001);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Jan 2024 02:26:10.8813 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 86db6224-ef13-4176-5397-08dc0e5ed8f5 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: SN1PEPF0002BA4D.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR12MB6288 Cache pci_dev obj during probe as part of struct hsmp_socket and use in amd_hsmp_rdwr(). This change will make it easier to support both non-ACPI and ACPI devices. Also add a check for sock_index agsint number of sockets in the hsmp_send_message(). Signed-off-by: Suma Hegde Signed-off-by: Naveen Krishna Chatradhi Reviewed-by: Hans de Goede Reviewed-by: Ilpo Järvinen --- Changes since v4: None Changes since v3: None Changes since v2: 1. Add "Reviewed-by: Ilpo Järvinen " Changes since v1: 1. Remove !sock check in hsmp_send_message() 2. Add "Reviewed-by: Hans de Goede " drivers/platform/x86/amd/hsmp.c | 42 +++++++++++++++++++-------------- 1 file changed, 24 insertions(+), 18 deletions(-) diff --git a/drivers/platform/x86/amd/hsmp.c b/drivers/platform/x86/amd/hsmp.c index 3c17b488f4f8..1a2abe4460f9 100644 --- a/drivers/platform/x86/amd/hsmp.c +++ b/drivers/platform/x86/amd/hsmp.c @@ -58,6 +58,7 @@ struct hsmp_socket { void __iomem *metric_tbl_addr; struct semaphore hsmp_sem; char name[HSMP_ATTR_GRP_NAME_SIZE]; + struct pci_dev *root; u16 sock_ind; }; @@ -71,17 +72,20 @@ struct hsmp_plat_device { static struct hsmp_plat_device plat_dev; -static int amd_hsmp_rdwr(struct pci_dev *root, u32 address, +static int amd_hsmp_rdwr(struct hsmp_socket *sock, u32 address, u32 *value, bool write) { int ret; - ret = pci_write_config_dword(root, HSMP_INDEX_REG, address); + if (!sock->root) + return -ENODEV; + + ret = pci_write_config_dword(sock->root, HSMP_INDEX_REG, address); if (ret) return ret; - ret = (write ? pci_write_config_dword(root, HSMP_DATA_REG, *value) - : pci_read_config_dword(root, HSMP_DATA_REG, value)); + ret = (write ? pci_write_config_dword(sock->root, HSMP_DATA_REG, *value) + : pci_read_config_dword(sock->root, HSMP_DATA_REG, value)); return ret; } @@ -95,7 +99,7 @@ static int amd_hsmp_rdwr(struct pci_dev *root, u32 address, * Returns 0 for success and populates the requested number of arguments. * Returns a negative error code for failure. */ -static int __hsmp_send_message(struct pci_dev *root, struct hsmp_message *msg) +static int __hsmp_send_message(struct hsmp_socket *sock, struct hsmp_message *msg) { unsigned long timeout, short_sleep; u32 mbox_status; @@ -104,7 +108,7 @@ static int __hsmp_send_message(struct pci_dev *root, struct hsmp_message *msg) /* Clear the status register */ mbox_status = HSMP_STATUS_NOT_READY; - ret = amd_hsmp_rdwr(root, SMN_HSMP_MSG_RESP, &mbox_status, HSMP_WR); + ret = amd_hsmp_rdwr(sock, SMN_HSMP_MSG_RESP, &mbox_status, HSMP_WR); if (ret) { pr_err("Error %d clearing mailbox status register\n", ret); return ret; @@ -113,7 +117,7 @@ static int __hsmp_send_message(struct pci_dev *root, struct hsmp_message *msg) index = 0; /* Write any message arguments */ while (index < msg->num_args) { - ret = amd_hsmp_rdwr(root, SMN_HSMP_MSG_DATA + (index << 2), + ret = amd_hsmp_rdwr(sock, SMN_HSMP_MSG_DATA + (index << 2), &msg->args[index], HSMP_WR); if (ret) { pr_err("Error %d writing message argument %d\n", ret, index); @@ -123,7 +127,7 @@ static int __hsmp_send_message(struct pci_dev *root, struct hsmp_message *msg) } /* Write the message ID which starts the operation */ - ret = amd_hsmp_rdwr(root, SMN_HSMP_MSG_ID, &msg->msg_id, HSMP_WR); + ret = amd_hsmp_rdwr(sock, SMN_HSMP_MSG_ID, &msg->msg_id, HSMP_WR); if (ret) { pr_err("Error %d writing message ID %u\n", ret, msg->msg_id); return ret; @@ -140,7 +144,7 @@ static int __hsmp_send_message(struct pci_dev *root, struct hsmp_message *msg) timeout = jiffies + msecs_to_jiffies(HSMP_MSG_TIMEOUT); while (time_before(jiffies, timeout)) { - ret = amd_hsmp_rdwr(root, SMN_HSMP_MSG_RESP, &mbox_status, HSMP_RD); + ret = amd_hsmp_rdwr(sock, SMN_HSMP_MSG_RESP, &mbox_status, HSMP_RD); if (ret) { pr_err("Error %d reading mailbox status\n", ret); return ret; @@ -175,7 +179,7 @@ static int __hsmp_send_message(struct pci_dev *root, struct hsmp_message *msg) */ index = 0; while (index < msg->response_sz) { - ret = amd_hsmp_rdwr(root, SMN_HSMP_MSG_DATA + (index << 2), + ret = amd_hsmp_rdwr(sock, SMN_HSMP_MSG_DATA + (index << 2), &msg->args[index], HSMP_RD); if (ret) { pr_err("Error %d reading response %u for message ID:%u\n", @@ -208,21 +212,19 @@ static int validate_message(struct hsmp_message *msg) int hsmp_send_message(struct hsmp_message *msg) { - struct hsmp_socket *sock = &plat_dev.sock[msg->sock_ind]; - struct amd_northbridge *nb; + struct hsmp_socket *sock; int ret; if (!msg) return -EINVAL; - - nb = node_to_amd_nb(msg->sock_ind); - if (!nb || !nb->root) - return -ENODEV; - ret = validate_message(msg); if (ret) return ret; + if (!plat_dev.sock || msg->sock_ind >= plat_dev.num_sockets) + return -ENODEV; + sock = &plat_dev.sock[msg->sock_ind]; + /* * The time taken by smu operation to complete is between * 10us to 1ms. Sometime it may take more time. @@ -233,7 +235,7 @@ int hsmp_send_message(struct hsmp_message *msg) if (ret < 0) return ret; - ret = __hsmp_send_message(nb->root, msg); + ret = __hsmp_send_message(sock, msg); up(&sock->hsmp_sem); @@ -500,6 +502,10 @@ static int hsmp_pltdrv_probe(struct platform_device *pdev) sema_init(&plat_dev.sock[i].hsmp_sem, 1); plat_dev.sock[i].sock_ind = i; + if (!node_to_amd_nb(i)) + return -ENODEV; + plat_dev.sock[i].root = node_to_amd_nb(i)->root; + /* Test the hsmp interface on each socket */ ret = hsmp_test(i, 0xDEADBEEF); if (ret) { From patchwork Sat Jan 6 02:25:24 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Suma Hegde X-Patchwork-Id: 13512645 Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11on2046.outbound.protection.outlook.com [40.107.223.46]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 65E6015B1 for ; Sat, 6 Jan 2024 02:26:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="rbfJonn4" ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=NVV3wm8cF/ItpTOP5h/rMOD4Oe7+vdj2mymToG1Y2L6NKZr5wZbNXXi8scSnhHmtpgfbRYHV+/8GxOaILmiXo1w+bh1l9JCikspZpyAcM1JlytLRQTe7adI6vaF4K4p1uodU25cr0XFqGlObIyjePG5ofgG4eoIC+KadKNRWKuWIHrJ/8g9R03xf1dvS0jpD/fpOhLFmM5D5v0rxHUD6dKsHxqSP6V7Ml4z3Bl3BjGu0qjPQB8eGFNQ7fnPiEnIX6nxCvStX4gNyJDPrHx9z52xTp9bqV376y1lzVUvRfgf5dJs46dlFUnXa3iiqTYY2xUmbDhXfNMqZWP22Ck+eXw== 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=TV7M241vojAtARgXeV6Ykw78so8GviOTVDdgifFYKWE=; b=N0ALSM4Erq0vl21uI+DWOZlrELXa19JNmhfgc7FpMSQHGtX5QneSHL7Bs42wNZTjhXe4kYxx9V7t0CI2XT86mP/8v1NhrbZA8mL5MVmT/xb1UrGUPNYa/RwiMYdl96kx53uPpJ5SF5Ls5RipnUUwLC3nsg5ooD6m1mfrCfBJJc80KWlvkz4vHaA1FpFGCx4lIq1iOaBLWcPY7L4kcPtZHmyU8ejF8OxJU1gWslvFts4ZJ7G88WQztLOHeVVGuCK5lHIuieat7tIfITjg+xNXrfpP/bJDIK7FzfHk4vi8edHfwP+Ndz7gTwiAutUlnhjc+dPDDIvVg2RzitM7RcJh5g== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=TV7M241vojAtARgXeV6Ykw78so8GviOTVDdgifFYKWE=; b=rbfJonn4hdUGqOPE6ZLA7Fg7GD3UlEmDrjkvhIK7c7PCjvF9DLQn/UGQUBAbpehyY2z/Z8nfmZN/D3OcYojDoIH+VvNv5VwT4qeLVR/NUIEen93GH0QyOQz9xbaQzGvImiiiIrze4rovp9MrIJzSwLXCb4gGtcKDGn94Q09b/Nw= Received: from SA9PR13CA0108.namprd13.prod.outlook.com (2603:10b6:806:24::23) by SA1PR12MB7365.namprd12.prod.outlook.com (2603:10b6:806:2ba::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.18; Sat, 6 Jan 2024 02:26:12 +0000 Received: from SN1PEPF0002BA4C.namprd03.prod.outlook.com (2603:10b6:806:24:cafe::ac) by SA9PR13CA0108.outlook.office365.com (2603:10b6:806:24::23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7181.9 via Frontend Transport; Sat, 6 Jan 2024 02:26:12 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by SN1PEPF0002BA4C.mail.protection.outlook.com (10.167.242.69) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.7159.9 via Frontend Transport; Sat, 6 Jan 2024 02:26:12 +0000 Received: from amd.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.34; Fri, 5 Jan 2024 20:26:10 -0600 From: Suma Hegde To: CC: , , Suma Hegde , Naveen Krishna Chatradhi Subject: [PATCH v5 03/11] platform/x86/amd/hsmp: Create static func to handle platdev Date: Sat, 6 Jan 2024 02:25:24 +0000 Message-ID: <20240106022532.1746932-3-suma.hegde@amd.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20240106022532.1746932-1-suma.hegde@amd.com> References: <20240106022532.1746932-1-suma.hegde@amd.com> Precedence: bulk X-Mailing-List: platform-driver-x86@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-ClientProxiedBy: SATLEXMB04.amd.com (10.181.40.145) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SN1PEPF0002BA4C:EE_|SA1PR12MB7365:EE_ X-MS-Office365-Filtering-Correlation-Id: 5ea2a81d-7e53-48cf-a888-08dc0e5eda0c X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: b2hkK0i4dblQCMSjoE37KgMwjmCJd1hY+LUivteY3raDxkq43v3gODTv53SX5Z8++Dho62bcdQr0qS2vPyBZuToJ1hB4FrS2pjMhlRme7ab58c7pL644lFb4vWcENLSdhlPe9FRHbumxHPpx0M9EDmkcjG+2FGkgSpxzPM5SRJpku8YkqowMdPfs9CUK6OBEQoirGL4+LeYxfm6iBm7V/nICSvmEaQDFHsKnCvN6nBgPtOUnaCI/Yp5PrqdHG33+qR1VZKu9qCWGD0L/ntEb/dOInEJizQiG//QoDEQV+XHZEAOhr4l1Zf4/uUCug1Y46J94lXSWrJiT4QAHqpaAFsDDyd2TVqyQQQFs7VyzxCN1h/Wrf7Zy972lvh+W8/jeLOOvTY7A0YkfVjO/6imj9Pp4I3ai7rp03iG4HiYztTLz8+ctiARFSAc6gY/Ps9JqVW9CGlDUdd7IsPFC9k7RoJXrm6Ja01SyHQpjaiB8EBQzGt6AdS4LFKw1NFMVU3BBZCc7HOyFrf3HkzoT2N4Fls0c0nDAmIQhwpqV8nzO4C7xuKHFXbzCKevZxQn+9AkX8OPYs8DkNtcr7myi8MdsXFAuMQw2OXB9Kfn6gZxwSxeVcng+PjvLT3FyhNV8dJSMfAccB/brLCsyqDppXG2kxznPhMO4UIV1ZAxM1hVF60dxIQbRNNfVF6XHSZTls9YN4oE1aHqRjdJlVG4FQvd455ZwVFvGQkgdjhWRjvLelu5qBqVKYnVuVpJapFQV+MZ1bvK0YQeGyZ0KTWZBG3infw== X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230031)(4636009)(376002)(136003)(396003)(39860400002)(346002)(230922051799003)(451199024)(1800799012)(82310400011)(64100799003)(186009)(36840700001)(46966006)(40470700004)(336012)(426003)(1076003)(2616005)(26005)(7696005)(478600001)(6666004)(47076005)(83380400001)(16526019)(5660300002)(2906002)(41300700001)(70206006)(70586007)(54906003)(316002)(6916009)(8936002)(44832011)(8676002)(4326008)(82740400003)(36860700001)(36756003)(86362001)(356005)(81166007)(40480700001)(40460700003)(36900700001);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Jan 2024 02:26:12.6651 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 5ea2a81d-7e53-48cf-a888-08dc0e5eda0c X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: SN1PEPF0002BA4C.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR12MB7365 Create a static function and call platform device alloc and add device, which will simplify handling acpi and plat device probing. Signed-off-by: Suma Hegde Signed-off-by: Naveen Krishna Chatradhi Reviewed-by: Hans de Goede Reviewed-by: Ilpo Järvinen --- Changes since v4: None Changes since v3: None Changes since v2: 1. Add "Reviewed-by: Ilpo Järvinen " Changes since v1: 1. Replace -1 with PLATFORM_DEVID_NONE in platform_device_alloc() 2. Add "Reviewed-by: Hans de Goede " drivers/platform/x86/amd/hsmp.c | 33 ++++++++++++++++++--------------- 1 file changed, 18 insertions(+), 15 deletions(-) diff --git a/drivers/platform/x86/amd/hsmp.c b/drivers/platform/x86/amd/hsmp.c index 1a2abe4460f9..e3354683b138 100644 --- a/drivers/platform/x86/amd/hsmp.c +++ b/drivers/platform/x86/amd/hsmp.c @@ -551,6 +551,21 @@ static struct platform_driver amd_hsmp_driver = { static struct platform_device *amd_hsmp_platdev; +static int hsmp_plat_dev_register(void) +{ + int ret; + + amd_hsmp_platdev = platform_device_alloc(DRIVER_NAME, PLATFORM_DEVID_NONE); + if (!amd_hsmp_platdev) + return -ENOMEM; + + ret = platform_device_add(amd_hsmp_platdev); + if (ret) + platform_device_put(amd_hsmp_platdev); + + return ret; +} + static int __init hsmp_plt_init(void) { int ret = -ENODEV; @@ -573,22 +588,10 @@ static int __init hsmp_plt_init(void) if (ret) return ret; - amd_hsmp_platdev = platform_device_alloc(DRIVER_NAME, PLATFORM_DEVID_NONE); - if (!amd_hsmp_platdev) { - ret = -ENOMEM; - goto drv_unregister; - } - - ret = platform_device_add(amd_hsmp_platdev); - if (ret) { - platform_device_put(amd_hsmp_platdev); - goto drv_unregister; - } - - return 0; + ret = hsmp_plat_dev_register(); + if (ret) + platform_driver_unregister(&amd_hsmp_driver); -drv_unregister: - platform_driver_unregister(&amd_hsmp_driver); return ret; } From patchwork Sat Jan 6 02:25:25 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Suma Hegde X-Patchwork-Id: 13512646 Received: from NAM10-BN7-obe.outbound.protection.outlook.com (mail-bn7nam10on2056.outbound.protection.outlook.com [40.107.92.56]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 93E1515A0 for ; Sat, 6 Jan 2024 02:26:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="KBBze7hw" ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=SwlSFuwqttRrDGL1ncS47NE1w/qyCEsuyqc7J6vrfqKvCPnAmt/9ZfcHoqmsvwWVuTGYNFlyBvYucVZA/Kbeng43RlQpEc4blVE/fJL6ngA63m0u7NkyD5kNRA3uJoJkW2RlHzkR377pbWj9iqKgIeml5m5H0dbinU9oM9VCgqAiImjERhCfQGLDuw7I3VGGLHMdWjbZzm4NbWCKp/XPPEZWsNNxOQ+AVRa445CnpxEeshh00mfeGiZC0Yfi8/Wq5sz3yGGVw5hu0iOfNbGadWZV5CXKnOhQt2qxQK17+srh3YzAa618zuDZ1gPPFD7Niobz6Fv7eg7yIHJN5+OJQw== 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=YhRF3hi4QRYRtNwh/bnYWtIZ3X7Ahgz3EXMV55BlGDs=; b=cfFga7iTQi6jo8RDmHTej9NAdwlkquIOqwCFfUjcrXmljwGuVNoXNaNhQdwA4/uIcvysD+xffi+mIlKxbwpWBfqUCGg8cdg4pThs9JoK+X6vBAE9IFsxZsquuZ2hiKMlFA14xL52FWKYn9TV+iZ38y8Z4V1vDGtZhswKLod/FeTzKtC8RIJppjDAd0FOXpaUarK+2CrG2sNPEYg+koIBLKzzNrevPl34WXJKJmfvHk9hFnOYkIcbiMWkGp797iGbWDmhZqNVTv9YU0/snL4ncdTvaWGJSIvjM5y07GYJwLw4knvqYmr+duO0fqKDY3D/dHW3l0ucC4VqCgW9eOowAQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=YhRF3hi4QRYRtNwh/bnYWtIZ3X7Ahgz3EXMV55BlGDs=; b=KBBze7hwbJ+gs7tuN4x6gL8dqGky8dy0HyEsqSfRlxF63hmrY5Ryk60S0eGbPIH6FuXHYA3TpyhgdyWWWRAqu0o6WU0siSkTzvAsXRBM3XpG8I3NysP9YnBv1UtZbR5+MKlnAk2cvzCr4mgvfNBOOwlO3IT/ZkLy6j/rBhnmJJY= Received: from SA0PR11CA0127.namprd11.prod.outlook.com (2603:10b6:806:131::12) by BL3PR12MB6593.namprd12.prod.outlook.com (2603:10b6:208:38c::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.17; Sat, 6 Jan 2024 02:26:14 +0000 Received: from SN1PEPF0002BA51.namprd03.prod.outlook.com (2603:10b6:806:131:cafe::c6) by SA0PR11CA0127.outlook.office365.com (2603:10b6:806:131::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.16 via Frontend Transport; Sat, 6 Jan 2024 02:26:14 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by SN1PEPF0002BA51.mail.protection.outlook.com (10.167.242.74) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.7159.9 via Frontend Transport; Sat, 6 Jan 2024 02:26:14 +0000 Received: from amd.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.34; Fri, 5 Jan 2024 20:26:12 -0600 From: Suma Hegde To: CC: , , Suma Hegde , Naveen Krishna Chatradhi Subject: [PATCH v5 04/11] platform/x86/amd/hsmp: Define a struct to hold mailbox regs Date: Sat, 6 Jan 2024 02:25:25 +0000 Message-ID: <20240106022532.1746932-4-suma.hegde@amd.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20240106022532.1746932-1-suma.hegde@amd.com> References: <20240106022532.1746932-1-suma.hegde@amd.com> Precedence: bulk X-Mailing-List: platform-driver-x86@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-ClientProxiedBy: SATLEXMB04.amd.com (10.181.40.145) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SN1PEPF0002BA51:EE_|BL3PR12MB6593:EE_ X-MS-Office365-Filtering-Correlation-Id: 3a0398ed-453f-45ed-8f4b-08dc0e5edb27 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: /s5ab1yZRaPFGElAla0JTBE6GuBEMbY3IpgeoqbQatDoqyCK0AevlfEMwPOjEKaEHhnOIllT9x9nq3WrIghJ1P7So/Jv9tNAMlY5Fn9mHSuoELR49gf7cIAR/OS2bRPJrxexyQ52MkkO+qWWRQEZN+P5IsR1vgMnSpWm9Vgfjk8jRFvwmJpiEurQvCYegsam7Vg/0JRr+KXNiwF5xAOKvcKGZA0NurscLau2okaEUVOzB8NN1V/GMigvyXXlM9NSmGMeoFIm8FVoJgqV/NZvwiQKfdjsOvX0Soqzq1dj0x1NpDH0CNOtk9Ecj5njF+rCosQ4jGZRyMFkHAk6LsVQKqbEc5Yu24HEOqcoVfU/sm9Yx3y0Zqb41EtRWumjsSdbv8n7QlgPl1e+8WKI1jMekfxxA9co3oB3H6yhIlfQ/hehkz3qYi98iIiC+M03O/gjbO5EgYwV06ukSlIoWlY8Eo2IFuFMTSreDEyskNgkR+5oSY7wyrk3Gn2BKKaw9nFNAxloKU00byuL4bBM4W+WhNAeDkbQ211ivPaBQeM/QFh3y0F76RgbgR3839tmyKMGgwlAE2sjIyRtvA7soE0eLL3zmhg3fKouu88oA6UbkzNQDYuyD2qO3j7bQEn45L5aTqi267Y01GElpw/6Mmt3zUKe841ig+/gp2/6L8O2PzFxd8MR4292nzKFtSGdWIE2UF8+O7ApW7Sa/gG3lEEY/LZX3pwc+dTwy0bfn3IjfVbC16F9+fiz30hgWHbDw2gd84rN0Ulc/AEf5ZBXCt3nAg== X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230031)(4636009)(39860400002)(136003)(396003)(376002)(346002)(230922051799003)(64100799003)(186009)(82310400011)(1800799012)(451199024)(40470700004)(36840700001)(46966006)(356005)(82740400003)(6916009)(70206006)(54906003)(36860700001)(81166007)(47076005)(316002)(86362001)(15650500001)(6666004)(2616005)(5660300002)(26005)(1076003)(7696005)(336012)(16526019)(426003)(70586007)(478600001)(2906002)(4326008)(36756003)(83380400001)(8676002)(44832011)(8936002)(40480700001)(40460700003)(41300700001)(36900700001);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Jan 2024 02:26:14.5816 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 3a0398ed-453f-45ed-8f4b-08dc0e5edb27 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: SN1PEPF0002BA51.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL3PR12MB6593 Define struct hsmp_mbaddr_info with register offsets and populate them during probe, which avoids the usage of macros in core functions. During ACPI probe, the same fields can be populated from ACPI table. Also move plat dev init to a static function. Signed-off-by: Suma Hegde Signed-off-by: Naveen Krishna Chatradhi Reviewed-by: Ilpo Järvinen --- Changes since v4: None Changes since v3: Rename init_socket_objects() to init_platform_device() Changes since v2: 1. Change initialize_platdev() name to init_socket_objects() 2. Use local variable sock to hold &plat_dev.sock[i] in init_socket_objects() 3. Add "Reviewed-by: Ilpo Järvinen " Changes since v1: Move hsmp_test() to initialize_platdev() drivers/platform/x86/amd/hsmp.c | 74 +++++++++++++++++++++++---------- 1 file changed, 52 insertions(+), 22 deletions(-) diff --git a/drivers/platform/x86/amd/hsmp.c b/drivers/platform/x86/amd/hsmp.c index e3354683b138..287eaa9b0dda 100644 --- a/drivers/platform/x86/amd/hsmp.c +++ b/drivers/platform/x86/amd/hsmp.c @@ -40,9 +40,10 @@ * register into the SMN_INDEX register, and reads/writes the SMN_DATA reg. * Below are required SMN address for HSMP Mailbox register offsets in SMU address space */ -#define SMN_HSMP_MSG_ID 0x3B10534 -#define SMN_HSMP_MSG_RESP 0x3B10980 -#define SMN_HSMP_MSG_DATA 0x3B109E0 +#define SMN_HSMP_BASE 0x3B00000 +#define SMN_HSMP_MSG_ID 0x0010534 +#define SMN_HSMP_MSG_RESP 0x0010980 +#define SMN_HSMP_MSG_DATA 0x00109E0 #define HSMP_INDEX_REG 0xc4 #define HSMP_DATA_REG 0xc8 @@ -53,8 +54,17 @@ #define HSMP_ATTR_GRP_NAME_SIZE 10 +struct hsmp_mbaddr_info { + u32 base_addr; + u32 msg_id_off; + u32 msg_resp_off; + u32 msg_arg_off; + u32 size; +}; + struct hsmp_socket { struct bin_attribute hsmp_attr; + struct hsmp_mbaddr_info mbinfo; void __iomem *metric_tbl_addr; struct semaphore hsmp_sem; char name[HSMP_ATTR_GRP_NAME_SIZE]; @@ -72,7 +82,7 @@ struct hsmp_plat_device { static struct hsmp_plat_device plat_dev; -static int amd_hsmp_rdwr(struct hsmp_socket *sock, u32 address, +static int amd_hsmp_rdwr(struct hsmp_socket *sock, u32 offset, u32 *value, bool write) { int ret; @@ -80,7 +90,8 @@ static int amd_hsmp_rdwr(struct hsmp_socket *sock, u32 address, if (!sock->root) return -ENODEV; - ret = pci_write_config_dword(sock->root, HSMP_INDEX_REG, address); + ret = pci_write_config_dword(sock->root, HSMP_INDEX_REG, + sock->mbinfo.base_addr + offset); if (ret) return ret; @@ -101,14 +112,17 @@ static int amd_hsmp_rdwr(struct hsmp_socket *sock, u32 address, */ static int __hsmp_send_message(struct hsmp_socket *sock, struct hsmp_message *msg) { + struct hsmp_mbaddr_info *mbinfo; unsigned long timeout, short_sleep; u32 mbox_status; u32 index; int ret; + mbinfo = &sock->mbinfo; + /* Clear the status register */ mbox_status = HSMP_STATUS_NOT_READY; - ret = amd_hsmp_rdwr(sock, SMN_HSMP_MSG_RESP, &mbox_status, HSMP_WR); + ret = amd_hsmp_rdwr(sock, mbinfo->msg_resp_off, &mbox_status, HSMP_WR); if (ret) { pr_err("Error %d clearing mailbox status register\n", ret); return ret; @@ -117,7 +131,7 @@ static int __hsmp_send_message(struct hsmp_socket *sock, struct hsmp_message *ms index = 0; /* Write any message arguments */ while (index < msg->num_args) { - ret = amd_hsmp_rdwr(sock, SMN_HSMP_MSG_DATA + (index << 2), + ret = amd_hsmp_rdwr(sock, mbinfo->msg_arg_off + (index << 2), &msg->args[index], HSMP_WR); if (ret) { pr_err("Error %d writing message argument %d\n", ret, index); @@ -127,7 +141,7 @@ static int __hsmp_send_message(struct hsmp_socket *sock, struct hsmp_message *ms } /* Write the message ID which starts the operation */ - ret = amd_hsmp_rdwr(sock, SMN_HSMP_MSG_ID, &msg->msg_id, HSMP_WR); + ret = amd_hsmp_rdwr(sock, mbinfo->msg_id_off, &msg->msg_id, HSMP_WR); if (ret) { pr_err("Error %d writing message ID %u\n", ret, msg->msg_id); return ret; @@ -144,7 +158,7 @@ static int __hsmp_send_message(struct hsmp_socket *sock, struct hsmp_message *ms timeout = jiffies + msecs_to_jiffies(HSMP_MSG_TIMEOUT); while (time_before(jiffies, timeout)) { - ret = amd_hsmp_rdwr(sock, SMN_HSMP_MSG_RESP, &mbox_status, HSMP_RD); + ret = amd_hsmp_rdwr(sock, mbinfo->msg_resp_off, &mbox_status, HSMP_RD); if (ret) { pr_err("Error %d reading mailbox status\n", ret); return ret; @@ -179,7 +193,7 @@ static int __hsmp_send_message(struct hsmp_socket *sock, struct hsmp_message *ms */ index = 0; while (index < msg->response_sz) { - ret = amd_hsmp_rdwr(sock, SMN_HSMP_MSG_DATA + (index << 2), + ret = amd_hsmp_rdwr(sock, mbinfo->msg_arg_off + (index << 2), &msg->args[index], HSMP_RD); if (ret) { pr_err("Error %d reading response %u for message ID:%u\n", @@ -487,24 +501,22 @@ static int hsmp_cache_proto_ver(void) return ret; } -static int hsmp_pltdrv_probe(struct platform_device *pdev) +static int init_platform_device(void) { + struct hsmp_socket *sock; int ret, i; - plat_dev.sock = devm_kzalloc(&pdev->dev, - (plat_dev.num_sockets * sizeof(struct hsmp_socket)), - GFP_KERNEL); - if (!plat_dev.sock) - return -ENOMEM; - plat_dev.dev = &pdev->dev; - for (i = 0; i < plat_dev.num_sockets; i++) { - sema_init(&plat_dev.sock[i].hsmp_sem, 1); - plat_dev.sock[i].sock_ind = i; - if (!node_to_amd_nb(i)) return -ENODEV; - plat_dev.sock[i].root = node_to_amd_nb(i)->root; + sock = &plat_dev.sock[i]; + sock->root = node_to_amd_nb(i)->root; + sock->sock_ind = i; + sock->mbinfo.base_addr = SMN_HSMP_BASE; + sock->mbinfo.msg_id_off = SMN_HSMP_MSG_ID; + sock->mbinfo.msg_resp_off = SMN_HSMP_MSG_RESP; + sock->mbinfo.msg_arg_off = SMN_HSMP_MSG_DATA; + sema_init(&sock->hsmp_sem, 1); /* Test the hsmp interface on each socket */ ret = hsmp_test(i, 0xDEADBEEF); @@ -516,6 +528,24 @@ static int hsmp_pltdrv_probe(struct platform_device *pdev) } } + return 0; +} + +static int hsmp_pltdrv_probe(struct platform_device *pdev) +{ + int ret; + + plat_dev.sock = devm_kzalloc(&pdev->dev, + (plat_dev.num_sockets * sizeof(struct hsmp_socket)), + GFP_KERNEL); + if (!plat_dev.sock) + return -ENOMEM; + plat_dev.dev = &pdev->dev; + + ret = init_platform_device(); + if (ret) + return ret; + plat_dev.hsmp_device.name = HSMP_CDEV_NAME; plat_dev.hsmp_device.minor = MISC_DYNAMIC_MINOR; plat_dev.hsmp_device.fops = &hsmp_fops; From patchwork Sat Jan 6 02:25:26 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Suma Hegde X-Patchwork-Id: 13512647 Received: from NAM10-MW2-obe.outbound.protection.outlook.com (mail-mw2nam10on2047.outbound.protection.outlook.com [40.107.94.47]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0482915A5 for ; Sat, 6 Jan 2024 02:26:20 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="WppfxJ8A" ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=GFT8M5J1/zqWfT9MGER/7QGCLgs33KRQyUHSg/Vrqqpdlv/7g1YyyAMvYJ3DH2dt00v54kDCGvzeLITCUr1G8wbFxJdIR6usgS3zUcKbJR4axXLVypr4I+tpcjfpcy0nRx9ZBfBeuDN97OhFO0YbY6P/QGTV6Sm2BnY2DFbHjnmnf/NEIGJnvtJaK6DI+2KUxu0+jG89Unff4aCKXboxP+mGkgdosvVNDXVm0t+EqdrnWcuLcM/yFLamFogBoYIGd3t2gYJRrm+VbGcCJ/L8/e1eJCrforo82UGylwsNGvmWWlNCQwUrzvhV2u49C1cw4STBKMfA+KH38fDwEo1i7Q== 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=kYs+h06LtWuYABVyyi4qprkUKy3aCkAEne9DV7PQ5Wc=; b=f0oISVRm0DfyejmTcIHqGyDiu7fnrIBj8qPX5hDQijQdqzo8fjHuo/LI8LXeh5gpOriYwZ06BZVIRWcdlbSehivL3SQK3k44iTXNl7eCwvp8XzAIIndKM2sjAEBhnUeimBCmweUIxt4+lmOFJvQDLcCXAY0moG8r4zGvJcV7VxjMJXtAmLVONaGuq3jOeRsBebF6Ygpq59BolQX+PLcwx+pSccKtPignY+9W6fWWNLeP3lw8/OvHdkHYPDFOMXQhWF36bIDp0wkboc2obLXMhglshaCmEAVtUJh4jQdVB08rzB0zzwFGQjqvcQer7DiB3rmCWWSAo4jmLquk4alrqg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=kYs+h06LtWuYABVyyi4qprkUKy3aCkAEne9DV7PQ5Wc=; b=WppfxJ8ABVK3+o63+BoA2lj629CVRBHbNUogAJ3GRiaro1e79EYZNaLlxbIGDmDsVC1cROMuOqWGKXPh9JZROI+ZKgOncasWOTUZzSlGPP3o7JtZipOwGE5nGk+vv8VpbZzNrfWwTY+iKkmlI9OBc3hKX5/q15Z/JdA/J7T0g2c= Received: from PH8PR07CA0034.namprd07.prod.outlook.com (2603:10b6:510:2cf::15) by DM4PR12MB5214.namprd12.prod.outlook.com (2603:10b6:5:395::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.17; Sat, 6 Jan 2024 02:26:17 +0000 Received: from SN1PEPF0002BA52.namprd03.prod.outlook.com (2603:10b6:510:2cf:cafe::96) by PH8PR07CA0034.outlook.office365.com (2603:10b6:510:2cf::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.18 via Frontend Transport; Sat, 6 Jan 2024 02:26:16 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by SN1PEPF0002BA52.mail.protection.outlook.com (10.167.242.75) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.7159.9 via Frontend Transport; Sat, 6 Jan 2024 02:26:16 +0000 Received: from amd.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.34; Fri, 5 Jan 2024 20:26:14 -0600 From: Suma Hegde To: CC: , , Suma Hegde , Naveen Krishna Chatradhi Subject: [PATCH v5 05/11] platform/x86/amd/hsmp: Move dev from platdev to hsmp_socket Date: Sat, 6 Jan 2024 02:25:26 +0000 Message-ID: <20240106022532.1746932-5-suma.hegde@amd.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20240106022532.1746932-1-suma.hegde@amd.com> References: <20240106022532.1746932-1-suma.hegde@amd.com> Precedence: bulk X-Mailing-List: platform-driver-x86@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-ClientProxiedBy: SATLEXMB04.amd.com (10.181.40.145) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SN1PEPF0002BA52:EE_|DM4PR12MB5214:EE_ X-MS-Office365-Filtering-Correlation-Id: abc6a0a3-df79-4dd2-5c5d-08dc0e5edc77 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: DbE/RUwUucUQ8UoRa7+atO469ygEQ+opQtLuWfsX1LO+aGsq4vg3DU8O6azw6CoMEnqfQuB7c92wJfbx81zt1sSJ14XM5JJ7bjsWDJSlMrmqvWuYQNgWqtQ0DQD5o6j+kWsn5TsWlRSKBiwJ9/DSIWya+LGMHGvdPQfkL26+/jV/Z2aTBRUlRUyFKYRrVgNHN7jk39XQ4ry7zFtzMksxOnNSbAid3tYzogND76T5vmvGVMqrLsiVEQSQLGoX66M00QXjFpSVvpGRZFxqOM6zEDMpW/Hse8InKLSGODjLSEnLe/J/8w/BDMkdlh1jpbp6TEuU/ivy2ZFGyQfa4hlKsLd7S9W+lB3AiHGXtmUowKzR/rnA0lFYL9vCKgzaXI9ayrBN0iK7vRZmQClah9JYx+LNSmyacFQzNwCBM9kZ5AOU1doPIef/pmWdwCgGZXizJzwiuJhXvRMOzVgMTSohcbiRMePVfjyvvtjx4df2cMdhaVmvFpbEigXD7d31IvvPmo37xrP5O4bKQ3pIcW44+p7/PNeSWCsEFNfwFxsxNXssH5IoDI7H8maf3XVj8cMMjSzfz4fH2nptoPIwhr0XK+icf8oUHDMJ3T24iDZRgeLgAlbX/QMDQeDu4Td5AkY+aY+fGoAmAixx/avpaua4AMlRJEVFYx0X7mjT674RJaqE3/bCGWA71eQCUlbXPFg3H6nlDsiexi4n1kZydRY2cbwLGStX6oFMLO/cSU+Y8Hsm0qlfry4GFkYltEM7wOUzQBQzRm8xPJXx7kymAd0QoA== X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230031)(4636009)(39860400002)(396003)(376002)(136003)(346002)(230922051799003)(82310400011)(1800799012)(64100799003)(186009)(451199024)(46966006)(40470700004)(36840700001)(26005)(426003)(336012)(1076003)(2616005)(478600001)(6666004)(7696005)(47076005)(83380400001)(16526019)(66574015)(2906002)(5660300002)(41300700001)(54906003)(6916009)(316002)(8936002)(8676002)(4326008)(44832011)(70586007)(70206006)(82740400003)(36756003)(36860700001)(86362001)(81166007)(356005)(40460700003)(40480700001)(36900700001);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Jan 2024 02:26:16.7198 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: abc6a0a3-df79-4dd2-5c5d-08dc0e5edc77 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: SN1PEPF0002BA52.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR12MB5214 On an ACPI enabled platforms the probe is called for each socket and the struct dev is different for each socket. This change will help in handling both ACPI and non-ACPI platforms. Also change pr_err() to dev_err() API. Signed-off-by: Suma Hegde Signed-off-by: Naveen Krishna Chatradhi Reviewed-by: Hans de Goede Reviewed-by: Ilpo Järvinen --- Changes since v4: None Changes since v3: None Changes since v2: 1. Edit commit message 2. Add "Reviewed-by: Ilpo Järvinen " Changes since v1: Add "Reviewed-by: Hans de Goede " drivers/platform/x86/amd/hsmp.c | 42 +++++++++++++++++---------------- 1 file changed, 22 insertions(+), 20 deletions(-) diff --git a/drivers/platform/x86/amd/hsmp.c b/drivers/platform/x86/amd/hsmp.c index 287eaa9b0dda..0bf94e2bd022 100644 --- a/drivers/platform/x86/amd/hsmp.c +++ b/drivers/platform/x86/amd/hsmp.c @@ -69,13 +69,13 @@ struct hsmp_socket { struct semaphore hsmp_sem; char name[HSMP_ATTR_GRP_NAME_SIZE]; struct pci_dev *root; + struct device *dev; u16 sock_ind; }; struct hsmp_plat_device { struct miscdevice hsmp_device; struct hsmp_socket *sock; - struct device *dev; u32 proto_ver; u16 num_sockets; }; @@ -278,8 +278,9 @@ static int hsmp_test(u16 sock_ind, u32 value) /* Check the response value */ if (msg.args[0] != (value + 1)) { - pr_err("Socket %d test message failed, Expected 0x%08X, received 0x%08X\n", - sock_ind, (value + 1), msg.args[0]); + dev_err(plat_dev.sock[sock_ind].dev, + "Socket %d test message failed, Expected 0x%08X, received 0x%08X\n", + sock_ind, (value + 1), msg.args[0]); return -EBADE; } @@ -358,12 +359,12 @@ static ssize_t hsmp_metric_tbl_read(struct file *filp, struct kobject *kobj, /* Do not support lseek(), reads entire metric table */ if (count < bin_attr->size) { - dev_err(plat_dev.dev, "Wrong buffer size\n"); + dev_err(sock->dev, "Wrong buffer size\n"); return -EINVAL; } if (!sock) { - dev_err(plat_dev.dev, "Failed to read attribute private data\n"); + dev_err(sock->dev, "Failed to read attribute private data\n"); return -EINVAL; } @@ -399,13 +400,13 @@ static int hsmp_get_tbl_dram_base(u16 sock_ind) */ dram_addr = msg.args[0] | ((u64)(msg.args[1]) << 32); if (!dram_addr) { - dev_err(plat_dev.dev, "Invalid DRAM address for metric table\n"); + dev_err(sock->dev, "Invalid DRAM address for metric table\n"); return -ENOMEM; } - sock->metric_tbl_addr = devm_ioremap(plat_dev.dev, dram_addr, + sock->metric_tbl_addr = devm_ioremap(sock->dev, dram_addr, sizeof(struct hsmp_metric_table)); if (!sock->metric_tbl_addr) { - dev_err(plat_dev.dev, "Failed to ioremap metric table addr\n"); + dev_err(sock->dev, "Failed to ioremap metric table addr\n"); return -ENOMEM; } return 0; @@ -453,14 +454,15 @@ static int hsmp_create_sysfs_interface(void) if (WARN_ON(plat_dev.num_sockets > U8_MAX)) return -ERANGE; - hsmp_attr_grps = devm_kzalloc(plat_dev.dev, sizeof(struct attribute_group *) * + hsmp_attr_grps = devm_kzalloc(plat_dev.sock[0].dev, sizeof(struct attribute_group *) * (plat_dev.num_sockets + 1), GFP_KERNEL); if (!hsmp_attr_grps) return -ENOMEM; /* Create a sysfs directory for each socket */ for (i = 0; i < plat_dev.num_sockets; i++) { - attr_grp = devm_kzalloc(plat_dev.dev, sizeof(struct attribute_group), GFP_KERNEL); + attr_grp = devm_kzalloc(plat_dev.sock[i].dev, sizeof(struct attribute_group), + GFP_KERNEL); if (!attr_grp) return -ENOMEM; @@ -468,7 +470,7 @@ static int hsmp_create_sysfs_interface(void) attr_grp->name = plat_dev.sock[i].name; /* Null terminated list of attributes */ - hsmp_bin_attrs = devm_kzalloc(plat_dev.dev, sizeof(struct bin_attribute *) * + hsmp_bin_attrs = devm_kzalloc(plat_dev.sock[i].dev, sizeof(struct bin_attribute *) * (NUM_HSMP_ATTRS + 1), GFP_KERNEL); if (!hsmp_bin_attrs) return -ENOMEM; @@ -482,7 +484,7 @@ static int hsmp_create_sysfs_interface(void) if (ret) return ret; } - return devm_device_add_groups(plat_dev.dev, hsmp_attr_grps); + return devm_device_add_groups(plat_dev.sock[0].dev, hsmp_attr_grps); } static int hsmp_cache_proto_ver(void) @@ -501,7 +503,7 @@ static int hsmp_cache_proto_ver(void) return ret; } -static int init_platform_device(void) +static int init_platform_device(struct device *dev) { struct hsmp_socket *sock; int ret, i; @@ -512,6 +514,7 @@ static int init_platform_device(void) sock = &plat_dev.sock[i]; sock->root = node_to_amd_nb(i)->root; sock->sock_ind = i; + sock->dev = dev; sock->mbinfo.base_addr = SMN_HSMP_BASE; sock->mbinfo.msg_id_off = SMN_HSMP_MSG_ID; sock->mbinfo.msg_resp_off = SMN_HSMP_MSG_RESP; @@ -521,9 +524,9 @@ static int init_platform_device(void) /* Test the hsmp interface on each socket */ ret = hsmp_test(i, 0xDEADBEEF); if (ret) { - pr_err("HSMP test message failed on Fam:%x model:%x\n", - boot_cpu_data.x86, boot_cpu_data.x86_model); - pr_err("Is HSMP disabled in BIOS ?\n"); + dev_err(dev, "HSMP test message failed on Fam:%x model:%x\n", + boot_cpu_data.x86, boot_cpu_data.x86_model); + dev_err(dev, "Is HSMP disabled in BIOS ?\n"); return ret; } } @@ -540,9 +543,8 @@ static int hsmp_pltdrv_probe(struct platform_device *pdev) GFP_KERNEL); if (!plat_dev.sock) return -ENOMEM; - plat_dev.dev = &pdev->dev; - ret = init_platform_device(); + ret = init_platform_device(&pdev->dev); if (ret) return ret; @@ -555,13 +557,13 @@ static int hsmp_pltdrv_probe(struct platform_device *pdev) ret = hsmp_cache_proto_ver(); if (ret) { - dev_err(plat_dev.dev, "Failed to read HSMP protocol version\n"); + dev_err(&pdev->dev, "Failed to read HSMP protocol version\n"); return ret; } ret = hsmp_create_sysfs_interface(); if (ret) - dev_err(plat_dev.dev, "Failed to create HSMP sysfs interface\n"); + dev_err(&pdev->dev, "Failed to create HSMP sysfs interface\n"); return misc_register(&plat_dev.hsmp_device); } From patchwork Sat Jan 6 02:25:27 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Suma Hegde X-Patchwork-Id: 13512648 Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11on2054.outbound.protection.outlook.com [40.107.223.54]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0864715AB for ; Sat, 6 Jan 2024 02:26:21 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="t3J2K5su" ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=NlHTVQmz4un3Yf7L7tYrg2SdHXkxz8xsLRpZ3O5e9sNc3mzL/Cx18EouSNfVNX5fhas6dE+GMbFSeH1zs+e+IFtShWYxnkbietI4B27PiNL/8mVg7XPm25vmNVih6nCRQUEkF/+X/PzsnShZrUznm8CxnwKs7HYhUoVYxOgKz/AXB4ZJC1VmSDO49cNsNhoReCKonZ7WtncdqCF798YIFKByu93Lc0FCX4cVjWgpNL5IoKlbdao5r1FEhSRVgvDwrYQz+VixRTvcJEtjM8c1P6E+0VfoN1kNzBTHROYOW/HT0gAgruDXeitr/jnaJfDHs8bZp5FXb7T2Thb1ksiVzQ== 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=y5zoM/nvw/CdaSL3AQrSqSRY+HNDa9mGiyhElilJiUw=; b=bf9q6tsLz8JK2NmYxSMcSF4BZwDRSo/lGTYouoRO6vmQFl/2iJeHmm2k85pTH64KKKMuc4oVOYbIMkYT7w4QrjcLH7yZtzmWcirIdYrg/1fMBVlk+seI732bmrR+EADgApvSRMrriNFLfH+Q+Ns3rV0IfxZW8kbZMQkRXRkPuVzKr1476AIAe9vFntEORQQXIUVqFVHs3lzX9OY0pqnuNdmH8Eer7xq2lptOY0NY2d2jR0ZTZsVd/5NiN4g1/yCGQR77GEiQvyDKr4UdhqctlfinQs4etueIIW+mB5VFk4fyX1ofsyL5SoWdFdYz9BnXjbm7AV6wT17UfZcFTXXX5g== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=y5zoM/nvw/CdaSL3AQrSqSRY+HNDa9mGiyhElilJiUw=; b=t3J2K5su9CGfsR6P6G5GQVLGJT3Ly2y6tfWlK6iew9naVIU90yJ3zxeYxYsknR9ohdLumlZf4jSmDFAqso+q6GaaUzAwsWSApgM2mvxseOP4QDEkqi30XpdSLQCqo+jg2V8+xhWVOzGUXFBvt+wWrwwgnFYG2lDUMLwpKWK7g0I= Received: from SA0PR11CA0143.namprd11.prod.outlook.com (2603:10b6:806:131::28) by MN0PR12MB6247.namprd12.prod.outlook.com (2603:10b6:208:3c1::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.17; Sat, 6 Jan 2024 02:26:18 +0000 Received: from SN1PEPF0002BA51.namprd03.prod.outlook.com (2603:10b6:806:131:cafe::40) by SA0PR11CA0143.outlook.office365.com (2603:10b6:806:131::28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.17 via Frontend Transport; Sat, 6 Jan 2024 02:26:18 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by SN1PEPF0002BA51.mail.protection.outlook.com (10.167.242.74) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.7159.9 via Frontend Transport; Sat, 6 Jan 2024 02:26:18 +0000 Received: from amd.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.34; Fri, 5 Jan 2024 20:26:16 -0600 From: Suma Hegde To: CC: , , Suma Hegde , Naveen Krishna Chatradhi Subject: [PATCH v5 06/11] platform/x86/amd/hsmp: Restructure sysfs group creation Date: Sat, 6 Jan 2024 02:25:27 +0000 Message-ID: <20240106022532.1746932-6-suma.hegde@amd.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20240106022532.1746932-1-suma.hegde@amd.com> References: <20240106022532.1746932-1-suma.hegde@amd.com> Precedence: bulk X-Mailing-List: platform-driver-x86@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-ClientProxiedBy: SATLEXMB04.amd.com (10.181.40.145) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SN1PEPF0002BA51:EE_|MN0PR12MB6247:EE_ X-MS-Office365-Filtering-Correlation-Id: 46dd90dd-cb2b-43a5-b082-08dc0e5edd6f X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 2N0nrgZ5ZcuMluUg0viMqNLJ8+a07Tu5DKnPzY4VN0iwg5yF2lYnfrQ2c5OyznDLn3OWjNxOaO21hrBmWjNxhMtxsFv/OVsbIwYKAw26zNZ+T1nrclAdYOSXYHAFQ9vvSfpTFN2HGzvnv0aIpw7oBUQCxccTf1S/QPl4OSKa7yAZo9zpD4IwXIfNbKkDPni/W8hEbGC4jv6w4ENr8Gzu5KVP90Qkp246pPsvGfRS9P3FKFU+AATI8vsuXent5rTTocjy9pQx+cSC4cdlhp7TN0Da05/+BzBCLwhT2Y9neWYILRdxmM9x+Yj3yOKofy/ldsINprP7dL/gUma8V71fNGGWlMc71VKIsNXu22laYTj581JsTYLzq8DsemIdatj4euDu8ROnpjKd+mlULJYmTn6A8I1RPx19uIq5xiuTH7fEsd5JzNC0yEimKuKYlV6yUV3O0kmir4S+RMGnGupS9ig18FFcNDQtRRgKt6mQfK+p8Sw2mXsJy+RN6XyFL2lKCGkMHWsPHc1ASj1B2IpUv4EERPKYwvwXJIDeGzu+s7oerjEKNJPLWe6FucZL+PQ+jWKOiDGFk5ZelYI9jpJaP2Su5JnCcZ/q+Nk8xQFJxt5DMEM0duERFM93/rQGSD8FXLWYZUDY+I9IXElZyxrUtTocbB1/H/e5m2qGwgqpQLiOZeQrN17OHJpn+kckB6it8I2BcjPjPUorRce+6BKXpsaJ8VPdTwBOwryn/RPws0Ei5XUrZiWsFrld5nNOLXUgd5f+yOYba+X0Fp4WRo0m3g== X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230031)(4636009)(396003)(376002)(39860400002)(136003)(346002)(230922051799003)(64100799003)(186009)(82310400011)(451199024)(1800799012)(40470700004)(36840700001)(46966006)(82740400003)(356005)(6916009)(70206006)(316002)(54906003)(70586007)(36860700001)(81166007)(47076005)(6666004)(86362001)(336012)(2616005)(1076003)(26005)(7696005)(426003)(16526019)(5660300002)(478600001)(2906002)(4326008)(36756003)(83380400001)(8676002)(44832011)(8936002)(40480700001)(40460700003)(41300700001)(36900700001);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Jan 2024 02:26:18.3785 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 46dd90dd-cb2b-43a5-b082-08dc0e5edd6f X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: SN1PEPF0002BA51.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN0PR12MB6247 Split the creation of array of attribute groups and array of attributes into different functions. This will ease the ACPI support. Signed-off-by: Suma Hegde Signed-off-by: Naveen Krishna Chatradhi Reviewed-by: Ilpo Järvinen --- Changes since v4: New patch, generated after splitting 6th patch in v4 series drivers/platform/x86/amd/hsmp.c | 44 ++++++++++++++++++--------------- 1 file changed, 24 insertions(+), 20 deletions(-) diff --git a/drivers/platform/x86/amd/hsmp.c b/drivers/platform/x86/amd/hsmp.c index 0bf94e2bd022..cf8e8d155155 100644 --- a/drivers/platform/x86/amd/hsmp.c +++ b/drivers/platform/x86/amd/hsmp.c @@ -442,49 +442,53 @@ static int hsmp_init_metric_tbl_bin_attr(struct bin_attribute **hattrs, u16 sock /* One bin sysfs for metrics table*/ #define NUM_HSMP_ATTRS 1 -static int hsmp_create_sysfs_interface(void) +static int hsmp_create_attr_list(struct attribute_group *attr_grp, + struct device *dev, u16 sock_ind) { - const struct attribute_group **hsmp_attr_grps; struct bin_attribute **hsmp_bin_attrs; + + /* Null terminated list of attributes */ + hsmp_bin_attrs = devm_kzalloc(dev, sizeof(struct bin_attribute *) * + (NUM_HSMP_ATTRS + 1), GFP_KERNEL); + if (!hsmp_bin_attrs) + return -ENOMEM; + + attr_grp->bin_attrs = hsmp_bin_attrs; + + return hsmp_init_metric_tbl_bin_attr(hsmp_bin_attrs, sock_ind); +} + +static int hsmp_create_sysfs_interface(struct device *dev) +{ + const struct attribute_group **hsmp_attr_grps; struct attribute_group *attr_grp; - int ret; u16 i; /* String formatting is currently limited to u8 sockets */ if (WARN_ON(plat_dev.num_sockets > U8_MAX)) return -ERANGE; - hsmp_attr_grps = devm_kzalloc(plat_dev.sock[0].dev, sizeof(struct attribute_group *) * + hsmp_attr_grps = devm_kzalloc(dev, sizeof(struct attribute_group *) * (plat_dev.num_sockets + 1), GFP_KERNEL); if (!hsmp_attr_grps) return -ENOMEM; /* Create a sysfs directory for each socket */ for (i = 0; i < plat_dev.num_sockets; i++) { - attr_grp = devm_kzalloc(plat_dev.sock[i].dev, sizeof(struct attribute_group), + attr_grp = devm_kzalloc(dev, sizeof(struct attribute_group), GFP_KERNEL); if (!attr_grp) return -ENOMEM; snprintf(plat_dev.sock[i].name, HSMP_ATTR_GRP_NAME_SIZE, "socket%u", (u8)i); - attr_grp->name = plat_dev.sock[i].name; - - /* Null terminated list of attributes */ - hsmp_bin_attrs = devm_kzalloc(plat_dev.sock[i].dev, sizeof(struct bin_attribute *) * - (NUM_HSMP_ATTRS + 1), GFP_KERNEL); - if (!hsmp_bin_attrs) - return -ENOMEM; - - attr_grp->bin_attrs = hsmp_bin_attrs; + attr_grp->name = plat_dev.sock[i].name; attr_grp->is_bin_visible = hsmp_is_sock_attr_visible; hsmp_attr_grps[i] = attr_grp; - /* Now create the leaf nodes */ - ret = hsmp_init_metric_tbl_bin_attr(hsmp_bin_attrs, i); - if (ret) - return ret; + hsmp_create_attr_list(attr_grp, dev, i); } - return devm_device_add_groups(plat_dev.sock[0].dev, hsmp_attr_grps); + + return devm_device_add_groups(dev, hsmp_attr_grps); } static int hsmp_cache_proto_ver(void) @@ -561,7 +565,7 @@ static int hsmp_pltdrv_probe(struct platform_device *pdev) return ret; } - ret = hsmp_create_sysfs_interface(); + ret = hsmp_create_sysfs_interface(&pdev->dev); if (ret) dev_err(&pdev->dev, "Failed to create HSMP sysfs interface\n"); From patchwork Sat Jan 6 02:25:28 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Suma Hegde X-Patchwork-Id: 13512650 Received: from NAM10-DM6-obe.outbound.protection.outlook.com (mail-dm6nam10on2069.outbound.protection.outlook.com [40.107.93.69]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A37B91381 for ; Sat, 6 Jan 2024 02:26:26 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="FADMcJDr" ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=CLSndt/HPAKIQsckd1nTCHeLpGPBcb1pHdyj5S1rO5elUW7T7ZzPVkgsHsEWeV+V9+JkW5JT+R1FJrCcR8Eap0CUqo2i3j7U7n9oTWzekjkvMryFQK3CGRdV+kW0Np8wz8vEgSsXK4miODv7bXtSx+oRmY+vJFEzCOSqKgKVPxpyDqnLo3ZSTZVGaZhPv5DGGtxy1Irfuqc9AwLezcSgUSgr1M1EDKEJd+cLnB+Gj2nHZrWLB4YSoS7VfuxPf73NIMhudddLcj3k6ll0C8E3a7t6b9yu/w+egbniCEGoIA0+YD6O2+WWIc4cAXvCP93jPgUSFoq+9hQGGytP6toe0g== 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=rDlJcFkz9NtX3nGZVA+zvi1caxQCJswBfUQ6O1Q1KyY=; b=lfgi0KJIjpl1d0g4wx5PUeYvEvi16NHVt65bq7/U4ZXU0NURChFln8Re0rpfksL2MsObnomifGpf13FEDcBvfs4dk6WApZ2mplM/aLcwjlPzKNVr7eUl4JyPnQv3EkFWWxxLy5bSb6CS7wdC/9cDVEKAnGEe0SBr8vsc1d5bvAkm6+Kd4CS40bzrVHVIH8Cr8aiMuYLYqbDP+7S4jcA9j0JCGYZpB/xakEA9sDEbJI96fphLdTREUhoil/lI0vMVHvB2JhESMQvefx5XQ+LkJ+sCMgJ870Lx3tQbbm3IJWX6dZbYm4371LYrVQKUQjplsaeW8dTd/NbytaueypKhKA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=rDlJcFkz9NtX3nGZVA+zvi1caxQCJswBfUQ6O1Q1KyY=; b=FADMcJDrZDAU/YP/GpS44RE4xnNzSYXjlFKgCaKM4khPxQmPGEFJBSbrwM3ReDWqsTsh0t60MpZazrD1vwU6AFoJYgZJcfnrTmh6JAn4cV8H2uXqFa+1Hv5DlIRkMu2slWiLGahZwfFrdHm+pbRsq4RdAd14is89Jqk+TIhhLRc= Received: from SN1PR12CA0094.namprd12.prod.outlook.com (2603:10b6:802:21::29) by SJ0PR12MB6878.namprd12.prod.outlook.com (2603:10b6:a03:483::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.18; Sat, 6 Jan 2024 02:26:20 +0000 Received: from SN1PEPF0002BA4B.namprd03.prod.outlook.com (2603:10b6:802:21:cafe::91) by SN1PR12CA0094.outlook.office365.com (2603:10b6:802:21::29) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.17 via Frontend Transport; Sat, 6 Jan 2024 02:26:20 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by SN1PEPF0002BA4B.mail.protection.outlook.com (10.167.242.68) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.7159.9 via Frontend Transport; Sat, 6 Jan 2024 02:26:20 +0000 Received: from amd.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.34; Fri, 5 Jan 2024 20:26:18 -0600 From: Suma Hegde To: CC: , , Suma Hegde , Naveen Krishna Chatradhi Subject: [PATCH v5 07/11] platform/x86/amd/hsmp: Add support for ACPI based probing Date: Sat, 6 Jan 2024 02:25:28 +0000 Message-ID: <20240106022532.1746932-7-suma.hegde@amd.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20240106022532.1746932-1-suma.hegde@amd.com> References: <20240106022532.1746932-1-suma.hegde@amd.com> Precedence: bulk X-Mailing-List: platform-driver-x86@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-ClientProxiedBy: SATLEXMB04.amd.com (10.181.40.145) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SN1PEPF0002BA4B:EE_|SJ0PR12MB6878:EE_ X-MS-Office365-Filtering-Correlation-Id: 4bbcb981-7ed0-49cf-81b4-08dc0e5edeaa X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: OEcRXeO/VVfzV7ToBzXdrh5rD7vV9svcsJHFq4wIj8NIM+OCU/IWaBVR/HOygwzDBTsC2v6icKhbRWqdGg2c1V4GcHFrkzP52TvsqkcM8qG4UVYH8dqHI0g7kmpGJmpv5VqnTMbRo3DKMh2T96Sjnq8lR4sNpBD4igZ/xz6llwNMv1DUodcHDyKQneU1PQPOxpx4OKPxmHxzYu66/bgGzmxLNcAit7v6tmlN2bg8gPQQJgy+mCAzNqtyjKey9GKqZ6OzVVjdg9qk9gtGS+uK7KWpPjp5mzqPC4qm67w3E42KkiCPO+zyNjx9J8sVsF3w6HcMDOo8S86TqNHSiLffjg8VjOgYM7NHf3tzZkQumYUEg3LLYA+hTnkACHyXMuJoaldoXGlsYb2gY3mJ6PXitf4YHLbkKYRT7DE3dJikzkQmIgznEOiOjrTBtDP7NPtbf7+ZC335fsxzz9WK23cqx7Fq4nVfeElkMW9pOY4O0puQ2JkwepXcd8JJKxrDKuFeotZJkLyKfGj92cfpOXcnK9B7PmtqU/PdfOg7wdMV+C3BgPDUjiwzZj4gaGdBU8eh9uctzXr5KzIePurlAY54om8ob+bxnLz1uOwo2jZRNsFE6EqCOg7kVXTs5ghl21xStxSDDQrUk4a0VsL3lI3TdY+TGO1J539X9XXu1wPw6UsV/CkOHPjggISpSFma/HE8sGQLidnR194NZrOJaJBlp2nMJz4u2NIPPfpydB9QOzSXTtfCnWnM6NnDbbPq+b6/ddmWk65trZ4+lJHEv4D2lA== X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230031)(4636009)(396003)(346002)(376002)(136003)(39860400002)(230922051799003)(186009)(451199024)(82310400011)(1800799012)(64100799003)(40470700004)(36840700001)(46966006)(70206006)(70586007)(47076005)(316002)(4326008)(8676002)(8936002)(26005)(1076003)(336012)(16526019)(478600001)(426003)(2616005)(83380400001)(6916009)(2906002)(82740400003)(5660300002)(41300700001)(30864003)(81166007)(7696005)(356005)(40480700001)(86362001)(36860700001)(6666004)(54906003)(36756003)(40460700003)(44832011)(36900700001);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Jan 2024 02:26:20.4417 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 4bbcb981-7ed0-49cf-81b4-08dc0e5edeaa X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: SN1PEPF0002BA4B.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR12MB6878 ACPI table provides mailbox base address and register offset information. The base address is provided as part of CRS method and mailbox offsets are provided through DSD table. Sockets are differentiated by UIDs. Signed-off-by: Suma Hegde Signed-off-by: Naveen Krishna Chatradhi Reviewed-by: Ilpo Järvinen --- Changes since v4: None, generated after splitting 6th patch in v4 series(6th and 7th patches of v5 is split from 6th patch in v4 series) Changes since v3: 1. Add hsmp_create_acpi_sysfs_if() and hsmp_create_non_acpi_sysfs_if() separately 2. Change hardcoded value 16 in is_acpi_hsmp_uuid() to UUID_SIZE 3. Change commit message Changes since v2: 1. Change EINVAL to ENODEV in hsmp_read_acpi_dsd() 2. Change EINVAL to ENOENT in hsmp_read_acpi_dsd() 3. Use resource_size() in hsmp_resource() Changes since v1: 1. Define amd_hsmp_acpi_rdwr() for doing mailbox memory mapped io 2. Add a check to see if mailbox register offsets are set in hsmp_read_acpi_dsd() 3. Add a check to see if sock->mbinfo.base_addr sockck->mbinfo.size are set in hsmp_read_acpi_crs() 4. Change order of the statements in switch case ACPI_RESOURCE_TYPE_FIXED_MEMORY32 in hsmp_resource() 5. Add hsmp_test() after hsmp_parse_acpi_table() call 6. Add r.end < r.start check in hsmp_resource() 7. Add !dsd error check in hsmp_read_acpi_dsd drivers/platform/x86/amd/hsmp.c | 351 +++++++++++++++++++++++++++++--- 1 file changed, 321 insertions(+), 30 deletions(-) diff --git a/drivers/platform/x86/amd/hsmp.c b/drivers/platform/x86/amd/hsmp.c index cf8e8d155155..723a92439438 100644 --- a/drivers/platform/x86/amd/hsmp.c +++ b/drivers/platform/x86/amd/hsmp.c @@ -18,9 +18,11 @@ #include #include #include +#include #define DRIVER_NAME "amd_hsmp" -#define DRIVER_VERSION "2.0" +#define DRIVER_VERSION "2.2" +#define ACPI_HSMP_DEVICE_HID "AMDI0097" /* HSMP Status / Error codes */ #define HSMP_STATUS_NOT_READY 0x00 @@ -54,6 +56,11 @@ #define HSMP_ATTR_GRP_NAME_SIZE 10 +/* These are the strings specified in ACPI table */ +#define MSG_IDOFF_STR "MsgIdOffset" +#define MSG_ARGOFF_STR "MsgArgOffset" +#define MSG_RESPOFF_STR "MsgRspOffset" + struct hsmp_mbaddr_info { u32 base_addr; u32 msg_id_off; @@ -66,6 +73,7 @@ struct hsmp_socket { struct bin_attribute hsmp_attr; struct hsmp_mbaddr_info mbinfo; void __iomem *metric_tbl_addr; + void __iomem *virt_base_addr; struct semaphore hsmp_sem; char name[HSMP_ATTR_GRP_NAME_SIZE]; struct pci_dev *root; @@ -78,12 +86,14 @@ struct hsmp_plat_device { struct hsmp_socket *sock; u32 proto_ver; u16 num_sockets; + bool is_acpi_device; + bool is_probed; }; static struct hsmp_plat_device plat_dev; -static int amd_hsmp_rdwr(struct hsmp_socket *sock, u32 offset, - u32 *value, bool write) +static int amd_hsmp_pci_rdwr(struct hsmp_socket *sock, u32 offset, + u32 *value, bool write) { int ret; @@ -101,8 +111,29 @@ static int amd_hsmp_rdwr(struct hsmp_socket *sock, u32 offset, return ret; } +static void amd_hsmp_acpi_rdwr(struct hsmp_socket *sock, u32 offset, + u32 *value, bool write) +{ + if (write) + iowrite32(*value, sock->virt_base_addr + offset); + else + *value = ioread32(sock->virt_base_addr + offset); +} + +static int amd_hsmp_rdwr(struct hsmp_socket *sock, u32 offset, + u32 *value, bool write) +{ + if (plat_dev.is_acpi_device) + amd_hsmp_acpi_rdwr(sock, offset, value, write); + else + return amd_hsmp_pci_rdwr(sock, offset, value, write); + + return 0; +} + /* - * Send a message to the HSMP port via PCI-e config space registers. + * Send a message to the HSMP port via PCI-e config space registers + * or by writing to MMIO space. * * The caller is expected to zero out any unused arguments. * If a response is expected, the number of response words should be greater than 0. @@ -349,6 +380,181 @@ static const struct file_operations hsmp_fops = { .compat_ioctl = hsmp_ioctl, }; +/* This is the UUID used for HSMP */ +static const guid_t acpi_hsmp_uuid = GUID_INIT(0xb74d619d, 0x5707, 0x48bd, + 0xa6, 0x9f, 0x4e, 0xa2, + 0x87, 0x1f, 0xc2, 0xf6); + +static inline bool is_acpi_hsmp_uuid(union acpi_object *obj) +{ + if (obj->type == ACPI_TYPE_BUFFER && obj->buffer.length == UUID_SIZE) + return guid_equal((guid_t *)obj->buffer.pointer, &acpi_hsmp_uuid); + + return false; +} + +static inline int hsmp_get_uid(struct device *dev, u16 *sock_ind) +{ + char *uid; + + /* + * UID (ID00, ID01..IDXX) is used for differentiating sockets, + * read it and strip the "ID" part of it and convert the remaining + * bytes to integer. + */ + uid = acpi_device_uid(ACPI_COMPANION(dev)); + + return kstrtou16((uid + 2), 10, sock_ind); +} + +static acpi_status hsmp_resource(struct acpi_resource *res, void *data) +{ + struct hsmp_socket *sock = data; + struct resource r; + + switch (res->type) { + case ACPI_RESOURCE_TYPE_FIXED_MEMORY32: + if (!acpi_dev_resource_memory(res, &r)) + return AE_ERROR; + if (!r.start || r.end < r.start || !(r.flags & IORESOURCE_MEM_WRITEABLE)) + return AE_ERROR; + sock->mbinfo.base_addr = r.start; + sock->mbinfo.size = resource_size(&r); + break; + case ACPI_RESOURCE_TYPE_END_TAG: + break; + default: + return AE_ERROR; + } + + return AE_OK; +} + +static int hsmp_read_acpi_dsd(struct hsmp_socket *sock) +{ + struct acpi_buffer buf = { ACPI_ALLOCATE_BUFFER, NULL }; + union acpi_object *guid, *mailbox_package; + union acpi_object *dsd; + acpi_status status; + int ret = 0; + int j; + + status = acpi_evaluate_object_typed(ACPI_HANDLE(sock->dev), "_DSD", NULL, + &buf, ACPI_TYPE_PACKAGE); + if (ACPI_FAILURE(status)) { + dev_err(sock->dev, "Failed to read mailbox reg offsets from DSD table, err: %s\n", + acpi_format_exception(status)); + return -ENODEV; + } + + dsd = buf.pointer; + + /* HSMP _DSD property should contain 2 objects. + * 1. guid which is an acpi object of type ACPI_TYPE_BUFFER + * 2. mailbox which is an acpi object of type ACPI_TYPE_PACKAGE + * This mailbox object contains 3 more acpi objects of type + * ACPI_TYPE_PACKAGE for holding msgid, msgresp, msgarg offsets + * these packages inturn contain 2 acpi objects of type + * ACPI_TYPE_STRING and ACPI_TYPE_INTEGER + */ + if (!dsd || dsd->type != ACPI_TYPE_PACKAGE || dsd->package.count != 2) { + ret = -EINVAL; + goto free_buf; + } + + guid = &dsd->package.elements[0]; + mailbox_package = &dsd->package.elements[1]; + if (!is_acpi_hsmp_uuid(guid) || mailbox_package->type != ACPI_TYPE_PACKAGE) { + dev_err(sock->dev, "Invalid hsmp _DSD table data\n"); + ret = -EINVAL; + goto free_buf; + } + + for (j = 0; j < mailbox_package->package.count; j++) { + union acpi_object *msgobj, *msgstr, *msgint; + + msgobj = &mailbox_package->package.elements[j]; + msgstr = &msgobj->package.elements[0]; + msgint = &msgobj->package.elements[1]; + + /* package should have 1 string and 1 integer object */ + if (msgobj->type != ACPI_TYPE_PACKAGE || + msgstr->type != ACPI_TYPE_STRING || + msgint->type != ACPI_TYPE_INTEGER) { + ret = -EINVAL; + goto free_buf; + } + + if (!strncmp(msgstr->string.pointer, MSG_IDOFF_STR, + msgstr->string.length)) { + sock->mbinfo.msg_id_off = msgint->integer.value; + } else if (!strncmp(msgstr->string.pointer, MSG_RESPOFF_STR, + msgstr->string.length)) { + sock->mbinfo.msg_resp_off = msgint->integer.value; + } else if (!strncmp(msgstr->string.pointer, MSG_ARGOFF_STR, + msgstr->string.length)) { + sock->mbinfo.msg_arg_off = msgint->integer.value; + } else { + ret = -ENOENT; + goto free_buf; + } + } + + if (!sock->mbinfo.msg_id_off || !sock->mbinfo.msg_resp_off || + !sock->mbinfo.msg_arg_off) + ret = -EINVAL; + +free_buf: + ACPI_FREE(buf.pointer); + return ret; +} + +static int hsmp_read_acpi_crs(struct hsmp_socket *sock) +{ + acpi_status status; + + status = acpi_walk_resources(ACPI_HANDLE(sock->dev), METHOD_NAME__CRS, + hsmp_resource, sock); + if (ACPI_FAILURE(status)) { + dev_err(sock->dev, "Failed to look up MP1 base address from CRS method, err: %s\n", + acpi_format_exception(status)); + return -EINVAL; + } + if (!sock->mbinfo.base_addr || !sock->mbinfo.size) + return -EINVAL; + + /* The mapped region should be un cached */ + sock->virt_base_addr = devm_ioremap_uc(sock->dev, sock->mbinfo.base_addr, + sock->mbinfo.size); + if (!sock->virt_base_addr) { + dev_err(sock->dev, "Failed to ioremap MP1 base address\n"); + return -ENOMEM; + } + + return 0; +} + +/* Parse the ACPI table to read the data */ +static int hsmp_parse_acpi_table(struct device *dev, u16 sock_ind) +{ + struct hsmp_socket *sock = &plat_dev.sock[sock_ind]; + int ret; + + sock->sock_ind = sock_ind; + sock->dev = dev; + plat_dev.is_acpi_device = true; + + sema_init(&sock->hsmp_sem, 1); + + /* Read MP1 base address from CRS method */ + ret = hsmp_read_acpi_crs(sock); + if (ret) + return ret; + + /* Read mailbox offsets from DSD table */ + return hsmp_read_acpi_dsd(sock); +} + static ssize_t hsmp_metric_tbl_read(struct file *filp, struct kobject *kobj, struct bin_attribute *bin_attr, char *buf, loff_t off, size_t count) @@ -458,7 +664,7 @@ static int hsmp_create_attr_list(struct attribute_group *attr_grp, return hsmp_init_metric_tbl_bin_attr(hsmp_bin_attrs, sock_ind); } -static int hsmp_create_sysfs_interface(struct device *dev) +static int hsmp_create_non_acpi_sysfs_if(struct device *dev) { const struct attribute_group **hsmp_attr_grps; struct attribute_group *attr_grp; @@ -491,13 +697,36 @@ static int hsmp_create_sysfs_interface(struct device *dev) return devm_device_add_groups(dev, hsmp_attr_grps); } -static int hsmp_cache_proto_ver(void) +static int hsmp_create_acpi_sysfs_if(struct device *dev) +{ + struct attribute_group *attr_grp; + u16 sock_ind; + int ret; + + attr_grp = devm_kzalloc(dev, sizeof(struct attribute_group), GFP_KERNEL); + if (!attr_grp) + return -ENOMEM; + + attr_grp->is_bin_visible = hsmp_is_sock_attr_visible; + + ret = hsmp_get_uid(dev, &sock_ind); + if (ret) + return ret; + + ret = hsmp_create_attr_list(attr_grp, dev, sock_ind); + if (ret) + return ret; + + return devm_device_add_group(dev, attr_grp); +} + +static int hsmp_cache_proto_ver(u16 sock_ind) { struct hsmp_message msg = { 0 }; int ret; msg.msg_id = HSMP_GET_PROTO_VER; - msg.sock_ind = 0; + msg.sock_ind = sock_ind; msg.response_sz = hsmp_msg_desc_table[HSMP_GET_PROTO_VER].response_sz; ret = hsmp_send_message(&msg); @@ -538,43 +767,102 @@ static int init_platform_device(struct device *dev) return 0; } +static const struct acpi_device_id amd_hsmp_acpi_ids[] = { + {ACPI_HSMP_DEVICE_HID, 0}, + {} +}; +MODULE_DEVICE_TABLE(acpi, amd_hsmp_acpi_ids); + static int hsmp_pltdrv_probe(struct platform_device *pdev) { + struct acpi_device *adev; + u16 sock_ind = 0; int ret; - plat_dev.sock = devm_kzalloc(&pdev->dev, - (plat_dev.num_sockets * sizeof(struct hsmp_socket)), - GFP_KERNEL); - if (!plat_dev.sock) - return -ENOMEM; - - ret = init_platform_device(&pdev->dev); - if (ret) - return ret; - - plat_dev.hsmp_device.name = HSMP_CDEV_NAME; - plat_dev.hsmp_device.minor = MISC_DYNAMIC_MINOR; - plat_dev.hsmp_device.fops = &hsmp_fops; - plat_dev.hsmp_device.parent = &pdev->dev; - plat_dev.hsmp_device.nodename = HSMP_DEVNODE_NAME; - plat_dev.hsmp_device.mode = 0644; + /* + * On ACPI supported BIOS, there is an ACPI HSMP device added for + * each socket, so the per socket probing, but the memory allocated for + * sockets should be contiguous to access it as an array, + * Hence allocate memory for all the sockets at once instead of allocating + * on each probe. + */ + if (!plat_dev.is_probed) { + plat_dev.sock = devm_kzalloc(&pdev->dev, + (plat_dev.num_sockets * sizeof(struct hsmp_socket)), + GFP_KERNEL); + if (!plat_dev.sock) + return -ENOMEM; + } + adev = ACPI_COMPANION(&pdev->dev); + if (adev && !acpi_match_device_ids(adev, amd_hsmp_acpi_ids)) { + ret = hsmp_get_uid(&pdev->dev, &sock_ind); + if (ret) + return ret; + if (sock_ind >= plat_dev.num_sockets) + return -EINVAL; + ret = hsmp_parse_acpi_table(&pdev->dev, sock_ind); + if (ret) { + dev_err(&pdev->dev, "Failed to parse ACPI table\n"); + return ret; + } + /* Test the hsmp interface */ + ret = hsmp_test(sock_ind, 0xDEADBEEF); + if (ret) { + dev_err(&pdev->dev, "HSMP test message failed on Fam:%x model:%x\n", + boot_cpu_data.x86, boot_cpu_data.x86_model); + dev_err(&pdev->dev, "Is HSMP disabled in BIOS ?\n"); + return ret; + } + } else { + ret = init_platform_device(&pdev->dev); + if (ret) { + dev_err(&pdev->dev, "Failed to init HSMP mailbox\n"); + return ret; + } + } - ret = hsmp_cache_proto_ver(); + ret = hsmp_cache_proto_ver(sock_ind); if (ret) { dev_err(&pdev->dev, "Failed to read HSMP protocol version\n"); return ret; } - ret = hsmp_create_sysfs_interface(&pdev->dev); + if (plat_dev.is_acpi_device) + ret = hsmp_create_acpi_sysfs_if(&pdev->dev); + else + ret = hsmp_create_non_acpi_sysfs_if(&pdev->dev); if (ret) dev_err(&pdev->dev, "Failed to create HSMP sysfs interface\n"); - return misc_register(&plat_dev.hsmp_device); + if (!plat_dev.is_probed) { + plat_dev.hsmp_device.name = HSMP_CDEV_NAME; + plat_dev.hsmp_device.minor = MISC_DYNAMIC_MINOR; + plat_dev.hsmp_device.fops = &hsmp_fops; + plat_dev.hsmp_device.parent = &pdev->dev; + plat_dev.hsmp_device.nodename = HSMP_DEVNODE_NAME; + plat_dev.hsmp_device.mode = 0644; + + ret = misc_register(&plat_dev.hsmp_device); + if (ret) + return ret; + + plat_dev.is_probed = true; + } + + return 0; + } static void hsmp_pltdrv_remove(struct platform_device *pdev) { - misc_deregister(&plat_dev.hsmp_device); + /* + * We register only one misc_device even on multi socket system. + * So, deregister should happen only once. + */ + if (plat_dev.is_probed) { + misc_deregister(&plat_dev.hsmp_device); + plat_dev.is_probed = false; + } } static struct platform_driver amd_hsmp_driver = { @@ -582,6 +870,7 @@ static struct platform_driver amd_hsmp_driver = { .remove_new = hsmp_pltdrv_remove, .driver = { .name = DRIVER_NAME, + .acpi_match_table = amd_hsmp_acpi_ids, }, }; @@ -624,9 +913,11 @@ static int __init hsmp_plt_init(void) if (ret) return ret; - ret = hsmp_plat_dev_register(); - if (ret) - platform_driver_unregister(&amd_hsmp_driver); + if (!plat_dev.is_acpi_device) { + ret = hsmp_plat_dev_register(); + if (ret) + platform_driver_unregister(&amd_hsmp_driver); + } return ret; } From patchwork Sat Jan 6 02:25:29 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Suma Hegde X-Patchwork-Id: 13512649 Received: from NAM10-DM6-obe.outbound.protection.outlook.com (mail-dm6nam10on2072.outbound.protection.outlook.com [40.107.93.72]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1AA8815AC for ; Sat, 6 Jan 2024 02:26:24 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="OgRPW+fV" ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=NAEDuZC180x9hhwtt3jkVcqKD1Y2A+oZWaTjtcQaMd4uDbG15p0SWLXc0+j4hsQFFL6PYch2JDIKEfgt1ue9MI6OzUtQ7ddJYrUTOkyRgVUUTtW2S3Bp5eibysGJZDMzucdC/NTWtIs1T5y/r+FJgVfWLtDECTBEb4eZDL1hEf3A6jXuYRXzDUM2lJQU6Kt65Vi81VKQ3/7WJwm8iFo0sYv3MLHQEMrI21a+v3k/23eUsoewXTYamq1jPNtq0w6Z+9K+b/hw9IqliZumEqEP1ghUPDTONOti0DADcildP4VHD0X9YejMC+hjfA2wHaQSIyFgjCbgH83istZWLW/JBg== 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=qpcgjt6Cmy2g5vQUcHP7YXaMTE7mwsO1jz6G1le5ToE=; b=iv+MSLoWOCQufRSe5gXFIYdB7x7uxQJ/37iEShw7tpU9PbOLwboZ8sBIVzZhGvups3n2uHpqetwtDaqLPsLddGtXF90sRpTuQe3PglJ5DGMURucga+S5reCEKPEqrXOwi0DnrjDmbIWfXQo9GXDoW6HsRfdRNzJKapkfdrAoMtY+EFtwWe2FIa2WLH6s9uqlqncA7YFhm4DZhLXPz1Fzvm5uYWMd0w/cj/fTyG4SkXlJfvOIVko9uERTs7jZRkovtygVIKeXL8XGfqPUx60Qh96d0KNyJ4mhp17PtuLUXHWYcmF9aXtNcw/61gpmfIF2VPSh42V/gLyCkTXZPKe+9Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=qpcgjt6Cmy2g5vQUcHP7YXaMTE7mwsO1jz6G1le5ToE=; b=OgRPW+fVo/KepIiyK6MmOX+H8BgyD6Tpp7mT6F2DmX8vpy7iV4zIaElsvaw1QGWEX+hhGL/EdRDqWFMB19P8KdqkjpH7Xwu7ixtl9zJIpcQrFEYbVdZUfqQnovplxnJfDUEdwp78dajTnXDdWKRqRIL2/7nZJCn1dR7jnLu4TTU= Received: from PH8PR07CA0018.namprd07.prod.outlook.com (2603:10b6:510:2cd::15) by IA1PR12MB7495.namprd12.prod.outlook.com (2603:10b6:208:419::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.17; Sat, 6 Jan 2024 02:26:22 +0000 Received: from SN1PEPF0002BA4F.namprd03.prod.outlook.com (2603:10b6:510:2cd:cafe::b) by PH8PR07CA0018.outlook.office365.com (2603:10b6:510:2cd::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.17 via Frontend Transport; Sat, 6 Jan 2024 02:26:22 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by SN1PEPF0002BA4F.mail.protection.outlook.com (10.167.242.72) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.7159.9 via Frontend Transport; Sat, 6 Jan 2024 02:26:22 +0000 Received: from amd.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.34; Fri, 5 Jan 2024 20:26:20 -0600 From: Suma Hegde To: CC: , , Suma Hegde , Naveen Krishna Chatradhi Subject: [PATCH v5 08/11] platform/x86/amd/hsmp: Non-ACPI support for AMD F1A_M00~0Fh Date: Sat, 6 Jan 2024 02:25:29 +0000 Message-ID: <20240106022532.1746932-8-suma.hegde@amd.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20240106022532.1746932-1-suma.hegde@amd.com> References: <20240106022532.1746932-1-suma.hegde@amd.com> Precedence: bulk X-Mailing-List: platform-driver-x86@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-ClientProxiedBy: SATLEXMB04.amd.com (10.181.40.145) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SN1PEPF0002BA4F:EE_|IA1PR12MB7495:EE_ X-MS-Office365-Filtering-Correlation-Id: 64fabe21-5b7b-40b9-0930-08dc0e5edfc0 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: ftmeJPWYBIWflZRNffye7qf1YeolYDJHfzRz3Ei5pSmclkiGpAmmmDJP+1kyOwriH12L6ykTxDUX3Zu5FYbc+yqNLEOt6Zb1wjGp3UXHtETuScWcIJNGngoT01X8Qwcu5GCofjKuLVIGFvgilFKxH+TEKp5zajSZ1FnwpghK53DMvAL+OuIDpOevpNmjLIcw2OTvHuqa4tjqfHhUKPDUebYkZ4OOwHJunm4cA/tSBp7XYlZoStIFbCVy8rV634hadmcfkSYZGeLDW/2aflXRORMQMMt7QV32srvaOJo4WmT4srkdBTnTkbBX9rP0hMaWTCuIFuN0ERX1bpQouWYOVRRiqM4vf6GW/DyczEBrH6aQltwIJ6rl/CkSJdgjPVamovYvV/UfsEEQ7y+hLQDSCGvxVrNfs3D8yLA+948jpB2isrNSZJYWO7JoRPQyK4vaMFVwEwPopvFC/jeqkK2FvA4mIH4JKLS06IrnT85j4YhBclktMpXkT9OgJyeq/jMA16xbmW+W7S1aGbW01ti3uJEIO7svnp8C4w4jgspxkHV4+IjY8tQ7LyAdLSLRoz7ppoIO/0p1z8pdYB0e5FweEV0lkBZeFh39il/TlJ7+saxoN5tJod02K8HixqVIYnOQ0eevPQLGEIRtOfH7+FTpHgMFxDn41OkATUHC2O2aKdoO3a5i8IIKK9FHgVA9TKjVX27CeFxtMx96ZN2uwJuQ6mcp5BeI+aSNE0I8WhPgj5wEHJ9GA/rjjsaAnrN3UcQZdNOx2UxyoX84MLRZB5iU+Q== X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230031)(4636009)(396003)(136003)(39860400002)(376002)(346002)(230922051799003)(82310400011)(186009)(451199024)(64100799003)(1800799012)(40470700004)(36840700001)(46966006)(426003)(2616005)(336012)(1076003)(26005)(16526019)(66574015)(36756003)(2906002)(47076005)(83380400001)(86362001)(81166007)(82740400003)(356005)(41300700001)(36860700001)(4326008)(8936002)(8676002)(40480700001)(40460700003)(44832011)(478600001)(54906003)(70586007)(70206006)(6916009)(316002)(7696005)(5660300002)(36900700001);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Jan 2024 02:26:22.2761 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 64fabe21-5b7b-40b9-0930-08dc0e5edfc0 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: SN1PEPF0002BA4F.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA1PR12MB7495 AMD EPYC family 0x1A and Model 0x0-0xF are having different mailbox message ID offset compared to previous platforms. In case of ACPI based BIOS, this information will be read from ACPI table, for non-ACPI BIOS, this needs to be #defined. Signed-off-by: Suma Hegde Signed-off-by: Naveen Krishna Chatradhi Reviewed-by: Hans de Goede Reviewed-by: Ilpo Järvinen --- Changes since v4: None Changes since v3: 1. Correct the spelling mistake of "transitional" word 2. Move sema_init() statement Changes since v2: 1. Change "non ACPI" to "non-ACPI" 2. Add "Reviewed-by: Ilpo Järvinen " Changes since v1: Add "Reviewed-by: Hans de Goede " drivers/platform/x86/amd/hsmp.c | 21 ++++++++++++++++++++- 1 file changed, 20 insertions(+), 1 deletion(-) diff --git a/drivers/platform/x86/amd/hsmp.c b/drivers/platform/x86/amd/hsmp.c index 723a92439438..05255102e0a0 100644 --- a/drivers/platform/x86/amd/hsmp.c +++ b/drivers/platform/x86/amd/hsmp.c @@ -44,6 +44,7 @@ */ #define SMN_HSMP_BASE 0x3B00000 #define SMN_HSMP_MSG_ID 0x0010534 +#define SMN_HSMP_MSG_ID_F1A_M0H 0x0010934 #define SMN_HSMP_MSG_RESP 0x0010980 #define SMN_HSMP_MSG_DATA 0x00109E0 @@ -736,6 +737,15 @@ static int hsmp_cache_proto_ver(u16 sock_ind) return ret; } +static inline bool is_f1a_m0h(void) +{ + if (boot_cpu_data.x86 == 0x1A && + (boot_cpu_data.x86_model >= 0x00 && boot_cpu_data.x86_model <= 0x0F)) + return true; + + return false; +} + static int init_platform_device(struct device *dev) { struct hsmp_socket *sock; @@ -749,7 +759,16 @@ static int init_platform_device(struct device *dev) sock->sock_ind = i; sock->dev = dev; sock->mbinfo.base_addr = SMN_HSMP_BASE; - sock->mbinfo.msg_id_off = SMN_HSMP_MSG_ID; + + /* + * This is a transitional change from non-ACPI to ACPI, only + * family 0x1A, model 0x00 platform is supported for both ACPI and non-ACPI. + */ + if (is_f1a_m0h()) + sock->mbinfo.msg_id_off = SMN_HSMP_MSG_ID_F1A_M0H; + else + sock->mbinfo.msg_id_off = SMN_HSMP_MSG_ID; + sock->mbinfo.msg_resp_off = SMN_HSMP_MSG_RESP; sock->mbinfo.msg_arg_off = SMN_HSMP_MSG_DATA; sema_init(&sock->hsmp_sem, 1); From patchwork Sat Jan 6 02:25:30 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Suma Hegde X-Patchwork-Id: 13512651 Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11on2074.outbound.protection.outlook.com [40.107.223.74]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B55B2139C for ; Sat, 6 Jan 2024 02:26:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="ejBORXMh" ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=JbQcovCp8bZqc9wo+cGS/rb4Ij9/9Tb8w+UrSVSt9t3NrYnBP5TkWEeI/cE7fmVLVrlpEm+gc+iaE5GK8DLZMQGazGKdF0C5Cx6xOhX1MT7igmIf5XySf+0Hv3ijiR2QL2Hvk5cienR5fpwyyBWRMjRJksI/g1Xq0Xk2e4wceFXY5p3/8LoFNi7BJHpnsr3Nx0Ccrf36eI5U3I5B+/oC6ApKZV6rWBHukg18lP7X2NrUn8ISiZRg7RuCbo735D2JoJOUleiE/w8Y/8P8zjr+RV1qmzzdiBOJfNXQ4tsoQhQ1m1jRI1ODzuZDvqjHTT1P6gm0Mn6i7ExaRGskfbjPlQ== 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=4lHpL47zzSexOlDEtXEHCpJQiKMGVGuS4T3Q3gEAj4Q=; b=YKxUHumlLGuj9EU3o0iRZxK/VDS4DbCWE9fcZhsoXcyGYmU6i2OxhdJkcSUXRT2YlSpCiv99DhPiqIS36NEm32EwVP6qn3I4J/Y57bbYvvMiichPJkxmX4HVC04+9OASIIjAENUaBLaFr8g61bnMhQNM7qulscc2+dKuZyMzfVFReK3eNb1y2kl9yAhlKHDI8PH4xQsm8Ql1isorwF31kAFwIBqB/v2O9ZeAVqnStmulfMrpgE9r3bPusuSeaQX+cPFTkQU7LWlM33D6QdxfQhk4QeudgB9nCdM6AyM4oY9G7mF90wmLMFFUdAZYvEvbTWs1ahDZT/RYIDgox8Hwpg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=4lHpL47zzSexOlDEtXEHCpJQiKMGVGuS4T3Q3gEAj4Q=; b=ejBORXMhhwP7eooWJuH/lVkuwUbhMBPE4xbTvK5jUi60kLAb5Y8zg3MCo+nXZqd9a7FTY3stZGtxz2As7+rH6oOsCisJZ6o/HNU1Fx0ul8j4CY346BJXffHA1G2QsnDgkF4rd7jHqoV4F33cJXnyP2e2XwO9bQouFFWrRc92HfA= Received: from SN6PR08CA0003.namprd08.prod.outlook.com (2603:10b6:805:66::16) by PH8PR12MB7277.namprd12.prod.outlook.com (2603:10b6:510:223::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.16; Sat, 6 Jan 2024 02:26:24 +0000 Received: from SN1PEPF0002BA4E.namprd03.prod.outlook.com (2603:10b6:805:66:cafe::10) by SN6PR08CA0003.outlook.office365.com (2603:10b6:805:66::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.15 via Frontend Transport; Sat, 6 Jan 2024 02:26:24 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by SN1PEPF0002BA4E.mail.protection.outlook.com (10.167.242.71) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.7159.9 via Frontend Transport; Sat, 6 Jan 2024 02:26:24 +0000 Received: from amd.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.34; Fri, 5 Jan 2024 20:26:22 -0600 From: Suma Hegde To: CC: , , Suma Hegde , Naveen Krishna Chatradhi Subject: [PATCH v5 09/11] platform/x86/amd/hsmp: Check num_sockets against MAX_AMD_SOCKETS Date: Sat, 6 Jan 2024 02:25:30 +0000 Message-ID: <20240106022532.1746932-9-suma.hegde@amd.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20240106022532.1746932-1-suma.hegde@amd.com> References: <20240106022532.1746932-1-suma.hegde@amd.com> Precedence: bulk X-Mailing-List: platform-driver-x86@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-ClientProxiedBy: SATLEXMB04.amd.com (10.181.40.145) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SN1PEPF0002BA4E:EE_|PH8PR12MB7277:EE_ X-MS-Office365-Filtering-Correlation-Id: 59d57dee-19b5-43dc-c091-08dc0e5ee0d7 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: ZfwLE6/5R+lxGPOlqKkCvpjr78onYxnRZZ0ge8y6O2Rk9K5EJRYp8WjXiGG2OGYHNiZQwCN7nQE/xCqWDs7UIRPpmYiikj7Ait7ZYIMY5LxjvecCKSX45Q3l1xhbrqq3YvHQ2TprnKbhhl5QMzC9cASqUSaWDCFdQbdQ9C9P5it8losnh8BwmNPxeVjuGqiPfoj8Vo6DAYVbQwoiR30ejF4v+Htl5LBpc4eBb/x5FWCw43W3lzRqPo8OaV3T9qf6PyN6BE2MxB8rs3LXLV1D8T5qHzl1/dJ6rc5L6i+m3ufEq3EtDxPyWBZYPlU2rpXb1arG6okD23vvC/3lhWOYGOq2TGyhnTaDhXONkIasOWuybX0gTMiDA/Q5Kp8SVRje2eXmIRWdJ6ox7qLVrCcSkl+v53taZuO7CrxcIUpTsayCiQblm+RnKxLaSfb9kmvhh4bf2LeHym1OvZotNO0l/5wbaO1A7TIkLBeDdePvUdNqWA/ar7Iafwf2421cSSdxXRzQgveKh/I+Ha/0qbawaNjHiWheljiJysurYcHHbAgYkQ4G1+kTOI698hK+PzewJjjd07zeTToUM9nPBHXIba/WMuRWm6dDMFqRBXRlMbTGah2wiNUulVud9IceJNxzaqu0GzQMogwdhzEWikWhH37YcD42MS/8JQ/rKzguH3FRRaJIejKhwfUV1GejlmHl/9o+dMGrojFAgzcQ68bpiAP4+x142xpXlFKHNjmEYIQbYPxxoQSC1rsdGhEAL28urLrTXrYURndgD+gIFvJ8CA== X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230031)(4636009)(376002)(346002)(136003)(396003)(39860400002)(230922051799003)(451199024)(64100799003)(186009)(82310400011)(1800799012)(46966006)(40470700004)(36840700001)(36756003)(40460700003)(86362001)(40480700001)(356005)(66899024)(81166007)(83380400001)(47076005)(2616005)(66574015)(26005)(1076003)(16526019)(426003)(336012)(82740400003)(70206006)(36860700001)(7696005)(6666004)(478600001)(70586007)(54906003)(6916009)(4326008)(44832011)(2906002)(5660300002)(41300700001)(316002)(8936002)(8676002)(36900700001);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Jan 2024 02:26:24.1249 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 59d57dee-19b5-43dc-c091-08dc0e5ee0d7 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: SN1PEPF0002BA4E.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH8PR12MB7277 AMD supports connecting up to 8 AMD EPYCs in a system. Hence, verify the num_sockets returned from amd_nb_num(). Also remove the WARN_ON() since the num_sockets is already verified. Signed-off-by: Suma Hegde Reviewed-by: Naveen Krishna Chatradhi Reviewed-by: Ilpo Järvinen --- Changes since v4: Add Reviewed-by: Ilpo Järvinen Changes since v3: New patch drivers/platform/x86/amd/hsmp.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/drivers/platform/x86/amd/hsmp.c b/drivers/platform/x86/amd/hsmp.c index 05255102e0a0..99bebb0ca5a9 100644 --- a/drivers/platform/x86/amd/hsmp.c +++ b/drivers/platform/x86/amd/hsmp.c @@ -62,6 +62,8 @@ #define MSG_ARGOFF_STR "MsgArgOffset" #define MSG_RESPOFF_STR "MsgRspOffset" +#define MAX_AMD_SOCKETS 8 + struct hsmp_mbaddr_info { u32 base_addr; u32 msg_id_off; @@ -671,10 +673,6 @@ static int hsmp_create_non_acpi_sysfs_if(struct device *dev) struct attribute_group *attr_grp; u16 i; - /* String formatting is currently limited to u8 sockets */ - if (WARN_ON(plat_dev.num_sockets > U8_MAX)) - return -ERANGE; - hsmp_attr_grps = devm_kzalloc(dev, sizeof(struct attribute_group *) * (plat_dev.num_sockets + 1), GFP_KERNEL); if (!hsmp_attr_grps) @@ -925,7 +923,7 @@ static int __init hsmp_plt_init(void) * if we have N SMN/DF interfaces that ideally means N sockets */ plat_dev.num_sockets = amd_nb_num(); - if (plat_dev.num_sockets == 0) + if (plat_dev.num_sockets == 0 || plat_dev.num_sockets > MAX_AMD_SOCKETS) return ret; ret = platform_driver_register(&amd_hsmp_driver); From patchwork Sat Jan 6 02:25:31 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Suma Hegde X-Patchwork-Id: 13512652 Received: from NAM11-BN8-obe.outbound.protection.outlook.com (mail-bn8nam11on2041.outbound.protection.outlook.com [40.107.236.41]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2328B15AC for ; Sat, 6 Jan 2024 02:26:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="u39JD7NI" ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=fmZAmdqcxUzck5etPUMtBz6wu9jekNxol7i/kiW5Ie48HnLbx4CXCZoDSRdn9oWW7a9gt9fhiwcAXMc2yBzWqHJQ4PANkCYMSnLwKglPfzSdtI60ozoiHtptFBxxm4F1g2V64v2XfAnqS5ULqrYGsWc1N01weXWkpkxuu9XnNX042icBIqvtuHxqD8MMg7AU9YzlYZH9Qwm9rSpo+gjxo4qD/DT1hk3Pd5DSRqX2YZd/51m3YyUxMa/HlsIoEha6g8jQ5HWbI7SX5uzAp0nyK+NeuGMLmxT06gHaK0lMQvAEiZDMgDgVIMEOBK6/twdigYn7euMPHI1mWGcRGMmaTQ== 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=PXWxTO79mGGb3y2ub16HhYUqVZkHS/sX+T+fM9Xd2O4=; b=CW636xtNjXZ6aSiw5OyGgiDCYgKqivjBkzK+IhbGhBVof0PlEndM/E4u0R4/pKVumd9ecGvDsRTaIFILEnXmeXEjcJ5f6UbiHgAuAGICkgiL8RyVXlLEgrd6o3jewfJGf9f6ShrFiyfqQGYOIQ0gqmFx0J7c2u7xZJAQ5617JZPZrH0jSjHg8KY3qrxzW2GoA2rAYkcOlJXSVIbP/HQmXsaLxzjytILWTWd8+W3rlrdETjsqJT0LDlwAWpSlobLoe1yCxRj+TzeoewExSbf+eYyR0yYKPY04S5qgINUWNhbcQxCD1wZYvxKQMxWNbvQI5Xx4H9UjmUjD38kX0QC9TQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=PXWxTO79mGGb3y2ub16HhYUqVZkHS/sX+T+fM9Xd2O4=; b=u39JD7NI+l2iLkQ9Gbn3SqaD1ouqPl/nP1Q0/hNsh2XdF/vvqHiTtCZICqITTqgH8ESzAriGHx44kJL8kvGtSwVP5MWKO2e6luguzLhI1mO0tfCy+6IY9n7wmj+SCRl/M4Ps3zjH3uR94lK7XoZDJqzFJqW9EIfprthmWMf7Iho= Received: from PH8PR07CA0010.namprd07.prod.outlook.com (2603:10b6:510:2cd::22) by IA1PR12MB8539.namprd12.prod.outlook.com (2603:10b6:208:446::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.15; Sat, 6 Jan 2024 02:26:26 +0000 Received: from SN1PEPF0002BA4F.namprd03.prod.outlook.com (2603:10b6:510:2cd:cafe::7d) by PH8PR07CA0010.outlook.office365.com (2603:10b6:510:2cd::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.17 via Frontend Transport; Sat, 6 Jan 2024 02:26:26 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by SN1PEPF0002BA4F.mail.protection.outlook.com (10.167.242.72) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.7159.9 via Frontend Transport; Sat, 6 Jan 2024 02:26:26 +0000 Received: from amd.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.34; Fri, 5 Jan 2024 20:26:24 -0600 From: Suma Hegde To: CC: , , Suma Hegde , Naveen Krishna Chatradhi Subject: [PATCH v5 10/11] platform/x86/amd/hsmp: Change devm_kzalloc() to devm_kcalloc() Date: Sat, 6 Jan 2024 02:25:31 +0000 Message-ID: <20240106022532.1746932-10-suma.hegde@amd.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20240106022532.1746932-1-suma.hegde@amd.com> References: <20240106022532.1746932-1-suma.hegde@amd.com> Precedence: bulk X-Mailing-List: platform-driver-x86@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-ClientProxiedBy: SATLEXMB04.amd.com (10.181.40.145) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SN1PEPF0002BA4F:EE_|IA1PR12MB8539:EE_ X-MS-Office365-Filtering-Correlation-Id: 44a2fbef-848a-4e61-6517-08dc0e5ee203 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: xocVcUM+/vMWapa8kqKMrdSl61MF4740jfCcSZ1Jqt3XiXfQb9Otljl4gxBbgeJ2aPqb/r4k0Nu5U0jIzWmQX0SeTJgHtQ3ZtumJQK48vGEmz1qI23RQCMkGIIq5laDqpe3aS92KjVbiNNj8mz7VUifY4e8TG/twR1KHR5Xy8prgz7ajmus09cCbo20lhxIKBh/x9nR/I3l2ABdiYioPY6DmGPc3D1wBfyT9+52+HQM8e7/MbiSTSyj3JujHGFte409xVwqrzyARY/OUlwGnpB6zeqfhsc7C4V6KiuZEthgHgY17gZPkRh2f6Zx+TGhfjo4Pi1EqsrphloFe05/9F8rQPKLzZHb2VM5jRYoGIzoeLEjzTuVbkFJiE3F+qFw0abqEjq5IQzhVlv/CwvjITYI8kI0rD1X8yag8wjA0Ym4glVtlt1PK1FnjOjqeWFB1f8cjCz0vbHIz4pYyEJdwJMWISy/mEPrp9/ljr+3lSz/xHYEBZd5NqOc6201hTdqUvaj7sDoW2B7yQ1bsMumaFyfDNqQHfWEo1Del0zMjiMiRHntplLmx1zKJsCX5JS/6mqdK6sgjSMwAcfpBu48skDbUnrN1i5dTIHvLCWI4UiJtXZ27kMyStauf+0RmR3AJctLQu9dTEl8Stghs2K8ITCTY2ruhyTSRRZAdB4WNWGRPTS/yfkqGYhhI54qHkK5hvg2HFLWcD0M2ZfDZEPQ2ermEgP7FvfrNuKHRi8hKdsYYw2EBDZM/713Qg0sYfxu4ahNNKZ/tBknoD5yvZVvamA== X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230031)(4636009)(376002)(39860400002)(346002)(136003)(396003)(230922051799003)(1800799012)(64100799003)(451199024)(186009)(82310400011)(46966006)(40470700004)(36840700001)(47076005)(1076003)(6666004)(336012)(26005)(426003)(478600001)(7696005)(70586007)(83380400001)(2906002)(16526019)(41300700001)(70206006)(54906003)(8936002)(4326008)(8676002)(6916009)(44832011)(5660300002)(316002)(2616005)(82740400003)(36860700001)(36756003)(356005)(86362001)(81166007)(40460700003)(40480700001)(36900700001);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Jan 2024 02:26:26.0574 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 44a2fbef-848a-4e61-6517-08dc0e5ee203 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: SN1PEPF0002BA4F.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA1PR12MB8539 Use the standard array allocation variant of devm memory allocation APIs. Signed-off-by: Suma Hegde Signed-off-by: Naveen Krishna Chatradhi --- Changes since v4: 1. Change sizeof(struct bin_attribute *) to sizeof(*hsmp_bin_attrs) 2. Change sizeof(struct attribute_group *) to sizeof(*hsmp_attr_grps) 3. Split some of the changes to 11th patch in this v5 series Changes since v3: New patch, based on Ilpos review comments and additional cosmetic changes. drivers/platform/x86/amd/hsmp.c | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/drivers/platform/x86/amd/hsmp.c b/drivers/platform/x86/amd/hsmp.c index 99bebb0ca5a9..ccf7cd8f98f6 100644 --- a/drivers/platform/x86/amd/hsmp.c +++ b/drivers/platform/x86/amd/hsmp.c @@ -657,8 +657,9 @@ static int hsmp_create_attr_list(struct attribute_group *attr_grp, struct bin_attribute **hsmp_bin_attrs; /* Null terminated list of attributes */ - hsmp_bin_attrs = devm_kzalloc(dev, sizeof(struct bin_attribute *) * - (NUM_HSMP_ATTRS + 1), GFP_KERNEL); + hsmp_bin_attrs = devm_kcalloc(dev, NUM_HSMP_ATTRS + 1, + sizeof(*hsmp_bin_attrs), + GFP_KERNEL); if (!hsmp_bin_attrs) return -ENOMEM; @@ -673,8 +674,9 @@ static int hsmp_create_non_acpi_sysfs_if(struct device *dev) struct attribute_group *attr_grp; u16 i; - hsmp_attr_grps = devm_kzalloc(dev, sizeof(struct attribute_group *) * - (plat_dev.num_sockets + 1), GFP_KERNEL); + hsmp_attr_grps = devm_kcalloc(dev, plat_dev.num_sockets + 1, + sizeof(*hsmp_attr_grps), + GFP_KERNEL); if (!hsmp_attr_grps) return -ENOMEM; @@ -804,8 +806,8 @@ static int hsmp_pltdrv_probe(struct platform_device *pdev) * on each probe. */ if (!plat_dev.is_probed) { - plat_dev.sock = devm_kzalloc(&pdev->dev, - (plat_dev.num_sockets * sizeof(struct hsmp_socket)), + plat_dev.sock = devm_kcalloc(&pdev->dev, plat_dev.num_sockets, + sizeof(struct hsmp_socket), GFP_KERNEL); if (!plat_dev.sock) return -ENOMEM; From patchwork Sat Jan 6 02:25:32 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Suma Hegde X-Patchwork-Id: 13512653 Received: from NAM12-DM6-obe.outbound.protection.outlook.com (mail-dm6nam12on2079.outbound.protection.outlook.com [40.107.243.79]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9583815A8 for ; Sat, 6 Jan 2024 02:26:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="q0Bs+LRN" ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=L+7MG62OTAHHZULZlb+3mQajqHdN0FDGQeHmQGM7Gq5tAZbgrK2xnE8AAGBpljIlt45cMr5QPZYD0NSkPw3M8sN8gQ5IGxWjPwT0LhW09PCfbb03CGt1MNhVhP+fHMVl51rNJ0LYmtF0U+Urux0s0h2VOVjmD+FdT8KYXdQiut93k2evV730v249jbT9xAPlvYcWme7hryC3t7LtcEk5ykWhBqZTmmASJjUcoYmi/p7BSKJ/xTUzfcW4oMha7gZBoccTC8P9xUGpP24MuZb0CIqdDpibsER8hemgPveGtAWn/FmE+osWy2sy5AYkhd/ywxTcAnlDGMYUPPM96+EAMw== 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=nKSI7O00tbADC6aIeiRZpsMRk14IN1sBAtV449z9D2U=; b=drph8DdFanNFl5gMa4K5eFrVywFI2ePkbixlPT2kGTQLG1g5nGbTfDT3p9c0HX4QSncm3JvFKRujdB4lLBR8rkmLOQjgLAXCf5aEIbsPiyAAIdqdZdDaL9Ssb4W/nLpqeAvMV18dHvQZXWrQ8XpcYNJd00RR9tQvCklGVSKLHtLSwMA0cPVUgIL/d8ObZ+7k1B/VrtZIcPvcugu1cocC0iMpZLV4kXJCrc3cVFR0WmD9w2F9V5INg/PARne/jz6wm/TJpWXOPVfwC0BRbNODRR/tfo76N7zHNAuqyquYuTPvEEvmNkqBZ8lC5y2szOlLY7HLKo0xITVC/aO0rOmLWQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=nKSI7O00tbADC6aIeiRZpsMRk14IN1sBAtV449z9D2U=; b=q0Bs+LRNx4xeG+OejKfuI1z4HtJIALHMSSjvKqf9/2MnYb8c9K7jTQgps0xwS+K8dO+hlWxeJwXTxgNHhmyHNLzac/y2Ggykvqw0x/tblmJnnP+fOPUBJXWxxfVRRrpylhDITFA7Qvo6WGD5LUN+PnoqcQ4yTbpx/ko2dCQjyr4= Received: from PH0PR07CA0030.namprd07.prod.outlook.com (2603:10b6:510:5::35) by CO6PR12MB5489.namprd12.prod.outlook.com (2603:10b6:303:139::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.17; Sat, 6 Jan 2024 02:26:28 +0000 Received: from SN1PEPF0002BA4D.namprd03.prod.outlook.com (2603:10b6:510:5:cafe::c1) by PH0PR07CA0030.outlook.office365.com (2603:10b6:510:5::35) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.17 via Frontend Transport; Sat, 6 Jan 2024 02:26:28 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by SN1PEPF0002BA4D.mail.protection.outlook.com (10.167.242.70) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.7159.9 via Frontend Transport; Sat, 6 Jan 2024 02:26:28 +0000 Received: from amd.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.34; Fri, 5 Jan 2024 20:26:25 -0600 From: Suma Hegde To: CC: , , Suma Hegde , Naveen Krishna Chatradhi Subject: [PATCH v5 11/11] platform/x86/amd/hsmp: Remove extra parenthesis and add a space Date: Sat, 6 Jan 2024 02:25:32 +0000 Message-ID: <20240106022532.1746932-11-suma.hegde@amd.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20240106022532.1746932-1-suma.hegde@amd.com> References: <20240106022532.1746932-1-suma.hegde@amd.com> Precedence: bulk X-Mailing-List: platform-driver-x86@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-ClientProxiedBy: SATLEXMB04.amd.com (10.181.40.145) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SN1PEPF0002BA4D:EE_|CO6PR12MB5489:EE_ X-MS-Office365-Filtering-Correlation-Id: 2dbb222d-8c3c-4346-4ff7-08dc0e5ee336 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: SDmCzHoghceJHdMoPPWpb07pFChhzMQ1sjcTywRUAFyjv9SrflEOnOoGcCFwAxbwJakrq16u9n7cvGmWOQ2BcL2ojoZOlpF1+PerDlmchNLSaXEYcyt4rIKuNQVTCH7bctm7FDhVfTUriCPEK+oYvXQzD3HgTOItoQtuDX2gFcSV+OaW5DDiPq/4015XVWSQKsn9Pnc3okZL2/4mjonL4kziF+ipSTG0HIU04ZoPR99punH6Ioksj7/jTvZRKxqWPT6BaSpc+8OzqOILO1eJkJYP6rDoHZkd1lw5KI3mTAFqow2vi0gRP0lSWvGG8rLRPOfdWGdAFHvyjcmDisRRn84EuHUG/inJVfp/vaQahZrP8jS2P0q5fz3GeWA8bW96NS/n/kz/0OTYEnxJF4lKNRhyLvcgGzFRdwf2WNeh0r47RUIe46Uw8LXvvG+/046QCKT9W7uh74g3yXP05SaC9knB0UwqIW4bbwwJJEO9aADPVHD3rzdY4kb08q9V5M1oDfweU1nAqFNxPuVkH5kjsnLzuIej+cPFBAeJ8fDhXoEY0a5n5wl+IO2EMqMSrJ7r+ou7b6as9eJG6XpbJBL+UanDNLngRDxO+AoDX3824p9TXjhrF8qIgMLpS96BFTYl0Mj2y6zuL1/zsM2J3Eap4Tw8nro/WQLv9qOn6m3ypio/Ynr6gWSt4FBywb6K+yIx7TwTA4RPBEcayv0GBFmnL8x9sI3Uxu/gL9PhYXdiZgSWEf/uJInotW+1aPX9M4mnkXPTuFsqRhfcaeRHRV6W9g== X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230031)(4636009)(396003)(376002)(346002)(39860400002)(136003)(230922051799003)(82310400011)(186009)(451199024)(1800799012)(64100799003)(36840700001)(46966006)(40470700004)(426003)(2616005)(336012)(26005)(1076003)(7696005)(6666004)(16526019)(44832011)(83380400001)(8676002)(316002)(8936002)(54906003)(36756003)(2906002)(4744005)(5660300002)(4326008)(478600001)(70206006)(70586007)(6916009)(82740400003)(356005)(81166007)(47076005)(86362001)(36860700001)(41300700001)(40480700001)(40460700003)(36900700001);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Jan 2024 02:26:28.0532 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 2dbb222d-8c3c-4346-4ff7-08dc0e5ee336 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: SN1PEPF0002BA4D.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO6PR12MB5489 Remove unnecessary parenthesis around hsmp_get_tbl_dram_base(). Signed-off-by: Suma Hegde Signed-off-by: Naveen Krishna Chatradhi Reviewed-by: Ilpo Järvinen --- Changes since v4: New patch, generated after splitting the 9th patch in v4 series drivers/platform/x86/amd/hsmp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/platform/x86/amd/hsmp.c b/drivers/platform/x86/amd/hsmp.c index ccf7cd8f98f6..99a34b48f78f 100644 --- a/drivers/platform/x86/amd/hsmp.c +++ b/drivers/platform/x86/amd/hsmp.c @@ -643,12 +643,12 @@ static int hsmp_init_metric_tbl_bin_attr(struct bin_attribute **hattrs, u16 sock hattrs[0] = hattr; if (plat_dev.proto_ver == HSMP_PROTO_VER6) - return (hsmp_get_tbl_dram_base(sock_ind)); + return hsmp_get_tbl_dram_base(sock_ind); else return 0; } -/* One bin sysfs for metrics table*/ +/* One bin sysfs for metrics table */ #define NUM_HSMP_ATTRS 1 static int hsmp_create_attr_list(struct attribute_group *attr_grp,