<snip> >> + >> +int si470x_disconnect_check(struct si470x_device *radio) >> +{ >> + return 0; >> +} > > Looks like this function is empty and it's called few times. Is it > good to make it inline? > Yes, this function is empty. It looks fine to me too. I will modify it, thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html