On Wed, Mar 05, 2025 at 11:50:27AM +0100, Csókás Bence wrote: > This duplicates a lot of `mchp_tc_count_function_write()`. I'd much rather > have this code in a separate function called something like > `mchp_tc_setup_channels()`, that, depending on `priv->qdec_mode`, sets up > the BMR, CCR and CMRs, and then have both probe() and function_write() call > it. Or alternatively, have probe() call function_write() at the end, but > that's not as nice. Hi Bence, I agree, the mchp_tc_count_function_write() could be cleaned up and divided into separate functions dedicated to configuring each mode (perhaps regmap_update_bits() could be leveraged too), but that would be a much more invasive update. For the sake of making backporting easy to address this particular issue, I've kept the changes here localized to just the probe() function. Once the fix is merged, someone can try tackling a more proper refactor of the mchp_tc_count_function_write() code. William Breathitt Gray
Attachment:
signature.asc
Description: PGP signature
