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 X-Spam-Level: X-Spam-Status: No, score=-0.4 required=3.0 tests=DKIM_ADSP_CUSTOM_MED, DKIM_INVALID,DKIM_SIGNED,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,HTML_MESSAGE,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D4E38C2D0C3 for ; Mon, 30 Dec 2019 20:11:31 +0000 (UTC) Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 9E72D20409 for ; Mon, 30 Dec 2019 20:11:31 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="nHu/QICP" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 9E72D20409 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Received: from localhost ([::1]:36254 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1im1Ny-0007eT-Qf for qemu-devel@archiver.kernel.org; Mon, 30 Dec 2019 15:11:30 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:40376) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1im1NJ-000779-Lr for qemu-devel@nongnu.org; Mon, 30 Dec 2019 15:10:51 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1im1NH-0001WK-RY for qemu-devel@nongnu.org; Mon, 30 Dec 2019 15:10:49 -0500 Received: from mail-il1-x143.google.com ([2607:f8b0:4864:20::143]:42987) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1im1NH-0001Vi-GB; Mon, 30 Dec 2019 15:10:47 -0500 Received: by mail-il1-x143.google.com with SMTP id t2so13339140ilq.9; Mon, 30 Dec 2019 12:10:47 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=c5Otgqe1JRl3ZBdsNri3GHgmpT8tEdVyUo3kmz6/T+E=; b=nHu/QICPy38pKXiJytNHC3k9WAaRmuFm03T2mS1YS6hjm0tceRB8a+VfPZ0wd2liRF jjLQLaXbagB/Az4DRNX7Zak7YLuseHIrUmkA2kk5hXHncZckWilFoK8mAjWYkQnsZH8u v6TZnybijrVOFe8l7cM7lzihSmrTp9C9xS52hzR5OC4MoFVr3dXST933W0INdPOREiK/ zwS7lXLFVNcG/q2dDCMKYLLf0xU4cH6jvjaS7ZvesxJj3jqxcUxiANcN38Ufv8oA/L49 cnlW4Sq0BzpN0S0KR8+5HiwPLpRGPD2bCl6ZQuLs9flBlPPeN4GKYx4DblJ+cMbaObsl l87Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=c5Otgqe1JRl3ZBdsNri3GHgmpT8tEdVyUo3kmz6/T+E=; b=OUwg/yJxvtDuf/0dtcTSRg6Jeypw2bZcdti6JZybh5qHoFV76CR4Jm7jdaAHdqZUcv INwiiEua2V355Uod6bR4WnaZ+5Qab70+OXM+kEnY9QUgQspC7MhiHlw8/4s7KBICzqNQ rNKmqckWYKJM81FR8lFll6FYIqqE5X7CjuKqcSGhi0qCD3/QWfSIuTszULAUjqMrpACW XtY4qd6TGmudFM3qDa+vclJyiGbltRAadYdaJg31RUPwyVEVIdcIrmiMELbOIzopwb4Q 36jg5HK+mDw1r/NdF5sBoXw68yVev38Hpi7hlu6w9oLBnMWS9/E+pKMZj0p4vfEu32b2 6KEA== X-Gm-Message-State: APjAAAUZFlrAzo55hDFSJUwwr9d6s+aNTf8Wvf7rxFhomSUEI4zH0TTh 5ptwyI2j5KUIKjTDpQ/GSEM8q7v4wov2k2Ma/zs= X-Google-Smtp-Source: APXvYqwWNYypmvf2xieWXZJPHdbhwYTQ0eFkR4W4S510235nkZJPeatnJa21ZYeFHrDZsW7NW2cN2NXdebWeMU7kHSw= X-Received: by 2002:a92:7509:: with SMTP id q9mr36693120ilc.67.1577736646489; Mon, 30 Dec 2019 12:10:46 -0800 (PST) MIME-Version: 1.0 References: <20191216233519.29030-1-nieklinnenbank@gmail.com> <9756419b-55bf-23a9-556a-d5bc5fb29331@redhat.com> In-Reply-To: <9756419b-55bf-23a9-556a-d5bc5fb29331@redhat.com> From: Niek Linnenbank Date: Mon, 30 Dec 2019 21:10:35 +0100 Message-ID: Subject: Re: [PATCH v2 00/10] Add Allwinner H3 SoC and Orange Pi PC Machine To: =?UTF-8?Q?Philippe_Mathieu=2DDaud=C3=A9?= Content-Type: multipart/alternative; boundary="0000000000005cfd1c059af17074" X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2607:f8b0:4864:20::143 X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , qemu-arm , QEMU Developers Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: "Qemu-devel" --0000000000005cfd1c059af17074 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Mon, Dec 30, 2019 at 3:56 PM Philippe Mathieu-Daud=C3=A9 wrote: > On 12/30/19 12:28 PM, Niek Linnenbank wrote: > > Hi, > > > > Here a short status report of this patch series. > > Good idea! > > > > > For V3 update I already prepared the following: > > - reworked all review comments from Philippe, except: > > - patch#8: question for the SID, whether command-line override is > > required (and how is the best way for machine-specific cli arg?) [1] > > Answered recently. > Thanks! > > > - added BootROM support, allows booting with only specifying -sd > > - added SDRAM controller driver, for U-Boot SPL > > - added Allwinner generic RTC driver (for both Cubieboard and OrangePi > > PC, supports sun4i, sun6i, sun7i) > > - small fixes for EMAC > > > > My current TODO: > > - integrate Philips acceptance tests in the series > > You can queue them in your series, adding your Signed-off-by tag after > mine. See: > > https://www.kernel.org/doc/html/latest/process/submitting-patches.html#si= gn-your-work-the-developer-s-certificate-of-origin > > The sign-off is a simple line at the end of the explanation for the > patch, which certifies that you wrote it or otherwise have the right to > pass it on as an open-source patch. > > See point (c). > > Ah that certainly helps. I'll read that page. > > - integrate Philips work for generalizing the Allwinner timer, and > > finish it > > We can also do that later, and get your work merged first. > Ok that sounds very good! Agreed, lets do the timer work later. > > > - test and fix BSD targets (NetBSD, FreeBSD) [2, 3] > > - further generalize the series to cover very similar SoCs: H2+, H5 > > > > Does anyone have more comments/requests for the V3 update? > > > > [1] https://lists.gnu.org/archive/html/qemu-devel/2019-12/msg04049.html > > [2] https://wiki.netbsd.org/ports/evbarm/allwinner/ > > [3] > > > https://wiki.freebsd.org/action/show/arm/Allwinner?action=3Dshow&redirect= =3DFreeBSD%2Farm%2FAllwinner > > --=20 Niek Linnenbank --0000000000005cfd1c059af17074 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable


