Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752896AbdC2E6a (ORCPT ); Wed, 29 Mar 2017 00:58:30 -0400 Received: from esa1.microchip.iphmx.com ([68.232.147.91]:46275 "EHLO esa1.microchip.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752371AbdC2E63 (ORCPT ); Wed, 29 Mar 2017 00:58:29 -0400 X-Greylist: delayed 333 seconds by postgrey-1.27 at vger.kernel.org; Wed, 29 Mar 2017 00:58:29 EDT X-IronPort-AV: E=Sophos;i="5.36,239,1486450800"; d="scan'208";a="748902" From: To: , CC: , , Subject: RE: [PATCH v2 08/11] ARM: at91: pm: Tie the USB clock mask to the pmc Thread-Topic: [PATCH v2 08/11] ARM: at91: pm: Tie the USB clock mask to the pmc Thread-Index: AQHSp7U3Ud16I1ji1ESthgvhg8tfu6GrQYVw Date: Wed, 29 Mar 2017 04:54:17 +0000 Message-ID: References: <20170328111938.21297-1-alexandre.belloni@free-electrons.com> <20170328111938.21297-9-alexandre.belloni@free-electrons.com> In-Reply-To: <20170328111938.21297-9-alexandre.belloni@free-electrons.com> Accept-Language: zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.10.76.4] Content-Type: text/plain; charset="gb2312" MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id v2T4wwnE009161 Content-Length: 4005 Lines: 130 > -----Original Message----- > From: Alexandre Belloni [mailto:alexandre.belloni@free-electrons.com] > Sent: 2017??3??28?? 19:20 > To: Nicolas Ferre - M43238 > Cc: linux-kernel@vger.kernel.org; linux-arm-kernel@lists.infradead.org; Boris > Brezillon ; Wenyou Yang - A41535 > ; Alexandre Belloni electrons.com> > Subject: [PATCH v2 08/11] ARM: at91: pm: Tie the USB clock mask to the pmc > > The USB clocks mask (uhp_udp_mask) depends on the pmc. Tie it to the pmc id > instead of the SoC. > > Signed-off-by: Alexandre Belloni Acked-by: Wenyou Yang > --- > arch/arm/mach-at91/pm.c | 37 +++++++++++++++++++++++-------------- > 1 file changed, 23 insertions(+), 14 deletions(-) > > diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c index > ddf62a006635..a7c047f0d21f 100644 > --- a/arch/arm/mach-at91/pm.c > +++ b/arch/arm/mach-at91/pm.c > @@ -442,31 +442,46 @@ static void __init at91_pm_sram_init(void) > &at91_pm_suspend_in_sram, > at91_pm_suspend_in_sram_sz); } > > +struct pmc_info { > + unsigned long uhp_udp_mask; > +}; > + > +static const struct pmc_info pmc_infos[] __initconst = { > + { .uhp_udp_mask = AT91RM9200_PMC_UHP | > AT91RM9200_PMC_UDP }, > + { .uhp_udp_mask = AT91SAM926x_PMC_UHP | > AT91SAM926x_PMC_UDP }, > + { .uhp_udp_mask = AT91SAM926x_PMC_UHP }, }; > + > static const struct of_device_id atmel_pmc_ids[] __initconst = { > - { .compatible = "atmel,at91rm9200-pmc" }, > - { .compatible = "atmel,at91sam9260-pmc" }, > - { .compatible = "atmel,at91sam9g45-pmc" }, > - { .compatible = "atmel,at91sam9n12-pmc" }, > - { .compatible = "atmel,at91sam9x5-pmc" }, > - { .compatible = "atmel,sama5d3-pmc" }, > - { .compatible = "atmel,sama5d2-pmc" }, > + { .compatible = "atmel,at91rm9200-pmc", .data = &pmc_infos[0] }, > + { .compatible = "atmel,at91sam9260-pmc", .data = &pmc_infos[1] }, > + { .compatible = "atmel,at91sam9g45-pmc", .data = &pmc_infos[2] }, > + { .compatible = "atmel,at91sam9n12-pmc", .data = &pmc_infos[1] }, > + { .compatible = "atmel,at91sam9x5-pmc", .data = &pmc_infos[1] }, > + { .compatible = "atmel,sama5d3-pmc", .data = &pmc_infos[1] }, > + { .compatible = "atmel,sama5d2-pmc", .data = &pmc_infos[1] }, > { /* sentinel */ }, > }; > > static void __init at91_pm_init(void (*pm_idle)(void)) { > struct device_node *pmc_np; > + const struct of_device_id *of_id; > + const struct pmc_info *pmc; > > if (at91_cpuidle_device.dev.platform_data) > platform_device_register(&at91_cpuidle_device); > > - pmc_np = of_find_matching_node(NULL, atmel_pmc_ids); > + pmc_np = of_find_matching_node_and_match(NULL, atmel_pmc_ids, > &of_id); > pm_data.pmc = of_iomap(pmc_np, 0); > if (!pm_data.pmc) { > pr_err("AT91: PM not supported, PMC not found\n"); > return; > } > > + pmc = of_id->data; > + pm_data.uhp_udp_mask = pmc->uhp_udp_mask; > + > if (pm_idle) > arm_pm_idle = pm_idle; > > @@ -487,35 +502,29 @@ void __init at91rm9200_pm_init(void) > */ > at91_ramc_write(0, AT91_MC_SDRAMC_LPR, 0); > > - pm_data.uhp_udp_mask = AT91RM9200_PMC_UHP | > AT91RM9200_PMC_UDP; > - > at91_pm_init(at91rm9200_idle); > } > > void __init at91sam9260_pm_init(void) > { > at91_dt_ramc(); > - pm_data.uhp_udp_mask = AT91SAM926x_PMC_UHP | > AT91SAM926x_PMC_UDP; > at91_pm_init(at91sam9_idle); > } > > void __init at91sam9g45_pm_init(void) > { > at91_dt_ramc(); > - pm_data.uhp_udp_mask = AT91SAM926x_PMC_UHP; > at91_pm_init(at91sam9_idle); > } > > void __init at91sam9x5_pm_init(void) > { > at91_dt_ramc(); > - pm_data.uhp_udp_mask = AT91SAM926x_PMC_UHP | > AT91SAM926x_PMC_UDP; > at91_pm_init(at91sam9_idle); > } > > void __init sama5_pm_init(void) > { > at91_dt_ramc(); > - pm_data.uhp_udp_mask = AT91SAM926x_PMC_UHP | > AT91SAM926x_PMC_UDP; > at91_pm_init(NULL); > } > -- > 2.11.0 Best Regards, Wenyou Yang