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 X-Spam-Level: X-Spam-Status: No, score=-8.5 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_PASS,URIBL_BLOCKED,USER_AGENT_MUTT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4D6BFC43381 for ; Thu, 28 Mar 2019 20:42:04 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 1D0CA20811 for ; Thu, 28 Mar 2019 20:42:04 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="uIkQUjey"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="tXOVtHwD" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1D0CA20811 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linaro.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=tS2Lda8TdxU1iayuiveVDChIS3lgQ4j4nQeNoyeL8DM=; b=uIkQUjeydFuVpC /QSgnERDLF7C5MLcKHlQ9lU1X8kDwAmsZlPYeBrDaUyKjCVSGHGIQcno3UMPlHNOVghPaeWwOUgQG kd2mw82abimsF511p5wECo2sVUt0u1e1/ghFmEzUWnXl4lUBPNWWhcCtqus9vYU78YkR84+jwd5vk 7r01fQ9ocH0515KP6fe/L/Zf6HYdnTqiKfS4OZVlKXlEniYbgJb7NYAga1pqhS7OKtgUJ4wDuIeIT dCHcCuDoFJ1MJZPzWWJxOlytdrpzL96lo19tDzB4F3FrqxKHtRYx5wtk+Z+lIPMTOJdHzn0ov8wvB PrE6NJAGlq8PJbsqoYhg==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1h9bqa-0001vP-C8; Thu, 28 Mar 2019 20:42:00 +0000 Received: from mail-pf1-x444.google.com ([2607:f8b0:4864:20::444]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1h9bqW-0001uj-Uv for linux-arm-kernel@lists.infradead.org; Thu, 28 Mar 2019 20:41:58 +0000 Received: by mail-pf1-x444.google.com with SMTP id t21so11909283pfe.2 for ; Thu, 28 Mar 2019 13:41:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=8By5OSotSuT4GJqP67uaJWemZ0EgShCc8LjQId4k3Rc=; b=tXOVtHwDPUUtgapGGO9kXhC64m6Muf9WfJvydhPhC4nkOGrxKkEx41Daatgf0ejBle IFImpN15g33BQTTAoDlV3ihA4wXCtldSyT3dy5jiluo8H+Jt2nn0OAQsEWvtIfgJHfme reHuIo62eIM9eldUVz3y5MY1L5zzXta5mIRbIg4cS5kr/u+5u8J6Cb9VOhhB162AexHW kWwt4YGW+8D1GgWEt0bUODXbzU9pYedUXKtJPvZ0quEkw1x3fY54r7Ygdu7KTkJmx6eT rL7sIgrLMkqs2sTJqFwzpRvUW+J3SVrnR7r9sleviRUy0NWp/JvA0F33aRoOa9w3lwDb UFzA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=8By5OSotSuT4GJqP67uaJWemZ0EgShCc8LjQId4k3Rc=; b=qUsjOxEnaQcyTTkcRJ5boaq9jEBmSBZI6FF4FydIc9yi6pgRJskq/UHbJM4HTuR2CI 2ylUyoT9Xu2K4IDONxZRdvIFik51r4NJRIwfJtYy88SmoR5lTXAuQnQuKYCQZvaN0BjI A7yv62BHDrybmxx8bgfVAjJDp37WRZzqbEH7he1lkIn/nClnNJrPvpoRljbiOkf5LlOp lcgC5j86feMQGXL7NQR0ZeXKtV2BQ107op3nsPGZTnJozK8m8XDA+xKkJebfd4lvqOQU UOqMpo86mIhV44OAGhzxuQKNOtr2CT00r4HQ9lmdiDSKzqzmqD+8R2sX3D0Ez0szbNVt 4YVg== X-Gm-Message-State: APjAAAU5Jpoe1dV+Dmx/Awfukpr/3NndBgVprArZ+rTi4s5uVxQcABd8 IY3b5WPFJjFgzedMe9DPZ3Mm8A== X-Google-Smtp-Source: APXvYqzuvOHSRpY6JBeyHb6ix0zBYaSPOK/RbkLGW9cxfwyk2KtngBHMB1hIV4IOfdJP6y+ZaUujKQ== X-Received: by 2002:a63:2c4a:: with SMTP id s71mr2995307pgs.373.1553805714899; Thu, 28 Mar 2019 13:41:54 -0700 (PDT) Received: from xps15 (S0106002369de4dac.cg.shawcable.net. [68.147.8.254]) by smtp.gmail.com with ESMTPSA id q11sm63621pgs.94.2019.03.28.13.41.53 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Thu, 28 Mar 2019 13:41:54 -0700 (PDT) Date: Thu, 28 Mar 2019 14:41:51 -0600 From: Mathieu Poirier To: Suzuki K Poulose Subject: Re: [PATCH 23/25] coresight: stm: ACPI support for parsing stimulus base Message-ID: <20190328204151.GA7163@xps15> References: <1553107783-3340-1-git-send-email-suzuki.poulose@arm.com> <1553107783-3340-24-git-send-email-suzuki.poulose@arm.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1553107783-3340-24-git-send-email-suzuki.poulose@arm.com> User-Agent: Mutt/1.9.4 (2018-02-28) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190328_134157_014038_33C6B654 X-CRM114-Status: GOOD ( 18.59 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: coresight@lists.linaro.org, "Rafael J. Wysocki" , linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org, robert.walker@arm.com, linux-arm-kernel@lists.infradead.org, mike.leach@linaro.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, Mar 20, 2019 at 06:49:40PM +0000, Suzuki K Poulose wrote: > The stimulus base for STM device must be listed as the second memory > resource, followed by the programming base address. Add support for > parsing the information for ACPI. > > Cc: Mathieu Poirier > Cc: "Rafael J. Wysocki" > Signed-off-by: Suzuki K Poulose > --- > drivers/hwtracing/coresight/coresight-stm.c | 43 +++++++++++++++++++++++++++++ > 1 file changed, 43 insertions(+) > > diff --git a/drivers/hwtracing/coresight/coresight-stm.c b/drivers/hwtracing/coresight/coresight-stm.c > index d94ae22..995443a 100644 > --- a/drivers/hwtracing/coresight/coresight-stm.c > +++ b/drivers/hwtracing/coresight/coresight-stm.c > @@ -16,6 +16,7 @@ > * (C) 2015-2016 Chunyan Zhang > */ > #include > +#include > #include > #include > #include > @@ -717,10 +718,52 @@ static inline int of_stm_get_stimulus_area(struct device *dev, > } > #endif > > +#ifdef CONFIG_ACPI > +static int acpi_stm_get_stimulus_area(struct device *dev, struct resource *res) > +{ > + int rc; > + bool found_base = false; > + struct resource_entry *rent; > + LIST_HEAD(res_list); > + > + struct acpi_device *adev = ACPI_COMPANION(dev); > + > + if (!adev) > + return -ENODEV; > + rc = acpi_dev_get_resources(adev, &res_list, NULL, NULL); > + if (rc < 0) > + return rc; > + > + rc = -ENOENT; > + list_for_each_entry(rent, &res_list, node) { > + if (resource_type(rent->res) != IORESOURCE_MEM) > + continue; > + if (found_base) { > + *res = *rent->res; > + rc = 0; > + break; > + } > + > + found_base = true; Is the ACPI binding crystal clear on the fact that the second resource region has to be for stimulus ports? > + } > + > + acpi_dev_free_resource_list(&res_list); > + return rc; > +} > +#else > +static inline int acpi_stm_get_stimulus_area(struct device *dev, > + struct resource *res) > +{ > + return -ENOENT; > +} > +#endif > + > static int stm_get_stimulus_area(struct device *dev, struct resource *res) > { > if (dev->of_node) Wouldn't it be better to use is_of_node()? > return of_stm_get_stimulus_area(dev, res); > + else if (is_acpi_node(dev->fwnode) is_acpi_device_node()? > + return acpi_stm_get_stimulus_area(dev, res); > return -ENOENT; > } > > -- > 2.7.4 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel