Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Feature requests · Rockbox frontpage

Tasklist

FS#8603 - iPod power management: voltage supply of dock connector pin17 (accessories)

Attached to Project: Rockbox
Opened by Andree Buschmann (Buschel) - Tuesday, 12 February 2008, 19:45 GMT+2
Last edited by Andree Buschmann (Buschel) - Monday, 14 April 2008, 19:00 GMT+2
Task Type Patches
Category Drivers
Status Assigned   Reopened
Assigned To Andree Buschmann (Buschel)
Player type iPod 5G
Severity Low
Priority Normal
Reported Version current build
Due in Version Undecided
Due Date Undecided
Percent Complete 0%
Private No

Details

Before the submission of the power management changes for PP-processors I was asked to not change the voltage supply of the PCF register D2REG for all targets (former svn did set it 3.3V).
After some runtime tests on a 5.5G 30GB it shows that this voltage supply results in ~1.5mA current consumption and is therefor of interest for further investigation.

v01-patch does simply switch leave the register unchanged. On a 5.5G 30GB (and a 4G Color/Photo) this is OFF, on other it is unknown yet.

v02-patch does switch it OFF, no matter what the PCF's default is.

I need some test support as I neither have any other iPods than my good old 5.5G 30GB, nor do I have any docking station or accessories. What is needed to be tested:

- What is the default value of D2REGC1 ("View PCF register" in the debug menu) for your player using v01-patch?
- Is there any influence on the "normal" rockbox behaviour (e.g. playing music/video, USB connect, USB charging, ...) using v01 or v02?
- Can anyone who owns or has access to any accessories or docking station check whether there is some change in behaviour with v01 or v02?

For iPod nano / Video it might be possible to enable / disable the voltage supply upon "dock detect" (GPIOA_INPUT_VAL & 0x10). But before doing so, I would really like to know whether this voltage supply is needed at all.
   dock_pin17_v01.patch (0.4 KiB)
   dock_pin17_v02.patch (0.4 KiB)
This task depends upon

Comment by Se Heon Ryu (cpu98) - Wednesday, 13 February 2008, 16:57 GMT+2
Nano with v01 , D2REGC1 value is 18 and plays music well, recharging or being recognized normally and I haven't seen any strange behavior.
with v2, D2REGC1 value is 18 again and same as above. So pin 17 is disabled by default on nano?
Comment by Andree Buschmann (Buschel) - Wednesday, 13 February 2008, 22:31 GMT+2
Yes, your results show that it is disabled by default. Can you test with any accessories?

For all testers: If you use v01-patch and D2REGC1 value is 18, there is no need to further test v02 -- the results will be the same.
Comment by Se Heon Ryu (cpu98) - Friday, 15 February 2008, 08:38 GMT+2
Unfortunately, I have no accessories.
But this patch also makes runtime longer. godspeed
Comment by Glenn (DancemasterGlenn) - Friday, 22 February 2008, 10:51 GMT+2
A rather unnecessary post on my part, but I just wanted to say I also have no accessories (nor have I actually tried to work with patches yet, instead of just test builds), else I would help you test this, Buschel. I've really appreciated the work you've been doing in improving battery life.
Comment by Glenn (DancemasterGlenn) - Friday, 22 February 2008, 10:54 GMT+2
Also, one more thing... my friend back home has a dock that works with Rockbox, so in a few weeks I may be able to test this patch out (hopefully by then I'll have figured out patches). Hopefully others will have stepped up to test by then, but if not, I'm game...
Comment by Jon (lemonman) - Wednesday, 05 March 2008, 21:36 GMT+2
Tested this on my 5G 30gb iPod. D2REGC1 shows 18 after applying v1. Function is unchanged, except for a few extra hours of battery life.

Unfortunately, I don't have any accessories to test.
Comment by Andree Buschmann (Buschel) - Saturday, 12 April 2008, 15:55 GMT+2
Synced against r17083.

- Leaves all non 5G-iPod untouched
- Does not enable PCF5060X_D2REGC1 for 5G iPods.
   dock_pin17_v03.patch (0.5 KiB)
Comment by Andree Buschmann (Buschel) - Monday, 14 April 2008, 19:02 GMT+2
Reopened due to bug report in http://forums.rockbox.org/index.php?topic=16392.0.
Comment by Andree Buschmann (Buschel) - Monday, 14 April 2008, 20:33 GMT+2
Results on 5.5G 80GB with radio remote:

without remote:
GPIOA_INPUT_VAL: 0x68
GPIOA_INPUT_VAL: 0x06

with remote (after transition):
GPIOA_INPUT_VAL: 0x78 -> 0x10
GPIOA_INPUT_VAL: 0x46 -> 0x40
Comment by Dave Hooper (stripwax) - Tuesday, 15 April 2008, 01:18 GMT+2
What do these results represent (why two readings for GPIOA_INPUT_VAL?)
Like the idea of enabling the voltage when a dock or accessory is detected. Thinking that the 'dock detect' logic is really an ADC connected to pin 21 . There must be a way to read that level so probably a range of values rather than a single dock detect flag?
http://www.hardwarebook.info/IPod_Dock
Comment by Andree Buschmann (Buschel) - Tuesday, 15 April 2008, 08:23 GMT+2
I am quite puzzled by the results in http://forums.rockbox.org/index.php?topic=16392.0. There were also tests done with r17110 (which has the power supply disabled) -- in this test the GPIO didn't change at all. Maybe the GPIO needs the supply for detection? If so, I need to find another way...
As far as I understood the _detailed_ accessory detection is done via resistors, this must be done via ADC in some way. So, another option would be to check the ADC channels. Nevertheless I am not sure whether such ADC-detection also needs the voltage supply on pin17...

I guess as a first step it would be fine to create a setting ("Accessory support: yes/no") to have the possibility to choose between battery runtime and functionality) until there is an automatic detection.
Comment by Andree Buschmann (Buschel) - Sunday, 20 April 2008, 21:20 GMT+2
With r17193 and r17196 a new setting was added to the system settings -> "Accessory Power Supply". It is disabled by default to save power and gain battery runtime. If you are using any iPod accessory which does not work, please first enable this setting.

Loading...