Hi Jonathan, thanks for your review. I have one comment. Jonathan Cameron <jic23@xxxxxxxxxx> schrieb am Sa, 06. Mai 17:04: > > + int scale; /* int part of scale */ > > + int scale2; /* nano part of scale */ > > + int offset; /* int part of offset */ > > + int offset2; /* nano part of offset */ > > + struct gpio_desc *gpiod_reset; /* reset */ > > + int irq; /* end of conversion irq */ > > Only needed in probe, no need for a copy in here. It's also used in mpr_read_pressure() to distinguish the two possible operation modes: - waiting for an interrupt - reading in a loop until status indicates data ready Andreas