=
On Mon, Dec 30, 2019 at 3:56 PM Phili= ppe Mathieu-Daud=C3=A9 <philmd@redh= at.com> wrote:
On 12/30/19 12:28 PM, Niek Linnenbank wrote:
> Hi,
>
> Here a short status report of this patch series.

Good idea!

>
> For V3 update I already prepared the following:
>=C2=A0 =C2=A0- reworked all review comments from Philippe, except:
>=C2=A0 =C2=A0=C2=A0 - patch#8: question for the SID, whether command-li= ne override is
> required (and how is the best way for machine-specific cli arg?) [1]
Answered recently.
Thanks!

> - added BootROM support, allows booting with only specifying -sd <I= MG>
> - added SDRAM controller driver, for U-Boot SPL
> - added Allwinner generic RTC driver (for both Cubieboard and OrangePi=
> PC, supports sun4i, sun6i, sun7i)
> - small fixes for EMAC
>
> My current TODO:
>=C2=A0 =C2=A0- integrate Philips acceptance tests in the series

You can queue them in your series, adding your Signed-off-by tag after
mine. See:
https://www.kernel.org/doc/html/latest/process/submi= tting-patches.html#sign-your-work-the-developer-s-certificate-of-origin=

=C2=A0 =C2=A0The sign-off is a simple line at the end of the explanation fo= r the
patch, which certifies that you wrote it or otherwise have the right to pass it on as an open-source patch.

See point (c).

Ah that certainly helps. I'll read that page.
=
=C2=A0
>=C2=A0 =C2=A0- integrate Philips work for generalizing the Allwinner ti= mer, and
> finish it

We can also do that later, and get your work merged first.
=

Ok that sounds very good! Agreed, lets do the timer wor= k later.
=C2=A0

>=C2=A0 =C2=A0- test and fix BSD targets (NetBSD, FreeBSD) [2, 3]
>=C2=A0 =C2=A0- further generalize the series to cover very similar SoCs= : H2+, H5
>
> Does anyone have more comments/requests for the V3 update?
>
> [1] https://lists.gnu.org/ar= chive/html/qemu-devel/2019-12/msg04049.html
> [2] https://wiki.netbsd.org/ports/evbarm/allwinn= er/
> [3]
> https://wiki.freebsd.org/action/show/arm/Allwinner?action=3Dsho= w&redirect=3DFreeBSD%2Farm%2FAllwinner



--
Niek Linnenbank

--0000000000005cfd1c059af17074--