[U-Boot] [PATCH 1/2] WIP: Disable SCSI on x86

[Bin Meng]

Hi Simon,

On Fri, Jun 9, 2017 at 8:27 PM, Simon Glass <sjg at chromium.org> wrote:
> Hi Bin,
>
> On 1 June 2017 at 17:04, Bin Meng <bmeng.cn at gmail.com> wrote:
>> Hi Simon,
>>
>> On Thu, Jun 1, 2017 at 9:14 PM, Simon Glass <sjg at chromium.org> wrote:
>>> Hi Bin,
>>>
>>> On 1 June 2017 at 01:27, Bin Meng <bmeng.cn at gmail.com> wrote:
>>>>
>>>> Hi Simon,
>>>>
>>>> On Wed, May 17, 2017 at 8:42 AM, Bin Meng <bmeng.cn at gmail.com> wrote:
>>>> > Hi Simon,
>>>> >
>>>> > On Sat, May 13, 2017 at 9:11 AM, Simon Glass <sjg at chromium.org> wrote:
>>>> >> Hi Bin,
>>>> >>
>>>> >> On 8 May 2017 at 01:05, Bin Meng <bmeng.cn at gmail.com> wrote:
>>>> >>> Hi Simon,
>>>> >>>
>>>> >>> On Mon, Apr 10, 2017 at 8:38 AM, Simon Glass <sjg at 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 <sjg at 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.

Regards,
Bin