* [PATCH 0/3] ARM: OMAP: SmartReflex driver
@ 2008-06-02 11:29 Kalle Jokiniemi
2008-06-02 12:31 ` Koen Kooi
0 siblings, 1 reply; 9+ messages in thread
From: Kalle Jokiniemi @ 2008-06-02 11:29 UTC (permalink / raw)
To: linux-omap; +Cc: ext-kalle.jokiniemi
Here is an updated version of the SmartReflex driver.
Patches 1 and 2 are same as before, but patch 3 has some new things:
- Changed the register accessing to use the prm_xxx_mod_reg{_bits}()
functions.
- Created a Kconfig entry for SmartReflex in "System type->TI OMAP
Implementations" menu.
Tested on 3430SDP.
Regards,
Kalle Jokiniemi
---
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH 0/3] ARM: OMAP: SmartReflex driver 2008-06-02 11:29 [PATCH 0/3] ARM: OMAP: SmartReflex driver Kalle Jokiniemi @ 2008-06-02 12:31 ` Koen Kooi 2008-06-02 14:58 ` TK, Pratheesh Gangadhar 0 siblings, 1 reply; 9+ messages in thread From: Koen Kooi @ 2008-06-02 12:31 UTC (permalink / raw) To: Kalle Jokiniemi; +Cc: linux-omap -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Op 2 jun 2008, om 13:29 heeft Kalle Jokiniemi het volgende geschreven: > Here is an updated version of the SmartReflex driver. > > Patches 1 and 2 are same as before, but patch 3 has some new things: > > - Changed the register accessing to use the prm_xxx_mod_reg{_bits}() > functions. > > - Created a Kconfig entry for SmartReflex in "System type->TI OMAP > Implementations" menu. > > Tested on 3430SDP. Compiles for beagleboard, but the board hangs after: echo -n 1 > /sys/power/sr_vdd1_autocomp echo -n 1 > /sys/power/sr_vdd2_autocomp But that's likely to be a beagleboard specific problem. regards, Koen -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.5 (Darwin) iD8DBQFIQ+g/MkyGM64RGpERAqwgAJ97AE4z0bIBZJs7nZcO1Npa8YkIcgCfY6KM 3Hhr4AIdmzNiozdqs1uQUgg= =rcQP -----END PGP SIGNATURE----- ^ permalink raw reply [flat|nested] 9+ messages in thread
* RE: [PATCH 0/3] ARM: OMAP: SmartReflex driver 2008-06-02 12:31 ` Koen Kooi @ 2008-06-02 14:58 ` TK, Pratheesh Gangadhar 2008-06-03 6:02 ` Kalle Jokiniemi 0 siblings, 1 reply; 9+ messages in thread From: TK, Pratheesh Gangadhar @ 2008-06-02 14:58 UTC (permalink / raw) To: Koen Kooi, Kalle Jokiniemi; +Cc: linux-omap Koen, You need to use NTargets from E-fuse for the correct operation. If E-fuse reads zero, then you are using a device without SmartReflex support or not guaranteed at least. #define CONTROL_FUSE_OPP1_VDD1 0x48002380 #define CONTROL_FUSE_OPP2_VDD1 0x48002384 #define CONTROL_FUSE_OPP3_VDD1 0x48002388 #define CONTROL_FUSE_OPP4_VDD1 0x4800238C #define CONTROL_FUSE_OPP5_VDD1 0x48002390 #define CONTROL_FUSE_OPP1_VDD2 0x48002394 #define CONTROL_FUSE_OPP2_VDD2 0x48002398 #define CONTROL_FUSE_OPP3_VDD2 0x4800239C #define CONTROL_FUSE_SR 0x480023A0 Those values present in the driver are meant for testing purposes only. Ideally in sr_enable, if current_nvalue reads zero we should return without enabling SmartReflex module. current_nvalue = sr_read_reg(sr, NVALUERECIPROCAL); if (nvalue_reciprocal == 0) { DPRINTK("OPP doesn't support SmartReflex\n"); return; } if (current_nvalue == nvalue_reciprocal) { DPRINTK("System is already at the desired voltage level\n"); return; } This driver needs modification to read NTargets from E-fuse rather than using hard coded values. Regards, Pratheesh -----Original Message----- From: linux-omap-owner@vger.kernel.org [mailto:linux-omap-owner@vger.kernel.org] On Behalf Of Koen Kooi Sent: Monday, June 02, 2008 6:02 PM To: Kalle Jokiniemi Cc: linux-omap Subject: Re: [PATCH 0/3] ARM: OMAP: SmartReflex driver -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Op 2 jun 2008, om 13:29 heeft Kalle Jokiniemi het volgende geschreven: > Here is an updated version of the SmartReflex driver. > > Patches 1 and 2 are same as before, but patch 3 has some new things: > > - Changed the register accessing to use the prm_xxx_mod_reg{_bits}() > functions. > > - Created a Kconfig entry for SmartReflex in "System type->TI OMAP > Implementations" menu. > > Tested on 3430SDP. Compiles for beagleboard, but the board hangs after: echo -n 1 > /sys/power/sr_vdd1_autocomp echo -n 1 > /sys/power/sr_vdd2_autocomp But that's likely to be a beagleboard specific problem. regards, Koen -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.5 (Darwin) iD8DBQFIQ+g/MkyGM64RGpERAqwgAJ97AE4z0bIBZJs7nZcO1Npa8YkIcgCfY6KM 3Hhr4AIdmzNiozdqs1uQUgg= =rcQP -----END PGP SIGNATURE----- -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 9+ messages in thread
* RE: [PATCH 0/3] ARM: OMAP: SmartReflex driver 2008-06-02 14:58 ` TK, Pratheesh Gangadhar @ 2008-06-03 6:02 ` Kalle Jokiniemi 2008-06-03 8:24 ` TK, Pratheesh Gangadhar 0 siblings, 1 reply; 9+ messages in thread From: Kalle Jokiniemi @ 2008-06-03 6:02 UTC (permalink / raw) To: ext TK, Pratheesh Gangadhar; +Cc: Koen Kooi, linux-omap Hi Pratheesh, On ma, 2008-06-02 at 20:28 +0530, ext TK, Pratheesh Gangadhar wrote: > Koen, > > You need to use NTargets from E-fuse for the correct operation. If E-fuse reads zero, then you are using a device without SmartReflex support or not guaranteed at least. > > #define CONTROL_FUSE_OPP1_VDD1 0x48002380 > #define CONTROL_FUSE_OPP2_VDD1 0x48002384 > #define CONTROL_FUSE_OPP3_VDD1 0x48002388 > #define CONTROL_FUSE_OPP4_VDD1 0x4800238C > #define CONTROL_FUSE_OPP5_VDD1 0x48002390 > > #define CONTROL_FUSE_OPP1_VDD2 0x48002394 > #define CONTROL_FUSE_OPP2_VDD2 0x48002398 > #define CONTROL_FUSE_OPP3_VDD2 0x4800239C > > #define CONTROL_FUSE_SR 0x480023A0 > > Those values present in the driver are meant for testing purposes only. Ideally in sr_enable, if current_nvalue reads zero we should return without enabling SmartReflex module. > > > current_nvalue = sr_read_reg(sr, NVALUERECIPROCAL); > > if (nvalue_reciprocal == 0) { > DPRINTK("OPP doesn't support SmartReflex\n"); > return; > } > > if (current_nvalue == nvalue_reciprocal) { > DPRINTK("System is already at the desired voltage level\n"); > return; > } This last if-statement is something that puzzled me while I was doing this patch-set. Shouldn't the SmartReflex be enabled even though we are at the desired voltage level at a given time. I mean, the operating conditions (temperature for one) could change over time and smartreflex will need to do it's magic in order to keep the device at a proper voltage level, right? > > This driver needs modification to read NTargets from E-fuse rather than using hard coded values. Tony, I can update this patch set, or send an additional patch that adds the efuse support. Which will you prefer? Regards, Kalle > -----Original Message----- > From: linux-omap-owner@vger.kernel.org [mailto:linux-omap-owner@vger.kernel.org] On Behalf Of Koen Kooi > Sent: Monday, June 02, 2008 6:02 PM > To: Kalle Jokiniemi > Cc: linux-omap > Subject: Re: [PATCH 0/3] ARM: OMAP: SmartReflex driver > > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA1 > > > Op 2 jun 2008, om 13:29 heeft Kalle Jokiniemi het volgende geschreven: > > > Here is an updated version of the SmartReflex driver. > > > > Patches 1 and 2 are same as before, but patch 3 has some new things: > > > > - Changed the register accessing to use the prm_xxx_mod_reg{_bits}() > > functions. > > > > - Created a Kconfig entry for SmartReflex in "System type->TI OMAP > > Implementations" menu. > > > > Tested on 3430SDP. > > Compiles for beagleboard, but the board hangs after: > > echo -n 1 > /sys/power/sr_vdd1_autocomp > echo -n 1 > /sys/power/sr_vdd2_autocomp > > But that's likely to be a beagleboard specific problem. > > regards, > > Koen > -----BEGIN PGP SIGNATURE----- > Version: GnuPG v1.4.5 (Darwin) > > iD8DBQFIQ+g/MkyGM64RGpERAqwgAJ97AE4z0bIBZJs7nZcO1Npa8YkIcgCfY6KM > 3Hhr4AIdmzNiozdqs1uQUgg= > =rcQP > -----END PGP SIGNATURE----- > -- > To unsubscribe from this list: send the line "unsubscribe linux-omap" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 9+ messages in thread
* RE: [PATCH 0/3] ARM: OMAP: SmartReflex driver 2008-06-03 6:02 ` Kalle Jokiniemi @ 2008-06-03 8:24 ` TK, Pratheesh Gangadhar 2008-06-03 15:10 ` Tony Lindgren 0 siblings, 1 reply; 9+ messages in thread From: TK, Pratheesh Gangadhar @ 2008-06-03 8:24 UTC (permalink / raw) To: Kalle Jokiniemi; +Cc: Koen Kooi, linux-omap Kalle, Yes, you are right. You can remove the second if statement. Regards, Pratheesh -----Original Message----- From: Kalle Jokiniemi [mailto:ext-kalle.jokiniemi@nokia.com] Sent: Tuesday, June 03, 2008 11:32 AM To: TK, Pratheesh Gangadhar Cc: Koen Kooi; linux-omap Subject: RE: [PATCH 0/3] ARM: OMAP: SmartReflex driver Hi Pratheesh, On ma, 2008-06-02 at 20:28 +0530, ext TK, Pratheesh Gangadhar wrote: > Koen, > > You need to use NTargets from E-fuse for the correct operation. If E-fuse reads zero, then you are using a device without SmartReflex support or not guaranteed at least. > > #define CONTROL_FUSE_OPP1_VDD1 0x48002380 > #define CONTROL_FUSE_OPP2_VDD1 0x48002384 > #define CONTROL_FUSE_OPP3_VDD1 0x48002388 > #define CONTROL_FUSE_OPP4_VDD1 0x4800238C > #define CONTROL_FUSE_OPP5_VDD1 0x48002390 > > #define CONTROL_FUSE_OPP1_VDD2 0x48002394 > #define CONTROL_FUSE_OPP2_VDD2 0x48002398 > #define CONTROL_FUSE_OPP3_VDD2 0x4800239C > > #define CONTROL_FUSE_SR 0x480023A0 > > Those values present in the driver are meant for testing purposes only. Ideally in sr_enable, if current_nvalue reads zero we should return without enabling SmartReflex module. > > > current_nvalue = sr_read_reg(sr, NVALUERECIPROCAL); > > if (nvalue_reciprocal == 0) { > DPRINTK("OPP doesn't support SmartReflex\n"); > return; > } > > if (current_nvalue == nvalue_reciprocal) { > DPRINTK("System is already at the desired voltage level\n"); > return; > } This last if-statement is something that puzzled me while I was doing this patch-set. Shouldn't the SmartReflex be enabled even though we are at the desired voltage level at a given time. I mean, the operating conditions (temperature for one) could change over time and smartreflex will need to do it's magic in order to keep the device at a proper voltage level, right? > > This driver needs modification to read NTargets from E-fuse rather than using hard coded values. Tony, I can update this patch set, or send an additional patch that adds the efuse support. Which will you prefer? Regards, Kalle > -----Original Message----- > From: linux-omap-owner@vger.kernel.org [mailto:linux-omap-owner@vger.kernel.org] On Behalf Of Koen Kooi > Sent: Monday, June 02, 2008 6:02 PM > To: Kalle Jokiniemi > Cc: linux-omap > Subject: Re: [PATCH 0/3] ARM: OMAP: SmartReflex driver > > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA1 > > > Op 2 jun 2008, om 13:29 heeft Kalle Jokiniemi het volgende geschreven: > > > Here is an updated version of the SmartReflex driver. > > > > Patches 1 and 2 are same as before, but patch 3 has some new things: > > > > - Changed the register accessing to use the prm_xxx_mod_reg{_bits}() > > functions. > > > > - Created a Kconfig entry for SmartReflex in "System type->TI OMAP > > Implementations" menu. > > > > Tested on 3430SDP. > > Compiles for beagleboard, but the board hangs after: > > echo -n 1 > /sys/power/sr_vdd1_autocomp > echo -n 1 > /sys/power/sr_vdd2_autocomp > > But that's likely to be a beagleboard specific problem. > > regards, > > Koen > -----BEGIN PGP SIGNATURE----- > Version: GnuPG v1.4.5 (Darwin) > > iD8DBQFIQ+g/MkyGM64RGpERAqwgAJ97AE4z0bIBZJs7nZcO1Npa8YkIcgCfY6KM > 3Hhr4AIdmzNiozdqs1uQUgg= > =rcQP > -----END PGP SIGNATURE----- > -- > To unsubscribe from this list: send the line "unsubscribe linux-omap" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 0/3] ARM: OMAP: SmartReflex driver 2008-06-03 8:24 ` TK, Pratheesh Gangadhar @ 2008-06-03 15:10 ` Tony Lindgren 2008-06-06 9:49 ` Kalle Jokiniemi 0 siblings, 1 reply; 9+ messages in thread From: Tony Lindgren @ 2008-06-03 15:10 UTC (permalink / raw) To: TK, Pratheesh Gangadhar; +Cc: Kalle Jokiniemi, Koen Kooi, linux-omap * TK, Pratheesh Gangadhar <pratheesh@ti.com> [080603 01:25]: > Kalle, > > Yes, you are right. You can remove the second if statement. > > Regards, > Pratheesh > > -----Original Message----- > From: Kalle Jokiniemi [mailto:ext-kalle.jokiniemi@nokia.com] > Sent: Tuesday, June 03, 2008 11:32 AM > To: TK, Pratheesh Gangadhar > Cc: Koen Kooi; linux-omap > Subject: RE: [PATCH 0/3] ARM: OMAP: SmartReflex driver > > Hi Pratheesh, > > On ma, 2008-06-02 at 20:28 +0530, ext TK, Pratheesh Gangadhar wrote: > > Koen, > > > > You need to use NTargets from E-fuse for the correct operation. If E-fuse reads zero, then you are using a device without SmartReflex support or not guaranteed at least. > > > > #define CONTROL_FUSE_OPP1_VDD1 0x48002380 > > #define CONTROL_FUSE_OPP2_VDD1 0x48002384 > > #define CONTROL_FUSE_OPP3_VDD1 0x48002388 > > #define CONTROL_FUSE_OPP4_VDD1 0x4800238C > > #define CONTROL_FUSE_OPP5_VDD1 0x48002390 > > > > #define CONTROL_FUSE_OPP1_VDD2 0x48002394 > > #define CONTROL_FUSE_OPP2_VDD2 0x48002398 > > #define CONTROL_FUSE_OPP3_VDD2 0x4800239C > > > > #define CONTROL_FUSE_SR 0x480023A0 > > > > Those values present in the driver are meant for testing purposes only. Ideally in sr_enable, if current_nvalue reads zero we should return without enabling SmartReflex module. > > > > > > current_nvalue = sr_read_reg(sr, NVALUERECIPROCAL); > > > > if (nvalue_reciprocal == 0) { > > DPRINTK("OPP doesn't support SmartReflex\n"); > > return; > > } > > > > if (current_nvalue == nvalue_reciprocal) { > > DPRINTK("System is already at the desired voltage level\n"); > > return; > > } > > This last if-statement is something that puzzled me while I was doing > this patch-set. Shouldn't the SmartReflex be enabled even though we are > at the desired voltage level at a given time. I mean, the operating > conditions (temperature for one) could change over time and smartreflex > will need to do it's magic in order to keep the device at a proper > voltage level, right? > > > > > > This driver needs modification to read NTargets from E-fuse rather than using hard coded values. > > Tony, I can update this patch set, or send an additional patch that adds > the efuse support. Which will you prefer? Well I guess updating it would be better if it's just defining them so we have a working patch rather than patch and fix cycle. Regards, Tony > > > Regards, > Kalle > > > > -----Original Message----- > > From: linux-omap-owner@vger.kernel.org [mailto:linux-omap-owner@vger.kernel.org] On Behalf Of Koen Kooi > > Sent: Monday, June 02, 2008 6:02 PM > > To: Kalle Jokiniemi > > Cc: linux-omap > > Subject: Re: [PATCH 0/3] ARM: OMAP: SmartReflex driver > > > > -----BEGIN PGP SIGNED MESSAGE----- > > Hash: SHA1 > > > > > > Op 2 jun 2008, om 13:29 heeft Kalle Jokiniemi het volgende geschreven: > > > > > Here is an updated version of the SmartReflex driver. > > > > > > Patches 1 and 2 are same as before, but patch 3 has some new things: > > > > > > - Changed the register accessing to use the prm_xxx_mod_reg{_bits}() > > > functions. > > > > > > - Created a Kconfig entry for SmartReflex in "System type->TI OMAP > > > Implementations" menu. > > > > > > Tested on 3430SDP. > > > > Compiles for beagleboard, but the board hangs after: > > > > echo -n 1 > /sys/power/sr_vdd1_autocomp > > echo -n 1 > /sys/power/sr_vdd2_autocomp > > > > But that's likely to be a beagleboard specific problem. > > > > regards, > > > > Koen > > -----BEGIN PGP SIGNATURE----- > > Version: GnuPG v1.4.5 (Darwin) > > > > iD8DBQFIQ+g/MkyGM64RGpERAqwgAJ97AE4z0bIBZJs7nZcO1Npa8YkIcgCfY6KM > > 3Hhr4AIdmzNiozdqs1uQUgg= > > =rcQP > > -----END PGP SIGNATURE----- > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-omap" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- > To unsubscribe from this list: send the line "unsubscribe linux-omap" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 0/3] ARM: OMAP: SmartReflex driver 2008-06-03 15:10 ` Tony Lindgren @ 2008-06-06 9:49 ` Kalle Jokiniemi 2008-06-06 11:01 ` Koen Kooi 0 siblings, 1 reply; 9+ messages in thread From: Kalle Jokiniemi @ 2008-06-06 9:49 UTC (permalink / raw) To: ext Tony Lindgren; +Cc: TK, Pratheesh Gangadhar, Koen Kooi, linux-omap Hi, > > Tony, I can update this patch set, or send an additional patch that adds > > the efuse support. Which will you prefer? > > Well I guess updating it would be better if it's just defining them so > we have a working patch rather than patch and fix cycle. It took a bit longer than I expected, but here's the updated smartreflex driver that uses the E-fuse values. Thanks to Pratheesh for helping out with the details of the implementation. First patch remains as it was. Some new register and regbit defs have been added to the second patch and the last patch now implements the E-fuse parameters. I also left in the software hard coded values under a "Smartreflex testing support" entry in the Kconfig. For testing purposes only. Again, tested on 3430SDP. Regards, Kalle > > Regards, > > Tony > > > > > > > > > > Regards, > > Kalle > > > > > > > -----Original Message----- > > > From: linux-omap-owner@vger.kernel.org [mailto:linux-omap-owner@vger.kernel.org] On Behalf Of Koen Kooi > > > Sent: Monday, June 02, 2008 6:02 PM > > > To: Kalle Jokiniemi > > > Cc: linux-omap > > > Subject: Re: [PATCH 0/3] ARM: OMAP: SmartReflex driver > > > > > > -----BEGIN PGP SIGNED MESSAGE----- > > > Hash: SHA1 > > > > > > > > > Op 2 jun 2008, om 13:29 heeft Kalle Jokiniemi het volgende geschreven: > > > > > > > Here is an updated version of the SmartReflex driver. > > > > > > > > Patches 1 and 2 are same as before, but patch 3 has some new things: > > > > > > > > - Changed the register accessing to use the prm_xxx_mod_reg{_bits}() > > > > functions. > > > > > > > > - Created a Kconfig entry for SmartReflex in "System type->TI OMAP > > > > Implementations" menu. > > > > > > > > Tested on 3430SDP. > > > > > > Compiles for beagleboard, but the board hangs after: > > > > > > echo -n 1 > /sys/power/sr_vdd1_autocomp > > > echo -n 1 > /sys/power/sr_vdd2_autocomp > > > > > > But that's likely to be a beagleboard specific problem. > > > > > > regards, > > > > > > Koen > > > -----BEGIN PGP SIGNATURE----- > > > Version: GnuPG v1.4.5 (Darwin) > > > > > > iD8DBQFIQ+g/MkyGM64RGpERAqwgAJ97AE4z0bIBZJs7nZcO1Npa8YkIcgCfY6KM > > > 3Hhr4AIdmzNiozdqs1uQUgg= > > > =rcQP > > > -----END PGP SIGNATURE----- > > > -- > > > To unsubscribe from this list: send the line "unsubscribe linux-omap" in > > > the body of a message to majordomo@vger.kernel.org > > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-omap" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 0/3] ARM: OMAP: SmartReflex driver 2008-06-06 9:49 ` Kalle Jokiniemi @ 2008-06-06 11:01 ` Koen Kooi 0 siblings, 0 replies; 9+ messages in thread From: Koen Kooi @ 2008-06-06 11:01 UTC (permalink / raw) To: Kalle Jokiniemi, linux-omap; +Cc: Beagle Board -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Op 6 jun 2008, om 11:49 heeft Kalle Jokiniemi het volgende geschreven: > Hi, > >>> Tony, I can update this patch set, or send an additional patch >>> that adds >>> the efuse support. Which will you prefer? >> >> Well I guess updating it would be better if it's just defining them >> so >> we have a working patch rather than patch and fix cycle. > > It took a bit longer than I expected, but here's the updated > smartreflex driver that uses the E-fuse values. Thanks to Pratheesh > for > helping out with the details of the implementation. > > First patch remains as it was. Some new register and regbit defs have > been added to the second patch and the last patch now implements the > E-fuse parameters. On my beagleboard: root@beagleboard:/sys/power# echo -n 1 > /sys/power/sr_vdd1_autocomp OPP3 doesn't support SmartReflex SR1: VDD autocomp not activated root@beagleboard:/sys/power# root@beagleboard:/sys/power# echo -n 1 > /sys/power/sr_vdd2_autocomp OPP3 doesn't support SmartReflex SR2: VDD autocomp not activated root@beagleboard:/sys/power# The board doesn't crash anymore :) regards, Koen > > > I also left in the software hard coded values under a "Smartreflex > testing support" entry in the Kconfig. For testing purposes only. > > Again, tested on 3430SDP. > > Regards, > Kalle > >> >> Regards, >> >> Tony >> >> >> >> >>> >>> >>> Regards, >>> Kalle >>> >>> >>>> -----Original Message----- >>>> From: linux-omap-owner@vger.kernel.org [mailto:linux-omap-owner@vger.kernel.org >>>> ] On Behalf Of Koen Kooi >>>> Sent: Monday, June 02, 2008 6:02 PM >>>> To: Kalle Jokiniemi >>>> Cc: linux-omap >>>> Subject: Re: [PATCH 0/3] ARM: OMAP: SmartReflex driver >>>> >>>> -----BEGIN PGP SIGNED MESSAGE----- >>>> Hash: SHA1 >>>> >>>> >>>> Op 2 jun 2008, om 13:29 heeft Kalle Jokiniemi het volgende >>>> geschreven: >>>> >>>>> Here is an updated version of the SmartReflex driver. >>>>> >>>>> Patches 1 and 2 are same as before, but patch 3 has some new >>>>> things: >>>>> >>>>> - Changed the register accessing to use the >>>>> prm_xxx_mod_reg{_bits}() >>>>> functions. >>>>> >>>>> - Created a Kconfig entry for SmartReflex in "System type->TI OMAP >>>>> Implementations" menu. >>>>> >>>>> Tested on 3430SDP. >>>> >>>> Compiles for beagleboard, but the board hangs after: >>>> >>>> echo -n 1 > /sys/power/sr_vdd1_autocomp >>>> echo -n 1 > /sys/power/sr_vdd2_autocomp >>>> >>>> But that's likely to be a beagleboard specific problem. >>>> >>>> regards, >>>> >>>> Koen >>>> -----BEGIN PGP SIGNATURE----- >>>> Version: GnuPG v1.4.5 (Darwin) >>>> >>>> iD8DBQFIQ+g/MkyGM64RGpERAqwgAJ97AE4z0bIBZJs7nZcO1Npa8YkIcgCfY6KM >>>> 3Hhr4AIdmzNiozdqs1uQUgg= >>>> =rcQP >>>> -----END PGP SIGNATURE----- >>>> -- >>>> To unsubscribe from this list: send the line "unsubscribe linux- >>>> omap" in >>>> the body of a message to majordomo@vger.kernel.org >>>> More majordomo info at http://vger.kernel.org/majordomo-info.html >>> -- >>> To unsubscribe from this list: send the line "unsubscribe linux- >>> omap" in >>> the body of a message to majordomo@vger.kernel.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html > -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.5 (Darwin) iD8DBQFISRkJMkyGM64RGpERAqdKAJ9FPOr/tYcEO17LkM8Ku8fdsD1AuwCgjDwa L3YD0guaALhLrntQ05vXwKc= =vkoy -----END PGP SIGNATURE----- ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 0/3] ARM: OMAP: SmartReflex driver @ 2008-05-30 14:12 Kalle Jokiniemi 0 siblings, 0 replies; 9+ messages in thread From: Kalle Jokiniemi @ 2008-05-30 14:12 UTC (permalink / raw) To: linux-omap; +Cc: ext-kalle.jokiniemi This patch set integrates smartreflex driver into linux-omap tree. The first patch adds Texas Instrumets' reference source and header files for smartreflex driver. Only checkpatch.pl fixes added. The second patch prepares required register and bit definitions to enable smartreflex integration. The last patch integrates the driver into linux-omap. Things to do: - Integration with DVFS, once an implementation is available. - efuse values should be taken into use instead of precalculated silicon characteristics Regards, Kalle Jokiniemi --- ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2008-06-06 11:01 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2008-06-02 11:29 [PATCH 0/3] ARM: OMAP: SmartReflex driver Kalle Jokiniemi 2008-06-02 12:31 ` Koen Kooi 2008-06-02 14:58 ` TK, Pratheesh Gangadhar 2008-06-03 6:02 ` Kalle Jokiniemi 2008-06-03 8:24 ` TK, Pratheesh Gangadhar 2008-06-03 15:10 ` Tony Lindgren 2008-06-06 9:49 ` Kalle Jokiniemi 2008-06-06 11:01 ` Koen Kooi -- strict thread matches above, loose matches on Subject: below -- 2008-05-30 14:12 Kalle Jokiniemi
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox