rpi5: Use devicetree to retrieve board revision 68/307168/2
authorIvan T. Ivanov <iivanov@suse.de>
Tue, 23 Jan 2024 08:07:55 +0000 (10:07 +0200)
committerMarek Szyprowski <m.szyprowski@samsung.com>
Thu, 14 Mar 2024 08:31:56 +0000 (09:31 +0100)
Firmware on RPi5 return error on board revision query
through firmware interface, but on the other hand it fills
"linux,revision" in "system" node, so use it to detect board
revision.

system {
linux,revision = <0xc04170>;
linux,serial = <0x6cf44e80 0x3c533ede>;
};

Reviewed-by: Matthias Brugger <mbrugger@suse.com>
Tested-by: Jens Maus <mail@jens-maus.de>
Tested-by: Darko Alavanja <darko.alavanja@konsulko.com>
Signed-off-by: Ivan T. Ivanov <iivanov@suse.de>
Signed-off-by: Matthias Brugger <mbrugger@suse.com>
[backport of the commit 08a5b4bd6cc14ad999a2d9a389f8b8b9dcea1682 from mainline]
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I9e5a1698456f03891ea0fb0c375f5386e8f447be

board/raspberrypi/rpi/rpi.c

index 1d08e56..1a5f645 100644 (file)
@@ -200,6 +200,11 @@ static const struct rpi_model rpi_models_new_scheme[] = {
                DTB_DIR "bcm2711-rpi-cm4.dtb",
                true,
        },
+       [0x17] = {
+               "5 Model B",
+               DTB_DIR "bcm2712-rpi-5-b.dtb",
+               true,
+       },
 };
 
 static const struct rpi_model rpi_models_old_scheme[] = {
@@ -507,15 +512,27 @@ static void get_board_revision(void)
        int ret;
        const struct rpi_model *models;
        uint32_t models_count;
+       ofnode node;
 
        BCM2835_MBOX_INIT_HDR(msg);
        BCM2835_MBOX_INIT_TAG(&msg->get_board_rev, GET_BOARD_REV);
 
        ret = bcm2835_mbox_call_prop(BCM2835_MBOX_PROP_CHAN, &msg->hdr);
        if (ret) {
-               printf("bcm2835: Could not query board revision\n");
                /* Ignore error; not critical */
-               return;
+               node = ofnode_path("/system");
+               if (!ofnode_valid(node)) {
+                       printf("bcm2835: Could not find /system node\n");
+                       return;
+               }
+
+               ret = ofnode_read_u32(node, "linux,revision", &revision);
+               if (ret) {
+                       printf("bcm2835: Could not find linux,revision\n");
+                       return;
+               }
+       } else {
+               revision = msg->get_board_rev.body.resp.rev;
        }
 
        /*
@@ -529,7 +546,6 @@ static void get_board_revision(void)
         * http://www.raspberrypi.org/forums/viewtopic.php?f=63&t=98367&start=250
         * http://www.raspberrypi.org/forums/viewtopic.php?f=31&t=20594
         */
-       revision = msg->get_board_rev.body.resp.rev;
        if (revision & 0x800000) {
                rev_scheme = 1;
                rev_type = (revision >> 4) & 0xff;