Re: [PATCH v2] pinctrl: starfive: use dynamic GPIO base allocation

[Ali Tariq <alitariq45892@gmail.com>]

Hi Emil,

Thanks for the review and feedback.

Quoting Emil Renner Berthing:
 > 1) The justification for why this is no longer needed should be in 
the commit
 > message, and not just in a random thread that will soon be forgotten.

My apologies for the confusion. I did add the removal of unused 
variables as a bullet point in the v2 changelog (which is in the email 
body), but I failed to include the necessary justification for this 
removal in the main commit message itself.

You are correct, the justification for removing `gc_base` and associated 
macros (because we now use `sfp->gc.base = -1` for dynamic allocation) 
belongs in the permanent commit message.

I will update the v2 commit message to clearly state the removal of 
macros and unused definitions (before "Signed-off-by") in the v3 submission.

Quoting Emil Renner Berthing:
 > 2) Please don't send new revisions as replies. Start a new mail thread.

Understood. I will send the next revision as a new thread.

Quoting Emil Renner Berthing:
 > The code changes look good though, so once that is fixed you can add
 >
 > Reviewed-by: Emil Renner Berthing <emil.renner.berthing@canonical.com>

Understood. I will include your Reviewed-by tag in the v3 patch.

Thanks again for the review!

-Ali