Re: [PATCH net] net: dsa: mt7530: Use NULL instead of plain integer

From: Sean Wang
Date: Tue Apr 03 2018 - 13:32:25 EST


On Tue, 2018-04-03 at 10:02 -0700, Florian Fainelli wrote:
> On 04/02/2018 07:18 PM, Sean Wang wrote:
> > On Mon, 2018-04-02 at 16:24 -0700, Florian Fainelli wrote:
> >> We would be passing 0 instead of NULL as the rsp argument to
> >> mt7530_fdb_cmd(), fix that.
> >>
> >
> > Acked-by: Sean Wang <sean.wang@xxxxxxxxxxxx>
> >
> > BTW, does the part of the commit message should be updated with "passing
> > NULL instead of 0"?
>
> I don't follow you, the commit message indicates what we were doing
> which implies it was wrong and fixes it. Would you want me to reword
> that part?

thanks for the explanation.

the commit message is clear and okay for me now