Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Fri, 29 Nov 2019 12:15:20 +0100
From:   Paul Cercueil <paul@crapouillou.net>
Subject: Re: Add initial support for Ingenic X1000 SoC and Y&A CU Neo board
 v6.
To:     Zhou Yanjie <zhouyanjie@zoho.com>
Cc:     linux-mips@vger.kernel.org, linux-kernel@vger.kernel.org,
        devicetree@vger.kernel.org, robh+dt@kernel.org,
        paul.burton@mips.com, paulburton@kernel.org, jhogan@kernel.org,
        mripard@kernel.org, shawnguo@kernel.org, mark.rutland@arm.com,
        syq@debian.org, ralf@linux-mips.org, heiko@sntech.de,
        icenowy@aosc.io, laurent.pinchart@ideasonboard.com,
        krzk@kernel.org, geert+renesas@glider.be,
        prasannatsmkumar@gmail.com, sernia.zhou@foxmail.com,
        zhenwenjin@gmail.com, 772753199@qq.com
Message-Id: <1575026120.3.1@crapouillou.net>
In-Reply-To: <1575017333-29020-1-git-send-email-zhouyanjie@zoho.com>
References: <1575017333-29020-1-git-send-email-zhouyanjie@zoho.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1; format=flowed
Content-Transfer-Encoding: quoted-printable
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Hi Zhou,

This series need some reorder.

You introduce the defconfig in patch [03/12], then modify it in=20
[06/12], [08/12] and [10/12]. This just makes the patchset hard to=20
review and less likely to be merged.

Simply:
- Add missing X1000 nodes (pdma, mmc, rtc) in one patch,
- then add the CU Neo devicetree & config with PDMA, MMC and RTC=20
enabled.

And where are patches [11/12] and [12/12]? I didn't receive them.

Cheers,
-Paul


Le ven., nov. 29, 2019 at 16:48, Zhou Yanjie <zhouyanjie@zoho.com> a=20
=E9crit :
> v5->v6:
> 1.Use "stdout-path =3D "serial2:115200n8"" instead "stdout-path =3D=20
> &uart2" in DT.
> 2.Remove "console=3DttyS2,115200n8" and "CONFIG_CMDLINE_OVERRIDE=3Dy"=20
> from defconfg.
> 3.Add watchdog timer DT node, and update defconfig for watchdog timer.
>=20
>=20

=