Re: [PATCH v1 6/8] watchdog: f71808e_wdt: consolidate variant handling into single array

2020-06-30 Thread Guenter Roeck
On Thu, Jun 11, 2020 at 09:17:47PM +0200, Ahmad Fatoum wrote: > Driver has two big switches: one to map hardware ID to an enum constant > one more to map the enum constant to the variant's pin configuration > routine. > > Drop the enum and give every variant an array entry identifying it. > This a

[PATCH v1 6/8] watchdog: f71808e_wdt: consolidate variant handling into single array

2020-06-11 Thread Ahmad Fatoum
Driver has two big switches: one to map hardware ID to an enum constant one more to map the enum constant to the variant's pin configuration routine. Drop the enum and give every variant an array entry identifying it. This arguably simplifies the code by making it a little more compact. Signed-of