[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <86243f0e-3503-4ccb-ba3a-afd2c24a5472@solid-run.com>
Date: Wed, 12 Nov 2025 12:00:09 +0000
From: Josua Mayer <josua@...id-run.com>
To: "Rob Herring (Arm)" <robh@...nel.org>
CC: Sascha Hauer <s.hauer@...gutronix.de>, Lad Prabhakar
<prabhakar.mahadev-lad.rj@...renesas.com>, Thierry Reding
<thierry.reding@...il.com>, "dri-devel@...ts.freedesktop.org"
<dri-devel@...ts.freedesktop.org>, "linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>, "linux-input@...r.kernel.org"
<linux-input@...r.kernel.org>, Mikhail Anikin <mikhail.anikin@...id-run.com>,
Fabio Estevam <festevam@...il.com>, "imx@...ts.linux.dev"
<imx@...ts.linux.dev>, Conor Dooley <conor+dt@...nel.org>, David Airlie
<airlied@...il.com>, Yazan Shhady <yazan.shhady@...id-run.com>, Shawn Guo
<shawnguo@...nel.org>, Jessica Zhang <jessica.zhang@....qualcomm.com>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>, Simona Vetter
<simona@...ll.ch>, Neil Armstrong <neil.armstrong@...aro.org>, Jon Nettleton
<jon@...id-run.com>, "devicetree@...r.kernel.org"
<devicetree@...r.kernel.org>, Krzysztof Kozlowski
<krzysztof.kozlowski@...aro.org>, Laurent Pinchart
<laurent.pinchart@...asonboard.com>, Maxime Ripard <mripard@...nel.org>,
Dmitry Torokhov <dmitry.torokhov@...il.com>, Thomas Zimmermann
<tzimmermann@...e.de>, Pengutronix Kernel Team <kernel@...gutronix.de>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, Krzysztof
Kozlowski <krzk+dt@...nel.org>
Subject: Re: [PATCH v2 00/11] arm64: dts: add description for solidrun imx8mp
hummingboard-iiot
Am 10.11.25 um 15:34 schrieb Rob Herring (Arm):
> On Fri, 07 Nov 2025 12:46:07 +0100, Josua Mayer wrote:
>> This patchset mainl adds description for 3 SolidRun boards:
>> - i.MX8MP Hummingboard IIoT
>> - SolidSense N8 Compact
>> - i.MX8MM Hummingboard Ripple
>>
>> This includes dt bindings and a range of bug-fixes:
>>
>> - dt bindings for the boards
>> - fix to dsi panel bindings referncing panel-common and adding port
>> property
>> - cosmetic fix to some solidrun imx8mp boards for regulator gpio
>> polarity
>> - fix dsi hdmi bridge on hummingboard pulse
>> - compile dtbs with symbols to support overlays
>> - gpiod_set_value _can_sleep conversion for panel and touchscreen
>> drivers
>>
>> Open Questions:
>>
>> - How to describe HX3 USB-2.0/3.0 Hub placed on a USB-2.0-only Bus
>> (affects imx8mm-hummingboard-ripple.dts)
>> - Is "description for" implied not only on dt-bindings patches, but also
>> dts? E.g. is this commit subject acceptable?:
>> "arm64: dts: add solidrun solidsense-n8 board"
>>
>> Signed-off-by: Josua Mayer <josua@...id-run.com>
>> ---
>> Changes in v2:
>> - fix spelling mistakes in commit descriptions.
>> - remove redundant "binding for" from subject:
>> https://elixir.bootlin.com/linux/v6.17-rc3/source/Documentation/devicetree/bindings/submitting-patches.rst
>> (Reported-by: Krzysztof Kozlowski <krzk@...nel.org>)
>> - remove one useless comment from imx8mm-sr-som.dtsi to fix long line
>> warning.
>> - change ronbo panel binding to inherit panel-common and switch
>> additionalProperties to unevaluatedProperties.
>> (Reported-by: Krzysztof Kozlowski <krzk@...nel.org>)
>> - add dt binding for winstar lvds panel
>> - fix dtbs_check for dsi & lvds panel addons
>> - change n8 board dts comment-style in header
>> - collected ack on solidrun boards bindings patch (patch 1 in the series)
>> (Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>)
>> - added standard led label property to hb-iiot multi-purpose leds
>> ensuring consistent names in case lp5562 driver starts using it.
>> - Link to v1: https://lore.kernel.org/r/20251027-imx8mp-hb-iiot-v1-0-683f86357818@solid-run.com
>>
>> ---
>> Josua Mayer (11):
>> dt-bindings: arm: fsl: Add various solidrun i.mx8m boards
>> dt-bindings: display: panel: ronbo,rb070d30: panel-common ref
>> dt-bindings: panel: lvds: add Winstar WF70A8SYJHLNGA
>> Input: ilitek_ts_i2c: fix warning with gpio controllers that sleep
>> drm/panel: ronbo-rb070d30: fix warning with gpio controllers that sleep
>> arm64: dts: imx8mp-hummingboard-pulse/cubox-m: fix vmmc gpio polarity
>> arm64: dts: imx8mp-hummingboard-pulse: fix mini-hdmi dsi port reference
>> arm64: dts: imx8mp-sr-som: build dtbs with symbols for overlay support
>> arm64: dts: add description for solidrun imx8mp hummingboard-iiot
>> arm64: dts: add description for solidrun solidsense-n8 board
>> arm64: dts: add description for solidrun i.mx8mm som and evb
>>
>> Documentation/devicetree/bindings/arm/fsl.yaml | 9 +
>> .../bindings/display/panel/panel-lvds.yaml | 2 +
>> .../bindings/display/panel/ronbo,rb070d30.yaml | 14 +-
>> arch/arm64/boot/dts/freescale/Makefile | 15 +
>> .../dts/freescale/imx8mm-hummingboard-ripple.dts | 335 ++++++++
>> arch/arm64/boot/dts/freescale/imx8mm-sr-som.dtsi | 395 ++++++++++
>> .../dts/freescale/imx8mn-solidsense-n8-compact.dts | 853 +++++++++++++++++++++
>> arch/arm64/boot/dts/freescale/imx8mp-cubox-m.dts | 2 +-
>> ...hummingboard-iiot-panel-dsi-WJ70N3TYJHMNG0.dtso | 69 ++
>> ...ummingboard-iiot-panel-lvds-WF70A8SYJHLNGA.dtso | 105 +++
>> .../imx8mp-hummingboard-iiot-rs485-a.dtso | 18 +
>> .../imx8mp-hummingboard-iiot-rs485-b.dtso | 18 +
>> .../dts/freescale/imx8mp-hummingboard-iiot.dts | 712 +++++++++++++++++
>> .../imx8mp-hummingboard-pulse-common.dtsi | 2 +-
>> .../imx8mp-hummingboard-pulse-mini-hdmi.dtsi | 11 +-
>> drivers/gpu/drm/panel/panel-ronbo-rb070d30.c | 8 +-
>> drivers/input/touchscreen/ilitek_ts_i2c.c | 4 +-
>> 17 files changed, 2548 insertions(+), 24 deletions(-)
>> ---
>> base-commit: 607bcb1e539a7f2b2ba6832ba96fe3ad73b7401c
>> change-id: 20251026-imx8mp-hb-iiot-525b03beea62
>>
>> Best regards,
>> --
>> Josua Mayer <josua@...id-run.com>
>>
>>
>>
>
> My bot found new DTB warnings on the .dts files added or changed in this
> series.
>
> Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings
> are fixed by another series. Ultimately, it is up to the platform
> maintainer whether these warnings are acceptable or not. No need to reply
> unless the platform maintainer has comments.
>
> If you already ran DT checks and didn't see these error(s), then
> make sure dt-schema is up to date:
>
> pip3 install dtschema --upgrade
>
>
> This patch series was applied (using b4) to base:
> Base: base-commit 607bcb1e539a7f2b2ba6832ba96fe3ad73b7401c not known, ignoring
> Base: attempting to guess base-commit...
> Base: tags/v6.18-rc1 (exact match)
> Base: tags/v6.18-rc1 (use --merge-base to override)
>
> If this is not the correct base, please add 'base-commit' tag
> (or use b4 which does this automatically)
>
> New warnings running 'make CHECK_DTBS=y for arch/arm64/boot/dts/freescale/' for 20251107-imx8mp-hb-iiot-v2-0-d8233ded999e@...id-run.com:
>
> arch/arm64/boot/dts/freescale/imx8mp-hummingboard-iiot.dtb: accelerometer@53 (adi,adxl345): 'interrupt-names' is a dependency of 'interrupts-extended'
> from schema $id: http://devicetree.org/schemas/iio/accel/adi,adxl345.yaml
Ack. Adding missing interrupt-name in v3 ...
Powered by blists - more mailing lists