Прошивка ROCKBOX для Hifiman'ов HM-60х и HM-801

[-=aiDARin=-]

Спустя долгих 5 лет, вопрос все-таки решен

[rusandy]

=Andrey=,
Я правильно понимаю, что пытаетесь собрать свежий билд под старую версию экрана?
Детально не смотрел, но вот, что по данной теме нашлось за пару минут (вдруг поможет):
1. Add support for HM-601 slims with new lcds.
2. Fix white screen bug

[=Andrey=]

Каким образом удалось исправить без исходников?

Метод научного тыка, сначала поигрался и версиями экрана, затем с задержками (udelay), и... попал в 10

[rusandy]

Метод научного тыка

Посмотрите по второй ссылке (из моего предыдущего поста), там видно что в процедуру инициализации экрана (lcd_display_init) mortalis добавил функцию сброса (reset_lcd), в которую специально добавлены пауза в 5000мс. Если для первой версии этого не достаточно и выпадает в белый экран, то можно попробовать увеличить эту паузу например в 1.5-2 раза, чтобы инициализация точно успела завершится. А так по идее все настройки у него уже учтены для обоих версий экрана.

[=Andrey=]

смотрите по второй ссылке (из моего предыдущего поста), там видно что в процедуру инициализации экрана (lcd_display_init) mortalis добавил функцию сброса (reset_lcd), в которую специально добавлены пауза в 5000мс. Если для первой версии этого не достаточно и выпадает в белый экран, то можно попробовать увеличить эту паузу например в 1.5-2 раза, чтобы инициализация точно успела завершится. А так по идее все настройки у него уже учтены для обоих версий экрана.

Именно это я и сделал, увеличил udelay, смутила задержка в 10мс... видимо в свежих 60х что-то удешевили, и оно перестало успевать за кодом)
Пробовал жестко прописывать V1, эффект любопытный, первый белый экран меняется на нормальную заставку, потом гаснет и появляется белый экран.
Была бы возможность отладить на ходу...

Удалил из reset_lcd вторую задержку и уменьшил первую, грузиться стало ощутимо быстрее

static void reset_lcd(void)
{
GPIO_PCCON |= (1<<7);
GPIO_PCDR &= ~(1<<7);
udelay(40); // ставил 20 - все ок, если <= 10, то привет белый экран
// udelay(10);
GPIO_PCDR |= (1<<7);
// udelay(5000);
}

Если кто-нибудь знает что происходит в этой функции, напишите комментарии плз, очень любопытно:

void udelay(unsigned usecs)
{
unsigned cycles_per_usec;
unsigned delay;

if (cpu_frequency == CPUFREQ_MAX) {
cycles_per_usec = (CPUFREQ_MAX + 999999) / 1000000;
} else {
cycles_per_usec = (CPUFREQ_NORMAL + 999999) / 1000000;
}

delay = (usecs * cycles_per_usec) / 5;

asm volatile(
"1: subs %0, %0, #1 \n" /* 1 cycle */
" nop \n" /* 1 cycle */
" bne 1b \n" /* 3 cycles */
: : "r"(delay)
);
}

[amoremio]

Отпишитесь, плиз, у кого заработало на новых ревизиях экрана...

[gemzza]

Если кто-нибудь знает что происходит в этой функции, напишите комментарии плз, очень любопытно:

Насколько я понял, дело обстоит следующим образом:

Код:

if (cpu_frequency == CPUFREQ_MAX) {
cycles_per_usec = (CPUFREQ_MAX + 999999) / 1000000;
} else {
cycles_per_usec = (CPUFREQ_NORMAL + 999999) / 1000000;
}

Здесь получаем кол-во циклов процессора за одну микросекунду. "Хитрая" операция "добавить 999999 а затем разделить на 1000000" делается для того, чтобы при делении получить чуть больший результат, чтобы при округлении результат "вдруг" не уменьшился на единицу. Хотя если взглянуть на определение CPUFREQ_MAX и CPUFREQ_NORMAL, расположенное в файле "firmware/target/arm/rk27xx/system-target.h"

Код:

#define CPUFREQ_DEFAULT  50000000
#define CPUFREQ_NORMAL   50000000
#define CPUFREQ_MAX     200000000

на мой взгляд это лишнее.

Далее

Код:

delay = (usecs * cycles_per_usec) / 5;

Вычисляется кол-во циклов, которые будут повторяться для задержки. Делим на 5 потому, что один цикл задержки занимает 5 тактов процессора, об этом собственно написано в комментариях к коду, который осуществляет задержку.

