From efbd2805e296a02929c299bc9091a2daae60592b Mon Sep 17 00:00:00 2001
From: Ivan Grokhotkov <ivan@espressif.com>
Date: Thu, 19 Jan 2017 00:32:10 +0800
Subject: [PATCH] ethernet: improve kconfig option descriptions

Also limit suggested EMAC task priority range to 22, as the top priority
should be used only by IPC tasks.
---
 components/ethernet/Kconfig | 41 ++++++++++++++++++++++++++-----------
 1 file changed, 29 insertions(+), 12 deletions(-)

diff --git a/components/ethernet/Kconfig b/components/ethernet/Kconfig
index 0ea94eaafd..663871f2a0 100644
--- a/components/ethernet/Kconfig
+++ b/components/ethernet/Kconfig
@@ -5,30 +5,47 @@ menuconfig ETHERNET
         Select this option to enable ethernet driver and show the submenu with ethernet features.
 
 config DMA_RX_BUF_NUM
-    int "DMA Rx Buf Num"
-    default  10
-    depends on ETHERNET
-    help
-        Dma rx buf num ,can not be 0 .
-
-config DMA_TX_BUF_NUM
-    int "DMA Tx Buf Num"
+    int "Number of DMA RX buffers"
+    range 1 10
     default 10
     depends on ETHERNET
     help
-        Dma tx Buf num ,can not be 0.
+        Number of DMA receive buffers. Each buffer is 1600 bytes.
+        Buffers are allocated statically.
+        Larger number of buffers increases throughput.
+
+config DMA_TX_BUF_NUM
+    int "Number of DMA RX buffers"
+    range 1 10
+    default 10
+    depends on ETHERNET
+    help
+        Number of DMA transmit buffers. Each buffer is 1600 bytes.
+        Buffers are allocated statically.
+        Larger number of buffers increases throughput.
 
 config EMAC_L2_TO_L3_RX_BUF_MODE
-    bool "L2 To L3 RX BUF COPY MODE"
+    bool "Enable copy between Layer2 and Layer3"
     default n
     depends on ETHERNET
     help
-        Receive Buf user copy mode or pointer mode.
+        If this options is selected, a copy of each received buffer will be created when
+        passing it from the Ethernet MAC (L2) to the IP stack (L3). Otherwise, IP stack
+        will receive pointers to the DMA buffers used by Ethernet MAC.
+        
+        When Ethernet MAC doesn't have any unused buffers left, it will drop incomming
+        packets (flow control may help with this problem, to some extent).
+        
+        The buffers for the IP stack are allocated from the heap, so the total number of
+        receive buffers is limited by the available heap size, if this option is selected.
+        
+        If unsure, choose n.
 
 config EMAC_TASK_PRIORITY
     int "EMAC_TASK_PRIORITY"
     default 20
+    range 3 22
     depends on ETHERNET
     help
-        Emac task priority ,suggest 3 ~ 23.
+        Ethernet MAC task priority.