diff options
| author | Koen Vandeputte <koen.vandeputte@ncentric.com> | 2019-08-09 09:51:29 +0200 |
|---|---|---|
| committer | Koen Vandeputte <koen.vandeputte@ncentric.com> | 2019-08-09 18:30:46 +0200 |
| commit | b417a0c48d63dde16384fb38f6cc95f49299091a (patch) | |
| tree | 6cfb5b100f0aaf0f4cbba36803e088baf4c9e830 /target/linux | |
| parent | 6ced31c32020ca095a9a5f9d45782896c7384551 (diff) | |
| download | upstream-b417a0c48d63dde16384fb38f6cc95f49299091a.tar.gz upstream-b417a0c48d63dde16384fb38f6cc95f49299091a.tar.bz2 upstream-b417a0c48d63dde16384fb38f6cc95f49299091a.zip | |
ar71xx/ath79: ag71xx: init rings with GFP_KERNEL
Upstream commit 246902bdf562d45ea3475fac64c93048a7a39f01
Which contains following explanation:
--
There is no need to use GFP_ATOMIC here, GFP_KERNEL should be enough.
The 'kcalloc()' just a few lines above, already uses GFP_KERNEL.
--
Looking at the code, all other descriptors also use plain GFP_KERNEL
Signed-off-by: Koen Vandeputte <koen.vandeputte@ncentric.com>
Diffstat (limited to 'target/linux')
| -rw-r--r-- | target/linux/ath79/files/drivers/net/ethernet/atheros/ag71xx/ag71xx_main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/target/linux/ath79/files/drivers/net/ethernet/atheros/ag71xx/ag71xx_main.c b/target/linux/ath79/files/drivers/net/ethernet/atheros/ag71xx/ag71xx_main.c index 3b2584c132..7dda45d936 100644 --- a/target/linux/ath79/files/drivers/net/ethernet/atheros/ag71xx/ag71xx_main.c +++ b/target/linux/ath79/files/drivers/net/ethernet/atheros/ag71xx/ag71xx_main.c @@ -277,7 +277,7 @@ static int ag71xx_rings_init(struct ag71xx *ag) return -ENOMEM; tx->descs_cpu = dma_alloc_coherent(&ag->pdev->dev, ring_size * AG71XX_DESC_SIZE, - &tx->descs_dma, GFP_ATOMIC); + &tx->descs_dma, GFP_KERNEL); if (!tx->descs_cpu) { kfree(tx->buf); tx->buf = NULL; |