On Wed, Mar 27, 2024 at 05:32:57AM +0000, Yoshihiro Shimoda wrote:
> > From: Bjorn Helgaas, Sent: Wednesday, March 27, 2024 5:49 AM
> > > static int rcar_gen4_pcie_get_resources(struct rcar_gen4_pcie *rcar)
> > > {
> > > + rcar->phy_base = devm_platform_ioremap_resource_byname(rcar->pdev, "phy");
> > > + if (IS_ERR(rcar->phy_base))
> > > + return PTR_ERR(rcar->base);
> >
> > I don't get it. This imposes a new requirement (presence of "phy"
> > resource) on the existing SoCs. That doesn't sound right.
>
> According to the dt-binding doc, the existing SoCs are also required
> for the "phy". That's why I didn't add any condition to simplify
> the code.
Is there anything that enforces that? Is it possible that DTs exist
in the field without it? We don't want to break any existing setup.
