On Mon, Jan 30, 2023 at 09:59:30PM +0100, Levente Révész wrote: > Hi All, > > The pca953x driver supports many chips. These all have the basic 4 > registers: input, output, invert and direction. Most of them have > additional registers for various functions, like pull-up and > pull-down control, interrupt status and mask, hardware debounce. > > Different chip types have various register layouts. These layouts differ > even in the basic 4 registers. > > This patch series > > 1. Cleans up chip type handling, and > 2. Replaces register address calculations. The proposed scheme > works with every register of every chip type. > > These changes make it possible to add support for extended functions for > more chip types, including: > > * Interrupt mask for PCA950X, PCA9698, PCA957X and XRA120X > * Interrupt status for PCA957X and XRA120X > * Bias for PCA957X and XRA120X > * Debounce for PCAL65XX and XRA120X > > References: > 1. Previous discussion about the chip types > https://lore.kernel.org/linux-gpio/Y1q52efyv93%2Fz8BC@xxxxxxxxxxxxxxxxxx/ > 2. An overview of pca953x chip types > https://lore.kernel.org/linux-gpio/67bba210-09ac-32fb-bb97-8bfc40c2c200@xxxxxxxxxx/ Thank you for the update, I'll look into the series later on. Just to tell you that your cover letter is from something else according to the email chain (the 1/3 isn't linked to 0/3). -- With Best Regards, Andy Shevchenko