Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 04 Jan 2014 16:23:12 -0500
From:      Nathan Whitehorn <nwhitehorn@freebsd.org>
To:        Zbigniew Bodek <zbb@semihalf.com>, "freebsd-arm@freebsd.org" <freebsd-arm@freebsd.org>
Cc:        br@freebsd.org, Olivier Houchard <cognet@FreeBSD.org>
Subject:   Re: RFC: ARM related fixes - GIC, cache line size, PCI FDT & AHCI
Message-ID:  <52C87BC0.2060304@freebsd.org>
In-Reply-To: <52B2D0F3.50100@semihalf.com>
References:  <52AF3D06.2000004@semihalf.com> <52AF760A.2030500@freebsd.org> <52B0900B.7020905@freebsd.org> <52B2D0F3.50100@semihalf.com>

next in thread | previous in thread | raw e-mail | index | archive | help
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



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