From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paulraj, Sandeep Date: Sat, 10 Oct 2009 13:24:13 -0500 Subject: [U-Boot] [PATCH v3] TI: OMAP3: Overo Tobi ethernet support In-Reply-To: <4AD0CB2D.2010003@googlemail.com> References: <20090911204750.GA22246@lixom.net> <20090923145556.GA28659@lixom.net> <20090926211434.GB21538@lixom.net> <5e088bd90910050656l265d6fav32fe3bd86fd06250@mail.gmail.com> <20091008065902.GA12935@lixom.net> <0554BEF07D437848AF01B9C9B5F0BC5D93164B86@dlee01.ent.ti.com> <4ACE1C78.9090901@gmail.com> <20091008192149.GA18486@lixom.net> <0554BEF07D437848AF01B9C9B5F0BC5D93268748@dlee01.ent.ti.com> <4AD0CB2D.2010003@googlemail.com> Message-ID: <0554BEF07D437848AF01B9C9B5F0BC5D93268795@dlee01.ent.ti.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de > > I'm a little confused :-) > > I realized when I was trying to apply this patch that this is already > part of the u-boot-ti and u-boot-arm trees. > > > > I am referring to > > http://git.denx.de/?p=u-boot/u-boot- > ti.git;a=commitdiff;h=4eb3af078267e103fb957cb831497cf7670fb3f4 > > > > The patch in this e-mail chain was supposed to fix a bug discovered > after Tom updated his tree. > > > > Since this patch was already part of u-boot-ti and u-boot-arm trees, I > don't see how this fixes a bug. > > It's easy ;) > > The applied patch has a bug that the chip detection is done too late. > It has to be done some lines earlier: > > http://lists.denx.de/pipermail/u-boot/2009-October/062150.html > > (which is against the patch already applied). > > So there are two options to deal with this: > > a) Apply > > http://lists.denx.de/pipermail/u-boot/2009-October/062150.html > > on top of the patch already in u-boot-ti (and u-boot-arm) > > or > > b) Revert > > http://git.denx.de/?p=u-boot/u-boot- > ti.git;a=commitdiff;h=4eb3af078267e103fb957cb831497cf7670fb3f4 > > and apply > > http://lists.denx.de/pipermail/u-boot/2009-October/062153.html > > instead which has the "make the chip detection earlier" fix from (a) > above incorporated. > > Best regards > > Dirk I have been enlightened :-) I have pushed this to u-boot-ti next http://git.denx.de/?p=u-boot/u-boot-ti.git;a=shortlog;h=refs/heads/next Thanks, Sandeep