Hi Stefan, On Sat, Jun 24, 2017 at 6:29 PM, Stefan Roese <s...@denx.de> wrote: > Hi Simon, Hi Bin, > > > On 12.06.2017 05:53, Simon Glass wrote: >> >> Hi Bin, >> >> On 9 June 2017 at 21:47, Bin Meng <bmeng...@gmail.com> wrote: >>> >>> Hi Simon, >>> >>> On Fri, Jun 9, 2017 at 8:27 PM, Simon Glass <s...@chromium.org> wrote: >>>> >>>> Hi Bin, >>>> >>>> On 1 June 2017 at 17:04, Bin Meng <bmeng...@gmail.com> wrote: >>>>> >>>>> Hi Simon, >>>>> >>>>> On Thu, Jun 1, 2017 at 9:14 PM, Simon Glass <s...@chromium.org> wrote: >>>>>> >>>>>> Hi Bin, >>>>>> >>>>>> On 1 June 2017 at 01:27, Bin Meng <bmeng...@gmail.com> wrote: >>>>>>> >>>>>>> >>>>>>> Hi Simon, >>>>>>> >>>>>>> On Wed, May 17, 2017 at 8:42 AM, Bin Meng <bmeng...@gmail.com> wrote: >>>>>>>> >>>>>>>> Hi Simon, >>>>>>>> >>>>>>>> On Sat, May 13, 2017 at 9:11 AM, Simon Glass <s...@chromium.org> >>>>>>>> wrote: >>>>>>>>> >>>>>>>>> Hi Bin, >>>>>>>>> >>>>>>>>> On 8 May 2017 at 01:05, Bin Meng <bmeng...@gmail.com> wrote: >>>>>>>>>> >>>>>>>>>> Hi Simon, >>>>>>>>>> >>>>>>>>>> On Mon, Apr 10, 2017 at 8:38 AM, Simon Glass <s...@chromium.org> >>>>>>>>>> wrote: >>>>>>>>>>> >>>>>>>>>>> This is not to be applied. It is needed to test using >>>>>>>>>>> CONFIG_DM_MMC on x86 >>>>>>>>>>> since SCSI is broken with this setup. >>>>>>>>>>> >>>>>>>>>>> Signed-off-by: Simon Glass <s...@chromium.org> >>>>>>>>>>> --- >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> When you said "SCSI is broken with this setup", did you mean GP# >>>>>>>>>> exception when boot up with a hard disk like below? >>>>>>>>> >>>>>>>>> >>>>>>>>> Yes that's right. >>>>>>>>> >>>>>>>> >>>>>>>> Do you plan to work on a patch soon? >>>>>>>> >>>>>>> >>>>>>> I tracked down the exception happens at >>>>>>> part_test_efi()->block_dread() function: >>>>>>> >>>>>>> if (!ops->read) >>>>>>> return -ENOSYS; >>>>>>> >>>>>>> Where ops points to nowhere, because underlying SATA device does not >>>>>>> have a DM driver associated. >>>>>>> >>>>>>> Could you please fix this? >>>>>> >>>>>> >>>>>> I've been hoping that someone else will take on this side of things, >>>>>> but perhaps I am optimistic. I'll see if I can take a look. >>>>> >>>>> >>>>> Thanks. I wanted to fix this myself however I have been busy on some >>>>> other stuff. >>>> >>>> >>>> I've sent a series which I think puts SCSI and SATA in a better >>>> position with respect to driver model. However even with that there is >>>> more work to do. >>>> >>> >>> Thanks for working on this! I will take a look. >>> >>> If we don't get enough time to sort out everything, we will have to >>> revert previous MMC DM conversion patch and leave that to next >>> release. >> >> >> Yes, let's see how it goes. I probably have time for one more spin if >> it is soon...but the last patch needs work. > > > As Andreas has pointed out here: > > https://lists.denx.de/pipermail/u-boot/2017-June/296337.html > > SCSI seems to be broken also on Armada 37xx (and perhaps other > platforms as well). Is it correct to assume, that this x86 SCSI > thread is about the same issue? > > I know that you are working on DM SCSI support, which is great of > course. But do you have an idea, if and how this could be fixed for > this upcoming release?
Simon already sent a series to convert SCSI to DM and applied to u-boot-dm. I also sent a series to fix some additional issues [1]. Can you or Andreas try on top of u-boot-dm plus my patches to see if the issue is fixed? [1] http://patchwork.ozlabs.org/patch/777312/ Regards, Bin _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
