On Mon, Feb 16, 2026 at 04:22:14PM +0000, Luca Leonardo Scorcia wrote: > Remove the dedicated "mediatek,mt8167-dsi" compatible from the device list and > describe it as compatible with mt2701 instead. It is safe to do so because:
You are not doing what you wrote. The dedicated mediatek,mt8167-dsi is still there. And if you want to describe mediatek,mt8167-dsi with OTHER compatible (mt2701), it is a NAK. It is wrong and not allowed by writing bindings doc. You just added fallback, didn't you? Please wrap commit message according to Linux coding style / submission process (neither too early nor over the limit): https://elixir.bootlin.com/linux/v6.4-rc1/source/Documentation/process/submitting-patches.rst#L597 Please run scripts/checkpatch.pl on the patches and fix reported warnings. After that, run also 'scripts/checkpatch.pl --strict' on the patches and (probably) fix more warnings. Some warnings can be ignored, especially from --strict run, but the code here looks like it needs a fix. Feel free to get in touch if the warning is not clear. > > - Bootloader doesn't rely on this single compatible; and Does not matter. You still CANNOT remove a compatible. If bootloader starts to rely on this single compatible, you add it back? No. > - There was never any upstreamed devicetree using this single compatible; and > - The MT8167 DSI Controller is fully compatible with the one found in MT2701. > > Fixes: 8867c4b39361 ("dt-bindings: display: mediatek: dsi: add documentation > for MT8167 SoC") > There is never a blank line between tags. > Signed-off-by: Luca Leonardo Scorcia <[email protected]> > --- > .../devicetree/bindings/display/mediatek/mediatek,dsi.yaml | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) Best regards, Krzysztof
