From patchwork Sun Jul 12 10:25:49 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Finn Thain X-Patchwork-Id: 494097 Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from lists.ozlabs.org (lists.ozlabs.org [103.22.144.68]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 5C72F1402B6 for ; Sun, 12 Jul 2015 21:09:23 +1000 (AEST) Received: from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3]) by lists.ozlabs.org (Postfix) with ESMTP id 2EA081A3678 for ; Sun, 12 Jul 2015 21:09:23 +1000 (AEST) X-Original-To: linuxppc-dev@lists.ozlabs.org Delivered-To: linuxppc-dev@lists.ozlabs.org Received: from kvm5.telegraphics.com.au (kvm5.telegraphics.com.au [98.124.60.144]) by lists.ozlabs.org (Postfix) with ESMTP id A61891A0CF6 for ; Sun, 12 Jul 2015 20:40:41 +1000 (AEST) Received: by kvm5.telegraphics.com.au (Postfix, from userid 502) id 38A3D27FEF; Sun, 12 Jul 2015 06:40:40 -0400 (EDT) Message-Id: <20150712102532.705917057@telegraphics.com.au> User-Agent: quilt/0.50-1 Date: Sun, 12 Jul 2015 20:25:49 +1000 From: Finn Thain To: , , , Geert Uytterhoeven Subject: [RFC v4 22/25] m68k/mac: Adopt naming and calling conventions for PRAM routines References: <20150712102527.356151908@telegraphics.com.au> Content-Disposition: inline; filename=mac68k-pram-function-names-and-signatures X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org Sender: "Linuxppc-dev" 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-12 20:24:53.000000000 +1000 +++ linux/arch/m68k/mac/misc.c 2015-07-12 20:25:20.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)