Re: [Xen-devel] [PATCH v2] xen/arm: platform: fix Raspberry Pi compatible string

[Stewart Hildebrand <Stewart.Hildebrand@xxxxxxxxxxxxxxx>]

On Tuesday, October 15, 2019 7:02 AM, Julien Grall wrote:
>Hi,

Hi!

>
>On 10/9/19 5:59 PM, Stewart Hildebrand wrote:
>> However, even with Xen looking for bcm2838, you wouldn't be able to
>> grab one of those releases and boot without running into other issues.
>> You'd still need a couple of additional patches at [4]. Currently the
>> only way that I'm aware of to successfully boot into dom0 and launch
>> domU is to build the dom0 kernel from source with the extra patches
>> applied found at [4].
>
>Is there any plan to upstream those patches?

Two of the patches are adding device tree nodes (xen,xen-bootargs,
etc), so those are unlikely to be upstreamed. One patch is a
hack/workaround to disable DMA for the SD card, also unlikely to be
upstreamed. Finally, the last patch adds a check to avoid invoking
xen_dma_ops recursively. This one could potentially be upstreamed,
though we haven't submitted or gathered any input from the linux or
raspberry pi communities yet.

We are tracking the following patch sets [5] [6] [7] for upstream, and
our current plan is basically to wait until they have been merged, try
booting the upstream kernel, then see if it is still necessary to apply
the DMA workarounds.

[5] https://patchwork.kernel.org/cover/11165395/
[6] https://patchwork.kernel.org/cover/11159379/
[7] https://patchwork.kernel.org/cover/11141719/

>> "If you are using a device tree with the old compatible string
>> brcm,bcm2838, you will need to upgrade your device tree to one that has
>> the new brcm,bcm2711 compatible string."
>
>That's fine with me.

OK, since it's simply a matter of adding this to the commit message,
can it be added on commit? Or should I spin a v3 with the updated
commit message?

Finally, @Juergen, are you willing to accept this patch for 4.13? Since
both upstream linux and downstream raspberry pi linux communities have
decided to use the bcm2711 convention, we don't currently match this
platform compatible. Meaning we can't currently boot on RPi4 without
this patch.

Stew
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel