From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexander Dahl Date: Mon, 25 Mar 2019 11:08:39 +0100 Subject: [U-Boot] [PATCH] ARM: at91: sama5d2: Wrap cpu detection to fix macb driver In-Reply-To: References: <20190322132554.5848-1-ada@thorsis.com> <23906734.aZzzLUK5Xj@ada> Message-ID: <5345167.Z57H08VWhP@ada> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: u-boot@lists.denx.de Hello Eugen, Am Montag, 25. März 2019, 09:46:02 CET schrieb Eugen.Hristev at microchip.com: > Are you sure there is a problem with this commit itself: > > net: macb: Fix GEM hardware detection > > as it looks , this commit fixes the interpretation of the MID register > w.r.t. the gem capability... not an issue there that your patch fixes. Well, without the changeset "net: macb: Fix GEM hardware detection" the actual problem I had, would not trigger, because the SAMA5D2 is not recognized having the gem capability. That's why I asked if that additional fixes line is correct. > More likely your patch fixes the fact that the sama5d2 soc was not > properly defining that macro, which was turned into a function, so your > patch fixes the code that changed the macro into a function, which broke > things... is my understanding correct ? My patch fixes that soc detection only. The behaviour on SAMA5D2 however is as follows: 1) Without both patches (as in v2019.01): SAMA5D2 mac is not detected as gem → soc detection in function 'gem_is_gigabit_capable()' is not relevant for gigabit detection -> ethernet works (maybe accidentally, gem actually used like old mac?) 2) With my patch only: mac is not detected as gem AND soc is detected as SAMA5D2 → no gigabit detected → ethernet might work as in 1), not tested though 3) With the gem hw detect fix only (as in v2019.04-rc3 already): mac is detected as gem, soc is not detected as SAMA5D2 → falsely detected gigabit capability → ethernet broken (can you reproduce this?) 4) With both patches: hw recognized as gem, soc recognized as SAMA5D2 → mac used as gem, but without gigabit caps → works for me on custom board Greets Alex