From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id ED802C7EE24 for ; Tue, 2 May 2023 01:28:30 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229863AbjEBB22 (ORCPT ); Mon, 1 May 2023 21:28:28 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33164 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231799AbjEBB21 (ORCPT ); Mon, 1 May 2023 21:28:27 -0400 Received: from mail-oa1-x36.google.com (mail-oa1-x36.google.com [IPv6:2001:4860:4864:20::36]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C6EFF1FEE for ; Mon, 1 May 2023 18:28:25 -0700 (PDT) Received: by mail-oa1-x36.google.com with SMTP id 586e51a60fabf-1924a23ddb7so11644400fac.0 for ; Mon, 01 May 2023 18:28:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ventanamicro.com; s=google; t=1682990905; x=1685582905; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=7mVH6+bWUcuKUF/qKwIgPR0gD6lIVpcYINT4JISpC94=; b=P9fppIss2OxfjULIXvdy+dUX9w7PEVRop13j86caNUW0bBs6C3/wQSOOYO3cSMJtjc ihPHtbEIVh4by3RpqOVqh6TeH5TRE4WC6Co4o4k3dMbyB0lgaMEEGof1fwKOkTEK1glf AuFI2se8mKywLPekgTqxP320UZ0MMV0U29UiLdWUptUQubbBRYA1uinapXGRIR2IvQiD 7lNS1CY5aalCpYBAShhxplpZZSpD2DKbUWgbHpZEn8cVN9e8x5T4y2vxao2EkDWbh7pS G4QZTcE6knoEB0E870EGK119fD4LHTea7plc+p6aDdf+8XHQSeTfxb3VXfJShGGoXlmp iT3A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1682990905; x=1685582905; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=7mVH6+bWUcuKUF/qKwIgPR0gD6lIVpcYINT4JISpC94=; b=cS5/0o9suT757zmCVEZuEUshi8V8lFo/KtEeHsuD+pIewbiXmdHPcNJinT2gYe7/J/ dB9MDGSlh+73K0GlWFNji0cYKCBCnlyXemC9GptSXmAHIZzWU70DzUlNWyI06CCGTL8K cOOW7y/FHyLZs3q2YVap1EXabEmv6yut6YCgQEbq4n3dIMEHFiK0VLVGmqN5CJd2dKLL F/tI1evXauO17nIbxerRorctTx6DwFBqX4s+WpCSQtVlO1XJ6a6SAa44SW6rjAaVKxLs e6wJbYLI2GVlRciC5L+4sazdX/0sxUorQ585NyHh+ucii5piB8RS4p6j+WSO20mRvxBl GE5A== X-Gm-Message-State: AC+VfDyx4Yv4JT5ulk4Wnms5rl8yF2DWTMRt7SBFdyCSGOirY18iBWib VekGYBEf9vNSOWvpULF9T2FQeg== X-Google-Smtp-Source: ACHHUZ6GSw2x3VDmZQX5TJwpmEs67OpnyMp9mprjr4us96mcsrPP3CnY/k1sZeNeHVZjlquWpmBpGQ== X-Received: by 2002:a05:6808:8f1:b0:38e:467f:fee3 with SMTP id d17-20020a05680808f100b0038e467ffee3mr6754306oic.29.1682990904854; Mon, 01 May 2023 18:28:24 -0700 (PDT) Received: from sunil-laptop ([2409:4071:6e87:a073:45c:9455:55ad:4703]) by smtp.gmail.com with ESMTPSA id p203-20020acaf1d4000000b0038cabfcb3ccsm12099517oih.15.2023.05.01.18.28.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 01 May 2023 18:28:24 -0700 (PDT) Date: Tue, 2 May 2023 06:58:07 +0530 From: Sunil V L To: Conor Dooley Cc: linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-riscv@lists.infradead.org, linux-acpi@vger.kernel.org, linux-crypto@vger.kernel.org, platform-driver-x86@vger.kernel.org, llvm@lists.linux.dev, Jonathan Corbet , Paul Walmsley , Palmer Dabbelt , Albert Ou , Len Brown , Daniel Lezcano , Thomas Gleixner , Weili Qian , Zhou Wang , Herbert Xu , Marc Zyngier , Maximilian Luz , Hans de Goede , Mark Gross , Nathan Chancellor , Nick Desaulniers , Tom Rix , "Rafael J . Wysocki" , "David S . Miller" , "Rafael J . Wysocki" , Andrew Jones Subject: Re: [PATCH V4 13/23] RISC-V: cpufeature: Add ACPI support in riscv_fill_hwcap() Message-ID: References: <20230404182037.863533-1-sunilvl@ventanamicro.com> <20230404182037.863533-14-sunilvl@ventanamicro.com> <20230429-voucher-tutor-715fd4f6c24e@spud> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230429-voucher-tutor-715fd4f6c24e@spud> Precedence: bulk List-ID: X-Mailing-List: linux-crypto@vger.kernel.org On Sat, Apr 29, 2023 at 11:31:20AM +0100, Conor Dooley wrote: > Hey Sunil, > > On Tue, Apr 04, 2023 at 11:50:27PM +0530, Sunil V L wrote: > > > @@ -103,14 +109,36 @@ void __init riscv_fill_hwcap(void) > > > > bitmap_zero(riscv_isa, RISCV_ISA_EXT_MAX); > > > > - for_each_of_cpu_node(node) { > > + if (!acpi_disabled) { > > + status = acpi_get_table(ACPI_SIG_RHCT, 0, &rhct); > > + if (ACPI_FAILURE(status)) > > + return; > > + } > > + > > + for_each_possible_cpu(cpu) { > > unsigned long this_hwcap = 0; > > DECLARE_BITMAP(this_isa, RISCV_ISA_EXT_MAX); > > const char *temp; > > > > - if (of_property_read_string(node, "riscv,isa", &isa)) { > > - pr_warn("Unable to find \"riscv,isa\" devicetree entry\n"); > > - continue; > > + if (acpi_disabled) { > > + node = of_cpu_device_node_get(cpu); > > + if (node) { > > + rc = of_property_read_string(node, "riscv,isa", &isa); > > + of_node_put(node); > > + if (rc) { > > + pr_warn("Unable to find \"riscv,isa\" devicetree entry\n"); > > + continue; > > + } > > + } else { > > + pr_warn("Unable to find cpu node\n"); > > + continue; > > I was poking at this the last few days and went back to look at the ACPI > code again. Is there a reason we don't do early-return here? IOW: > > node = of_cpu_device_node_get(cpu); > if (!node) { > pr_warn() > continue; > } > > rc = of_property_read_string(node, "riscv,isa", &isa); > of_node_put(node); > if (rc) { > pr_warn(); > continue; > } > This looks better. Will update when I send the next revision of the series. Thank you!, Conor.