Далее собственно сам код, который осуществляет задержку

Код:

asm volatile(
"1: subs %0, %0, #1 \n" /* 1 cycle */
" nop \n" /* 1 cycle */
" bne 1b \n" /* 3 cycles */
: : "r"(delay)
);

Алгоритм задержки простейший:
1. Вычитаем из delay единицу (занимает 1 такт процессора): инструкция subs %0, %0, #1
2. Ничего не делаем один такт: инструкция nop
3. Проверяем, если в регистре не 0, т.е. delay > 0, возвращаемся к шагу 1 (занимает 3 такта процессора): инструкция bne 1b

[=Andrey=]

gemzza, Большое спасибо, все понятно)

[rusandy]

видимо в свежих 60х что-то удешевили, и оно перестало успевать за кодом)

Очень даже возможно, тут где-то была инфа на форуме, что какие-то партии собирали из того, что под рукой было (даже зарядные другие в коробке были).

Была бы возможность отладить на ходу...

Я в таких случаях делаю сразу несколько сборок с разными параметрами, чтобы потом просто поочередно закидывать на карту и смотреть результат.

[=Andrey=]

Текущие результаты экспериментов:

static void reset_lcd(void)
{
GPIO_PCCON |= (1<<7);
GPIO_PCDR &= ~(1<<7);

udelay(200);
// < 13 = белый экран на V2, < 50 + microSD = ATA error -4 на V1 (один раз словил при задержке 100, по этому остановился на 200)

GPIO_PCDR |= (1<<7);

udelay(2000)
// < 843 = белый экран на V1, с 843 до ~880 инициализация цвета(экран темнеет до нормы), затем всё ок.
}
задержки из других функций выпилил, никаких изменений не заметил

на руках 3 мужика, два 601 с разными экранами и один свеженький 603, это вариант пашет на всех без глюков.

[-=aiDARin=-]

Рад, что у всех работает, залил правку в основную ветку...

Не совсем понял, где это находится?

[=Andrey=]

Не совсем понял, где это находится?

Пока нигде, я поторопился с сообщением, локально закомитил и думал что всё на github улетело)))
Нужен кто-то с правами на репозиторий rockbox

[rusandy]

Текущие результаты экспериментов:

Вы будете смеяться, но смотрите что нашлось.. полтора года прошло, всем просто было лень попробовать))

[=Andrey=]

rusandy, это я видел, ветку всю перечитал... даже где-то на буржуйских форумах на куски этого кода натыкался

[gemzza]

на github

Так репозиторий рокбокса вроде как не на гитхабе. Если действительно хотите запушить Ваши изменения в рокбокс, Вам необходимо создать код ревью на сайте gerrit.rockbox.org, а затем написать кому-нибудь из ревьюверов, которые имеют право заливать.

---------- Добавлено в 09:43 ---------- Предыдущее сообщение было размещено в 09:42 ----------

Вы будете смеяться, но смотрите что нашлось.. полтора года прошло, всем просто было лень попробовать))

Это было так давно, что даже я об этом забыл. Учитывая, что с тех пор ни 60x ни 801 мужиков у меня так и не появилось, это нормально

[=Andrey=]

Так репозиторий рокбокса вроде как не на гитхабе. Если действительно хотите запушить Ваши изменения в рокбокс, Вам необходимо создать код ревью на сайте gerrit.rockbox.org, а затем написать кому-нибудь из ревьюверов, которые имеют право заливать.

Спасибо, займусь на досуге

[rusandy]

Учитывая, что с тех пор ни 60x ни 801 мужиков у меня так и не появилось, это нормально

Это не нормально, мужик должен быть в каждой семье Если не секрет, то на чем остановились из портатива?

[gemzza]

Это не нормально, мужик должен быть в каждой семье Если не секрет, то на чем остановились из портатива?

Мне и себя в качестве мужика хватает.

Мой текущий портативный в подписи. Но Миша вроде обещал ещё чего-то сотворить

[-=aiDARin=-]

gemzza, с ММ6 итак уже выше крыши), только вот пока непонятно с ОУ-шками, никак определиться не могут, жду финальный вариант).

[sanmigel]

Камрады программисты, посмотрите, плиз формирование мастерклока i2s в 601 мужике, на последних страницы темы про мм6 описал. Скважность в рокбоксе корявенькая получается, а в стоке все ок.
Собсно

http://player.ru/showthread.php?p=2687095