From: Stefan Wahren <wahrenst@gmx.net>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Peter Robinson <pbrobinson@gmail.com>,
"Ivan T. Ivanov" <iivanov@suse.de>
Cc: 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: Wed, 27 Mar 2024 07:49:38 +0100 [thread overview]
Message-ID: <b044eec1-ae64-4de7-9ab0-e3fe756131ef@gmx.net> (raw)
In-Reply-To: <20240326195807.15163-1-laurent.pinchart@ideasonboard.com>
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?
>
> [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
>
next parent reply other threads:[~2024-03-27 6:49 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20240326195807.15163-1-laurent.pinchart@ideasonboard.com>
2024-03-27 6:49 ` Stefan Wahren [this message]
2024-03-27 23:37 ` [PATCH v2 0/3] dt-bindings: arm: bcm: raspberrypi,bcm2835-firmware: Drive-by fixes Laurent Pinchart
2024-03-28 7:38 ` Ivan T. Ivanov
2024-04-02 8:58 ` Ivan T. Ivanov
2024-04-02 19:52 ` Stefan Wahren
2024-04-02 20:08 ` Laurent Pinchart
2024-04-02 20:18 ` Florian Fainelli
2024-04-02 20:32 ` Laurent Pinchart
2024-04-02 20:52 ` Florian Fainelli
2024-04-02 20:22 ` Stefan Wahren
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=b044eec1-ae64-4de7-9ab0-e3fe756131ef@gmx.net \
--to=wahrenst@gmx.net \
--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=laurent.pinchart@ideasonboard.com \
--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 \
/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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox