Re: [PATCH v3 04/10] Revert "usb: phy: tegra: don't call intotegra-ehci directly"

[Alan Stern]

On Fri, 10 May 2013, BVG Rao wrote:

> On 10 May 2013 19:56, "Alan Stern" <stern@xxxxxxxxxxxxxxxxxxx> wrote:
> >
> > On Fri, 10 May 2013, Venu Byravarasu wrote:
> >
> > > This reverts commit ee5d5499edb94cd03738a52a7e234b139da8fd72.
> > >
> > > Signed-off-by: Venu Byravarasu <vbyravarasu@xxxxxxxxxx>
> > > ---
> > > As PHY is being registered as separate driver, solution proposed with
> > > this patch cannot work anymore.
> > > Hence pushed patch 10 of this series to address the issue.
> >
> > You can't just revert a change like this -- doing so will recreate the
> > original problem that ee5d5499ed was meant to fix.
> >
> > You have to fix the problem _first_.  Then you can revert that commit.
> 
> Hi Alan,
> 
> I addressed that as part of patch 10 of the series.

I know -- that's what I was complaining about.  Patch 4 reverts the old
fix, and the new fix isn't added until patch 10.  So anybody who builds
a kernel using only patches 1 - 5 (for example) will end up with a
broken system.  This can easily happen during bisection testing.

You need to add the new fix either before or in the same patch as where 
the old fix is reverted.

Alan Stern

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/