On 20 May 2015 at 08:26, Simon Glass <s...@chromium.org> wrote:
> +Wolfgang
>
> On 18 May 2015 at 19:40, Bin Meng <bmeng...@gmail.com> wrote:
>> Hi Simon,
>>
>> Reply this thread to the v2 patch.
>>
>> On Tue, May 19, 2015 at 5:39 AM, Simon Glass <s...@chromium.org> wrote:
>>> Hi Bin,
>>>
>> On Sat, May 16, 2015 at 9:33 AM, Bin Meng <bmeng...@gmail.com> wrote:
>>>> The legacy IDE device has a BAR4 (Bus Master Interface BAR) which
>>>> needs to be configured.
>>>>
>>>> Signed-off-by: Bin Meng <bmeng...@gmail.com>
>>>>
>>>> ---
>>>>
>>>> Changes in v2:
>>>> - Correct a typo in the commit message title
>>>>
>>>>  drivers/pci/pci_auto.c | 14 --------------
>>>>  1 file changed, 14 deletions(-)
>>>
>>> I don't see how this code change relates to your commit message...
>>>
>>
>> The legacy IDE controller reports PI (offset 9 in the configuration
>> space) as 0x80, so the codes here will skip (!(0x80 & 0x05)) its
>> configuration (ie: BAR4 won't be assigned to a valid memory window)
>
> OK I see. It looks like this code has been there since the dawn of
> U-Boot. I added Wolfgang in case we are missing something.
>
> Reviewed-by: Simon Glass <s...@chromium.org>

Acked-by: Simon Glass <s...@chromium.org>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to