"Hiremath, Vaibhav" <hvaibhav@xxxxxx> writes: >> -----Original Message----- >> From: Hilman, Kevin >> Sent: Thursday, December 01, 2011 5:41 AM >> To: Hiremath, Vaibhav >> Cc: linux-omap@xxxxxxxxxxxxxxx; tony@xxxxxxxxxxx; paul@xxxxxxxxx; linux- >> arm-kernel@xxxxxxxxxxxxxxxxxxx; Cousson, Benoit; Mohammed, Afzal >> Subject: Re: [RFC PATCH 01/11] arm:omap:am33xx: Add voltage domain data >> >> Vaibhav Hiremath <hvaibhav@xxxxxx> writes: >> >> > From: Afzal Mohammed <afzal@xxxxxx> >> > >> > Currently dummy voltage domain data is being created >> > in order to succeed boot process. >> > Nothing has been done w.r.t actual hardware (voltage control). >> > >> > Signed-off-by: Afzal Mohammed <afzal@xxxxxx> >> > Signed-off-by: Vaibhav Hiremath <hvaibhav@xxxxxx> >> >> [...] >> >> > --- /dev/null >> > +++ b/arch/arm/mach-omap2/voltagedomains33xx_data.c >> > @@ -0,0 +1,37 @@ >> > +/* >> > + * AM33XX voltage domain data >> > + * >> > + * Copyright (C) 2011 Texas Instruments, Inc. >> > + * >> > + * This program is free software; you can redistribute it and/or modify >> > + * it under the terms of the GNU General Public License version 2 as >> > + * published by the Free Software Foundation. >> > + */ >> > +#include <linux/kernel.h> >> > +#include <linux/init.h> >> > + >> > +#include "voltage.h" >> > + >> > +static struct voltagedomain am33xx_voltdm_mpu = { >> > + .name = "mpu", >> > +}; >> >> According to the TRM, there is no MPU voltage domain. Only CORE and RTC >> exist. >> >> However, looking at the data manual (SPRS717 Oct 2011), it seems that >> some packages have a separate VDD_MPU while others have a combined >> VDD_MPU and VDD_CORE. >> >> This should be clarified and documented in the changelog. >> > > > No, we do have VDD_MPU specifically documented in TRM (SPRUH73 - October2011). > > The TRM is accessible at - http://www.ti.com/lit/ug/spruh73/spruh73.pdf That link doesn't work for me, I think you meant http://www.ti.com/lit/ug/spruh73a/spruh73a.pdf which is the document I'm reading. Specifically it says: 8.1.1.10 Voltage and Power Domains The following table shows how the device core logic is partitioned into two core logic voltage domains and four power domains. The table lists which voltage and power domain a functional module belongs. Table 8-25. Core Logic Voltage and Power Domains Logic Voltage Domain Name | Module --------------------------------------------- CORE | All Core Modules RTC | RTC 8.1.1.10.1 Voltage Domains The core logic is divided into two voltage domains: VDD_CORE and VDD_RTC. However, in reading other parts of the TRM, it clearly references VDD_MPU and the data manual also references a separate VDD_MPU. My point in this comment was that since the TRM is rather unclear on this, the changelog should be more verbose, and take care to clear up any thing that's wrong or unclear in the TRM. Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html