public inbox for linux-omap@vger.kernel.org
 help / color / mirror / Atom feed
From: Tony Lindgren <tony@atomide.com>
To: Kalle Jokiniemi <ext-kalle.jokiniemi@nokia.com>
Cc: linux-omap@vger.kernel.org,
	ext Peter 'p2' De Schrijver <peter.de-schrijver@nokia.com>
Subject: Re: [PATCH] Load triton2 scripts.
Date: Mon, 18 Aug 2008 16:31:44 +0300	[thread overview]
Message-ID: <20080818133143.GK17827@atomide.com> (raw)
In-Reply-To: <1218103945.6414.3.camel@kj-ubuntu>

* Kalle Jokiniemi <ext-kalle.jokiniemi@nokia.com> [080807 13:10]:
> Hi Tony,
> 
> This patch should be safe to apply now as the sys off mode pin polarity
> was fixed in the patch "Add early init for voltage controller
> configuration and off mode polarity" (index a6cfc46..fc72c11 100644).
> 
> I compile & boot tested it on SDP board, on both plain master branch and
> with Jouni's workarounds. Works for me.

Hmm, just to confirm... Isn't this one already applied modified with
commits 07667f11a9864671bcec35dd0ee36e47cb3f675c and
3238fb156cf77b537b2df8d897fe62a7feab4939? Can you please check and
let me know if something still needs to be patched.

Thanks,

Tony

