diff mbox series

[v2,2/4] PCI: of: Drop 'No bus range found' message

Message ID 20250113231557.441289-3-helgaas@kernel.org (mailing list archive)
State Accepted
Delegated to: Bjorn Helgaas
Headers show
Series PCI: Simplify bus range parsing | expand

Commit Message

Bjorn Helgaas Jan. 13, 2025, 11:15 p.m. UTC
From: Bjorn Helgaas <bhelgaas@google.com>

The typical bus range for a host bridge is [bus 00-ff], and
devm_of_pci_get_host_bridge_resources() defaults to that unless DT contains
a "bus-range" property.

devm_of_pci_get_host_bridge_resources() previously emitted a message when
there was no "bus-range" property, but that seems unnecessary for this
common situation.  Remove the message.

Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
---
 drivers/pci/of.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Sathyanarayanan Kuppuswamy Jan. 15, 2025, 12:31 a.m. UTC | #1
On 1/13/25 3:15 PM, Bjorn Helgaas wrote:
> From: Bjorn Helgaas <bhelgaas@google.com>
>
> The typical bus range for a host bridge is [bus 00-ff], and
> devm_of_pci_get_host_bridge_resources() defaults to that unless DT contains
> a "bus-range" property.
>
> devm_of_pci_get_host_bridge_resources() previously emitted a message when
> there was no "bus-range" property, but that seems unnecessary for this
> common situation.  Remove the message.
>
> Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
> ---

Reviewed-by: Kuppuswamy Sathyanarayanan 
<sathyanarayanan.kuppuswamy@linux.intel.com>

>   drivers/pci/of.c | 2 --
>   1 file changed, 2 deletions(-)
>
> diff --git a/drivers/pci/of.c b/drivers/pci/of.c
> index 2f579b691f8e..48b9274b846e 100644
> --- a/drivers/pci/of.c
> +++ b/drivers/pci/of.c
> @@ -346,8 +346,6 @@ static int devm_of_pci_get_host_bridge_resources(struct device *dev,
>   		bus_range->start = busno;
>   		bus_range->end = bus_max;
>   		bus_range->flags = IORESOURCE_BUS;
> -		dev_info(dev, "  No bus range found for %pOF, using %pR\n",
> -			 dev_node, bus_range);
>   	} else {
>   		if (bus_range->end > bus_range->start + bus_max)
>   			bus_range->end = bus_range->start + bus_max;
diff mbox series

Patch

diff --git a/drivers/pci/of.c b/drivers/pci/of.c
index 2f579b691f8e..48b9274b846e 100644
--- a/drivers/pci/of.c
+++ b/drivers/pci/of.c
@@ -346,8 +346,6 @@  static int devm_of_pci_get_host_bridge_resources(struct device *dev,
 		bus_range->start = busno;
 		bus_range->end = bus_max;
 		bus_range->flags = IORESOURCE_BUS;
-		dev_info(dev, "  No bus range found for %pOF, using %pR\n",
-			 dev_node, bus_range);
 	} else {
 		if (bus_range->end > bus_range->start + bus_max)
 			bus_range->end = bus_range->start + bus_max;