[v2,0/4] PCI: Allow Thunderbolt to work with resources from pci=hpmemsize

Message

Nicholas Johnson Jan. 15, 2020, 5:56 p.m. UTC

Changes:
	- Add Mika Westerberg's Reviewed-by: tags that he gave for 1, 2
	- Change commit description slightly in 1
	- Change commit description more substantially in 3, 4
	- Fix indentation in 3, oops. I thought I ran checkpatch.pl on
	  it, clearly I was asleep or missed that one
	- Update 4 to remove usage of reset_resource() which Bjorn says
	  he considers to be depricated.

I have not heard back from Bjorn in a while from my latest replies, so I
am moving ahead with PATCH v2. I have tried to act on his initial
feedback the best I can.

Apply this series after series:
"PCI: Fix failure to assign BARs with alignment >1M with Thunderbolt"

Nicholas Johnson (4):
  PCI: In extend_bridge_window() change available to new_size
  PCI: Rename extend_bridge_window() to adjust_bridge_window()
  PCI: Change extend_bridge_window() to set resource size directly
  PCI: Allow extend_bridge_window() to shrink resource if necessary

 drivers/pci/setup-bus.c | 41 ++++++++++++++++++++---------------------
 1 file changed, 20 insertions(+), 21 deletions(-)

Comments

Bjorn Helgaas Jan. 30, 2020, 8:55 p.m. UTC | #1

On Wed, Jan 15, 2020 at 05:56:42PM +0000, Nicholas Johnson wrote:
> Changes:
> 	- Add Mika Westerberg's Reviewed-by: tags that he gave for 1, 2
> 	- Change commit description slightly in 1
> 	- Change commit description more substantially in 3, 4
> 	- Fix indentation in 3, oops. I thought I ran checkpatch.pl on
> 	  it, clearly I was asleep or missed that one
> 	- Update 4 to remove usage of reset_resource() which Bjorn says
> 	  he considers to be depricated.
> 
> I have not heard back from Bjorn in a while from my latest replies, so I
> am moving ahead with PATCH v2. I have tried to act on his initial
> feedback the best I can.
> 
> Apply this series after series:
> "PCI: Fix failure to assign BARs with alignment >1M with Thunderbolt"
> 
> Nicholas Johnson (4):
>   PCI: In extend_bridge_window() change available to new_size
>   PCI: Rename extend_bridge_window() to adjust_bridge_window()
>   PCI: Change extend_bridge_window() to set resource size directly
>   PCI: Allow extend_bridge_window() to shrink resource if necessary
> 
>  drivers/pci/setup-bus.c | 41 ++++++++++++++++++++---------------------
>  1 file changed, 20 insertions(+), 21 deletions(-)

Applied to pci/resource for v5.6, thanks!