Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 4 Jan 2014 22:28:32 +0100
From:      Zbigniew Bodek <zbb@freebsd.org>
To:        Nathan Whitehorn <nwhitehorn@freebsd.org>
Cc:        Ruslan Bukin <br@freebsd.org>, "freebsd-arm@freebsd.org" <freebsd-arm@freebsd.org>, Olivier Houchard <cognet@freebsd.org>
Subject:   Re: RFC: ARM related fixes - GIC, cache line size, PCI FDT & AHCI
Message-ID:  <CALF_TxnxeNNTRbDjFnRnhFTeKjsuvkWzXivYtqJH-orAfbA7PA@mail.gmail.com>
In-Reply-To: <52C87BC0.2060304@freebsd.org>
References:  <52AF3D06.2000004@semihalf.com> <52AF760A.2030500@freebsd.org> <52B0900B.7020905@freebsd.org> <52B2D0F3.50100@semihalf.com> <52C87BC0.2060304@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
2014/1/4 Nathan Whitehorn <nwhitehorn@freebsd.org>:
> On 12/19/13 05:56, Zbigniew Bodek wrote:
>> On 17.12.2013 18:55, Nathan Whitehorn wrote:
>>> On 12/16/13 15:52, Nathan Whitehorn wrote:
>>>> On 12/16/13 11:48, Zbigniew Bodek wrote:
>>>>> Hello Everyone.
>>>>>
>>>>> We would like to submit some new patches recently developed by Semihalf.
>>>>>
>>>>> You can find them here:
>>>>> http://people.freebsd.org/~zbb/Semihalf/12.2013/
>>>>>
>>>>> Detailed description is available in the commit logs but in general:
>>>>>
>>>>> -- 0001-Resolve-cache-line-size-using-CP15.patch
>>>>>     - use cache line size acquired in runtime
>>>>>
>>>>> -- 0002-GIC-polarity-and-level-support.patch
>>>>>     - suport for setting trigger level and polarity in GIC
>>>>>
>>>>> -- 0003-Add-PCI-FDT-interrupt-trigger-polarity-parsing.patch
>>>>>     - trigger and polarity parsing for PCI FDT interrupts
>>>>>
>>>>> -- 0004-Do-not-attach-to-bridges-in-AHCI-driver.patch
>>>>> -- 0005-Use-only-mapped-BIOs-on-ARM.patch
>>>>>     - Two patches enabling the AHCI driver on ARM chips
>>>>>
>>>>>
>>>>> We will appreciate if you could post your comments and/or remarks by the
>>>>> end of this week when we plan to commit the changes.
>>>>>
>>>>> Best regards
>>>>> zbb
>>>>> _______________________________________________
>>>>> freebsd-arm@freebsd.org mailing list
>>>>> http://lists.freebsd.org/mailman/listinfo/freebsd-arm
>>>>> To unsubscribe, send any mail to "freebsd-arm-unsubscribe@freebsd.org"
>>>> Can you hold off on
>>>> 0003-Add-PCI-FDT-interrupt-trigger-polarity-parsing.patch for the time
>>>> being? I'm restructuring this code at the moment.
>>>> -Nathan
>>>> _______________________________________________
>>>> freebsd-arm@freebsd.org mailing list
>>>> http://lists.freebsd.org/mailman/listinfo/freebsd-arm
>>>> To unsubscribe, send any mail to "freebsd-arm-unsubscribe@freebsd.org"
>>> This is done now. I have not updated the ARM code, because I don't know
>>> how it is supposed to work, but you can take a look at r259513 to
>>> powerpc/ofw/ofw_pci.c to see how the new stuff works. It relies on some
>>> device (nexus, for example) implementing OFW_BUS_CONFIG_INTR(), which
>>> takes an IRQ and a sense code, but that seems to be wrapped up in a lot
>>> of ARM-specific stuff.
>>>
>>> If you like, I can just write a piece of bridge code, but I don't want
>>> to interfere with in-flight things on your end.
>>
>> Hello Nathan.
>>
>> We will wait for your refactoring to finish so no worries. We can skip
>> this patch for now, the more that there were some comments to this one.
>>
>> Thanks and best regards
>> zbb
>>
>
> My restructuring is now done. I suspect ARM may want something like
> PowerPC's pic_if.m in the future, but the interfaces are now in place
> (ofw_bus_map_intr(), ofw_bus_config_intr()) to make that transition
> fairly painless later on.
> -Nathan

Thank you very much Nathan.

Best regards
zbb



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CALF_TxnxeNNTRbDjFnRnhFTeKjsuvkWzXivYtqJH-orAfbA7PA>