Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751717AbaG0E7j (ORCPT ); Sun, 27 Jul 2014 00:59:39 -0400 Received: from mail-vc0-f177.google.com ([209.85.220.177]:56423 "EHLO mail-vc0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751521AbaG0E7g convert rfc822-to-8bit (ORCPT ); Sun, 27 Jul 2014 00:59:36 -0400 MIME-Version: 1.0 In-Reply-To: <53D23192.4000908@rock-chips.com> References: <1405774529-26027-1-git-send-email-caesar.wang@rock-chips.com> <1405774529-26027-3-git-send-email-caesar.wang@rock-chips.com> <20140721085001.GG8843@ulmo> <53CD0E82.6030901@rock-chips.com> <20140721132723.GH15238@ulmo> <53D23192.4000908@rock-chips.com> Date: Sat, 26 Jul 2014 21:59:35 -0700 X-Google-Sender-Auth: EyKHsWW6Tkafi3CQ8kjxMLO8Muw Message-ID: Subject: Re: [PATCH v2 2/2] pwm: add this patch to support the new pwm of Rockchip SoCs From: Doug Anderson To: caesar Cc: Thierry Reding , linux-pwm@vger.kernel.org, "devicetree@vger.kernel.org" , "linux-doc@vger.kernel.org" , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org caesar, On Fri, Jul 25, 2014 at 3:29 AM, caesar wrote: > Hi Thierry, > > > > > 在 2014年07月21日 21:27, Thierry Reding 写道: >> >> On Mon, Jul 21, 2014 at 08:58:42PM +0800, caesar wrote: >> >>> 于 2014年07月21日 16:50, Thierry Reding 写道: >>>> >>>> On Sat, Jul 19, 2014 at 08:55:29PM +0800, Caesar Wang wrote: >> >> [...] >>>>> >>>>> struct rockchip_pwm_chip *pc; >>>>> struct resource *r; >>>>> int ret; >>>>> @@ -119,7 +182,10 @@ static int rockchip_pwm_probe(struct >>>>> platform_device *pdev) >>>>> return -ENOMEM; >>>>> r = platform_get_resource(pdev, IORESOURCE_MEM, 0); >>>>> - pc->base = devm_ioremap_resource(&pdev->dev, r); >>>>> + if (!strcmp(of_id->compatible, "rockchip,vop-pwm")) >>>>> + pc->base = devm_ioremap(&pdev->dev, r->start, >>>>> resource_size(r)); >>>>> + else >>>>> + pc->base = devm_ioremap_resource(&pdev->dev, r); >>>> >>>> Sorry, this still isn't an option. You really shouldn't remap I/O >>>> regions that other drivers may be using. You hinted at a shared register >>>> space during the review of the initial version. Can you provide more >>>> detail about what exactly the memory map looks like of the rk3288? Is >>>> there some kind of technical reference manual that I could look at? Or >>>> do you have a device tree extract that shows what the memory map looks >>>> like? >>>> >>>> Thierry >>> >>> Maybe,you can look at the ARM: dts: rk3288: >>> >>> https://github.com/rkchrome/kernel/blob/master/arch/arm/boot/dts/rk3288.dtsi >>> There is some lcdc and vop-pwm map address for rk3288. >>> >>> ,and you can look at the vop-introduce.pdf and vop-register.pdf in Annex. >>> >>> Maybe,I should put the vop-pwm in lcdc driver,but I don't hope do so it. >>> >>> Could you give a suggestion to solve it? Thanks. >> >> It looks like you could turn the lcdc device into an MFD device so that >> it can instantiate two devices, one for the display controller, the >> other for the PWM. Or perhaps it would even work with only a single >> child device. >> >> The device tree would become something like this: >> >> lcdc@ff930000 { >> compatible = "rockchip,rk3288-lcdc"; >> ... >> >> pwm@ff9301a0 { >> compatible = "rockchip,vop-pwm"; >> ... >> }; >> }; >> >> And your driver would do something like: >> >> static const struct resource pwm_resources[] = { >> { >> .start = 0x1a0, >> .end = 0x1af, >> .flags = IORESOURCE_MEM, >> }, >> }; >> >> static const struct mfd_cell subdevices[] = { >> { >> .name = "pwm", >> .id = 1, >> .of_compatible = "rockchip,vop-pwm", >> .num_resources = ARRAY_SIZE(pwm_resources), >> .resources = pwm_resources, >> }, >> }; >> >> static int lcdc_probe(struct platform_device *pdev) >> { >> struct resource *regs; >> ... >> >> regs = platform_get_resource(pdev, IORESOURCE_MEM, 0); >> >> ... >> >> err = mfd_add_devices(&pdev->dev, 0, subdevices, >> ARRAY_SIZE(subdevices), >> regs, NULL, NULL); >> ... >> } >> >> Thierry > > Sorry,I might a little trouble for the changes. > > The driver changes only for lcdc? If that is the case,I suddenly don't know > how to do it ? > > Maybe,I didn't say it clearly. > > lcdc0: lcdc@ff930000 | vop0pwm: pwm@ff9301a0 > reg = <0xff930000 0x10000> | reg = <0xff9301a0 0x10>; > > The lcdc has to add resource's address from 0xff930000 to 0xff93ffff. > > When the pwm driver is loading vop0pwm. the "devm_ioremap_resource()" will > be used in probe(); > > I think it will be occur a fail. because the resource [mem > 0xff9301a0-0xff9301af] has be requested by lcdc. > =>rockchip-pwm ff9301a0.pwm: can't request region for resource [mem > 0xff9301a0-0xff9301af] > > If I do the changes in pwm driver,do you have a other suggestion for it? > thanks.:-) Sorry if this is stupid (and I haven't tried it), but does "ranges" help solve this problem? AKA: lcdc@ff930000 { compatible = "rockchip,rk3288-lcdc"; reg = <0xff930000 0x10000>; #address-cells = <2>; #size-cells = <1>; ranges = <0 0xff9301a0 0x10>; ... pwm@0,0 { compatible = "rockchip,vop-pwm"; reg = <0 0 0x10>; ... }; }; Does that avoid the failure? The lcdc driver would need to call of_platform_populate() to make the PWM show up. -- 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/