diff options
author | Jens Arnold <amiconn@rockbox.org> | 2008-12-12 07:48:01 +0000 |
---|---|---|
committer | Jens Arnold <amiconn@rockbox.org> | 2008-12-12 07:48:01 +0000 |
commit | 0ad97d13fc52b28de566dc0ddaf7245583eec2cc (patch) | |
tree | 7b91b56ddbaf2f55133c15aabdd65ab102be0af4 /firmware/drivers/ata.c | |
parent | c99ad8f2b47277d01474095203d531a04269d37e (diff) | |
download | rockbox-0ad97d13fc52b28de566dc0ddaf7245583eec2cc.tar.gz rockbox-0ad97d13fc52b28de566dc0ddaf7245583eec2cc.zip |
Reclaim a bit of binsize on hdd based archoses by making the bit test unsigned. On SH1, gcc optimizes this by using a right shift, and a signed '1' made it use an arithmetic right-shift-by-n, which is a libgcc function on SH1. This was the only place in the core using it. No effect on other architectures.
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@19399 a1c6a512-1295-4272-9138-f99709370657
Diffstat (limited to 'firmware/drivers/ata.c')
-rw-r--r-- | firmware/drivers/ata.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/firmware/drivers/ata.c b/firmware/drivers/ata.c index e3fa3e8958..edef507e0d 100644 --- a/firmware/drivers/ata.c +++ b/firmware/drivers/ata.c | |||
@@ -1197,7 +1197,7 @@ static int set_features(void) | |||
1197 | } | 1197 | } |
1198 | 1198 | ||
1199 | for (i=0; i < (int)(sizeof(features)/sizeof(features[0])); i++) { | 1199 | for (i=0; i < (int)(sizeof(features)/sizeof(features[0])); i++) { |
1200 | if (identify_info[features[i].id_word] & (1 << features[i].id_bit)) { | 1200 | if (identify_info[features[i].id_word] & (1u << features[i].id_bit)) { |
1201 | SET_REG(ATA_FEATURE, features[i].subcommand); | 1201 | SET_REG(ATA_FEATURE, features[i].subcommand); |
1202 | SET_REG(ATA_NSECTOR, features[i].parameter); | 1202 | SET_REG(ATA_NSECTOR, features[i].parameter); |
1203 | SET_REG(ATA_COMMAND, CMD_SET_FEATURES); | 1203 | SET_REG(ATA_COMMAND, CMD_SET_FEATURES); |