From:   Damien Le Moal <Damien.LeMoal@wdc.com>
To:     Palmer Dabbelt <palmer@rivosinc.com>,
        "linux-riscv@lists.infradead.org" <linux-riscv@lists.infradead.org>
CC:     Paul Walmsley <paul.walmsley@sifive.com>,
        Palmer Dabbelt <palmer@dabbelt.com>,
        "aou@eecs.berkeley.edu" <aou@eecs.berkeley.edu>,
        Anup Patel <Anup.Patel@wdc.com>,
        "heinrich.schuchardt@canonical.com" 
        <heinrich.schuchardt@canonical.com>,
        Atish Patra <Atish.Patra@wdc.com>,
        "bin.meng@windriver.com" <bin.meng@windriver.com>,
        "sagar.kadam@sifive.com" <sagar.kadam@sifive.com>,
        "axboe@kernel.dk" <axboe@kernel.dk>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 12/12] RISC-V: defconfigs: Remove redundant K210 DT source
Thread-Topic: [PATCH 12/12] RISC-V: defconfigs: Remove redundant K210 DT
 source
Thread-Index: AQHX3WTpsbliyan1bkCwPrZcgVMvjA==
Date:   Sun, 21 Nov 2021 23:41:18 +0000
Message-ID: <DM6PR04MB70816CBD9FFA383FEE8DDC67E79E9@DM6PR04MB7081.namprd04.prod.outlook.com>
References: <20211119164413.29052-1-palmer@rivosinc.com>
 <20211119164413.29052-13-palmer@rivosinc.com>
Accept-Language: en-US
Content-Language: en-US
wdcipoutbound: EOP-TRUE
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?qrx6hCRPutq9Cb8JK6rRPeAeexbXIQLQcRo3Lms9rUrgnWuYiKJToY+emeLI?=
 =?us-ascii?Q?jHvn47wnQVhwbgYAUssG34awx1tAIGFGa7ipsqkHfUNXI9I4wzryUe6vxzCg?=
 =?us-ascii?Q?Ij+kVXaaFAEACAl13Jk4h6lyoBUYaqU297t2xTV7cv0D/klklk3sfgahP4SP?=
 =?us-ascii?Q?5xn12meJJdAEVIF68v7/VlLPj9L/KhqZ5UsNRzfcZvBsL0dTmJDxgYYeEvWB?=
 =?us-ascii?Q?8AvrcVSl96c3wvONZCqj5e0+moVQ0q9+punsb657j9tI8KbKrloj6UgRtrTc?=
 =?us-ascii?Q?Ph9RkK9TDVHBc86dFqjcExIphlNb8loFmLre+kZrnP9MrHHyCr6LU+gs427I?=
 =?us-ascii?Q?lRQGE7pTAZPaCcV2F5deLQu2t6+IWEPNeiHG/fqkjhyqn8McGOEun+eT6+Qc?=
 =?us-ascii?Q?ND1RgP0BsyPmBgTFJnxMjoQeTzE0U19mMIlhvwBT4MjkZOWrE4YIbkUdI9se?=
 =?us-ascii?Q?ELYT0SqqbnnP+u4wMrAKarYj61Dn8oVh0RqM4aJVwx5M4JLwf9U46YdRSK+s?=
 =?us-ascii?Q?Mwru9gNYl7sHKJiFrANCBCIvGVDQ7txX6Sc7y2HYtwyxZGD3K5rSi09HILw0?=
 =?us-ascii?Q?WnLHPe2aal3jPqoPBb1hNWqLnDN8RF12+cXbrJ242LvKSXYXXmvxp0r5pVIx?=
 =?us-ascii?Q?bjafZOk3sN1CPDjVTgQqI19axBRvuzZFRaFyX05Fh+3iToJWR/6vSx/vV2mJ?=
 =?us-ascii?Q?jSW1zLkTIjUhK8Wo9s971WhLFnErssiP9M/9euNAnBelqaTbXZW0uUoEWdkB?=
 =?us-ascii?Q?8C61ZCVK0MW9p7K4iIIyKHrmMAWTg+Ml2AoZHIcU5/QG/zpA2bnNdAxwGO8w?=
 =?us-ascii?Q?VCJo2ZYUdOxzpaYuBVqjS9pf4EO2gPBvrFTOLXHmM8PKROirPlsiVHGHTLtL?=
 =?us-ascii?Q?fcPVBPNqR5wOU006g84MgqtmLMW5USSuGP1/CEt+RSqRNnb+dV5HSBSJGja5?=
 =?us-ascii?Q?A8uRoh0C4ShfTtiitJVY9/0bwakNWQuh3y0G2QlsVR4OmXaZMh4kDOPX8PIY?=
 =?us-ascii?Q?gtt7GUsXLVx66khJ1wM/4KSYjItklQlKNe421p5FiYfvL0la/6MHLZT5U27R?=
 =?us-ascii?Q?zQyWo8nqzrgL7MRiMUkE+7CLSbdqxBVgCwFry6n8i0/5MOWw3j1avRbNs7D4?=
 =?us-ascii?Q?OFJ7RZ+t1BwoIiL/JV1soi/yeb6+GQHL0pb/zg/Gpi+okO/1qHthv4YKy8S1?=
 =?us-ascii?Q?/jV8uTY4LCikNKik/o//I+j9FYKqRaV960yi2CKAI6xGra6IN4ok0dziskrR?=
 =?us-ascii?Q?05n4pWZIEPqFFcafRvuwPwSGuU6SJDdeu+JuFS9kIsj2sFL1Sv406WOaDp0e?=
 =?us-ascii?Q?Gl7KRDRql6VMQSuCgxmx+OZg95oFI6NR/dXMoDHiHtT6yaqs3bvkIYJTugdk?=
 =?us-ascii?Q?CFsYL8xj8181Lfi61TlisdJ3nLAY67flXKyHnsl3nDMoWnd2DlPHQLaSYtAf?=
 =?us-ascii?Q?1cp2GIhHq+9VnaNwTXdggm8IcBu6GiEl29vxr0Ie/FMM8Hdi6YHUzyT7xpWX?=
 =?us-ascii?Q?884slYbtVbaLSuz0894Y1tfjvKoMA2eCzlhR4IeUYpjwMQxQ1QJgGnthOrme?=
 =?us-ascii?Q?8jfiac+7Y5CzEuBTMR+8wfI/8SPshztj/Qr5Xp/JWH6h3dC1XcKECJPMRoUp?=
 =?us-ascii?Q?5wXthrIrVolQgIjOJx5g6Vhz5Sdi/2sA9G/2VEEUyjkahA/O7aKPWJ3nyDT1?=
 =?us-ascii?Q?UVLN7prUosXKX4GIN0TfZ0poe2I=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: wdc.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DM6PR04MB7081.namprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e12ae080-b983-49ca-5659-08d9ad486a75
