> > -/* Match table for of_platform binding */ No longer true? > static const struct of_device_id adi_axi_adc_of_match[] = { > - { .compatible = "adi,axi-adc-10.0.a", .data = &adi_axi_adc_10_0_a_info }, > - { /* end of list */ } Leave this terminator as it was. It's not important to have, but no need to add churn here given the comment does no harm. > + { .compatible = "adi,axi-adc-10.0.a", .data = &adc_generic }, > + { } > }; > MODULE_DEVICE_TABLE(of, adi_axi_adc_of_match); > >