> 
> regards,
> Kalle
> 
> On ma, 2008-07-21 at 19:17 +0300, ext Peter 'p2' De Schrijver wrote: 
> > Signed-off-by: Peter 'p2' De Schrijver <peter.de-schrijver@nokia.com>
> > ---
> >  drivers/i2c/chips/Makefile        |    2 +-
> >  drivers/i2c/chips/twl4030-power.c |  337 +++++++++++++++++++++++++++++++++++++
> >  2 files changed, 338 insertions(+), 1 deletions(-)
> >  create mode 100644 drivers/i2c/chips/twl4030-power.c
> > 
> > diff --git a/drivers/i2c/chips/Makefile b/drivers/i2c/chips/Makefile
> > index 1f81ebd..a44e617 100644
> > --- a/drivers/i2c/chips/Makefile
> > +++ b/drivers/i2c/chips/Makefile
> > @@ -23,7 +23,7 @@ obj-$(CONFIG_GPIOEXPANDER_OMAP)	+= gpio_expander_omap.o
> >  obj-$(CONFIG_MENELAUS)		+= menelaus.o
> >  obj-$(CONFIG_SENSORS_TSL2550)	+= tsl2550.o
> >  obj-$(CONFIG_SENSORS_TSL2563)	+= tsl2563.o
> > -obj-$(CONFIG_TWL4030_CORE)	+= twl4030-core.o twl4030-pwrirq.o
> > +obj-$(CONFIG_TWL4030_CORE)	+= twl4030-core.o twl4030-pwrirq.o twl4030-power.o
> >  obj-$(CONFIG_TWL4030_GPIO)	+= twl4030-gpio.o
> >  obj-$(CONFIG_TWL4030_USB)	+= twl4030-usb.o
> >  obj-$(CONFIG_TWL4030_POWEROFF)	+= twl4030-poweroff.o
> > diff --git a/drivers/i2c/chips/twl4030-power.c b/drivers/i2c/chips/twl4030-power.c
> > new file mode 100644
> > index 0000000..195c3c4
> > --- /dev/null
> > +++ b/drivers/i2c/chips/twl4030-power.c
> > @@ -0,0 +1,337 @@
> > +/*
> > + * linux/drivers/i2c/chips/twl4030-power.c
> > + *
> > + * Handle TWL4030 Power initialization
> > + *
> > + * Copyright (C) 2008 Nokia Corporation
> > + * Copyright (C) 2006 Texas Instruments, Inc
> > + *
> > + * Written by 	Kalle Jokiniemi
> > + *		Peter De Schrijver <peter.de-schrijver@nokia.com>
> > + *
> > + * This file is subject to the terms and conditions of the GNU General
> > + * Public License. See the file "COPYING" in the main directory of this
> > + * archive for more details.
> > + *
> > + * This program is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> > + * GNU General Public License for more details.
> > + *
> > + * You should have received a copy of the GNU General Public License
> > + * along with this program; if not, write to the Free Software
> > + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
> > + */
> > +
> > +#include <linux/module.h>
> > +#include <linux/pm.h>
> > +#include <linux/i2c/twl4030.h>
> > +
> > +#define PWR_P1_SW_EVENTS	0x10
> > +#define PWR_DEVOFF	(1<<0)
> > +
> > +#define PHY_TO_OFF_PM_MASTER(p)		(p - 0x36)
> > +#define PHY_TO_OFF_PM_RECIEVER(p)	(p - 0x5b)
> > +
> > +/* resource - hfclk */
> > +#define R_HFCLKOUT_DEV_GRP 	PHY_TO_OFF_PM_RECIEVER(0xe6)
> > +
> > +/* PM events */
> > +#define R_P1_SW_EVENTS		PHY_TO_OFF_PM_MASTER(0x46)
> > +#define R_P2_SW_EVENTS		PHY_TO_OFF_PM_MASTER(0x47)
> > +#define R_P3_SW_EVENTS		PHY_TO_OFF_PM_MASTER(0x48)
> > +#define R_CFG_P1_TRANSITION	PHY_TO_OFF_PM_MASTER(0x36)
> > +#define R_CFG_P2_TRANSITION	PHY_TO_OFF_PM_MASTER(0x37)
> > +#define R_CFG_P3_TRANSITION	PHY_TO_OFF_PM_MASTER(0x38)
> > +
> > +#define LVL_WAKEUP	0x08
> > +
> > +#define ENABLE_WARMRESET (1<<4)
> > +
> > +/* sequence script */
> > +
> > +#define END_OF_SCRIPT		0x3f
> > +
> > +#define R_SEQ_ADD_A2S		PHY_TO_OFF_PM_MASTER(0x55)
> > +#define R_SEQ_ADD_SA12		PHY_TO_OFF_PM_MASTER(0x56)
> > +#define	R_SEQ_ADD_S2A3		PHY_TO_OFF_PM_MASTER(0x57)
> > +#define	R_SEQ_ADD_WARM		PHY_TO_OFF_PM_MASTER(0x58)
> > +#define R_MEMORY_ADDRESS	PHY_TO_OFF_PM_MASTER(0x59)
> > +#define R_MEMORY_DATA		PHY_TO_OFF_PM_MASTER(0x5a)
> > +
> > +/* Power bus message definitions */
> > +
> > +#define DEV_GRP_NULL		0x0
> > +#define DEV_GRP_P1		0x1
> > +#define DEV_GRP_P2		0x2
> > +#define DEV_GRP_P3		0x4
> > +
> > +#define RES_GRP_RES		0x0
> > +#define RES_GRP_PP		0x1
> > +#define RES_GRP_RC		0x2
> > +#define RES_GRP_PP_RC		0x3
> > +#define RES_GRP_PR		0x4
> > +#define RES_GRP_PP_PR		0x5
> > +#define RES_GRP_RC_PR		0x6
> > +#define RES_GRP_ALL		0x7
> > +
> > +#define RES_TYPE2_R0		0x0
> > +
> > +#define RES_TYPE_ALL		0x7
> > +
> > +#define RES_STATE_WRST		0xF
> > +#define RES_STATE_ACTIVE	0xE
> > +#define RES_STATE_SLEEP		0x8
> > +#define RES_STATE_OFF		0x0
> > +
> > +/*
> > +*	Power Bus Message Format
> > +*
> > +*	Broadcast Message (16 Bits)
> > +*	DEV_GRP[15:13] MT[12]  RES_GRP[11:9]  RES_TYPE2[8:7] RES_TYPE[6:4]
> > +*	RES_STATE[3:0]
> > +*
> > +*	Singular Message (16 Bits)
> > +*	DEV_GRP[15:13] MT[12]  RES_ID[11:4]  RES_STATE[3:0]
> > +*
> > +*/
> > +
> > +#define MSG_BROADCAST(devgrp, grp, type, type2, state) \
> > +	(devgrp << 13 | 1 << 12 | grp << 9 | type2 << 7 | type << 4 | state)
> > +
> > +#define MSG_SINGULAR(devgrp, id, state) \
> > +	(devgrp << 13 | 0 << 12 | id << 4 | state)
> > +
> > +#define R_PROTECT_KEY		0x0E
> > +#define KEY_1			0xC0
> > +#define KEY_2			0x0C
> > +
> > +struct triton_ins {
> > +	u16 pmb_message;
> > +	u8 delay;
> > +};
> > +
> > +
> > +#define CONFIG_DISABLE_HFCLK	1
> > +
> > +#if defined(CONFIG_MACH_OMAP_3430SDP) || defined(CONFIG_MACH_OMAP_3430LABRADOR)
> > +
> > +struct triton_ins sleep_on_seq[] __initdata = {
> > +	{MSG_SINGULAR(DEV_GRP_P1, 0xf, RES_STATE_OFF), 4},
> > +	{MSG_SINGULAR(DEV_GRP_P1, 0x10, RES_STATE_OFF), 2},
> > +#ifdef CONFIG_DISABLE_HFCLK
> > +	{MSG_SINGULAR(DEV_GRP_P1, 0x7, RES_STATE_OFF), 3},
> > +	{MSG_SINGULAR(DEV_GRP_P1, 0x19, RES_STATE_OFF), 3},
> > +#endif /* #ifdef CONFIG_DISABLE_HFCLK */
> > +};
> > +
> > +struct triton_ins sleep_off_seq[] __initdata = {
> > +#ifndef CONFIG_DISABLE_HFCLK
> > +	{MSG_SINGULAR(DEV_GRP_P1, 0xf, RES_STATE_ACTIVE), 4},
> > +	{MSG_SINGULAR(DEV_GRP_P1, 0x10, RES_STATE_ACTIVE), 2},
> > +#else
> > +	{MSG_SINGULAR(DEV_GRP_P1, 0x7, RES_STATE_ACTIVE), 0x30},
> > +	{MSG_SINGULAR(DEV_GRP_P1, 0xf, RES_STATE_ACTIVE), 0x30},
> > +	{MSG_SINGULAR(DEV_GRP_P1, 0x10, RES_STATE_ACTIVE), 0x37},
> > +	{MSG_SINGULAR(DEV_GRP_P1, 0x19, RES_STATE_ACTIVE), 3},
> > +#endif /* #ifndef CONFIG_DISABLE_HFCLK */
> > +};
> > +
> > +struct triton_ins t2_wrst_seq[] __initdata = {
> > +	{MSG_SINGULAR(DEV_GRP_NULL, 0x1b, RES_STATE_OFF), 2},
> > +	{MSG_SINGULAR(DEV_GRP_P1, 0xf, RES_STATE_WRST), 15},
> > +	{MSG_SINGULAR(DEV_GRP_P1, 0x10, RES_STATE_WRST), 15},
> > +	{MSG_SINGULAR(DEV_GRP_P1, 0x7, RES_STATE_WRST), 0x60},
> > +	{MSG_SINGULAR(DEV_GRP_P1, 0x19, RES_STATE_ACTIVE), 2},
> > +	{MSG_SINGULAR(DEV_GRP_NULL, 0x1b, RES_STATE_ACTIVE), 2},
> > +};
> > +#else
> > +struct triton_ins sleep_on_seq[] __initdata = {
> > +	{MSG_BROADCAST(DEV_GRP_NULL, RES_GRP_RC, RES_TYPE_ALL, RES_TYPE2_R0,
> > +			RES_STATE_SLEEP), 4},
> > +	{MSG_BROADCAST(DEV_GRP_NULL, RES_GRP_ALL, RES_TYPE_ALL, RES_TYPE_R7,
> > +			RES_STATE_SLEEP), 4},
> > +};
> > +
> > +struct triton_ins sleep_off_seq[] __initdata = {
> > +	{MSG_SINGULAR(DEV_GRP_NULL, 0x17, RES_STATE_ACTIVE), 0x30},
> > +	{MSG_BROADCAST(DEV_GRP_NULL, RES_GRP_PP_PR, RES_TYPE_ALL, RES_TYPE2_R0,
> > +			RES_STATE_ACTIVE), 0x37},
> > +	{MSG_BROADCAST(DEV_GRP_NULL, RES_GRP_ALL, RES_TYPE_ALL, RES_TYPE2_R0,
> > +			RES_STATE_ACTIVE), 0x2},
> > +};
> > +#endif
> > +
> > +static int __init twl4030_write_script_byte(u8 address, u8 byte)
> > +{
> > +	int err;
> > +
> > +	err = twl4030_i2c_write_u8(TWL4030_MODULE_PM_MASTER, address,
> > +					R_MEMORY_ADDRESS);
> > +	err |= twl4030_i2c_write_u8(TWL4030_MODULE_PM_MASTER, byte,
> > +					R_MEMORY_DATA);
> > +
> > +	return err;
> > +}
> > +
> > +static int __init twl4030_write_script_ins(u8 address, u16 pmb_message,
> > +						u8 delay, u8 next)
> > +{
> > +	int err = 0;
> > +
> > +	address *= 4;
> > +	err |= twl4030_write_script_byte(address++, pmb_message >> 8);
> > +	err |= twl4030_write_script_byte(address++, pmb_message & 0xff);
> > +	err |= twl4030_write_script_byte(address++, delay);
> > +	err |= twl4030_write_script_byte(address++, next);
> > +
> > +	return err;
> > +}
> > +
> > +static int __init twl4030_write_script(u8 address, struct triton_ins *script,
> > +					int len)
> > +{
> > +	int err = 0;
> > +
> > +	for (; len; len--, address++, script++) {
> > +		if (len == 1)
> > +			err |= twl4030_write_script_ins(address,
> > +							script->pmb_message,
> > +							script->delay,
> > +							END_OF_SCRIPT);
> > +		else
> > +			err |= twl4030_write_script_ins(address,
> > +							script->pmb_message,
> > +							script->delay,
> > +							address + 1);
> > +	}
> > +
> > +	return err;
> > +}
> > +
> > +static int __init config_sleep_wake_sequence(void)
> > +{
> > +	int err = 0;
> > +	u8 data;
> > +
> > +	/*
> > +	 * CLKREQ is pulled high on the 2430SDP, therefore, we need to take
> > +	 * it out of the HFCLKOUT DEV_GRP for P1 else HFCLKOUT can't be stopped.
> > +	 */
> > +
> > +	err |= twl4030_i2c_write_u8(TWL4030_MODULE_PM_RECEIVER,
> > +				  0x20, R_HFCLKOUT_DEV_GRP);
> > +
> > +	/* Set ACTIVE to SLEEP SEQ address in T2 memory*/
> > +	err |= twl4030_i2c_write_u8(TWL4030_MODULE_PM_MASTER, 0x2B,
> > +				  R_SEQ_ADD_A2S);
> > +
> > +	/* Set SLEEP to ACTIVE SEQ address for P1 and P2 */
> > +	err |= twl4030_i2c_write_u8(TWL4030_MODULE_PM_MASTER, 0x2F,
> > +				  R_SEQ_ADD_SA12);
> > +
> > +	/* Set SLEEP to ACTIVE SEQ address for P3 */
> > +	err |= twl4030_i2c_write_u8(TWL4030_MODULE_PM_MASTER, 0x2F,
> > +				  R_SEQ_ADD_S2A3);
> > +
> > +	/* Install Active->Sleep (A2S) sequence */
> > +	err |= twl4030_write_script(0x2B, sleep_on_seq,
> > +					ARRAY_SIZE(sleep_on_seq));
> > +
> > +	/* Install Sleep->Active (S2A) sequence */
> > +	err |= twl4030_write_script(0x2F, sleep_off_seq,
> > +					ARRAY_SIZE(sleep_off_seq));
> > +
> > +#if defined(CONFIG_MACH_OMAP_3430SDP) || defined(CONFIG_MACH_OMAP_3430LABRADOR)
> > +	/* Disabling AC charger effect on sleep-active transitions */
> > +	err |= twl4030_i2c_read_u8(TWL4030_MODULE_PM_MASTER, &data,
> > +					R_CFG_P1_TRANSITION);
> > +	data &= 0x0;
> > +	err |= twl4030_i2c_write_u8(TWL4030_MODULE_PM_MASTER, data ,
> > +					R_CFG_P1_TRANSITION);
> > +#endif
> > +	/* P1/P2/P3 LVL_WAKEUP should be on LEVEL */
> > +	err |= twl4030_i2c_write_u8(TWL4030_MODULE_PM_MASTER, LVL_WAKEUP,
> > +					R_P1_SW_EVENTS);
> > +	err |= twl4030_i2c_write_u8(TWL4030_MODULE_PM_MASTER, LVL_WAKEUP,
> > +					R_P2_SW_EVENTS);
> > +	err |= twl4030_i2c_write_u8(TWL4030_MODULE_PM_MASTER, LVL_WAKEUP,
> > +				R_P3_SW_EVENTS);
> > +
> > +	if (err)
> > +		printk(KERN_ERR "TWL4030 sleep-wake sequence config error\n");
> > +
> > +	return err;
> > +}
> > +
> > +
> > +/* Programming the WARMRESET Sequence on TRITON */
> > +static int __init config_warmreset_sequence(void)
> > +{
> > +
> > +	int e = 0;
> > +	u8 rd_data;
> > +
> > +	if (!ARRAY_SIZE(t2_wrst_seq))
> > +		return 0;
> > +
> > +	/* Set WARM RESET SEQ address for P1 */
> > +	e |= twl4030_i2c_write_u8(TWL4030_MODULE_PM_MASTER, 0x38,
> > +				R_SEQ_ADD_WARM);
> > +
> > +	/* Install Warm Reset sequence */
> > +	e |= twl4030_write_script(0x38, t2_wrst_seq,
> > +					ARRAY_SIZE(t2_wrst_seq));
> > +
> > +	/* P1/P2/P3 enable WARMRESET */
> > +	e |= twl4030_i2c_read_u8(TWL4030_MODULE_PM_MASTER, &rd_data,
> > +				R_P1_SW_EVENTS);
> > +	rd_data |= ENABLE_WARMRESET;
> > +	e |= twl4030_i2c_write_u8(TWL4030_MODULE_PM_MASTER, rd_data,
> > +				R_P1_SW_EVENTS);
> > +
> > +	e |= twl4030_i2c_read_u8(TWL4030_MODULE_PM_MASTER, &rd_data,
> > +				R_P2_SW_EVENTS);
> > +	rd_data |= ENABLE_WARMRESET;
> > +	e |= twl4030_i2c_write_u8(TWL4030_MODULE_PM_MASTER, rd_data,
> > +				R_P2_SW_EVENTS);
> > +
> > +	e |= twl4030_i2c_read_u8(TWL4030_MODULE_PM_MASTER, &rd_data,
> > +				R_P3_SW_EVENTS);
> > +	rd_data |= ENABLE_WARMRESET;
> > +	e |= twl4030_i2c_write_u8(TWL4030_MODULE_PM_MASTER, rd_data,
> > +				R_P3_SW_EVENTS);
> > +
> > +	if (e)
> > +		printk(KERN_ERR
> > +			"TWL4030 Power Companion Warmreset seq config error\n");
> > +	return e;
> > +}
> > +
> > +static int __init twl4030_power_init(void)
> > +{
> > +	int err = 0;
> > +
> > +	err = twl4030_i2c_write_u8(TWL4030_MODULE_PM_MASTER, KEY_1,
> > +				R_PROTECT_KEY);
> > +	err |= twl4030_i2c_write_u8(TWL4030_MODULE_PM_MASTER, KEY_2,
> > +				R_PROTECT_KEY);
> > +
> > +	if (err)
> > +		return err;
> > +
> > +	err = config_sleep_wake_sequence();
> > +	if (err)
> > +		return err;
> > +
> > +	err = config_warmreset_sequence();
> > +	if (err)
> > +		return err;
> > +
> > +	err = twl4030_i2c_write_u8(TWL4030_MODULE_PM_MASTER, 0, R_PROTECT_KEY);
> > +
> > +	return err;
> > +
> > +}
> > +
> > +module_init(twl4030_power_init);

  reply	other threads:[~2008-08-18 13:31 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-21 16:17 RFC: OMAP3 SYS_OFF_MODE support Peter 'p2' De Schrijver
2008-07-21 16:17 ` [PATCH] Set correct off mode polarity Peter 'p2' De Schrijver
2008-07-21 16:17   ` [PATCH] Enable SYSOFFMODE use Peter 'p2' De Schrijver
2008-07-21 16:17     ` [PATCH] Add SYSOFFMODE option Peter 'p2' De Schrijver
2008-07-21 16:17       ` [PATCH] Load triton2 scripts Peter 'p2' De Schrijver
2008-08-07 10:12         ` Kalle Jokiniemi
2008-08-18 13:31           ` Tony Lindgren [this message]
2008-08-19  7:02             ` Kalle Jokiniemi
2008-08-19  7:13               ` Tony Lindgren
2008-07-22 12:05 ` OMAP3 SYS_OFF_MODE support TK, Pratheesh Gangadhar
2008-07-22 14:26   ` Rangasamy, Devaraj
2008-07-23 13:42     ` Peter 'p2' De Schrijver
2008-07-24  7:30       ` Rangasamy, Devaraj
2008-07-24 13:37         ` Peter 'p2' De Schrijver
2008-08-25 10:27 ` RFC: " Kalle Jokiniemi
  -- strict thread matches above, loose matches on Subject: below --
2008-08-25 10:31 [PATCH] Load triton2 scripts Kalle Jokiniemi
2008-09-11  0:08 ` Tony Lindgren
2008-09-11  3:20   ` Steve Sakoman
2008-09-11  8:26     ` Koen Kooi
2008-09-11  8:50 ` Kevin Hilman
2008-07-21 16:02 [PATCH] Set correct off mode polarity Peter 'p2' De Schrijver
2008-07-21 16:02 ` [PATCH] Enable SYSOFFMODE use Peter 'p2' De Schrijver
2008-07-21 16:02   ` [PATCH] Add SYSOFFMODE option Peter 'p2' De Schrijver
2008-07-21 16:02     ` [PATCH] Load triton2 scripts Peter 'p2' De Schrijver
2008-07-21 17:32       ` Felipe Balbi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20080818133143.GK17827@atomide.com \
    --to=tony@atomide.com \
    --cc=ext-kalle.jokiniemi@nokia.com \
    --cc=linux-omap@vger.kernel.org \
    --cc=peter.de-schrijver@nokia.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox