Rockbox

The Portalplayer i2c controller can read/write up to 4 bytes of data at a time. The current svn code does multi-byte writes (sends), however, when reading, it always requests 1 byte at a time.

While working on the Philips hdd1630 radio (tea5767 connected via i2c), I found that it doesn’t like this method of i2c communication. When reading from the tea5767, you need get 4 bytes at once or else, with the current code, you only get the first byte over and over again.

Since this potentially impacts all Portalplayer devices, I’ll put the patch here for some testing just to be cautious. Looks like effected drivers are pcf50605 (ipods), rtc_e8564 (h10 & hdd1630), and as3514 devices (sansas). I tested it on the hdd1630 (PP5022B), sansa c200 (PP5022C), and m:robe 100 (PP5020E) with seemingly no bad results. I suspect that the PP502x players are fine, but I’m unsure about the ipods, particularly PP5002’s.