An issue[1] related to how the V4L2_FWNODE_BUS_TYPE_PARALLEL flag is mis-used was found in recent addition to the anx7625 driver. As used currently CPI (camera parallel interface) and DPI (display parallel interface) would share the V4L2_FWNODE_BUS_TYPE_PARALLEL enum. I think that would be perfectly functional, but it is not what V4L2_FWNODE_BUS_TYPE_PARALLEL is documented to represent. As far as I can see it's only intended to represent CPI. Instead of having V4L2_FWNODE_BUS_TYPE_PARALLEL represent two standards, I think they should be split. And possibly V4L2_FWNODE_BUS_TYPE_PARALLEL should be renamed for CPI, but that is a separate story. This would provide for the neatest and most legible solution. If this solution is implemented, this range would be incorrect. Additionally the snippet reverted in 2/2 of this series would no longer be valid. As it stands V4L2_FWNODE_BUS_TYPE_PARALLEL was used to represent DPI due to not being caught in the review process. In order to not introduce this issue into the ABI, let's revert the changes to the anx7625 dt-binding related to this. [1] https://lore.kernel.org/all/YiTruiCIkyxs3jTC@xxxxxxxxxxxxxxxxxxxxxxxxxx/ Robert Foss (2): dt-bindings: drm/bridge: anx7625: Revert DPI support Revert "arm64: dts: mt8183: jacuzzi: Fix bus properties in anx's DSI endpoint" .../display/bridge/analogix,anx7625.yaml | 19 +------------------ .../dts/mediatek/mt8183-kukui-jacuzzi.dtsi | 2 -- 2 files changed, 1 insertion(+), 20 deletions(-) -- 2.32.0
