Messages in this thread | | | Date | Thu, 21 Nov 2013 17:39:03 +0200 | From | "ivan.khoronzhuk" <> | Subject | Re: [PATCH] clk: keystone: gate: don't use reserved bits |
| |
On 11/21/2013 05:35 PM, Shilimkar, Santosh wrote: > Sorry for top posting. The user-guide is not upto date.... > > State > 4:0 > R > Actual state > Current Power Domain State. > Only PSM states Off and On are “key states” that users should consider in normal run time situation. States with bit 4 = 1 are transitional states helpful for debug if PSM is stuck in those states. > > Regards, > Santosh > > ________________________________________ > From: Khoronzhuk, Ivan > Sent: Thursday, November 21, 2013 10:30 AM > To: Shilimkar, Santosh > Cc: Mike Turquette; linux-arm-kernel@lists.infradead.org; linux-kernel@vger.kernel.org; Strashko, Grygorii; Khoronzhuk, Ivan > Subject: [PATCH] clk: keystone: gate: don't use reserved bits > > According to TRM http://www.ti.com/lit/ug/sprugv4b/sprugv4b.pdf > the Power Domain Status Register (PDSTAT) has 0-1 bits for power > domain status, but PDSTAT_STATE_MASK is defined with 0x1F. In that > case we operate with reserved bits. So correct PDSTAT_STATE_MASK > to be 0x03. > > Signed-off-by: Ivan Khoronzhuk <ivan.khoronzhuk@ti.com> > --- > drivers/clk/keystone/gate.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/clk/keystone/gate.c b/drivers/clk/keystone/gate.c > index 1f333bc..995ae80 100644 > --- a/drivers/clk/keystone/gate.c > +++ b/drivers/clk/keystone/gate.c > @@ -35,7 +35,7 @@ > > #define MDSTAT_STATE_MASK 0x3f > #define MDSTAT_MCKOUT BIT(12) > -#define PDSTAT_STATE_MASK 0x1f > +#define PDSTAT_STATE_MASK 0x03 > #define MDCTL_FORCE BIT(31) > #define MDCTL_LRESET BIT(8) > #define PDCTL_NEXT BIT(0) > -- > 1.7.9.5 >
Ok
-- Regards, Ivan Khoronzhuk -- 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/
| |