On Thu, Jul 06, 2023 at 02:12:12PM +0000, Hawkins, Nick wrote: > Greetings Krzysztof, > > Thank you for the feedback. I see that due to a patch conflict I > reintroduced some of the alignment issues you had me fix in > a previous version. This was a mistake and I will correct this. > > > > v5: > > > *Removed use of gpio-gxp in favor of just supporting > > > hpe,gxp-gpio-pl for now as the full gpio-gxp will > > > require a much larger patchset > > > Bindings describe hardware, not drivers, and should be rather complete. > > This patch is intended to still cover the hardware interface between our > BMC and our CPLD which gathers GPIO for us. The part of the binding I > removed was a completely separate interface with different mechanisms > for reading GPIOs. With that said I could keep these two interfaces > separate in yaml files: Having a yaml for hpe,gxp-gpio and another for > hpe,gxp-gpio-pl. Would this be a better approach? If they are independent (and it sounds like they are), then yes. Rob
