len0rd/rockbox
1
0
Fork 1
mirror of https://github.com/Rockbox/rockbox.git synced 2025-10-13 10:07:38 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

ipod6g: CE-ATA uses a 4K block size at minimum

Browse source 
Therefore, turn on MAX_VARIABLE_LOG_SECTOR so we can support 512B and 4K
sector sizes.  Additionally, correct the interpretation of identify info
word 106 in CE-ATA mode.

Change-Id: I24dc7dd4a8617fcb60ed87c0c1be98d00dbdfa30
This commit is contained in:
Solomon Peachy 2025-09-16 14:59:01 -04:00
parent 4960a3b753
commit 9644bb5d9d
2 changed files with 11 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
firmware/export/config/ipod6g.h
View file

@ -203,7 +203,7 @@
#define MAX_PHYS_SECTOR_SIZE 4096
/* define this if we want to support 512n and 4Kn drives */
//#define MAX_VARIABLE_LOG_SECTOR 4096
#define MAX_VARIABLE_LOG_SECTOR 4096
#define HAVE_HARDWARE_CLICK

10
firmware/target/arm/s5l8702/ipod6g/storage_ata-6g.c
View file

@ -1170,9 +1170,19 @@ int ata_init(void)
/* Logical sector size */
if ((identify_info[106] & 0xd000) == 0x5000) /* B14, B12 */
log_sector_size = (identify_info[117] | (identify_info[118] << 16)) * 2;
else if (ceata)
log_sector_size = 1 << identify_info[106];
else
log_sector_size = 512;
#ifndef MAX_VARIABLE_LOG_SECTOR
if (log_sector_size != SECTOR_SIZE)
panicf("Bad logical sector size (%ld)", log_sector_size);
#else
if (log_sector_size > MAX_VARIABLE_LOG_SECTOR)
panicf("Logical sector size too large (%ld)", log_sector_size);
#endif
#ifdef MAX_PHYS_SECTOR_SIZE
rc = ata_get_phys_sector_mult();
if (IS_ERR(rc))