Re: [PATCH 1/3] Introduce seperate IRQs for the TWL4030 power block

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Feb 14, 2008 at 09:35:32AM -0600, ext Felipe Balbi wrote:
> 
> 
> On Thu, 14 Feb 2008 17:13:39 +0200, "Peter 'p2' De Schrijver"
> <peter.de-schrijver@xxxxxxxxx> wrote:
> > This patch introduces seperate IRQs for the TWL4030 power block
> > interrupt source, similar to the IRQs for the TWL4030 GPIO block.
> > 
> > Signed-off-by: Peter 'p2' De Schrijver <peter.de-schrijver@xxxxxxxxx>
> > ---
> >  drivers/i2c/chips/Kconfig              |    4 +
> >  drivers/i2c/chips/Makefile             |    3 +-
> >  drivers/i2c/chips/twl4030_core.c       |    3 +-
> >  drivers/i2c/chips/twl4030_pwrirq.c     |  230
> > ++++++++++++++++++++++++++++++++
> >  include/asm-arm/arch-omap/board-rx51.h |    5 +-
> >  include/asm-arm/arch-omap/twl4030.h    |   10 ++
> >  6 files changed, 252 insertions(+), 3 deletions(-)
> >  create mode 100644 drivers/i2c/chips/twl4030_pwrirq.c
> > 
> > diff --git a/drivers/i2c/chips/Kconfig b/drivers/i2c/chips/Kconfig
> > index f8b4e54..fa817b7 100644
> > --- a/drivers/i2c/chips/Kconfig
> > +++ b/drivers/i2c/chips/Kconfig
> > @@ -164,6 +164,10 @@ config TWL4030_POWEROFF
> >  	bool "TWL4030 device poweroff"
> >  	depends on TWL4030_CORE
> > 
> > +config TWL4030_PWRBUTTON
> > +        bool "TWL4030 Power button Driver"
> > +	        depends on TWL4030_CORE
> > +
> >  config SENSORS_M41T00
> >  	tristate "ST M41T00 RTC chip (DEPRECATED)"
> >  	depends on PPC32
> > diff --git a/drivers/i2c/chips/Makefile b/drivers/i2c/chips/Makefile
> > index 72a053d..94d7bd8 100644
> > --- a/drivers/i2c/chips/Makefile
> > +++ b/drivers/i2c/chips/Makefile
> > @@ -17,10 +17,11 @@ obj-$(CONFIG_SENSORS_TLV320AIC23) += tlv320aic23.o
> >  obj-$(CONFIG_GPIOEXPANDER_OMAP)	+= gpio_expander_omap.o
> >  obj-$(CONFIG_MENELAUS)		+= menelaus.o
> >  obj-$(CONFIG_SENSORS_TSL2550)	+= tsl2550.o
> > -obj-$(CONFIG_TWL4030_CORE)	+= twl4030_core.o
> > +obj-$(CONFIG_TWL4030_CORE)	+= twl4030_core.o  twl4030_pwrirq.o
> >  obj-$(CONFIG_TWL4030_GPIO)	+= twl4030_gpio.o
> >  obj-$(CONFIG_TWL4030_USB)	+= twl4030_usb.o
> >  obj-$(CONFIG_TWL4030_POWEROFF)	+= twl4030_poweroff.o
> > +obj-$(CONFIG_TWL4030_PWRBUTTON) += twl4030_pwrbutton.o
> >  obj-$(CONFIG_RTC_X1205_I2C)	+= x1205.o
> > 
> >  ifeq ($(CONFIG_I2C_DEBUG_CHIP),y)
> > diff --git a/drivers/i2c/chips/twl4030_core.c
> > b/drivers/i2c/chips/twl4030_core.c
> > index e733195..24dd29f 100644
> > --- a/drivers/i2c/chips/twl4030_core.c
> > +++ b/drivers/i2c/chips/twl4030_core.c
> > @@ -678,12 +678,13 @@ static int power_companion_init(void)
> >  		case 38400000 : ctrl = HFCLK_FREQ_38p4_MHZ; break;
> >  	}
> > 
> > +#if 0
> >  	ctrl |= HIGH_PERF_SQ;
> >  	e |= unprotect_pm_master();
> >  			/* effect->MADC+USB ck en */
> >  	e |= twl4030_i2c_write_u8(TWL4030_MODULE_PM_MASTER, ctrl, R_CFG_BOOT);
> >  	e |= protect_pm_master();
> > -
> > +#endif
> >  	return e;
> >  }
> > 
> > diff --git a/drivers/i2c/chips/twl4030_pwrirq.c
> > b/drivers/i2c/chips/twl4030_pwrirq.c
> > new file mode 100644
> > index 0000000..765a6b0
> > --- /dev/null
> > +++ b/drivers/i2c/chips/twl4030_pwrirq.c
> > @@ -0,0 +1,230 @@
> > +/*
> > + * twl4030_pwrirq.c - handle power interrupts from TWL3040
> > + *
> > + * Copyright (C) 2008 Nokia Corporation
> > + *
> > + * Written by Peter De Schrijver <peter.de-schrijver@xxxxxxxxx>
> > + *
> > + * 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/kernel_stat.h>
> > +#include <linux/module.h>
> > +#include <linux/interrupt.h>
> > +#include <linux/irq.h>
> > +#include <linux/i2c.h>
> > +#include <linux/random.h>
> > +#include <linux/kthread.h>
> > +#include <asm/arch/twl4030.h>
> > +
> > +#define PWR_ISR1 0
> > +#define PWR_IMR1 1
> > +#define PWR_SIH_CTRL 7
> > +#define PWR_SIH_CTRL_COR (1<<2)
> > +
> > +static u8 twl4030_pwrirq_mask;
> > +static u8 twl4030_pwrirq_pending_unmask;
> > +
> > +static struct task_struct *twl4030_pwrirq_unmask_thread;
> > +
> > +static void twl4030_pwrirq_ack(unsigned int irq) {}
> > +
> > +static void twl4030_pwrirq_disableint(unsigned int irq) {}
> > +
> > +static void twl4030_pwrirq_enableint(unsigned int irq)
> > +{
> > +	twl4030_pwrirq_pending_unmask |= 1 << (irq - IH_TWL4030_PWRBASE);
> > +	if (twl4030_pwrirq_unmask_thread &&
> > +		twl4030_pwrirq_unmask_thread->state != TASK_RUNNING)
> > +		wake_up_process(twl4030_pwrirq_unmask_thread);
> > +}
> > +
> > +static struct irq_chip twl4030_pwrirq_chip = {
> > +	.name	= "twl4030-pwr",
> > +	.ack	= twl4030_pwrirq_ack,
> > +	.mask	= twl4030_pwrirq_disableint,
> > +	.unmask	= twl4030_pwrirq_enableint,
> > +};
> > +
> > +static void do_twl4030_pwrmodule_irq(unsigned int irq, irq_desc_t *desc)
> > +{
> > +	struct irqaction *action;
> > +	const unsigned int cpu = smp_processor_id();
> > +
> > +	desc->status |= IRQ_LEVEL;
> > +
> > +	if (!desc->depth) {
> > +		kstat_cpu(cpu).irqs[irq]++;
> > +
> > +		action = desc->action;
> > +		if (action) {
> > +			int ret;
> > +			int status = 0;
> > +			int retval = 0;
> > +
> > +			do {
> > +				ret = action->handler(irq, action->dev_id);
> > +				if (ret == IRQ_HANDLED)
> > +					status |= action->flags;
> > +				retval |= ret;
> > +				action = action->next;
> > +			} while (action);
> > +
> > +			if (status & IRQF_SAMPLE_RANDOM)
> > +				add_interrupt_randomness(irq);
> > +
> > +			if (retval != IRQ_HANDLED)
> > +				printk(KERN_ERR "ISR for TWL4030 power module"
> > +					" irq %d can't handle interrupt\n",
> > +					irq);
> > +		} else {
> > +			local_irq_disable();
> > +			twl4030_pwrirq_mask |= 1 << (irq - IH_TWL4030_PWRBASE);
> > +			local_irq_enable();
> > +			twl4030_i2c_write_u8(TWL4030_MODULE_INT,
> > +						twl4030_pwrirq_mask, PWR_IMR1);
> > +		}
> > +	}
> > +}
> > +
> > +static void do_twl4030_pwrirq(unsigned int irq, irq_desc_t *desc)
> > +{
> > +	const unsigned int cpu = smp_processor_id();
> > +
> > +	desc->status |= IRQ_LEVEL;
> > +
> > +	desc->chip->ack(irq);
> > +
> > +	if (!desc->depth) {
> > +		int ret;
> > +		int module_irq;
> > +		u8 pwr_isr;
> > +
> > +		kstat_cpu(cpu).irqs[irq]++;
> > +
> > +		ret = twl4030_i2c_read_u8(TWL4030_MODULE_INT, &pwr_isr,
> > +						PWR_ISR1);
> > +		if (ret) {
> > +			printk(KERN_WARNING
> > +				"I2C error %d while reading TWL4030"
> > +				"INT PWR_ISR1 register\n", ret);
> > +			return;
> > +		}
> > +
> > +		for (module_irq = IH_TWL4030_PWRBASE; pwr_isr != 0;
> > +			module_irq++, pwr_isr >>= 1) {
> > +			if (pwr_isr & 1) {
> > +				irq_desc_t *d = irq_desc + module_irq;
> > +
> > +				local_irq_disable();
> > +
> > +				d->handle_irq(module_irq, d);
> > +
> > +				local_irq_enable();
> > +			}
> > +		}
> > +
> > +		desc->chip->unmask(irq);
> > +	}
> > +}
> > +
> > +static int twl4030_pwrirq_thread(void *data)
> > +{
> > +	current->flags |= PF_NOFREEZE;
> > +
> > +	while (!kthread_should_stop()) {
> > +		u8 local_unmask;
> > +
> > +		local_irq_disable();
> > +		local_unmask = twl4030_pwrirq_pending_unmask;
> > +		twl4030_pwrirq_pending_unmask = 0;
> > +		local_irq_enable();
> > +
> > +		twl4030_pwrirq_mask &= ~local_unmask;
> > +
> > +		twl4030_i2c_write_u8(TWL4030_MODULE_INT, twl4030_pwrirq_mask,
> > +					PWR_IMR1);
> > +
> > +		local_irq_disable();
> > +		if (!twl4030_pwrirq_pending_unmask)
> > +			set_current_state(TASK_INTERRUPTIBLE);
> > +		local_irq_enable();
> > +
> > +		schedule();
> > +	}
> > +	set_current_state(TASK_RUNNING);
> > +	return 0;
> > +}
> > +
> > +static int __init twl4030_pwrirq_init(void)
> > +{
> > +	int i, err;
> > +	u8 val;
> > +
> > +	twl4030_pwrirq_mask = 0xff;
> > +	twl4030_pwrirq_pending_unmask = 0;
> > +
> > +	err = twl4030_i2c_write_u8(TWL4030_MODULE_INT, twl4030_pwrirq_mask,
> > +					PWR_IMR1);
> > +	if (err)
> > +		return err;
> > +
> > +	/* Enable clear on read */
> > +
> > +	err = twl4030_i2c_write_u8(TWL4030_MODULE_INT, PWR_SIH_CTRL_COR,
> > +					PWR_SIH_CTRL);
> > +	if (err)
> > +		return err;
> > +
> > +	twl4030_pwrirq_unmask_thread = kthread_create(twl4030_pwrirq_thread,
> > +		NULL, "twl4030 pwrirq");
> > +	if (!twl4030_pwrirq_unmask_thread) {
> > +		printk(KERN_ERR
> > +			"%s: could not create twl4030 pwrirq unmask thread!\n",
> > +				__FUNCTION__);
> > +		return -ENOMEM;
> > +	}
> > +
> > +	for (i = IH_TWL4030_PWRBASE; i < IH_TWL4030_PWRBASE_END; i++) {
> > +		set_irq_chip(i, &twl4030_pwrirq_chip);
> > +		set_irq_handler(i, do_twl4030_pwrmodule_irq);
> > +		set_irq_flags(i, IRQF_VALID);
> > +	}
> > +
> > +	set_irq_chained_handler(TWL4030_MODIRQ_PWR, do_twl4030_pwrirq);
> > +
> > +	return 0;
> > +}
> > +
> > +static void __exit twl4030_pwrirq_exit(void)
> > +{
> > +
> > +	int i;
> > +
> > +	set_irq_handler(TWL4030_MODIRQ_PWR, NULL);
> > +	set_irq_flags(TWL4030_MODIRQ_PWR, 0);
> > +
> > +	for (i = IH_TWL4030_PWRBASE; i < IH_TWL4030_PWRBASE_END; i++) {
> > +		set_irq_handler(i, NULL);
> > +		set_irq_flags(i, 0);
> > +	}
> > +
> > +	if (twl4030_pwrirq_unmask_thread) {
> > +		kthread_stop(twl4030_pwrirq_unmask_thread);
> > +		twl4030_pwrirq_unmask_thread = NULL;
> > +	}
> > +}
> > +
> > +subsys_initcall(twl4030_pwrirq_init);
> > +module_exit(twl4030_pwrirq_exit);
> > diff --git a/include/asm-arm/arch-omap/board-rx51.h
> > b/include/asm-arm/arch-omap/board-rx51.h
> > index c8aa586..8fe09af 100644
> > --- a/include/asm-arm/arch-omap/board-rx51.h
> > +++ b/include/asm-arm/arch-omap/board-rx51.h
> 
> I can't see this file on linux-omap tree. Could you remake the patch and
> send again ?
> 

Just ignore these patches.

Cheers,

Peter.
-
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

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux