Message ID | 20200523163837.407592-15-marek.vasut+renesas@gmail.com |
---|---|
State | New |
Headers | show |
Series | [01/30] net: eepro100: Remove EEPRO100_SROM_WRITE | expand |
On Sat, May 23, 2020 at 7:42 PM Marek Vasut <marek.vasut at gmail.com> wrote: > > Remove all the remaining use of the 'volatile' keyword, as this is > no longer required. All the accesses which might have needed this > use of 'volatile' have been repaired properly. > > Signed-off-by: Marek Vasut <marek.vasut+renesas at gmail.com> > --- > drivers/net/eepro100.c | 42 +++++++++++++++++++++--------------------- > 1 file changed, 21 insertions(+), 21 deletions(-) > > diff --git a/drivers/net/eepro100.c b/drivers/net/eepro100.c > index 89bfcfba0a..f3bcb0dfe4 100644 > --- a/drivers/net/eepro100.c > +++ b/drivers/net/eepro100.c > @@ -103,13 +103,13 @@ > > /* Receive frame descriptors. */ > struct eepro100_rxfd { > - volatile u16 status; > - volatile u16 control; > - volatile u32 link; /* struct eepro100_rxfd * */ > - volatile u32 rx_buf_addr; /* void * */ > - volatile u32 count; > + u16 status; > + u16 control; > + u32 link; /* struct eepro100_rxfd * */ > + u32 rx_buf_addr; /* void * */ > + u32 count; > > - volatile u8 data[PKTSIZE_ALIGN]; > + u8 data[PKTSIZE_ALIGN]; > }; > > #define RFD_STATUS_C 0x8000 /* completion of received frame */ > @@ -136,17 +136,17 @@ struct eepro100_rxfd { > #define RFD_RX_TCO 0x0001 /* TCO indication */ > > /* Transmit frame descriptors */ > -struct eepro100_txfd { /* Transmit frame descriptor set. */ > - volatile u16 status; > - volatile u16 command; > - volatile u32 link; /* void * */ > - volatile u32 tx_desc_addr; /* Always points to the tx_buf_addr element. */ > - volatile s32 count; > - > - volatile u32 tx_buf_addr0; /* void *, frame to be transmitted. */ > - volatile s32 tx_buf_size0; /* Length of Tx frame. */ > - volatile u32 tx_buf_addr1; /* void *, frame to be transmitted. */ > - volatile s32 tx_buf_size1; /* Length of Tx frame. */ > +struct eepro100_txfd { /* Transmit frame descriptor set. */ > + u16 status; > + u16 command; > + u32 link; /* void * */ > + u32 tx_desc_addr; /* Always points to the tx_buf_addr element. */ > + s32 count; > + > + u32 tx_buf_addr0; /* void *, frame to be transmitted. */ > + s32 tx_buf_size0; /* Length of Tx frame. */ > + u32 tx_buf_addr1; /* void *, frame to be transmitted. */ > + s32 tx_buf_size1; /* Length of Tx frame. */ > }; > > #define TXCB_CMD_TRANSMIT 0x0004 /* transmit command */ > @@ -160,10 +160,10 @@ struct eepro100_txfd { /* Transmit frame descriptor set. */ > #define TXCB_COUNT_EOF 0x8000 > > /* The Speedo3 Rx and Tx frame/buffer descriptors. */ > -struct descriptor { /* A generic descriptor. */ > - volatile u16 status; > - volatile u16 command; > - volatile u32 link; /* struct descriptor * */ > +struct descriptor { /* A generic descriptor. */ > + u16 status; > + u16 command; > + u32 link; /* struct descriptor * */ > > unsigned char params[0]; > }; > -- > 2.25.1 > Reviewed-By: Ramon Fried <rfried.dev at gmail.com>
diff --git a/drivers/net/eepro100.c b/drivers/net/eepro100.c index 89bfcfba0a..f3bcb0dfe4 100644 --- a/drivers/net/eepro100.c +++ b/drivers/net/eepro100.c @@ -103,13 +103,13 @@ /* Receive frame descriptors. */ struct eepro100_rxfd { - volatile u16 status; - volatile u16 control; - volatile u32 link; /* struct eepro100_rxfd * */ - volatile u32 rx_buf_addr; /* void * */ - volatile u32 count; + u16 status; + u16 control; + u32 link; /* struct eepro100_rxfd * */ + u32 rx_buf_addr; /* void * */ + u32 count; - volatile u8 data[PKTSIZE_ALIGN]; + u8 data[PKTSIZE_ALIGN]; }; #define RFD_STATUS_C 0x8000 /* completion of received frame */ @@ -136,17 +136,17 @@ struct eepro100_rxfd { #define RFD_RX_TCO 0x0001 /* TCO indication */ /* Transmit frame descriptors */ -struct eepro100_txfd { /* Transmit frame descriptor set. */ - volatile u16 status; - volatile u16 command; - volatile u32 link; /* void * */ - volatile u32 tx_desc_addr; /* Always points to the tx_buf_addr element. */ - volatile s32 count; - - volatile u32 tx_buf_addr0; /* void *, frame to be transmitted. */ - volatile s32 tx_buf_size0; /* Length of Tx frame. */ - volatile u32 tx_buf_addr1; /* void *, frame to be transmitted. */ - volatile s32 tx_buf_size1; /* Length of Tx frame. */ +struct eepro100_txfd { /* Transmit frame descriptor set. */ + u16 status; + u16 command; + u32 link; /* void * */ + u32 tx_desc_addr; /* Always points to the tx_buf_addr element. */ + s32 count; + + u32 tx_buf_addr0; /* void *, frame to be transmitted. */ + s32 tx_buf_size0; /* Length of Tx frame. */ + u32 tx_buf_addr1; /* void *, frame to be transmitted. */ + s32 tx_buf_size1; /* Length of Tx frame. */ }; #define TXCB_CMD_TRANSMIT 0x0004 /* transmit command */ @@ -160,10 +160,10 @@ struct eepro100_txfd { /* Transmit frame descriptor set. */ #define TXCB_COUNT_EOF 0x8000 /* The Speedo3 Rx and Tx frame/buffer descriptors. */ -struct descriptor { /* A generic descriptor. */ - volatile u16 status; - volatile u16 command; - volatile u32 link; /* struct descriptor * */ +struct descriptor { /* A generic descriptor. */ + u16 status; + u16 command; + u32 link; /* struct descriptor * */ unsigned char params[0]; };
Remove all the remaining use of the 'volatile' keyword, as this is no longer required. All the accesses which might have needed this use of 'volatile' have been repaired properly. Signed-off-by: Marek Vasut <marek.vasut+renesas at gmail.com> --- drivers/net/eepro100.c | 42 +++++++++++++++++++++--------------------- 1 file changed, 21 insertions(+), 21 deletions(-)