On Tue, Nov 15, 2022 at 05:54:12PM +0000, Niyas Sait wrote: > This is a proposal for adding ACPI support to pin controller. > > The patch supports following resources introduced in ACPI from v6.2 > > - PinFunction > - PinConfig > - PinGroupFunction > - PinGroupConfig > - PinGroup > > The patch has been tested on NXP I.MX8MP Plus platform with ACPI. Cover letter doesn't point to the ASL code you are using. I have some questions about that: 1) why do you need specific _DSD() for the pin mappings? 2) wgy you need vendor data for some of Pin*() resources? -- With Best Regards, Andy Shevchenko