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
| ||
|
Date: Wed, 18 Aug 2010 12:02:57 +0530 From: "Gopinath, Thara" <thara@...com> To: "felipe.balbi@...ia.com" <felipe.balbi@...ia.com>, Samuel Ortiz <sameo@...ux.intel.com> CC: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, "linux-omap@...r.kernel.org" <linux-omap@...r.kernel.org>, Tony Lindgren <tony@...mide.com>, Andrew Morton <akpm@...ux-foundation.org> Subject: RE: [PATCH 2/4] mfd: twl-core: switch over to defines in twl.h >>-----Original Message----- >>From: linux-omap-owner@...r.kernel.org [mailto:linux-omap-owner@...r.kernel.org] On Behalf Of >>felipe.balbi@...ia.com >>Sent: Wednesday, August 18, 2010 11:50 AM >>To: Samuel Ortiz >>Cc: linux-kernel@...r.kernel.org; linux-omap@...r.kernel.org; Tony Lindgren; Andrew Morton; Felipe >>Balbi >>Subject: [PATCH 2/4] mfd: twl-core: switch over to defines in twl.h >> >>From: Felipe Balbi <felipe.balbi@...ia.com> >> >>use the new definitions on twl header for code >>consistency. >> >>Signed-off-by: Felipe Balbi <felipe.balbi@...ia.com> >>--- >> drivers/mfd/twl-core.c | 21 +++++++++------------ >> 1 files changed, 9 insertions(+), 12 deletions(-) >> >>diff --git a/drivers/mfd/twl-core.c b/drivers/mfd/twl-core.c >>index 720e099..9689ecf 100644 >>--- a/drivers/mfd/twl-core.c >>+++ b/drivers/mfd/twl-core.c >>@@ -202,12 +202,6 @@ >> >> /* Few power values */ >> #define R_CFG_BOOT 0x05 >>-#define R_PROTECT_KEY 0x0E >>- >>-/* access control values for R_PROTECT_KEY */ >>-#define KEY_UNLOCK1 0xce >>-#define KEY_UNLOCK2 0xec >>-#define KEY_LOCK 0x00 >> >> /* some fields in R_CFG_BOOT */ >> #define HFCLK_FREQ_19p2_MHZ (1 << 0) >>@@ -846,8 +840,8 @@ static inline int __init protect_pm_master(void) >> { >> int e = 0; >> >>- e = twl_i2c_write_u8(TWL_MODULE_PM_MASTER, KEY_LOCK, >>- R_PROTECT_KEY); >>+ e = twl_i2c_write_u8(TWL4030_MODULE_PM_MASTER, 0, >>+ TWL4030_PM_MASTER_PROTECT_KEY); Hello Felipe, R_PROTECT_KEY offset is 0xE where as the new TWL4030_PM_MASTER_PROTECT_KEY is defined as 0xd. I have not checked the trm to see which is correct. But I hope this is a conscious change. Else we will end up breaking the functionality. Also if this is a conscious change may be you should mention it in the patch description. This comment applies to all other places where R_PROTECT_KEY has been replaced with TWL4030_PM_MASTER_PROTECT_KEY in this patch as well as in patch 3/4 and patch 4/4. Regards Thara >> return e; >> } >> >>@@ -855,10 +849,13 @@ static inline int __init unprotect_pm_master(void) >> { >> int e = 0; >> >>- e |= twl_i2c_write_u8(TWL_MODULE_PM_MASTER, KEY_UNLOCK1, >>- R_PROTECT_KEY); >>- e |= twl_i2c_write_u8(TWL_MODULE_PM_MASTER, KEY_UNLOCK2, >>- R_PROTECT_KEY); >>+ e |= twl_i2c_write_u8(TWL4030_MODULE_PM_MASTER, >>+ TWL4030_PM_MASTER_KEY_CFG1, >>+ TWL4030_PM_MASTER_PROTECT_KEY); >>+ e |= twl_i2c_write_u8(TWL4030_MODULE_PM_MASTER, >>+ TWL4030_PM_MASTER_KEY_CFG2, >>+ TWL4030_PM_MASTER_PROTECT_KEY); >>+ >> return e; >> } >> >>-- >>1.7.2.1.6.g61bf12 >> >>-- >>To unsubscribe from this list: send the line "unsubscribe linux-omap" in >>the body of a message to majordomo@...r.kernel.org >>More majordomo info at http://vger.kernel.org/majordomo-info.html -- 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/
Powered by blists - more mailing lists