X-MS-Exchange-CrossTenant-originalarrivaltime: 21 Nov 2021 23:41:18.4294
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: b61c8803-16f3-4c35-9b17-6f65f441df86
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: c108DeCsqMHlBvU8gTNk4VNsEZeryv7yb6GsY19XrzNuhCDIlEKkrj6GvMH3pRBYKwPPADp+HDpOXJBGqKPPzw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR04MB4923
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 2021/11/20 1:45, Palmer Dabbelt wrote:=0A=
> From: Palmer Dabbelt <palmer@rivosinc.com>=0A=
> =0A=
> The "k210_generic" DT has been the default in Kconfig since 7d96729a9e7=
=0A=
> ("riscv: Update Canaan Kendryte K210 device tree"), so drop it from the=
=0A=
> defconfigs to avoid diff with savedefconfig.=0A=
> =0A=
> Signed-off-by: Palmer Dabbelt <palmer@rivosinc.com>=0A=
> ---=0A=
>  arch/riscv/configs/nommu_k210_defconfig        | 1 -=0A=
>  arch/riscv/configs/nommu_k210_sdcard_defconfig | 1 -=0A=
>  2 files changed, 2 deletions(-)=0A=
> =0A=
> diff --git a/arch/riscv/configs/nommu_k210_defconfig b/arch/riscv/configs=
/nommu_k210_defconfig=0A=
> index 89ab76349ea8..e8ceab678e8b 100644=0A=
> --- a/arch/riscv/configs/nommu_k210_defconfig=0A=
> +++ b/arch/riscv/configs/nommu_k210_defconfig=0A=
> @@ -29,7 +29,6 @@ CONFIG_EMBEDDED=3Dy=0A=
>  CONFIG_SLOB=3Dy=0A=
>  # CONFIG_MMU is not set=0A=
>  CONFIG_SOC_CANAAN=3Dy=0A=
> -CONFIG_SOC_CANAAN_K210_DTB_SOURCE=3D"k210_generic"=0A=
>  CONFIG_MAXPHYSMEM_2GB=3Dy=0A=
>  CONFIG_SMP=3Dy=0A=
>  CONFIG_NR_CPUS=3D2=0A=
> diff --git a/arch/riscv/configs/nommu_k210_sdcard_defconfig b/arch/riscv/=
configs/nommu_k210_sdcard_defconfig=0A=
> index 84b87f8bfc8f..46aa3879f19c 100644=0A=
> --- a/arch/riscv/configs/nommu_k210_sdcard_defconfig=0A=
> +++ b/arch/riscv/configs/nommu_k210_sdcard_defconfig=0A=
> @@ -21,7 +21,6 @@ CONFIG_EMBEDDED=3Dy=0A=
>  CONFIG_SLOB=3Dy=0A=
>  # CONFIG_MMU is not set=0A=
>  CONFIG_SOC_CANAAN=3Dy=0A=
> -CONFIG_SOC_CANAAN_K210_DTB_SOURCE=3D"k210_generic"=0A=
>  CONFIG_MAXPHYSMEM_2GB=3Dy=0A=
>  CONFIG_SMP=3Dy=0A=
>  CONFIG_NR_CPUS=3D2=0A=
> =0A=
=0A=
Then maybe also delete the arch/riscv/boot/dts/canaan/k210_generic.dts file=
 too=0A=
? It is redundant now that it is not being referenced...=0A=
=0A=
-- =0A=
Damien Le Moal=0A=
Western Digital Research=0A=