On Mon, Jan 30, 2023 at 11:18:30PM +0200, Sakari Ailus wrote: > On Fri, Jan 27, 2023 at 12:08:39PM +0200, Andy Shevchenko wrote: > > On Thu, Jan 26, 2023 at 12:40:55AM +0200, Sakari Ailus wrote: ... > > > + /* Collect the devices that have a _CRS CSI-2 resource */ > > > + acpi_walk_namespace(ACPI_TYPE_DEVICE, handle, ACPI_UINT32_MAX, > > > > Other serial buses limit the DEPTH by 32, why do we need more here? > > I'm using the same value as for scanning devices to be added. Effectively > this is not a practical limit but it doesn't need to be. Hmm... Theoretically one may supply a malformed table, quite compact, but with huge amount of "fake" devices. I would limit this the same way. If we got a report of the depth not being enough we may bump based on the reality. This also may limit scanning time. > > > + scan_check_crs_csi2, NULL, &crs_csi2_handles, NULL); -- With Best Regards, Andy Shevchenko
