On Fri, Jun 07, 2019 at 05:44:06PM +0200, Paul Cercueil wrote: > Right now none of the Ingenic-based boards probe this driver from > devicetree. This driver defined three compatible strings for the exact > same behaviour. Before these strings are used, we can remove two of > them. > > Signed-off-by: Paul Cercueil <p...@crapouillou.net>
No hard opinion on my side on this one. Keeping the unused compatibles around (maybe with a comment to prefer ingenic,jz4740-pwm) is what I would have done. If you're really sure nobody used the two removed ones this is fine. The binding already requests that ingenic,jz4740-pwm is to be used. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | http://www.pengutronix.de/ |