[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v6 11/41] Temporarily disable unimplemented rpi4b devices
From: |
Peter Maydell |
Subject: |
Re: [PATCH v6 11/41] Temporarily disable unimplemented rpi4b devices |
Date: |
Mon, 26 Feb 2024 13:39:58 +0000 |
On Mon, 26 Feb 2024 at 13:35, Philippe Mathieu-Daudé <philmd@linaro.org> wrote:
>
> On 26/2/24 01:02, Sergey Kambalin wrote:
> > +static void raspi4_modify_dtb(const struct arm_boot_info *info, void *fdt)
> > +{
> > + uint64_t ram_size;
> > +
> > + /* Temporarily disable following devices until they are implemented */
> > + const char *nodes_to_remove[] = {
> > + "brcm,bcm2711-pcie",
> > + "brcm,bcm2711-rng200",
> > + "brcm,bcm2711-thermal",
> > + "brcm,bcm2711-genet-v5",
> > + };
> > +
> > + for (int i = 0; i < ARRAY_SIZE(nodes_to_remove); i++) {
> > + const char *dev_str = nodes_to_remove[i];
> > +
> > + int offset = fdt_node_offset_by_compatible(fdt, -1, dev_str);
> > + if (offset >= 0) {
> > + if (!fdt_nop_node(fdt, offset)) {
>
> Peter, I remember a discussion where you wre not keen on altering DTB
> for non-Virt machines.
>
> Since these devices are all implemented at the end of the series, why
> not add the devices then the raspi4 board at the end, so this patch is
> not even required?
I'm not super-keen on it, but as you say it goes away once all
the devices are implemented, so I'm not too worried.
Doing it this way around would let us take the first 11 patches
in the series into git now (they've all been reviewed), which
gives us (I think) a functional raspi4 with some missing devices,
which seems useful in the interim until the rest of the series
gets reviewed and committed.
-- PMM
- [PATCH v6 06/41] Add BCM2838 GPIO stub, (continued)
- [PATCH v6 06/41] Add BCM2838 GPIO stub, Sergey Kambalin, 2024/02/25
- [PATCH v6 09/41] Add GPIO and SD to BCM2838 periph, Sergey Kambalin, 2024/02/25
- [PATCH v6 14/41] Add BCM2838 PCIE host, Sergey Kambalin, 2024/02/25
- [PATCH v6 15/41] Enable BCM2838 PCIE, Sergey Kambalin, 2024/02/25
- [PATCH v6 07/41] Implement BCM2838 GPIO functionality, Sergey Kambalin, 2024/02/25
- [PATCH v6 08/41] Connect SD controller to BCM2838 GPIO, Sergey Kambalin, 2024/02/25
- [PATCH v6 05/41] Add GIC-400 to BCM2838 SoC, Sergey Kambalin, 2024/02/25
- [PATCH v6 11/41] Temporarily disable unimplemented rpi4b devices, Sergey Kambalin, 2024/02/25
[PATCH v6 13/41] Add BCM2838 PCIE Root Complex, Sergey Kambalin, 2024/02/25
[PATCH v6 03/41] Split out raspi machine common part, Sergey Kambalin, 2024/02/25
[PATCH v6 10/41] Introduce Raspberry PI 4 machine, Sergey Kambalin, 2024/02/25
[PATCH v6 12/41] Add memory region for BCM2837 RPiVid ASB, Sergey Kambalin, 2024/02/25
[PATCH v6 16/41] Add RPi4 RNG200, Sergey Kambalin, 2024/02/25
[PATCH v6 17/41] Implement BCM2838 thermal sensor, Sergey Kambalin, 2024/02/25