Support and General Use > Hardware

Memory access noise on Clip+

<< < (4/6) > >>

Gunbuster:
Hi bertrik,

I tried your new build, at startup it say Ver. 4fc4f4e-130712

Sadly I can't hear any improvement with both of my devices.

Thank you for your hard work and please don't hesitate to share new builds if you have other clues about this issue.

b0hoon:
Hi.

I have a clip+, variant 0 and have no ANY noise with unbranded 2GB card, class 4. Rockbox version is 8723132-120830, so after 3.8. So my guess is that it depends on the hardware (sd card).

Gunbuster:
Hi b0hoon,

It could be the card yes, as I have already stated I have two memory cards, a Sandisk class 4 and a Trenscend class 10 and I can't hear any difference one from another.

Two different brand, two different speed, the only redundancy is the size (32gb.)

So the size of the card could be one of the root of the problem, I will try to have access on a different card size this weekend to test this theory.

Still it don't explain why two different version of rockbox can sound different with the same hardware (both player and SD card.)

Edit : I just tried with a friend's 16gb class 10 (also Transcend), and still no difference.

Gunbuster:
So based on the research of bertrik and gevaerts I tried to mess with the sources and make build of rockbox with those patch reverted, and I found the culprit.

* http://git.rockbox.org/?p=rockbox.git;a=commit;h=8351a1aa9db98bab1f66274ee7b7e33839a31c60

What I did is :


--- Code: ---Launch the VM
git clone git://git.rockbox.org/rockbox ./Desktop/rockbox
cd ./Desktop/rockbox
mkdir build
cd build
../tools/configure
62
patch -p0 -R < 8351a1aa9db98bab1f66274ee7b7e33839a31c60
--- End code ---

However when Applied the patch it said :


--- Code: -----------------------------
|--- a/apps/plugins/SOURCES
|+++ b/apps/plugins/SOURCES
--------------------------
File to patch: ../apps/plugins/SOURCES
patching file ../apps/plugins/SOURCES
Unreversed patch detected!  Ignore -R? [n]
--- End code ---

So only this patch has been applied


--- Code: ------ a/firmware/target/arm/as3525/sd-as3525v2.c
+++ b/firmware/target/arm/as3525/sd-as3525v2.c
@@ -549,40 +549,11 @@ static int sd_init_card(const int drive)
     /*  Card back to full speed  */
     MCI_CLKDIV &= ~(0xFF);    /* CLK_DIV_0 : bits 7:0 = 0x00 */
 
-    if (sd_v2)
-    {
-        /* Attempt to switch cards to HS timings, non HS cards just ignore this */
-        /*  CMD7 w/rca: Select card to put it in TRAN state */
-        if(!send_cmd(drive, SD_SELECT_CARD, card_info[drive].rca, MCI_RESP, &response))
-            return -7;
-
-        if(sd_wait_for_tran_state(drive))
-            return -8;
-
-        /* CMD6 */
-        if(!send_cmd(drive, SD_SWITCH_FUNC, 0x80fffff1, MCI_RESP, &response))
-            return -9;
-
-        /* This delay is a bit of a hack, but seems to fix card detection
-           problems with some SD cards (particularly 16 GB and bigger cards).
-           Preferably we should handle this properly instead of using a delay,
-           see also FS#11870. */
-        sleep(HZ/10);
-
-        /*  We need to go back to STBY state now so we can read csd */
-        /*  CMD7 w/rca=0:  Deselect card to put it in STBY state */
-        if(!send_cmd(drive, SD_DESELECT_CARD, 0, MCI_NO_RESP, NULL))
-            return -10;
-    }
-
     /* CMD9 send CSD */
     if(!send_cmd(drive, SD_SEND_CSD, card_info[drive].rca,
                  MCI_RESP|MCI_LONG_RESP, card_info[drive].csd))
         return -11;
 
-    /* Another delay hack, see FS#11798 */
-    mci_delay();
-
     sd_parse_csd(&card_info[drive]);
 
     if(drive == INTERNAL_AS3525) /* The OF is stored in the first blocks */
--- End code ---


Edit : For anyone who want to test the build :
https://www.dropbox.com/s/vn3m816omnfli7e/rockbox.zip

404_user_not_found:
You should try a new patch http://gerrit.rockbox.org/r/#/c/512/

Navigation

[0] Message Index

[#] Next page

[*] Previous page