Re: [PATCH v2 00/10] Add Allwinner H3 SoC and Orange Pi PC Machine

[Philippe Mathieu-Daudé]

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.

> - added BootROM support, allows booting with only specifying -sd <IMG>
> - 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#sign-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).

>   - integrate Philips work for generalizing the Allwinner timer, and 
> finish it

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

>   - 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=show&redirect=FreeBSD%2Farm%2FAllwinner