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=-2.3 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=no 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 547FCECE587 for ; Tue, 1 Oct 2019 17:31:48 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 33AB12053B for ; Tue, 1 Oct 2019 17:31:48 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727263AbfJARbr (ORCPT ); Tue, 1 Oct 2019 13:31:47 -0400 Received: from foss.arm.com ([217.140.110.172]:55214 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727228AbfJARbr (ORCPT ); Tue, 1 Oct 2019 13:31:47 -0400 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id B2069337; Tue, 1 Oct 2019 10:31:46 -0700 (PDT) Received: from [10.1.197.116] (unknown [10.1.197.116]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id A01A13F706; Tue, 1 Oct 2019 10:31:40 -0700 (PDT) Subject: Re: [PATCHv9 2/3] arm64: dts: qcom: msm8998: Add Coresight support To: Jeffrey Hugo , Sai Prakash Ranjan Cc: Mark Rutland , Rajendra Nayak , Mathieu Poirier , Alexander Shishkin , MSM , Marc Gonzalez , lkml , Bjorn Andersson , David Brown , Andy Gross , Sibi Sankar , Leo Yan , linux-arm-kernel@lists.infradead.org References: <90114e06825e537c3aafd3de5c78743a9de6fadc.1564550873.git.saiprakash.ranjan@codeaurora.org> <16212a577339204e901cf4eefa5e82f1@codeaurora.org> From: Suzuki K Poulose Message-ID: <450756ff-f897-7825-3424-6d5645fa9db9@arm.com> Date: Tue, 1 Oct 2019 18:31:38 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.1.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-arm-msm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org On 01/10/2019 18:14, Jeffrey Hugo wrote: > On Tue, Oct 1, 2019 at 11:04 AM Sai Prakash Ranjan > wrote: >> >> On 2019-10-01 09:13, Jeffrey Hugo wrote: >>> Sai, >>> >>> This patch breaks boot on the 835 laptops. However, I haven't seen >>> the same issue on the MTP. I wonder, is coresight expected to work >>> with production fused devices? I wonder if thats the difference >>> between the laptop and MTP that is causing the issue. >>> >>> Let me know what I can do to help debug. >>> >> >> I did test on MSM8998 MTP and didn't face any issue. I am guessing this >> is the same issue which you reported regarding cpuidle? Coresight ETM > > Yes, its the same issue. Right now, I need both patches reverted to boot. > >> and cpuidle do not work well together since ETMs share the same power >> domain as CPU and they might get turned off when CPU enters idle states. >> Can you try with cpuidle.off=1 cmdline or just remove idle states from >> DT to confirm? If this is the issue, then we can try the below patch >> from Andrew Murray for ETM save and restore: >> >> https://patchwork.kernel.org/patch/11097893/ > > Is there still value in testing this if the idle states are removed, > yet the coresight nodes still cause issues? > > Funny enough, I'm using the arm64 defconfig which doesn't seem to > select CONFIG_CORESIGHT, so I'm not even sure what would be binding to > the DT devices... That looks like potentially missing Power domain support, either in the kernel or from the firmware. The Coresight components are also AMBA devices (with primecell compatible) and thus the AMBA bus layer does some probing to check the PIDRx registers to match the driver. The AMBA layer does try to get the power to the component, but someone is lying that it is powered. Suzuki 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=-2.3 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,USER_AGENT_SANE_1 autolearn=no 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 972D2ECE587 for ; Tue, 1 Oct 2019 17:32:01 +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 665192053B for ; Tue, 1 Oct 2019 17:32:01 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="iiUUUDUG" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 665192053B Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=arm.com 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-Type: Content-Transfer-Encoding:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=Mr46U5Fhbi6Oe0Y2tr6j8UmPQ7bI6f/UZIfQ9LxsmBo=; b=iiUUUDUGnDlDwTG4q9LYDrF0O CF2YqWE3v70ynw9WK9mHs1I3enuDvQLgNr84K1TWw91q1qqG9UDA2GmhO39RBhz7h+JDqvsHuYlnT 3WrcwG+Un9taXSVl6lfnVist2KOr+mXEGM8UafFY8CuwdEeUyvFf/Y4a7pj3OKxFWUd1FmZXrk1mR Qfj+YGZQsyZo/8FAfbdVeQo3UKzr0p91JOqI3KKnclMYf6toSxB/wjLrN50CB7fi39f5csswtr89y Ql/h8ceOC5cPxXiKGi5fcZ72upPyHzmEeJq6tlkRj3jmftSgy8m2kKVNr1h11u65X4HRGQ2REQMRh GoYrUBPaA==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.2 #3 (Red Hat Linux)) id 1iFM08-0005v8-6l; Tue, 01 Oct 2019 17:31:52 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.92.2 #3 (Red Hat Linux)) id 1iFM04-0005uI-VN for linux-arm-kernel@lists.infradead.org; Tue, 01 Oct 2019 17:31:50 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id B2069337; Tue, 1 Oct 2019 10:31:46 -0700 (PDT) Received: from [10.1.197.116] (unknown [10.1.197.116]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id A01A13F706; Tue, 1 Oct 2019 10:31:40 -0700 (PDT) Subject: Re: [PATCHv9 2/3] arm64: dts: qcom: msm8998: Add Coresight support To: Jeffrey Hugo , Sai Prakash Ranjan References: <90114e06825e537c3aafd3de5c78743a9de6fadc.1564550873.git.saiprakash.ranjan@codeaurora.org> <16212a577339204e901cf4eefa5e82f1@codeaurora.org> From: Suzuki K Poulose Message-ID: <450756ff-f897-7825-3424-6d5645fa9db9@arm.com> Date: Tue, 1 Oct 2019 18:31:38 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.1.0 MIME-Version: 1.0 In-Reply-To: Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20191001_103149_053316_1F0D54DF X-CRM114-Status: GOOD ( 19.52 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Mark Rutland , Rajendra Nayak , Mathieu Poirier , Marc Gonzalez , Alexander Shishkin , MSM , lkml , Bjorn Andersson , David Brown , Andy Gross , Sibi Sankar , Leo Yan , linux-arm-kernel@lists.infradead.org Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 01/10/2019 18:14, Jeffrey Hugo wrote: > On Tue, Oct 1, 2019 at 11:04 AM Sai Prakash Ranjan > wrote: >> >> On 2019-10-01 09:13, Jeffrey Hugo wrote: >>> Sai, >>> >>> This patch breaks boot on the 835 laptops. However, I haven't seen >>> the same issue on the MTP. I wonder, is coresight expected to work >>> with production fused devices? I wonder if thats the difference >>> between the laptop and MTP that is causing the issue. >>> >>> Let me know what I can do to help debug. >>> >> >> I did test on MSM8998 MTP and didn't face any issue. I am guessing this >> is the same issue which you reported regarding cpuidle? Coresight ETM > > Yes, its the same issue. Right now, I need both patches reverted to boot. > >> and cpuidle do not work well together since ETMs share the same power >> domain as CPU and they might get turned off when CPU enters idle states. >> Can you try with cpuidle.off=1 cmdline or just remove idle states from >> DT to confirm? If this is the issue, then we can try the below patch >> from Andrew Murray for ETM save and restore: >> >> https://patchwork.kernel.org/patch/11097893/ > > Is there still value in testing this if the idle states are removed, > yet the coresight nodes still cause issues? > > Funny enough, I'm using the arm64 defconfig which doesn't seem to > select CONFIG_CORESIGHT, so I'm not even sure what would be binding to > the DT devices... That looks like potentially missing Power domain support, either in the kernel or from the firmware. The Coresight components are also AMBA devices (with primecell compatible) and thus the AMBA bus layer does some probing to check the PIDRx registers to match the driver. The AMBA layer does try to get the power to the component, but someone is lying that it is powered. Suzuki _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel