From mboxrd@z Thu Jan 1 00:00:00 1970 From: Krzysztof Kozlowski Subject: Re: [PATCH] ARM: dts: exynos5422-odroid*: remove fimd node Date: Tue, 24 Nov 2015 12:50:58 +0900 Message-ID: <5653DEA2.6060802@samsung.com> References: <1447151049-25370-1-git-send-email-m.szyprowski@samsung.com> <56428F34.7060003@samsung.com> <56531B1F.9050502@osg.samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: Received: from mailout4.w1.samsung.com ([210.118.77.14]:37889 "EHLO mailout4.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752635AbbKXDvF (ORCPT ); Mon, 23 Nov 2015 22:51:05 -0500 Received: from eucpsbgm2.samsung.com (unknown [203.254.199.245]) by mailout4.w1.samsung.com (Oracle Communications Messaging Server 7.0.5.31.0 64bit (built May 5 2014)) with ESMTP id <0NYA00IIAW11YP00@mailout4.w1.samsung.com> for linux-samsung-soc@vger.kernel.org; Tue, 24 Nov 2015 03:51:01 +0000 (GMT) In-reply-to: <56531B1F.9050502@osg.samsung.com> Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Javier Martinez Canillas , Marek Szyprowski , linux-samsung-soc@vger.kernel.org Cc: Kukjin Kim , Bartlomiej Zolnierkiewicz On 23.11.2015 22:56, Javier Martinez Canillas wrote: > Hello Krzysztof, > > On 11/10/2015 09:43 PM, Krzysztof Kozlowski wrote: > > [snip] > >> >> >> BTW, do you know why we don't have EXYNOS_IOMMU enabled in defconfig? >> Any reasons against? >> > > It was explicitly disabled by commit 6562f3bd396a ("ARM: exynos_defconfig: > Disable IOMMU support") because Exynos IOMMU support was broken and caused > a BUG on boot, the discussion of the patch is [0]. Right, now I remember. > But I just tested booting a v4.4-rc2 kernel on an Exynos5800 Peach Pi with > Exynos IOMMU enabled and the machine boots, display is working and > /sys/kernel/iommu_grups/*/devices shows that the devices were correctly > attached to an IOMMU group so things seems to have been sorted out now. > > So it seems that EXYNOS_IOMMU could be enabled again. It would be good to > give such a patch a spin at kernelci before posting IMHO though just to be > sure there are no issues remaining. Yes for enabling. No for testing only on kernelci. Booting is not a sufficient test in this case. I would expect testing also display - at least some frame buffer console on DP or HDMI (or whatever output could be generated... Xorg/Wayland would be better of course). You need it because display and camera (including complementary modules like JPEG, MFC etc) are actually the only users of Exynos IOMMU in mainline. Best regards, Krzysztof