Adopt the existing *_read_byte and *_write_byte naming convention. Rename via_pram_readbyte and via_pram_writebyte to avoid confusion. Adjust calling conventions of mac_pram_* functions to match the arch_nvram_ops struct methods. Signed-off-by: Finn Thain --- Changes since v1: - Don't introduce the arch_nvram_ops struct in this patch, even if it would form a logical progression. Since the struct would get replaced later on, some might see it as churn. Changes since v2: - Rename via_pram_send() and via_pram_recv() as via_rtc_send() and via_rtc_recv() resp. --- arch/m68k/mac/misc.c | 91 +++++++++++++++++++++++++-------------------------- 1 file changed, 46 insertions(+), 45 deletions(-) Index: linux/arch/m68k/mac/misc.c =================================================================== --- linux.orig/arch/m68k/mac/misc.c 2015-07-25 17:42:33.000000000 +1000 +++ linux/arch/m68k/mac/misc.c 2015-07-25 17:45:53.000000000 +1000 @@ -61,7 +61,7 @@ static void cuda_write_time(long data) cuda_poll(); } -static __u8 cuda_read_pram(int offset) +static unsigned char cuda_pram_read_byte(int offset) { struct adb_request req; if (cuda_request(&req, NULL, 4, CUDA_PACKET, CUDA_GET_PRAM, @@ -72,7 +72,7 @@ static __u8 cuda_read_pram(int offset) return req.reply[3]; } -static void cuda_write_pram(int offset, __u8 data) +static void cuda_pram_write_byte(unsigned char data, int offset) { struct adb_request req; if (cuda_request(&req, NULL, 5, CUDA_PACKET, CUDA_SET_PRAM, @@ -84,8 +84,8 @@ static void cuda_write_pram(int offset, #else #define cuda_read_time() 0 #define cuda_write_time(n) -#define cuda_read_pram NULL -#define cuda_write_pram NULL +#define cuda_pram_read_byte NULL +#define cuda_pram_write_byte NULL #endif #ifdef CONFIG_ADB_PMU68K @@ -116,7 +116,7 @@ static void pmu_write_time(long data) pmu_poll(); } -static __u8 pmu_read_pram(int offset) +static unsigned char pmu_pram_read_byte(int offset) { struct adb_request req; if (pmu_request(&req, NULL, 3, PMU_READ_NVRAM, @@ -127,7 +127,7 @@ static __u8 pmu_read_pram(int offset) return req.reply[3]; } -static void pmu_write_pram(int offset, __u8 data) +static void pmu_pram_write_byte(unsigned char data, int offset) { struct adb_request req; if (pmu_request(&req, NULL, 4, PMU_WRITE_NVRAM, @@ -139,8 +139,8 @@ static void pmu_write_pram(int offset, _ #else #define pmu_read_time() 0 #define pmu_write_time(n) -#define pmu_read_pram NULL -#define pmu_write_pram NULL +#define pmu_pram_read_byte NULL +#define pmu_pram_write_byte NULL #endif #if 0 /* def CONFIG_ADB_MACIISI */ @@ -169,7 +169,7 @@ static void maciisi_write_time(long data (data >> 8) & 0xFF, data & 0xFF); } -static __u8 maciisi_read_pram(int offset) +static unsigned char maciisi_pram_read_byte(int offset) { struct adb_request req; if (maciisi_request(&req, NULL, 4, CUDA_PACKET, CUDA_GET_PRAM, @@ -178,7 +178,7 @@ static __u8 maciisi_read_pram(int offset return req.reply[3]; } -static void maciisi_write_pram(int offset, __u8 data) +static void maciisi_pram_write_byte(unsigned char data, int offset) { struct adb_request req; maciisi_request(&req, NULL, 5, CUDA_PACKET, CUDA_SET_PRAM, @@ -187,8 +187,8 @@ static void maciisi_write_pram(int offse #else #define maciisi_read_time() 0 #define maciisi_write_time(n) -#define maciisi_read_pram NULL -#define maciisi_write_pram NULL +#define maciisi_pram_read_byte NULL +#define maciisi_pram_write_byte NULL #endif /* @@ -198,7 +198,7 @@ static void maciisi_write_pram(int offse * the RTC should be enabled. */ -static __u8 via_pram_readbyte(void) +static __u8 via_rtc_recv(void) { int i,reg; __u8 data; @@ -225,7 +225,7 @@ static __u8 via_pram_readbyte(void) return data; } -static void via_pram_writebyte(__u8 data) +static void via_rtc_send(__u8 data) { int i,reg,bit; @@ -262,17 +262,17 @@ static void via_pram_command(int command via1[vBufB] = (via1[vBufB] | VIA1B_vRTCClk) & ~VIA1B_vRTCEnb; if (command & 0xFF00) { /* extended (two-byte) command */ - via_pram_writebyte((command & 0xFF00) >> 8); - via_pram_writebyte(command & 0xFF); + via_rtc_send((command & 0xFF00) >> 8); + via_rtc_send(command & 0xFF); is_read = command & 0x8000; } else { /* one-byte command */ - via_pram_writebyte(command); + via_rtc_send(command); is_read = command & 0x80; } if (is_read) { - *data = via_pram_readbyte(); + *data = via_rtc_recv(); } else { - via_pram_writebyte(*data); + via_rtc_send(*data); } /* All done, disable the RTC */ @@ -282,12 +282,12 @@ static void via_pram_command(int command local_irq_restore(flags); } -static __u8 via_read_pram(int offset) +static unsigned char via_pram_read_byte(int offset) { return 0; } -static void via_write_pram(int offset, __u8 data) +static void via_pram_write_byte(unsigned char data, int offset) { } @@ -453,50 +453,51 @@ void pmu_shutdown(void) *------------------------------------------------------------------- */ -void mac_pram_read(int offset, __u8 *buffer, int len) +unsigned char mac_pram_read_byte(int addr) { - __u8 (*func)(int); - int i; + unsigned char (*func)(int); switch(macintosh_config->adb_type) { case MAC_ADB_IISI: - func = maciisi_read_pram; break; + func = maciisi_pram_read_byte; + break; case MAC_ADB_PB1: case MAC_ADB_PB2: - func = pmu_read_pram; break; + func = pmu_pram_read_byte; + break; case MAC_ADB_CUDA: - func = cuda_read_pram; break; + func = cuda_pram_read_byte; + break; default: - func = via_read_pram; - } - if (!func) - return; - for (i = 0 ; i < len ; i++) { - buffer[i] = (*func)(offset++); + func = via_pram_read_byte; } + + if (func) + return (*func)(addr); + return 0xff; } -void mac_pram_write(int offset, __u8 *buffer, int len) +void mac_pram_write_byte(unsigned char val, int addr) { - void (*func)(int, __u8); - int i; + void (*func)(unsigned char, int); switch(macintosh_config->adb_type) { case MAC_ADB_IISI: - func = maciisi_write_pram; break; + func = maciisi_pram_write_byte; + break; case MAC_ADB_PB1: case MAC_ADB_PB2: - func = pmu_write_pram; break; + func = pmu_pram_write_byte; + break; case MAC_ADB_CUDA: - func = cuda_write_pram; break; + func = cuda_pram_write_byte; + break; default: - func = via_write_pram; - } - if (!func) - return; - for (i = 0 ; i < len ; i++) { - (*func)(offset++, buffer[i]); + func = via_pram_write_byte; } + + if (func) + (*func)(val, addr); } void mac_poweroff(void) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/