On Fri, Sep 06, 2019 at 03:06:03PM +0800, shifu0704@thundersoft.com wrote: > + - ti,left-slot: - Sets TDM RX left time slots. > + - ti,right-slot: - Sets TDM RX right time slots. This looks like it's duplicating things that are normally done with the set_tdm_slot() callback. Otherwise the binding looks good.