On Tue, Apr 01, 2025 at 10:17:04AM +0100, Marc Zyngier wrote:
> In order to decouple ecam config space creation from probing via
> pci_host_common_probe(), allow the private pointer to be populated
> via the device drvdata pointer.
>
> Crucially, this is set before calling ops->init(), allowing that
> particular callback to have access to probe data.
>
> This should have no impact on existing code which ignores the
> current value of cfg->priv.
>
> Reviewed-by: Rob Herring (Arm) <robh@xxxxxxxxxx>
> Acked-by: Alyssa Rosenzweig <alyssa@xxxxxxxxxxxxx>
> Tested-by: Janne Grunau <j@xxxxxxxxxx>
> Signed-off-by: Marc Zyngier <maz@xxxxxxxxxx>
Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx>
- Mani
> ---
> drivers/pci/ecam.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/pci/ecam.c b/drivers/pci/ecam.c
> index 260b7de2dbd57..2c5e6446e00ee 100644
> --- a/drivers/pci/ecam.c
> +++ b/drivers/pci/ecam.c
> @@ -84,6 +84,8 @@ struct pci_config_window *pci_ecam_create(struct device *dev,
> goto err_exit_iomap;
> }
>
> + cfg->priv = dev_get_drvdata(dev);
> +
> if (ops->init) {
> err = ops->init(cfg);
> if (err)
> --
> 2.39.2
>
--
மணிவண்ணன் சதாசிவம்
