From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 9E774C433EF for ; Fri, 29 Apr 2022 14:28:53 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 83CE583A9F; Fri, 29 Apr 2022 16:28:51 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=web.de Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (1024-bit key; secure) header.d=web.de header.i=@web.de header.b="d7xqQidj"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 12EC283F04; Fri, 29 Apr 2022 16:28:49 +0200 (CEST) Received: from mout.web.de (mout.web.de [212.227.17.11]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 209DB81A02 for ; Fri, 29 Apr 2022 16:28:46 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=web.de Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=smoch@web.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=web.de; s=dbaedf251592; t=1651242505; bh=CyxBwUq9+hAvVInai/SqDyf4exXflYYu+vYarwLeO4w=; h=X-UI-Sender-Class:Date:Subject:To:Cc:References:From:In-Reply-To; b=d7xqQidjNHphUNk46P9F5C1q6Uk6wYKs4EI6K2R2xxJyvRW5YdQ3kjXjGjOaV0A2K MRXE23z7bl+MPr/r+Hf8m0Sa5Fe+2tJzDC5A+romiqy5leZeYbbO3HrSBarJf1p/IA HVklbz3pR5cr0Fy5i7DQsx8Iw5MA6MRH9jRvVS5o= X-UI-Sender-Class: c548c8c5-30a9-4db5-a2e7-cb6cb037b8f9 Received: from [10.9.8.2] ([62.227.160.233]) by smtp.web.de (mrweb105 [213.165.67.124]) with ESMTPSA (Nemesis) id 1MGQGF-1necj61sIH-00GpRW; Fri, 29 Apr 2022 16:28:25 +0200 Message-ID: <19bb57bf-d749-eac2-9ed0-db75a78f8ff0@web.de> Date: Fri, 29 Apr 2022 16:28:17 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.7.0 Subject: Re: [PATCH] imx: drop CONFIG_MXC_UART_BASE Content-Language: en-GB To: "Peng Fan (OSS)" , sbabic@denx.de, festevam@gmail.com, Heiko Schocher , Peng Fan , Adrian Alonso , Arkadiusz Karas , Marcel Ziswiler Cc: u-boot@lists.denx.de References: <20220429131040.19097-1-peng.fan@oss.nxp.com> From: Soeren Moch In-Reply-To: <20220429131040.19097-1-peng.fan@oss.nxp.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: quoted-printable X-Provags-ID: V03:K1:c144GAr6xt0CvgDv7odvH6+GeQxqohZ5Qg/e0FI4farqZe9T9bU 11lDujCufpliTDfp3N60XowMrTSiCP6Rpq6cIyCqQqIOl4ID3tcW6sHEz1v7rJm0kokoGvh yyjr9TxEeSHf0tAmsj5vDz7XtG0O1SPaxOV+g2jd+SK2fu1hOdz8qAibF6irFvLDEWXgcZe PZEdSCfPbVD/K2gBqkT2g== X-UI-Out-Filterresults: notjunk:1;V03:K0:XAJcP3/sZOM=:LAKnXhoHxzr9DAuSjhhX9r cJ8qaBw20zkJAG9PsZ9F6R0DNBslAybqSldPARPJT77T61iKLP3mLCK4MjWEQe0pWvSiTy4Gd khSeaqblQQLJ/+FPcR0mGKginO9zRGSWEdRo8o+fNMIvz3Mv89jGGOV3K++hU56A4gAsJX3p2 6YNkYaRQciygimBz8N/Hd/UKt3BoQ9jRXVijw6rviyC+64TNdDr+4C+jdQg0qwfGSGSRDXX3B DI1e5oSdmWvLYMK75/JgWSveQCuGtG6A0Ubu6GPKxLcBB8Kd2H1muSMRgG/iN1EWlnKouO+Ey ie+8mPXYFyafHq3+BKqXBdJMjNd6g1BCtu0KKBNFE70l145hHvP5eRevSmW27FlyY0FNAN8yn EJ/5o7wYPM7KkrGGZhoz9RyROEkpE9q/0JjUl5muVUtafEmeSYf1Z2+ZOua0slUjZ41xikTmy HQln7VYLb3xA4KH+FhC93vK4z5OJbgIB1f/YOz7iGOka3OHsHSYNI5tqKjftLk6ZKOALciF4a HyFXqmMiZXvhqNBHjp2i2N3FV9E1Scp7/CD8nWMwfcBUWFiUhQEveKqkIvrckvIcUJde3NqhY As+ZsMkPbf4XvHkf4MLTjma1C4a24RqOtJxc1EWOdf372o57pcz4B18ryBDQhnyUZGVeGUIAP 3W+CwaOX91H/URl89IAdwnfqleDWXNvKityS293TZSzlMmScCIV5rOtQajEpnUb7MWOSrWV3Y 3degaLtf5Ka0cEsurD6yJC2qEax6JEIlP5FNtKl/6GEGeNY27OuB6OWEpNITrgsKblvWEMfyx 67GDIx6qBh5/dmcDsuV6J6dq9v/jgpcfdOfZOWRZ+B2qGovR6Zn+VFP8dtR9csXUpKkoo25Uk IoUyH7sIsNxS27XU3CeVX3s29yvyRkbosdCqIVHkrbnowBBPdY8XwuiOdG0KTh85pZzn+64Kd dWm6mzdbVmIxPVgVsG4woy7FgDLNTUHhKbldlYsYYMsl/kEYgc4Be+VmHE4jlcYZVqQTnV1Qy XMkPHpWAyAdKux7YIZ8y07DMmzjPlUBnchuKlmTkrxn60Nh2G1fNcxtvyFnFQL9aXg== X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.5 at phobos.denx.de X-Virus-Status: Clean On 29.04.22 15:10, Peng Fan (OSS) wrote: > From: Peng Fan > > Since these boards has CONFIG_DM_SERIAL and/or CONFIG_SPL_DM_SERIAL, > the legacy macro no need to be defined. > > Signed-off-by: Peng Fan > --- > include/configs/aristainetos2.h | 2 -- > include/configs/imx8mm_evk.h | 1 - > include/configs/imx8mn_evk.h | 1 - > include/configs/imx8mp_evk.h | 1 - > include/configs/mx7dsabresd.h | 1 - > include/configs/somlabs_visionsom_6ull.h | 1 - > include/configs/tbs2910.h | 1 - > include/configs/verdin-imx8mm.h | 1 - > include/configs/verdin-imx8mp.h | 1 - > 9 files changed, 10 deletions(-) > > diff --git a/include/configs/aristainetos2.h b/include/configs/aristaine= tos2.h > index 8ee97f1d4e3..27adae5d6bf 100644 > --- a/include/configs/aristainetos2.h > +++ b/include/configs/aristainetos2.h > @@ -14,10 +14,8 @@ > #define CONFIG_HOSTNAME "aristainetos2" > > #if (CONFIG_SYS_BOARD_VERSION =3D=3D 5) > -#define CONFIG_MXC_UART_BASE UART2_BASE > #define CONSOLE_DEV "ttymxc1" > #elif (CONFIG_SYS_BOARD_VERSION =3D=3D 6) > -#define CONFIG_MXC_UART_BASE UART1_BASE > #define CONSOLE_DEV "ttymxc0" > #endif > > diff --git a/include/configs/imx8mm_evk.h b/include/configs/imx8mm_evk.h > index 42b78485cfc..7dd66527c3d 100644 > --- a/include/configs/imx8mm_evk.h > +++ b/include/configs/imx8mm_evk.h > @@ -68,7 +68,6 @@ > #define PHYS_SDRAM 0x40000000 > #define PHYS_SDRAM_SIZE 0x80000000 /* 2GB DDR */ > > -#define CONFIG_MXC_UART_BASE UART2_BASE_ADDR > > /* Monitor Command Prompt */ > #define CONFIG_SYS_CBSIZE 2048 > diff --git a/include/configs/imx8mn_evk.h b/include/configs/imx8mn_evk.h > index 034132225c6..b20e0a5553d 100644 > --- a/include/configs/imx8mn_evk.h > +++ b/include/configs/imx8mn_evk.h > @@ -75,7 +75,6 @@ > #define PHYS_SDRAM 0x40000000 > #define PHYS_SDRAM_SIZE 0x80000000 /* 2GB DDR */ > > -#define CONFIG_MXC_UART_BASE UART2_BASE_ADDR > > /* Monitor Command Prompt */ > #define CONFIG_SYS_CBSIZE 2048 > diff --git a/include/configs/imx8mp_evk.h b/include/configs/imx8mp_evk.h > index cc8d65cb54e..cbdd490a300 100644 > --- a/include/configs/imx8mp_evk.h > +++ b/include/configs/imx8mp_evk.h > @@ -80,7 +80,6 @@ > #define PHYS_SDRAM_2 0x100000000 > #define PHYS_SDRAM_2_SIZE 0xC0000000 /* 3 GB */ > > -#define CONFIG_MXC_UART_BASE UART2_BASE_ADDR > > /* Monitor Command Prompt */ > #define CONFIG_SYS_CBSIZE 2048 > diff --git a/include/configs/mx7dsabresd.h b/include/configs/mx7dsabresd= .h > index aaad232f0e4..62e62c6701f 100644 > --- a/include/configs/mx7dsabresd.h > +++ b/include/configs/mx7dsabresd.h > @@ -12,7 +12,6 @@ > > #define PHYS_SDRAM_SIZE SZ_1G > > -#define CONFIG_MXC_UART_BASE UART1_IPS_BASE_ADDR > > #ifdef CONFIG_IMX_BOOTAUX > /* Set to QSPI1 A flash at default */ > diff --git a/include/configs/somlabs_visionsom_6ull.h b/include/configs/= somlabs_visionsom_6ull.h > index d4761296c75..e6aad386efe 100644 > --- a/include/configs/somlabs_visionsom_6ull.h > +++ b/include/configs/somlabs_visionsom_6ull.h > @@ -16,7 +16,6 @@ > /* SPL options */ > #include "imx6_spl.h" > > -#define CONFIG_MXC_UART_BASE UART1_BASE > > /* MMC Configs */ > #ifdef CONFIG_FSL_USDHC > diff --git a/include/configs/tbs2910.h b/include/configs/tbs2910.h > index b7a94812f35..7af182524a6 100644 > --- a/include/configs/tbs2910.h > +++ b/include/configs/tbs2910.h > @@ -25,7 +25,6 @@ > #define CONFIG_SYS_BOOTMAPSZ 0x10000000 > > /* Serial console */ Since the whole section becomes empty, please also remove this comment and the empty line. Thanks for the cleanup, Soeren > -#define CONFIG_MXC_UART_BASE UART1_BASE /* select UART1/UART2 */ > > /* Framebuffer */ > #define CONFIG_IMX_HDMI > diff --git a/include/configs/verdin-imx8mm.h b/include/configs/verdin-im= x8mm.h > index da3dc95f9ee..3d7f5a3380e 100644 > --- a/include/configs/verdin-imx8mm.h > +++ b/include/configs/verdin-imx8mm.h > @@ -84,7 +84,6 @@ > #define PHYS_SDRAM_SIZE SZ_2G /* 2GB DDR */ > > /* UART */ > -#define CONFIG_MXC_UART_BASE UART1_BASE_ADDR > > /* Monitor Command Prompt */ > #define CONFIG_SYS_CBSIZE SZ_2K > diff --git a/include/configs/verdin-imx8mp.h b/include/configs/verdin-im= x8mp.h > index 7b7407752c1..b166e750c27 100644 > --- a/include/configs/verdin-imx8mp.h > +++ b/include/configs/verdin-imx8mp.h > @@ -101,7 +101,6 @@ > #define PHYS_SDRAM_2_SIZE (SZ_4G + SZ_1G) > > /* UART */ > -#define CONFIG_MXC_UART_BASE UART3_BASE_ADDR > > /* Monitor Command Prompt */ > #define CONFIG_SYS_CBSIZE SZ_2K