On Tue, Feb 7, 2023 at 12:50 PM Rob Herring <robh@xxxxxxxxxx> wrote: > > On Mon, Feb 06, 2023 at 07:50:13AM -0600, Danny Kaehn wrote: > > This is a USB HID device which includes an I2C controller and 8 GPIO pins. > > > > The binding allows describing the chip's gpio and i2c controller in DT > > using the subnodes named "gpio" and "i2c", respectively. This is > > intended to be used in configurations where the CP2112 is permanently > > connected in hardware. > > My comments on v3 still apply. Please slow down your pace of sending new > versions so folks have change to review. > > Rob Thanks for the correction -- I definitely see how that could be frustrating / problematic. (until now I'd thought the ideal case was that comments were addressed as soon as possible, so that folks wouldn't be reviewing now-obsolete code that might change due to other comments anyways) Thanks, Danny Kaehn