Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752822AbbEDNvC (ORCPT ); Mon, 4 May 2015 09:51:02 -0400 Received: from mail-vn0-f41.google.com ([209.85.216.41]:45823 "EHLO mail-vn0-f41.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751252AbbEDNuz (ORCPT ); Mon, 4 May 2015 09:50:55 -0400 MIME-Version: 1.0 In-Reply-To: <55477413.2020908@opensource.altera.com> References: <1430713734-12175-1-git-send-email-dinguyen@opensource.altera.com> <55477413.2020908@opensource.altera.com> Date: Mon, 4 May 2015 22:50:54 +0900 X-Google-Sender-Auth: M-JGqcyXMiPSVL1kY6Y5q2x0xhM Message-ID: Subject: Re: [PATCH] dmaengine: pl300: enable the clock to PL330 dma From: Krzysztof Kozlowski To: Dinh Nguyen Cc: Krzysztof Kozlowski , dan.j.williams@intel.com, dinh.linux@gmail.com, vinod.koul@intel.com, dmaengine@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2640 Lines: 75 2015-05-04 22:28 GMT+09:00 Dinh Nguyen : > Hi Krzystof, > > On 5/4/15 12:30 AM, Krzysztof Kozlowski wrote: >> 2015-05-04 13:28 GMT+09:00 : >>> From: Dinh Nguyen >>> >>> Turn on the clock to the PL330 DMA if there is a clock node provided. >> >> Why? There is no explanation in the patch for this important question - why? >> >> Amba bus already does this and provide a wrapper function. >> Additionally that would mess up with runtime PM and clock >> enable/disable. > > I don't see the clock for the DMA getting turned on at all, which is why > after the kernel has booted, the filesystem tries to open up a serial > port using DMA and the system hangs. The failure is seen here: > > http://arm-soc.lixom.net/bootlogs/next/next-20150504/socfpga-arm-multi_v7_defconfig.html Thanks! The amba bus and pl330 should enable the clock and then disable it after probing: static int amba_probe(struct device *dev) { ... ret = amba_get_enable_pclk(pcdev); ... I wonder why do you think it is not enabled at all? > > This only happens with the multi_v7_defconfig, because the PL330 DMA is > getting built into the kernel, while the socfpga_defconfig does not > enable the PL330. It makes sense. If pl330 driver is not enabled then necessary clocks are turned on by bootloader. Probing pl330 effectively disables the clock (if DMA is not used). > The DTS for the socfpga platform looks like this: > > pdma: pdma@ffe01000 { > compatible = "arm,pl330", "arm,primecell"; > reg = <0xffe01000 0x1000>; > interrupts = <0 104 4>, > <0 105 4>, > ... > #dma-cells = <1>; > #dma-channels = <8>; > #dma-requests = <32>; > clocks = <&l4_main_clk>; > clock-names = "apb_pclk"; > }; > > Perhaps I have the wrong designation for clock-names and the amba bus is > not able to pick up the correct clock? I have two ideas: 1. Is this really the clock for the DMA? If DMA is not used then disabling it should be OK. 2. Disabling the clock may effectively disable its parent or grandparent if there are not more users. Maybe some other driver needs these parents to be enabled? This was the issue for at least one similar error (on Exynos boards). Best regards, Krzysztof -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/