On Do, 2023-12-07 at 12:10 -0500, Frank Li wrote: > On Wed, Dec 06, 2023 at 05:52:23PM +0100, Philipp Zabel wrote: [...] > > I wonder whether we should just defer the check whether apps/pciephy > > resets should be used to the device tree validation, and make this an > > unconditional call to get an optional reset control: > > > > imx6_pcie->apps_reset = devm_reset_control_get_optional_exclusive(dev, "apps"); > > I think double check here is neccesary. No sure if dts file version > binding yaml was exactly matched driver. Sometime user use old version dts. > Double check here will help identify the dts problem. Makes sense to me, Reviewed-by: Philipp Zabel <p.zabel@xxxxxxxxxxxxxx> regards Philipp
