Hi Anson, On Mon, Jul 16, 2018 at 3:14 AM, Anson Huang <anson.huang@xxxxxxx> wrote: > OK, since i.MX7D has same GPIO type as i.MX35, to make it simple, I just added > a flag to indicate whether it supports save/restore, only i.MX7D enables In imx7s.dtsi the gpio nodes have: compatible = "fsl,imx7d-gpio", "fsl,imx35-gpio"; If you add fsl,imx7d-gpio entry in the gpio driver, then the match will be done against "fsl,imx7d-gpio" since it is more specific. Then in the mx8 dts you can add: compatible = "fsl,imx8m-gpio", "fsl,imx7d-gpio"; and it will also match the more generic "fsl,imx7d-gpio" compatible. > the flag now, since other i.MX8 SoCs' dts is NOT upstreamed yet, so I did NOT add > support for i.MX8 SoCs, please help review V2 patch, thanks. I did not see the v2. Did you put me on Cc? -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html