Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1374588

Re: [PATCH 2/2] pci, acpi: free IO resource during shutdown

From Tomasz Nowicki <tn@semihalf.com>
Newsgroups linux.kernel
Subject Re: [PATCH 2/2] pci, acpi: free IO resource during shutdown
Date 2016-04-09 10:50 +0200
Message-ID <rlWKC-2bt-11@gated-at.bofh.it> (permalink)
References (3 earlier) <rlme6-7wf-5@gated-at.bofh.it> <rlpYm-1GR-7@gated-at.bofh.it> <rlvAJ-68q-1@gated-at.bofh.it> <rlyyB-8iG-9@gated-at.bofh.it> <rlIHE-7v9-13@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On 08.04.2016 19:46, Sinan Kaya wrote:
> Hi Tomasz,
>
> On 4/8/2016 2:51 AM, Tomasz Nowicki wrote:
>> Hi Sinan,
>>
>> On 08.04.2016 05:40, Sinan Kaya wrote:
>>> Hi Tomasz,
>>>
>>> On 4/7/2016 5:41 PM, Bjorn Helgaas wrote:
>>>>>> You say this is undoing the effect of pci_remap_iospace(), but that's
>>>>>>>> only called by native drivers and the generic (OF) driver, not by
>>>>>>>> pci_root.c.
>>>>>>
>>>>>> See the ACPI root bridge driver above.
>>>> If this is a fix to patches that haven't been merged yet, we need to
>>>> squash the fix into the patches.
>>>>
>>>
>>> Can you merge these to two patches to your series for the next post?
>>>
>>> I need to remove weak on the first patch per direction from Bjorn and
>>> fix the function comments. You could as well do this while you are merging.
>>>
>>> Let me know what your preference is.
>>>
>>
>> Please do necessary fixes for your patches and send me the repo reference link. I will merge these to my patch set. Thanks!
>>
>> Tomasz
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-pci" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
> I posted the updated patch here. Changes are:
>
> - I squashed these two patches together per Bjorn's request.
> - Removed the weak declarations from both remap and unmap calls.
> - Fixed the doxygen document to match the actual parameters.
>
> https://us.codeaurora.org/cgit/quic/server/kernel/commit/?h=okaya/pciunmapv2&id=6120a5f0c5e6d757f18a076917fa202e2c9823d0
>

Thanks Sinan!

Tomasz

Back to linux.kernel | Previous | NextPrevious in thread | Find similar | Unroll thread


Thread

Re: [PATCH 2/2] pci, acpi: free IO resource during shutdown Bjorn Helgaas <helgaas@kernel.org> - 2016-04-07 18:10 +0200
  Re: [PATCH 2/2] pci, acpi: free IO resource during shutdown Sinan Kaya <okaya@codeaurora.org> - 2016-04-07 19:50 +0200
    Re: [PATCH 2/2] pci, acpi: free IO resource during shutdown Bjorn Helgaas <helgaas@kernel.org> - 2016-04-07 23:50 +0200
      Re: [PATCH 2/2] pci, acpi: free IO resource during shutdown Sinan Kaya <okaya@codeaurora.org> - 2016-04-08 05:50 +0200
        Re: [PATCH 2/2] pci, acpi: free IO resource during shutdown Tomasz Nowicki <tn@semihalf.com> - 2016-04-08 09:00 +0200
          Re: [PATCH 2/2] pci, acpi: free IO resource during shutdown Sinan Kaya <okaya@codeaurora.org> - 2016-04-08 19:50 +0200
            Re: [PATCH 2/2] pci, acpi: free IO resource during shutdown Tomasz Nowicki <tn@semihalf.com> - 2016-04-09 10:50 +0200

csiph-web