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=-14.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED 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 46803C55178 for ; Wed, 21 Oct 2020 15:43:08 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id C85A22224E for ; Wed, 21 Oct 2020 15:43:07 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="T/6wmvdV" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2502640AbgJUPnH (ORCPT ); Wed, 21 Oct 2020 11:43:07 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48418 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2502639AbgJUPnF (ORCPT ); Wed, 21 Oct 2020 11:43:05 -0400 Received: from mail-ot1-x342.google.com (mail-ot1-x342.google.com [IPv6:2607:f8b0:4864:20::342]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 11E60C0613D2 for ; Wed, 21 Oct 2020 08:43:04 -0700 (PDT) Received: by mail-ot1-x342.google.com with SMTP id h62so2215758oth.9 for ; Wed, 21 Oct 2020 08:43:04 -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; bh=GRVTZ0+G5+e7Gux+3d1S4AXSA+7idOeLoopuLqwFUjs=; b=T/6wmvdVjJux3vjPa6gCiRYLDSGONafaoP0Hfv0kpZ48rFqbTg3Pr/qWh0hlL/KkFY Oh/Gt8/kOhs1KrwiNXy1MfwL54nyLeVfZk7ZqI4Ko3gIWXiL+XKisctpDz6D2VAbDHFY INL+FxemggHg6K66O6u4hMI9Sj0gFunje3+vKmfyRQ2cAc8Et7Wc9cdEMchWYpjaMyNm HkcOrNLwDANAWTz7tGP0WiRW8PzeelzC85+2VJczoAdf1eMtfyMvf2yhgqiCJRQWNmkP LXaIh8BDDSOSKxMvs4IZJTFM8rlVRwRA3gQrtgHCjdH8Xehzq8mow1SwnVOhlp5racst n60Q== 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; bh=GRVTZ0+G5+e7Gux+3d1S4AXSA+7idOeLoopuLqwFUjs=; b=CusE4qpB8xzI1lcOR2FQXCryJjR6hJGQPdR64qATJuBzgcQu6yhVfUAf0+CURYOzXy i2DgdjK7V5Q1UilJSZLVlVWRYQgVx8UcpkuFUSXjahzOJVWEou6y016AQhxpuVOH3eks ZR9tWlyRWhAyH1GPFEEa2Nljdrh35C5pc4qPQcweSB7zfAVt1cdqaY5Lw7xkJ4ZBch7t YZCmwEQQxJvhsCnJvfJq0fQNHEg9qEhntYlpin67IjvtB1tTNVt04fMY0R2hhEmt2T7j WXd+0tkEcJM7m/sH/KHkP2onHu0ey2enK3THRrwKJLJHpJjQO8Yqv4ArqgwgtO8Gz7B3 SR7w== X-Gm-Message-State: AOAM531sJzdj+Jt2Ke0pWKYKbRyalBkflim2DuRneES36BfnmhjOcRSu rHMhVOTPVrm4U8gxjXEmSLTL2A== X-Google-Smtp-Source: ABdhPJxTplvkcmM/KoCmkb4Jc8WHOFJRl8ThWa3noqUWCPbYK9ytlkapINHoqlxpCiqpGztyBSJVhA== X-Received: by 2002:a05:6830:2314:: with SMTP id u20mr3092642ote.259.1603294983226; Wed, 21 Oct 2020 08:43:03 -0700 (PDT) Received: from builder.lan (104-57-184-186.lightspeed.austtx.sbcglobal.net. [104.57.184.186]) by smtp.gmail.com with ESMTPSA id n1sm670421otr.16.2020.10.21.08.43.02 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 21 Oct 2020 08:43:02 -0700 (PDT) Date: Wed, 21 Oct 2020 10:38:01 -0500 From: Bjorn Andersson To: Vinod Koul Cc: Andy Gross , Rob Herring , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [RFT PATCH] arm64: dts: sdm845: Add iommus property to qup Message-ID: <20201021153801.GG6705@builder.lan> References: <20201020150301.3259814-1-bjorn.andersson@linaro.org> <20201021065033.GC9746@vkoul-mobl> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20201021065033.GC9746@vkoul-mobl> Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org On Wed 21 Oct 01:50 CDT 2020, Vinod Koul wrote: > Hi Bjorn, Steve, > > On 20-10-20, 08:03, Bjorn Andersson wrote: > > From: Stephen Boyd > > > > The SMMU that sits in front of the QUP needs to be programmed properly > > so that the i2c geni driver can allocate DMA descriptors. Failure to do > > this leads to faults when using devices such as an i2c touchscreen where > > the transaction is larger than 32 bytes and we use a DMA buffer. > > So I tried this and results in reboot for me when I use it in GSI > driver in RB3. I seem to already have the iommu properties [1] but values > are different > > [1]: https://git.linaro.org/people/vinod.koul/kernel.git/commit/?h=topic/gsi3-db&id=385edef08d55432a40fc9a8ae1f49248d5417a92 Thanks for the testing, I was hoping to cover the two related streams in one go - but in a discussion with Sai yesterday we concluded that it would be better to just pinpoint the one stream that we know is relevant for us. Hopefully this is the cause for the crash you're seeing as well... I will resend this in line with our discussion. Thanks, Bjorn > > > arm-smmu 15000000.iommu: Unexpected global fault, this could be serious > > arm-smmu 15000000.iommu: GFSR 0x00000002, GFSYNR0 0x00000002, GFSYNR1 0x000006c0, GFSYNR2 0x00000000 > > > > Add the right SID and mask so this works. > > > > Signed-off-by: Stephen Boyd > > [bjorn: Define for second QUP as well] > > Signed-off-by: Bjorn Andersson > > --- > > arch/arm64/boot/dts/qcom/sdm845.dtsi | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi > > index 8eb5a31346d2..7d635bc919cb 100644 > > --- a/arch/arm64/boot/dts/qcom/sdm845.dtsi > > +++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi > > @@ -811,6 +811,7 @@ qupv3_id_0: geniqup@8c0000 { > > clock-names = "m-ahb", "s-ahb"; > > clocks = <&gcc GCC_QUPV3_WRAP_0_M_AHB_CLK>, > > <&gcc GCC_QUPV3_WRAP_0_S_AHB_CLK>; > > + iommus = <&apps_smmu 0x0 0x3>; > > #address-cells = <2>; > > #size-cells = <2>; > > ranges; > > @@ -1119,6 +1120,7 @@ qupv3_id_1: geniqup@ac0000 { > > clock-names = "m-ahb", "s-ahb"; > > clocks = <&gcc GCC_QUPV3_WRAP_1_M_AHB_CLK>, > > <&gcc GCC_QUPV3_WRAP_1_S_AHB_CLK>; > > + iommus = <&apps_smmu 0x6c0 0x3>; > > #address-cells = <2>; > > #size-cells = <2>; > > ranges; > > -- > > 2.28.0 > > -- > ~Vinod