On Mon, Oct 11, 2021 at 8:43 AM Markus Schneider-Pargmann msp@baylibre.com wrote:
Hi,
On Mon, Oct 11, 2021 at 08:36:18AM -0500, Rob Herring wrote:
On Mon, 11 Oct 2021 11:46:18 +0200, Markus Schneider-Pargmann wrote:
DP_INTF is a similar functional block to mediatek,dpi but is different in that it serves the DisplayPort controller on mediatek SoCs and uses different clocks. Therefore this patch creates a new binding file for this functional block.
Signed-off-by: Markus Schneider-Pargmann msp@baylibre.com
Notes: Changes v3 -> v4: - Fixed clock names in the example as the clock patch series is merged into next now - Add missing ports decleration to the example
Changes v1 -> v2: - Move the devicetree binding from mediatek,dpi into its own binding file..../display/mediatek/mediatek,dpintf.yaml | 86 +++++++++++++++++++ 1 file changed, 86 insertions(+) create mode 100644 Documentation/devicetree/bindings/display/mediatek/mediatek,dpintf.yaml
My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' on your patch (DT_CHECKER_FLAGS is new in v5.13):
yamllint warnings/errors:
dtschema/dtc warnings/errors: Documentation/devicetree/bindings/display/mediatek/mediatek,dpintf.example.dts:21:18: fatal error: dt-bindings/clock/mt8195-clk.h: No such file or directory 21 | #include <dt-bindings/clock/mt8195-clk.h> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
The mt8195 clock series was already merged and is available in linux-next. I checked with make dt_binding_check before sending.
The bot has no way of knowing that, and the dependency is not called out in this patch for me to know that when reviewing the failures.
Rob