Re: [3/3] pinctrl: mcp23s08: Combine two function calls in mcp23s08_dbg_show()

From: SF Markus Elfring
Date: Wed Nov 29 2017 - 09:04:43 EST


>> * Print a line break together with other data in a single function call.
>>
>> * Adjust indentation.
â
>
> This does not apply.
> Possibly depends on 2/3.

Partly, yes.


How do you think about the following possibilities?

* Omit two context lines at the end from this update step.
or
* Refer to the jump label âdoneâ (instead of renaming it to âunlockâ before).


Would you try once more to integrate the shown small change?

Regards,
Markus