Re: [PATCH v2 0/4] bulk: Replace TARGET_FMT_plx by HWADDR_PRIx

[BALATON Zoltan]

On Fri, 20 Jan 2023, Thomas Huth wrote:
> On 11/01/2023 09.39, Philippe Mathieu-Daudé wrote:
> > Since v1:
> > - Fix checkpatch style violations
> > - Use HWADDR_PRIx instead of HWADDR_FMT_plx (Zoltan)
> >
> > Supersedes: <20230110212947.34557-1-philmd@linaro.org>
> >              "bulk: Rename TARGET_FMT_plx -> HWADDR_FMT_plx"
> >
> > Philippe Mathieu-Daudé (4):
> >    hw: Remove hardcoded tabs (code style)
> >    bulk: Coding style fixes
> >    bulk: Replace TARGET_FMT_plx -> HWADDR_PRIx
> >    bulk: Prefix '0x' to hex values displayed with HWADDR_PRIx format
>
> Big sorry, I picked up v1 for my last pull request before I saw that there is 
> a v2. But IMHO it's ok to have a separate macro with a %016 included, so I'd 
> rather tend to keep HWADDR_FMT_plx. Anyway, if you consider the other changes 
> in your series important enough, please rebase them. Sorry again for the 
> additional work that this might cause.

I don't mind too much so maybe isn't worth the churn to have another patch 
for this. I'll let Philippe decide what he wants to do with it.

Regards,
BALATON Zoltan