From: Paul Kocialkowski <paul.kocialkowski@bootlin.com> To: "H. Nikolaus Schaller" <hns@goldelico.com> Cc: Robin Murphy <robin.murphy@arm.com>, Yong Wu <yong.wu@mediatek.com>, Rob Herring <robh@kernel.org>, Frank Rowand <frowand.list@gmail.com>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, linux-arm-kernel <linux-arm-kernel@lists.infradead.org>, Discussions about the Letux Kernel <letux-kernel@openphoenux.org> Subject: Re: BOG: commit 89c7cb1608ac3 ("of/device: Update dma_range_map only when dev has valid dma-ranges") seems to break Pinephone display or LCDC Date: Tue, 2 Feb 2021 14:42:52 +0100 [thread overview] Message-ID: <YBlW3ES1E5SwUOpJ@aptenodytes> (raw) In-Reply-To: <E45AE13E-6A74-43E6-A7AE-B0D425B10344@goldelico.com> [-- Attachment #1: Type: text/plain, Size: 2695 bytes --] Hi Nikolaus, On Tue 02 Feb 21, 11:50, H. Nikolaus Schaller wrote: > Hi Paul, > > > Am 02.02.2021 um 10:56 schrieb Paul Kocialkowski <paul.kocialkowski@bootlin.com>: > > > > Hi Nikolaus, > > > > On Tue 02 Feb 21, 10:18, H. Nikolaus Schaller wrote: > >> Hi, > >> since v5.11-rc6 my Pinephone display shows some moiré pattern. > >> > >> I did a bisect between v5.11-rc5 and v5.11-rc6 and it told me that > >> the commit mentioned in the subject is the reason. > >> > >> Reverting it makes the display work again and re-reverting fail again. > >> > >> IMHO it seems as if the display DMA of the pinephone (allwinner suni-a54) > >> got influenced and stopped to scan the framebuffer. > >> > >> The only dma-ranges I could find are defined here: > >> > >> arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi > >> > >> dma-ranges = <0x00000000 0x40000000 0xc0000000>; > >> > >> but I can't tell if they are "valid" or not. > >> > >> Any insights are welcome. And please direct to the right people/mailing lists > >> if they are missing. > > > > This may not be strictly the same thing, but is this patch in your tree: > > https://patchwork.kernel.org/project/linux-arm-kernel/patch/20210115175831.1184260-2-paul.kocialkowski@bootlin.com/ > > > > If not, it's worth a try to add it. > > No, it hasn't arrived in v5.11-rc6 (or linux-next) yet. > > But it fixes the issue. Great! The patch should already be on its way to the next RC. And nice to see you're active on Pinephone as well! You might remember me from the Replicant project, as I've worked a bit on the GTA04 :) > great and many thanks, > Nikolaus Cheers, Paul > > If it is, it's worth doing a revert. > > > > My understanding is like DE2 does not need a particular DMA range and has DRAM > > starting at 0x40000000 (just like the CPU) but it will map DRAM in a loop > > before and after this address. > > > > I suspect the issue shows because the pinephone has 2 GiB RAM while for other > > boards with < 2 GiB RAM, removing 0x40000000 to the DMA addresses still points > > to the same location. So IMO the MBUS dma-ranges shouldn't apply to DE2. > > I think this is already the case in dt, but the mbus driver may add it if you > > don't have that patch. > > > > I think I have a few A64 boards around, but probably not with 2 GiB RAM. > > If adding the patch doesn't help, I'll try to make a few test. > > > > Cheers! > > > > Paul > > > > -- > > Paul Kocialkowski, Bootlin > > Embedded Linux and kernel engineering > > https://bootlin.com > -- Paul Kocialkowski, Bootlin Embedded Linux and kernel engineering https://bootlin.com [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Paul Kocialkowski <paul.kocialkowski@bootlin.com> To: "H. Nikolaus Schaller" <hns@goldelico.com> Cc: Rob Herring <robh@kernel.org>, Frank Rowand <frowand.list@gmail.com>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Yong Wu <yong.wu@mediatek.com>, Discussions about the Letux Kernel <letux-kernel@openphoenux.org>, Robin Murphy <robin.murphy@arm.com>, linux-arm-kernel <linux-arm-kernel@lists.infradead.org> Subject: Re: BOG: commit 89c7cb1608ac3 ("of/device: Update dma_range_map only when dev has valid dma-ranges") seems to break Pinephone display or LCDC Date: Tue, 2 Feb 2021 14:42:52 +0100 [thread overview] Message-ID: <YBlW3ES1E5SwUOpJ@aptenodytes> (raw) In-Reply-To: <E45AE13E-6A74-43E6-A7AE-B0D425B10344@goldelico.com> [-- Attachment #1.1: Type: text/plain, Size: 2695 bytes --] Hi Nikolaus, On Tue 02 Feb 21, 11:50, H. Nikolaus Schaller wrote: > Hi Paul, > > > Am 02.02.2021 um 10:56 schrieb Paul Kocialkowski <paul.kocialkowski@bootlin.com>: > > > > Hi Nikolaus, > > > > On Tue 02 Feb 21, 10:18, H. Nikolaus Schaller wrote: > >> Hi, > >> since v5.11-rc6 my Pinephone display shows some moiré pattern. > >> > >> I did a bisect between v5.11-rc5 and v5.11-rc6 and it told me that > >> the commit mentioned in the subject is the reason. > >> > >> Reverting it makes the display work again and re-reverting fail again. > >> > >> IMHO it seems as if the display DMA of the pinephone (allwinner suni-a54) > >> got influenced and stopped to scan the framebuffer. > >> > >> The only dma-ranges I could find are defined here: > >> > >> arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi > >> > >> dma-ranges = <0x00000000 0x40000000 0xc0000000>; > >> > >> but I can't tell if they are "valid" or not. > >> > >> Any insights are welcome. And please direct to the right people/mailing lists > >> if they are missing. > > > > This may not be strictly the same thing, but is this patch in your tree: > > https://patchwork.kernel.org/project/linux-arm-kernel/patch/20210115175831.1184260-2-paul.kocialkowski@bootlin.com/ > > > > If not, it's worth a try to add it. > > No, it hasn't arrived in v5.11-rc6 (or linux-next) yet. > > But it fixes the issue. Great! The patch should already be on its way to the next RC. And nice to see you're active on Pinephone as well! You might remember me from the Replicant project, as I've worked a bit on the GTA04 :) > great and many thanks, > Nikolaus Cheers, Paul > > If it is, it's worth doing a revert. > > > > My understanding is like DE2 does not need a particular DMA range and has DRAM > > starting at 0x40000000 (just like the CPU) but it will map DRAM in a loop > > before and after this address. > > > > I suspect the issue shows because the pinephone has 2 GiB RAM while for other > > boards with < 2 GiB RAM, removing 0x40000000 to the DMA addresses still points > > to the same location. So IMO the MBUS dma-ranges shouldn't apply to DE2. > > I think this is already the case in dt, but the mbus driver may add it if you > > don't have that patch. > > > > I think I have a few A64 boards around, but probably not with 2 GiB RAM. > > If adding the patch doesn't help, I'll try to make a few test. > > > > Cheers! > > > > Paul > > > > -- > > Paul Kocialkowski, Bootlin > > Embedded Linux and kernel engineering > > https://bootlin.com > -- Paul Kocialkowski, Bootlin Embedded Linux and kernel engineering https://bootlin.com [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] [-- Attachment #2: Type: text/plain, Size: 176 bytes --] _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2021-02-02 13:46 UTC|newest] Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-02-02 9:18 BOG: commit 89c7cb1608ac3 ("of/device: Update dma_range_map only when dev has valid dma-ranges") seems to break Pinephone display or LCDC H. Nikolaus Schaller 2021-02-02 9:18 ` H. Nikolaus Schaller 2021-02-02 9:56 ` Paul Kocialkowski 2021-02-02 9:56 ` Paul Kocialkowski 2021-02-02 10:50 ` H. Nikolaus Schaller 2021-02-02 10:50 ` H. Nikolaus Schaller 2021-02-02 13:42 ` Paul Kocialkowski [this message] 2021-02-02 13:42 ` Paul Kocialkowski [not found] ` <13D0EFBB-E2A1-4BA3-9F99-CF2B85127074@goldelico.com> 2021-02-05 15:10 ` Paul Kocialkowski 2021-02-05 15:10 ` Paul Kocialkowski 2021-02-06 17:59 ` [Letux-kernel] " Andreas Kemnade 2021-02-06 17:59 ` Andreas Kemnade
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=YBlW3ES1E5SwUOpJ@aptenodytes \ --to=paul.kocialkowski@bootlin.com \ --cc=frowand.list@gmail.com \ --cc=hns@goldelico.com \ --cc=letux-kernel@openphoenux.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=robh@kernel.org \ --cc=robin.murphy@arm.com \ --cc=yong.wu@mediatek.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.