Hi Peng,

On Mon, Sep 19, 2016 at 11:23 AM, Peng Fan <van.free...@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 <s...@chromium.org>
>>Cc: Fabio Estevam <fabio.este...@nxp.com>
>>Cc: Stefano Babic <sba...@denx.de>
>>Cc: Peng Fan <peng....@nxp.com>
>>Cc: Matteo Lisi <matteo.l...@engicam.com>
>>Cc: Michael Trimarchi <mich...@amarulasolutions.com>
>>Signed-off-by: Jagan Teki <ja...@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 
> 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'

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

Reply via email to