[PATCH 1/7] drivers: net: fsl-mc: remove useless assignment of variable

Ramon Fried rfried.dev at gmail.com
Thu Jan 12 20:01:15 CET 2023


On Thu, 12 Jan 2023, 12:58 Ioana Ciornei, <ioana.ciornei at nxp.com> wrote:

> On Tue, Jan 10, 2023 at 07:04:00PM +0200, Ramon Fried wrote:
> > On Thu, Jan 5, 2023 at 5:03 PM Ioana Ciornei <ioana.ciornei at nxp.com>
> wrote:
> > >
> > > The cur_ptr variable is set to the start of the log buffer but then
> it's
> > > not used. Just remove the assignment altogether.
> > >
> > > Signed-off-by: Ioana Ciornei <ioana.ciornei at nxp.com>
> > > ---
> > >  drivers/net/fsl-mc/mc.c | 1 -
> > >  1 file changed, 1 deletion(-)
> > >
> > > diff --git a/drivers/net/fsl-mc/mc.c b/drivers/net/fsl-mc/mc.c
> > > index 6b36860187ca..66fcb48ebd55 100644
> > > --- a/drivers/net/fsl-mc/mc.c
> > > +++ b/drivers/net/fsl-mc/mc.c
> > > @@ -1796,7 +1796,6 @@ static void mc_dump_log(void)
> > >         if (size > bytes_end) {
> > >                 print_k_bytes(cur_ptr, &bytes_end);
> > >
> > > -               cur_ptr = buf;
> > >                 size -= bytes_end;
> > >         }
> > >
> > > --
> > > 2.25.1
> > >
> > Reviewed-by: Ramon Fried <rfried.dev at gmail.com>
>
> Thanks for the reviews!
>
> These kind of network patches go through the u-boot-net tree or is there
> another path?
>
Here is fine.

>
> Ioana
>


More information about the U-Boot mailing list