On Fri, May 19, 2017 at 1:56 AM, Romain Perier <romain.perier@xxxxxxxxxxxxx> wrote: > Hello, > > > Le 16/05/2017 à 13:18, Mark Brown a écrit : >> On Mon, May 15, 2017 at 04:41:38PM -0500, Rob Herring wrote: >>> On Fri, May 12, 2017 at 03:22:25PM +0200, Romain Perier wrote: >>>> This commit adds a compatible string for everest,es8323. This is an >>>> audio codec that is compatible with es8328 and can be found for example >>>> on the Firefly-RK3288 board. >>> If it is compatible with the es8328, then that should be a fallback and >>> you don't need the driver change. >> While people don't strictly need the driver change it doesn't do any >> harm either and encourages people to get the information into the DT >> that it's a different chip. Thinking about it it might be good to have >> a way for something to validate if fallback compatibles are being listed >> when they should - the drivers could provide the information fairly >> easily I guess, or it could go into the binding docs once we have a >> schema format. >> >> Even if it doesn't currently make a difference to software I'd rather >> get the information in there for mixed signal devices like audio CODECs >> - it's not that unknown to find later that supposedly register identical >> chips have some differences in the analog which we want to care about. > So, what should I do for this patch, finally ? fallback or driver change ? Well, you could do both. However, there's not much point if you do the driver change. So: Acked-by: Rob Herring <robh@xxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html