Simon, Heiko, On Tue, Jul 12, 2016 at 1:22 PM, Bin Meng <bmeng...@gmail.com> wrote: > Hi Simon, > > On Tue, Jun 28, 2016 at 9:44 PM, Stefan Roese <s...@denx.de> wrote: >> This patch adds support for the SMBus block read/write functionality. >> Other protocols like the SMBus quick command need to get added >> if this is needed. >> >> This patch also removed the SMBus related defines from the Ivybridge >> pch.h header. As they are integrated in this driver and should be >> used from here. This change is added in this patch to avoid compile >> breakage to keep the source git bisectable. >> >> Tested on a congatec BayTrail board to configure the SMSC2513 USB >> hub. >> >> Signed-off-by: Stefan Roese <s...@denx.de> >> Cc: Bin Meng <bmeng...@gmail.com> >> Cc: Simon Glass <s...@chromium.org> >> Cc: Heiko Schocher <h...@denx.de> >> --- >> Simon, I'm not sure if this change breaks your Ivybridge targets >> using the probe part of this driver. Could you please let me >> know if this works? Or let me know what needs changes here? >> > > Can you test this patch on Ivybridge to see if it breaks anything? Is > SMBus used on Ivybridge for the memory initialization (eg: reading > SPD?) >
Do you have any review comments before it's applied? I believe this needs to be tested on Ivybridge at least. Regards, Bin _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot