Re: IA32 (2.6.9-rc3 - 2004-09-29.21.30) - 10 New warnings (gcc3.2.2)

From: Alan Cox
Date: Thu Sep 30 2004 - 11:08:03 EST


On Iau, 2004-09-30 at 15:32, John Cherry wrote:
> drivers/net/wan/pc300_tty.c:704: warning: passing arg 1 of `tty_ldisc_ref' from incompatible pointer type

I've got a patch for this and the other 2 64bit warnings in that file.
- Fix the ldisc ref bug
- Fix the 64bit pointer cast warnings
- Fix the coding style from "Alan bracketing" to "Linus bracketing"


--- ../linux.vanilla-2.6.9rc3/drivers/net/wan/pc300_tty.c 2004-09-30 16:13:09.075305760 +0100
+++ drivers/net/wan/pc300_tty.c 2004-09-30 17:02:18.400940112 +0100
@@ -192,13 +192,14 @@
*/
void cpc_tty_init(pc300dev_t *pc300dev)
{
- int port, aux;
+ unsigned long port;
+ int aux;
st_cpc_tty_area * cpc_tty;

/* hdlcX - X=interface number */
port = pc300dev->dev->name[4] - '0';
if (port >= CPC_TTY_NPORTS) {
- printk("%s-tty: invalid interface selected (0-%i): %i",
+ printk("%s-tty: invalid interface selected (0-%i): %li",
pc300dev->dev->name,
CPC_TTY_NPORTS-1,port);
return;
@@ -682,7 +683,8 @@
*/
static void cpc_tty_rx_work(void * data)
{
- int port, i, j;
+ unsigned long port;
+ int i, j;
st_cpc_tty_area *cpc_tty;
volatile st_cpc_rx_buf * buf;
char flags=0,flg_rx=1;
@@ -693,18 +695,15 @@

for (i=0; (i < 4) && flg_rx ; i++) {
flg_rx = 0;
- port = (int) data;
+ port = (unsigned long) data;
for (j=0; j < CPC_TTY_NPORTS; j++) {
cpc_tty = &cpc_tty_area[port];

if ((buf=cpc_tty->buf_rx.first) != 0) {
-
- if(cpc_tty->tty)
- {
- ld = tty_ldisc_ref(cpc_tty);
- if(ld)
- {
- if (ld->receive_buf)) {
+ if(cpc_tty->tty) {
+ ld = tty_ldisc_ref(cpc_tty->tty);
+ if(ld) {
+ if (ld->receive_buf) {
CPC_TTY_DBG("%s: call line disc. receive_buf\n",cpc_tty->name);
ld->receive_buf(cpc_tty->tty, (char *)(buf->data), &flags, buf->size);
}