Re: [PATCH] staging: et131x: Fix errors caused by phydev->addr accesses before initialisation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Aug 09, 2014 at 09:39:17PM -0700, Greg KH wrote:
> On Tue, Aug 05, 2014 at 11:57:06PM +0100, Mark Einon wrote:
> > Fix two reported bugs, caused by et131x_adapter->phydev->addr being accessed
> > before it is initialised, by:
> >
> > - letting et131x_mii_write() take a phydev address, instead of using the one
> >   stored in adapter by default. This is so et131x_mdio_write() can use it's own
> >   addr value.
> > - removing implementation of et131x_mdio_reset(), as it's not needed.
> > - moving a call to et131x_disable_phy_coma() in et131x_pci_setup(), which uses
> >   phydev->addr, until after the mdiobus has been registered.
> >
> > Link: https://bugzilla.kernel.org/show_bug.cgi?id=80751
> > Link: https://bugzilla.kernel.org/show_bug.cgi?id=77121
> > Cc: stable@xxxxxxxxxxxxxxx
> > Signed-off-by: Mark Einon <mark.einon@xxxxxxxxx>
> > ---
> >  drivers/staging/et131x/et131x.c | 68 ++++++++++++++++-------------------------
> >  1 file changed, 27 insertions(+), 41 deletions(-)
>
> I got 3 copies of this patch, all look to be different?
>
> I don't know what to do, so I've deleted them all, please resend the
> version you want me to apply.

Your email server rejected the first two as I sent them from home, not
authenticating myself with my email server. I assumed you didn't receive them.
I resent the patch from a different address, which may explain the differences.

Sorry for the noise.

Cheers,

Mark
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux