Hi Marek, On Tue, Oct 29, 2024 at 5:16 PM Marek Vasut <marex@xxxxxxx> wrote: > So you wouldn't have to write three compatible strings for the 6sl/sll , > but only two ? I.e. this: > > compatible = "fsl,imx6sl-lcdif", "fsl,imx28-lcdif"; > compatible = "fsl,imx6sll-lcdif", "fsl,imx28-lcdif"; i.MX6SL and i.MX6SLL have a more advanced LCDIF version than the one on i.MX28. The LCDIF block on i.MX6SL/i.MX6SLL has both overlay and CRC support, corresponding to the MXSFB_V6 of the mxsfb_drv driver. That's why it is better to have the compatibles as: compatible = "fsl,imx6sl-lcdif", "fsl,imx6sx-lcdif", "fsl,imx28-lcdif"; compatible = "fsl,imx6sll-lcdif", "fsl,imx6sx-lcdif", "fsl,imx28-lcdif"; This way they can match with a more specific fsl,imx6sx-lcdif entry. If an OS does not implement the fsl,imx6sx-lcdif match, it can still match with fsl,imx28-lcdif as a fallback, keeping the DT compatibility. I missed updating the binding example. I will wait for more feedback before sending a v4. Thanks
