[U-Boot] [PATCH v2 02/17] serial: Kconfig: Add MXC_UART entry

[Jagan Teki]

Hi Peng,

On Mon, Sep 19, 2016 at 11:23 AM, Peng Fan <van.freenix at gmail.com> wrote:
> Hi Jagan,
> On Sat, Sep 17, 2016 at 02:18:34AM +0530, Jagan Teki wrote:
>>Added kconfig for MXC_UART driver.
>>
>>Cc: Simon Glass <sjg at chromium.org>
>>Cc: Fabio Estevam <fabio.estevam at nxp.com>
>>Cc: Stefano Babic <sbabic at denx.de>
>>Cc: Peng Fan <peng.fan at nxp.com>
>>Cc: Matteo Lisi <matteo.lisi at engicam.com>
>>Cc: Michael Trimarchi <michael at amarulasolutions.com>
>>Signed-off-by: Jagan Teki <jagan at amarulasolutions.com>
>>---
>> drivers/serial/Kconfig | 7 +++++++
>> 1 file changed, 7 insertions(+)
>>
>>diff --git a/drivers/serial/Kconfig b/drivers/serial/Kconfig
>>index ab5df70..9abf158 100644
>>--- a/drivers/serial/Kconfig
>>+++ b/drivers/serial/Kconfig
>>@@ -295,6 +295,13 @@ config FSL_LPUART
>>         Select this to enable a Low Power UART for Freescale VF610 and
>>         QorIQ Layerscape devices.
>>
>>+config MXC_UART
>>+      bool "IMX serial port support"
>>+      depends on MX6
>
> Not only MX6. I think this line could be removed.
> Add "selects MXC_UART" in arch/arm/cpu/armv7/mx6/Kconfig or just add "CONFIG_MXC_UART"
> in defconfig.

It's not a better approach to select driver config, since it's in
drivers/ directory and any new user will looking in to into it for
config entry to select and proper help text to understand better.

The reason I've added MX6 is my defconfig entries are tested for MX6
arch related if some wanted this driver config in their defconfig and
with MX7 or something they simply update the 'depends on'

thanks!
-- 
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.