All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: "Pali Rohár" <pali@kernel.org>
Cc: "Bin Meng" <bmeng.cn@gmail.com>, "Simon Glass" <sjg@chromium.org>,
	"Stefan Roese" <sr@denx.de>, "Marek Behún" <kabel@kernel.org>,
	u-boot@lists.denx.de
Subject: Re: [PATCH 2/4] pci: Extend 'pci regions' command with bus number
Date: Sat, 29 Jan 2022 13:48:49 -0500	[thread overview]
Message-ID: <20220129184849.GS7515@bill-the-cat> (raw)
In-Reply-To: <20220117153840.31173-3-pali@kernel.org>

[-- Attachment #1: Type: text/plain, Size: 891 bytes --]

On Mon, Jan 17, 2022 at 04:38:38PM +0100, Pali Rohár wrote:

> 'pci regions' currently prints only region information from bus 0 which
> belongs to controller 0. Parser for 'pci regions' cmdline currently ignores
> any additional arguments and so U-Boot always uses bus 0.
> 
> Regions are stored in controller (not on the bus) and therefore to retrieve
> controller from the bus, it is needed to call pci_get_controller() which
> returns root bus. Because bus 0 is root bus, current code worked fine for
> controller 0.
> 
> Extend cmdline parser for 'pci regions' to allows specifying bus number,
> extend pci_show_regions() code to accept also non-zero bus number and
> print bus ranges for which is regions configuration assigned.
> 
> Signed-off-by: Pali Rohár <pali@kernel.org>
> Reviewed-by: Stefan Roese <sr@denx.de>

Applied to u-boot/master, thanks!

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]

  parent reply	other threads:[~2022-01-29 18:49 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-17 15:38 [PATCH 0/4] pci: Extend 'pci' and 'pci regions' commands Pali Rohár
2022-01-17 15:38 ` [PATCH 1/4] pci: Fix setting controller's last_busno Pali Rohár
2022-01-25 15:49   ` Stefan Roese
2022-01-29 18:48   ` Tom Rini
2022-01-17 15:38 ` [PATCH 2/4] pci: Extend 'pci regions' command with bus number Pali Rohár
2022-01-25 15:53   ` Stefan Roese
2022-01-25 16:50     ` Pali Rohár
2022-01-26 15:43       ` Stefan Roese
2022-01-29 18:48   ` Tom Rini [this message]
2022-01-17 15:38 ` [PATCH 3/4] pci: Add checks for valid cmdline arguments Pali Rohár
2022-01-25 15:53   ` Stefan Roese
2022-01-29 18:48   ` Tom Rini
2022-01-17 15:38 ` [PATCH 4/4] pci: Extend 'pci' command with bus option '*' Pali Rohár
2022-01-25 15:53   ` Stefan Roese
2022-01-29 18:48   ` Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220129184849.GS7515@bill-the-cat \
    --to=trini@konsulko.com \
    --cc=bmeng.cn@gmail.com \
    --cc=kabel@kernel.org \
    --cc=pali@kernel.org \
    --cc=sjg@chromium.org \
    --cc=sr@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.