On Sat, 30 Mar 2019 13:01:41 +0100 Luc Van Oostenryck <luc.vanoostenryck@xxxxxxxxx> wrote: > OK, I see. IMO, it would be better to have 2 patches for this: > one for the redundant assignment to pin6 and anotther one for > the IS_ENABLED() change but feel free, if needed, to add my I agree with it being separate patches. I'm big on the "one patch accomplishes one task" ideology. -- Steve