From: Laurent Pinchart <laurent.pinchart@ideasonboard.com> To: Stefan Wahren <wahrenst@gmx.net> Cc: Peter Robinson <pbrobinson@gmail.com>, "Ivan T. Ivanov" <iivanov@suse.de>, Dave Stevenson <dave.stevenson@raspberrypi.com>, Naushir Patuck <naush@raspberrypi.com>, Bartosz Golaszewski <brgl@bgdev.pl>, Broadcom internal kernel review list <bcm-kernel-feedback-list@broadcom.com>, Conor Dooley <conor+dt@kernel.org>, Florian Fainelli <florian.fainelli@broadcom.com>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Linus Walleij <linus.walleij@linaro.org>, Nicolas Saenz Julienne <nsaenz@kernel.org>, Ray Jui <rjui@broadcom.com>, Rob Herring <robh@kernel.org>, Scott Branden <sbranden@broadcom.com>, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-rpi-kernel@lists.infradead.org, "u-boot@lists.denx.de" <u-boot@lists.denx.de> Subject: Re: [PATCH v2 0/3] dt-bindings: arm: bcm: raspberrypi,bcm2835-firmware: Drive-by fixes Date: Thu, 28 Mar 2024 01:37:00 +0200 [thread overview] Message-ID: <20240327233700.GA21080@pendragon.ideasonboard.com> (raw) In-Reply-To: <b044eec1-ae64-4de7-9ab0-e3fe756131ef@gmx.net> On Wed, Mar 27, 2024 at 07:49:38AM +0100, Stefan Wahren wrote: > Hi, > > [add Peter and Ivan] > > Am 26.03.24 um 20:58 schrieb Laurent Pinchart: > > Hello, > > > > This small series includes a few drive-by fixes for DT validation > > errors. > > > > The first patch has been posted previously in v1 ([1], and now addresses > > a small review comment. I think it's good to go. > > > > The next two patches address the same issue as "[PATCH 1/2] dt-bindings: > > arm: bcm: raspberrypi,bcm2835-firmware: Add missing properties" ([2]), > > but this time with a (hopefully) correct approach. Patch 2/3 starts by > > fixing the raspberrypi-bcm2835-firmware driver, removing the need for DT > > properties that are specified in bcm2835-rpi.dtsi but not documented in > > the corresponding bindings. Patch 3/3 can then drop those properties, > > getting rid of the warnings. > > since this series drops properties from the device tree, does anyone > have the chance to test it with a recent U-Boot? I don't have U-Boot running with my RPi, so I would appreciate if someone could help :-) > > [1] https://lore.kernel.org/linux-arm-kernel/20240326004902.17054-3-laurent.pinchart@ideasonboard.com/ > > [2] https://lore.kernel.org/linux-arm-kernel/20240326004902.17054-2-laurent.pinchart@ideasonboard.com/ > > > > Laurent Pinchart (3): > > dt-bindings: arm: bcm: raspberrypi,bcm2835-firmware: Add gpio child > > node > > firmware: raspberrypi: Use correct device for DMA mappings > > ARM: dts: bcm283x: Drop unneeded properties in the bcm2835-firmware > > node > > > > .../arm/bcm/raspberrypi,bcm2835-firmware.yaml | 30 +++++++++++++++++++ > > .../gpio/raspberrypi,firmware-gpio.txt | 30 ------------------- > > arch/arm/boot/dts/broadcom/bcm2835-rpi.dtsi | 4 --- > > drivers/firmware/raspberrypi.c | 7 +++-- > > 4 files changed, 34 insertions(+), 37 deletions(-) > > delete mode 100644 Documentation/devicetree/bindings/gpio/raspberrypi,firmware-gpio.txt > > -- Regards, Laurent Pinchart
WARNING: multiple messages have this Message-ID (diff)
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com> To: Stefan Wahren <wahrenst@gmx.net> Cc: Peter Robinson <pbrobinson@gmail.com>, "Ivan T. Ivanov" <iivanov@suse.de>, Dave Stevenson <dave.stevenson@raspberrypi.com>, Naushir Patuck <naush@raspberrypi.com>, Bartosz Golaszewski <brgl@bgdev.pl>, Broadcom internal kernel review list <bcm-kernel-feedback-list@broadcom.com>, Conor Dooley <conor+dt@kernel.org>, Florian Fainelli <florian.fainelli@broadcom.com>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Linus Walleij <linus.walleij@linaro.org>, Nicolas Saenz Julienne <nsaenz@kernel.org>, Ray Jui <rjui@broadcom.com>, Rob Herring <robh@kernel.org>, Scott Branden <sbranden@broadcom.com>, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-rpi-kernel@lists.infradead.org, "u-boot@lists.denx.de" <u-boot@lists.denx.de> Subject: Re: [PATCH v2 0/3] dt-bindings: arm: bcm: raspberrypi,bcm2835-firmware: Drive-by fixes Date: Thu, 28 Mar 2024 01:37:00 +0200 [thread overview] Message-ID: <20240327233700.GA21080@pendragon.ideasonboard.com> (raw) In-Reply-To: <b044eec1-ae64-4de7-9ab0-e3fe756131ef@gmx.net> On Wed, Mar 27, 2024 at 07:49:38AM +0100, Stefan Wahren wrote: > Hi, > > [add Peter and Ivan] > > Am 26.03.24 um 20:58 schrieb Laurent Pinchart: > > Hello, > > > > This small series includes a few drive-by fixes for DT validation > > errors. > > > > The first patch has been posted previously in v1 ([1], and now addresses > > a small review comment. I think it's good to go. > > > > The next two patches address the same issue as "[PATCH 1/2] dt-bindings: > > arm: bcm: raspberrypi,bcm2835-firmware: Add missing properties" ([2]), > > but this time with a (hopefully) correct approach. Patch 2/3 starts by > > fixing the raspberrypi-bcm2835-firmware driver, removing the need for DT > > properties that are specified in bcm2835-rpi.dtsi but not documented in > > the corresponding bindings. Patch 3/3 can then drop those properties, > > getting rid of the warnings. > > since this series drops properties from the device tree, does anyone > have the chance to test it with a recent U-Boot? I don't have U-Boot running with my RPi, so I would appreciate if someone could help :-) > > [1] https://lore.kernel.org/linux-arm-kernel/20240326004902.17054-3-laurent.pinchart@ideasonboard.com/ > > [2] https://lore.kernel.org/linux-arm-kernel/20240326004902.17054-2-laurent.pinchart@ideasonboard.com/ > > > > Laurent Pinchart (3): > > dt-bindings: arm: bcm: raspberrypi,bcm2835-firmware: Add gpio child > > node > > firmware: raspberrypi: Use correct device for DMA mappings > > ARM: dts: bcm283x: Drop unneeded properties in the bcm2835-firmware > > node > > > > .../arm/bcm/raspberrypi,bcm2835-firmware.yaml | 30 +++++++++++++++++++ > > .../gpio/raspberrypi,firmware-gpio.txt | 30 ------------------- > > arch/arm/boot/dts/broadcom/bcm2835-rpi.dtsi | 4 --- > > drivers/firmware/raspberrypi.c | 7 +++-- > > 4 files changed, 34 insertions(+), 37 deletions(-) > > delete mode 100644 Documentation/devicetree/bindings/gpio/raspberrypi,firmware-gpio.txt > > -- Regards, Laurent Pinchart _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2024-03-27 23:37 UTC|newest] Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top 2024-03-26 19:58 [PATCH v2 0/3] dt-bindings: arm: bcm: raspberrypi,bcm2835-firmware: Drive-by fixes Laurent Pinchart 2024-03-26 19:58 ` Laurent Pinchart 2024-03-26 19:58 ` [PATCH v2 1/3] dt-bindings: arm: bcm: raspberrypi,bcm2835-firmware: Add gpio child node Laurent Pinchart 2024-03-26 19:58 ` Laurent Pinchart 2024-04-04 16:46 ` Florian Fainelli 2024-04-04 16:46 ` Florian Fainelli 2024-03-26 19:58 ` [PATCH v2 2/3] firmware: raspberrypi: Use correct device for DMA mappings Laurent Pinchart 2024-03-26 19:58 ` Laurent Pinchart 2024-04-04 16:46 ` Florian Fainelli 2024-04-04 16:46 ` Florian Fainelli 2024-03-26 19:58 ` [PATCH v2 3/3] ARM: dts: bcm283x: Drop unneeded properties in the bcm2835-firmware node Laurent Pinchart 2024-03-26 19:58 ` Laurent Pinchart 2024-04-04 16:47 ` Florian Fainelli 2024-04-04 16:47 ` Florian Fainelli 2024-03-27 6:49 ` [PATCH v2 0/3] dt-bindings: arm: bcm: raspberrypi,bcm2835-firmware: Drive-by fixes Stefan Wahren 2024-03-27 6:49 ` Stefan Wahren 2024-03-27 23:37 ` Laurent Pinchart [this message] 2024-03-27 23:37 ` Laurent Pinchart 2024-03-28 7:38 ` Ivan T. Ivanov 2024-04-02 8:58 ` Ivan T. Ivanov 2024-04-02 8:58 ` Ivan T. Ivanov 2024-04-02 19:52 ` Stefan Wahren 2024-04-02 19:52 ` Stefan Wahren 2024-04-02 20:08 ` Laurent Pinchart 2024-04-02 20:08 ` Laurent Pinchart 2024-04-02 20:18 ` Florian Fainelli 2024-04-02 20:18 ` Florian Fainelli 2024-04-02 20:32 ` Laurent Pinchart 2024-04-02 20:32 ` Laurent Pinchart 2024-04-02 20:52 ` Florian Fainelli 2024-04-02 20:52 ` Florian Fainelli 2024-04-02 20:22 ` Stefan Wahren 2024-04-02 20:22 ` Stefan Wahren 2024-04-02 20:22 ` Florian Fainelli 2024-04-02 20:22 ` Florian Fainelli
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20240327233700.GA21080@pendragon.ideasonboard.com \ --to=laurent.pinchart@ideasonboard.com \ --cc=bcm-kernel-feedback-list@broadcom.com \ --cc=brgl@bgdev.pl \ --cc=conor+dt@kernel.org \ --cc=dave.stevenson@raspberrypi.com \ --cc=devicetree@vger.kernel.org \ --cc=florian.fainelli@broadcom.com \ --cc=iivanov@suse.de \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=linus.walleij@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-rpi-kernel@lists.infradead.org \ --cc=naush@raspberrypi.com \ --cc=nsaenz@kernel.org \ --cc=pbrobinson@gmail.com \ --cc=rjui@broadcom.com \ --cc=robh@kernel.org \ --cc=sbranden@broadcom.com \ --cc=u-boot@lists.denx.de \ --cc=wahrenst@gmx.net \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.