@@ -290,9 +290,11 @@ static void octeon_mgmt_clean_tx_buffers(struct octeon_mgmt *p)/* Read the hardware TX timestamp if one was recorded */if(unlikely(re.s.tstamp)){structskb_shared_hwtstampsts;+u64ns;+memset(&ts,0,sizeof(ts));/* Read the timestamp */-u64ns=cvmx_read_csr(CVMX_MIXX_TSTAMP(p->port));+ns=cvmx_read_csr(CVMX_MIXX_TSTAMP(p->port));/* Remove the timestamp from the FIFO */cvmx_write_csr(CVMX_MIXX_TSCTL(p->port),0);/* Tell the kernel about the timestamp */
From: David Daney <hidden> Date: 2014-09-08 16:36:31
On 09/08/2014 08:01 AM, Aaro Koskinen wrote:
Fix the following compiler warning:
drivers/net/ethernet/octeon/octeon_mgmt.c: In function 'octeon_mgmt_clean_tx_buffers':
drivers/net/ethernet/octeon/octeon_mgmt.c:295:4: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
u64 ns = cvmx_read_csr(CVMX_MIXX_TSTAMP(p->port));
^
Signed-off-by: Aaro Koskinen <redacted>
For some reason we missed this when the initial patch was created.
Thanks for fixing it...
Acked-by: David Daney <redacted>
@@ -290,9 +290,11 @@ static void octeon_mgmt_clean_tx_buffers(struct octeon_mgmt *p)/* Read the hardware TX timestamp if one was recorded */if(unlikely(re.s.tstamp)){structskb_shared_hwtstampsts;+u64ns;+memset(&ts,0,sizeof(ts));/* Read the timestamp */-u64ns=cvmx_read_csr(CVMX_MIXX_TSTAMP(p->port));+ns=cvmx_read_csr(CVMX_MIXX_TSTAMP(p->port));/* Remove the timestamp from the FIFO */cvmx_write_csr(CVMX_MIXX_TSCTL(p->port),0);/* Tell the kernel about the timestamp */
Fix the following compiler warning:
drivers/net/ethernet/octeon/octeon_mgmt.c: In function 'octeon_mgmt_clean_tx_buffers':
drivers/net/ethernet/octeon/octeon_mgmt.c:295:4: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
u64 ns = cvmx_read_csr(CVMX_MIXX_TSTAMP(p->port));
^
Signed-off-by: Aaro Koskinen <redacted>