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

Stefan Roese sr at denx.de
Sat Jun 24 10:29:08 UTC 2017


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.cn at gmail.com> wrote:
>> 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.
> 
> 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?

Thanks,
Stefan


More information about the U-Boot mailing list