From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from Atcsqr.andestech.com (60-248-80-70.hinet-ip.hinet.net [60.248.80.70]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2AF7F2E3716 for ; Mon, 11 Aug 2025 12:17:16 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=60.248.80.70 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1754914641; cv=none; b=hYBgo15viCE+XNd/FOYmyQRsebjgu6n0GnAltgrOrVCH2QDV9BDy6Kk4u8Jouj7k8cmRK67+R+/ijczI+waOtbkEjcYvBRpZIyffGKUNvhbGCDQkXIV/pP+7klU3FLlV8Qb8s5qxaW/tvsaPXcPYuXjNclsq3Y7mFDmODTA1mow= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1754914641; c=relaxed/simple; bh=HEAZHFL2C+Cyd28LoJJKuBXif+PFRZHVXWb1Vheykjs=; h=Date:From:To:CC:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=p5B+IHVwqlGXf7FzWFVxRdDfqvyYHms0HBu9O7nkJ/8sFhzYfPm8X23uqPi6nURj0hqfK5gCc1tB7QrLiD7z789X1DpdzVUfdEHIYiPyBMDZA6QyT9aC8KQd+iZwxRQA7Khuz46LBBv6PE2KURxmGDaX5wdajCGGH7Mt8wGMDK0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=andestech.com; spf=pass smtp.mailfrom=andestech.com; arc=none smtp.client-ip=60.248.80.70 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=andestech.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=andestech.com Received: from Atcsqr.andestech.com (localhost [127.0.0.2] (may be forged)) by Atcsqr.andestech.com with ESMTP id 57BBmAJi099964 for ; Mon, 11 Aug 2025 19:48:10 +0800 (+08) (envelope-from ycliang@andestech.com) Received: from mail.andestech.com (ATCPCS34.andestech.com [10.0.1.134]) by Atcsqr.andestech.com with ESMTPS id 57BBlI3X099746 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=OK); Mon, 11 Aug 2025 19:47:18 +0800 (+08) (envelope-from ycliang@andestech.com) Received: from swlinux02 (10.0.15.183) by ATCPCS34.andestech.com (10.0.1.134) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Mon, 11 Aug 2025 19:47:18 +0800 Date: Mon, 11 Aug 2025 19:47:15 +0800 From: Leo Liang To: Yixun Lan CC: Niu Zhihong , , Subject: Re: [PATCH] board: spacemit: Add SpacemiT MUSE Pi Pro Message-ID: References: <20250728082547.454505-1-zhihong@nzhnb.com> <20250728100607-GYA777015@gentoo> Precedence: bulk X-Mailing-List: spacemit@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20250728100607-GYA777015@gentoo> User-Agent: Mutt/2.2.10 (e0e92c31) (2023-03-25) X-ClientProxiedBy: ATCPCS33.andestech.com (10.0.1.100) To ATCPCS34.andestech.com (10.0.1.134) X-DKIM-Results: atcpcs34.andestech.com; dkim=none; X-DNSRBL: X-SPAM-SOURCE-CHECK: pass X-MAIL:Atcsqr.andestech.com 57BBmAJi099964 On Mon, Jul 28, 2025 at 06:06:07PM +0800, Yixun Lan wrote: > [EXTERNAL MAIL] > > Hi Niu, > > You need to split the patch, and prepare a cover letter.. > > Have to say, generally, this patch isn't really different comparing > with current in tree bananapi-f3, so neither it is useful.. > > I personally would expect to see more drivers support for K1 SoC, > then figure out how to have more boards enabled.. > (I'd propose to not merge this patch, but if maintainer is ok with > this, then I'd have no objection) Hi Niu and Yixun, I am OK merging the patch for new board, but as Yixun and Yao's suggestion, there are some modifications need to be made. Please follow the advice and re-spin a patchset, thanks! Best regards, Leo > > CC spacemit mailing list for more wide comments > > On 16:25 Mon 28 Jul , Niu Zhihong wrote: > > MUSE Pi Pro uses spacemit K1, > > a new generation of octa-core 64-bit RISC-V processor, > > which includes octa-core X60. > > > > Features tested on a MUSE Pi Pro 16GB: > > - SD-card boot > > - eMMC boot > this brings some confusions, as currently there is no SD/eMMC driver support > for K1, only means it's able to boot from SD/eMMC (with brom support) > > > > > Signed-off-by: Niu Zhihong > > --- > > arch/riscv/Kconfig | 4 ++ > > arch/riscv/dts/Makefile | 1 + > > arch/riscv/dts/k1-muse-pi-pro.dts | 27 ++++++++ > > board/spacemit/muse-pi-pro/Kconfig | 37 +++++++++++ > > board/spacemit/muse-pi-pro/MAINTAINERS | 6 ++ > > board/spacemit/muse-pi-pro/Makefile | 3 + > > board/spacemit/muse-pi-pro/board.c | 8 +++ > > configs/muse-pi-pro_defconfig | 6 ++ > > doc/board/spacemit/index.rst | 1 + > > doc/board/spacemit/muse-pi-pro.rst | 85 ++++++++++++++++++++++++++ > > include/configs/muse-pi-pro.h | 11 ++++ > > 11 files changed, 189 insertions(+) > > create mode 100644 arch/riscv/dts/k1-muse-pi-pro.dts > > create mode 100644 board/spacemit/muse-pi-pro/Kconfig > > create mode 100644 board/spacemit/muse-pi-pro/MAINTAINERS > > create mode 100644 board/spacemit/muse-pi-pro/Makefile > > create mode 100644 board/spacemit/muse-pi-pro/board.c > > create mode 100644 configs/muse-pi-pro_defconfig > > create mode 100644 doc/board/spacemit/muse-pi-pro.rst > > create mode 100644 include/configs/muse-pi-pro.h