linux-kernel - Re: [PATCH] power: max8925_power. Do not detect ac insert if handled by other code

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Fri, 26 Aug 2011 16:29:21 -0700
From:	Philip Rakity <prakity@...vell.com>
To:	"cbou@...l.ru" <cbou@...l.ru>,
	David Woodhouse <dwmw2@...radead.org>
CC:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Haojian Zhuang <hzhuang1@...vell.com>
Subject: Re: [PATCH] power: max8925_power.  Do not detect ac insert if
 handled by other code 


loop maintainer
On Aug 24, 2011, at 3:27 PM, Philip Rakity wrote:

> On brownstone rev 4 ac-insert detect is handled by vbus.
> 
> allow the platform code to configure the disabling of insert
> by setting no_insert_detect.
> 
> Signed-off-by: Philip Rakity <prakity@...vell.com>
> ---
> drivers/power/max8925_power.c |    8 ++++++--
> include/linux/mfd/max8925.h   |    1 +
> 2 files changed, 7 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/power/max8925_power.c b/drivers/power/max8925_power.c
> index dba07f3..fd32c08 100644
> --- a/drivers/power/max8925_power.c
> +++ b/drivers/power/max8925_power.c
> @@ -79,6 +79,7 @@ struct max8925_power_info {
> 	unsigned		topoff_threshold:2;
> 	unsigned		fast_charge:3;
> 	unsigned		no_temp_support:1;
> +	unsigned		no_insert_detect:1;
> 
> 	int (*set_charger) (int);
> };
> @@ -366,8 +367,10 @@ static __devinit int max8925_init_charger(struct max8925_chip *chip,
> 	int ret;
> 
> 	REQUEST_IRQ(MAX8925_IRQ_VCHG_DC_OVP, "ac-ovp");
> -	REQUEST_IRQ(MAX8925_IRQ_VCHG_DC_F, "ac-remove");
> -	REQUEST_IRQ(MAX8925_IRQ_VCHG_DC_R, "ac-insert");
> +	if (!info->no_insert_detect) {
> +		REQUEST_IRQ(MAX8925_IRQ_VCHG_DC_F, "ac-remove");
> +		REQUEST_IRQ(MAX8925_IRQ_VCHG_DC_R, "ac-insert");
> +	}
> 	REQUEST_IRQ(MAX8925_IRQ_VCHG_USB_OVP, "usb-ovp");
> 	REQUEST_IRQ(MAX8925_IRQ_VCHG_USB_F, "usb-remove");
> 	REQUEST_IRQ(MAX8925_IRQ_VCHG_USB_R, "usb-insert");
> @@ -482,6 +485,7 @@ static __devinit int max8925_power_probe(struct platform_device *pdev)
> 	info->fast_charge = pdata->fast_charge;
> 	info->set_charger = pdata->set_charger;
> 	info->no_temp_support = pdata->no_temp_support;
> +	info->no_insert_detect = pdata->no_insert_detect;
> 
> 	max8925_init_charger(chip, info);
> 	return 0;
> diff --git a/include/linux/mfd/max8925.h b/include/linux/mfd/max8925.h
> index 69ec8f0..e742e04 100644
> --- a/include/linux/mfd/max8925.h
> +++ b/include/linux/mfd/max8925.h
> @@ -224,6 +224,7 @@ struct max8925_power_pdata {
> 	unsigned	topoff_threshold:2;
> 	unsigned	fast_charge:3;	/* charge current */
> 	unsigned	no_temp_support:1; /* set if no temperature detect */
> +	unsigned	no_insert_detect:1; /* set if no ac insert detect */
> };
> 
> /*
> -- 
> 1.7.6
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/