11/10/2017 02:03, Don Provan: > From: Shreyansh Jain [mailto:shreyansh.j...@nxp.com] > > > > ... > > > > > From the application's point of view, the potential error here > > > is that the device it wants to use isn't available. I don't see that > > > either the init function or the probe function will have enough > > > information to understand that application-level problem, so > > > they should leave it to the application to detect it. > > > > I think I understand you comment but just want to cross check again: > > Scan or probe error should simply be ignored by EAL layer and let the > > application take stance when it detects that the device it was looking > > for is missing. Is my understanding correct? > > > > I am trying to come a conclusion so that this patch can either be > > modified or pushed as it is. If the above understanding is correct, I > > don't see any changes required in the patch. > > Yes, I agree my comments support the patch as is. > -don
Applied, thanks for the discussion