Re: [PATCH 1/2] staging: et131x: Replace rxdma csr register magic numbers with defines

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Dec 30, 2012 at 08:26:54PM +0000, mark.einon@xxxxxxxxx wrote:
> @@ -881,15 +880,16 @@ static void et131x_rx_dma_disable(struct et131x_adapter *adapter)
>  {
>  	u32 csr;
>  	/* Setup the receive dma configuration register */
> -	writel(0x00002001, &adapter->regs->rxdma.csr);
> +	writel(ET_RXDMA_CSR_HALT | ET_RXDMA_CSR_FBR0_ENABLE,
> +	       &adapter->regs->rxdma.csr);
>  	csr = readl(&adapter->regs->rxdma.csr);

This is a behavior change.  ET_RXDMA_CSR_FBR1_ENABLE was intended
instead of ET_RXDMA_CSR_FBR0_ENABLE.

> +#define ET_RXDMA_CSR_FBR0_ENABLE	0x0400
> +#define ET_RXDMA_CSR_FBR1_ENABLE	0x2000

regards,
dan carpenter
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel


[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux