> > +static void tty_slave_release(struct device *dev) {
> > + struct tty_slave *tts = to_tty_slave(dev);
> > +
> > + kfree(tts);
> > + /* Test code to see if slave device get released */
> > + BUG();
> > +}
>
> You send me a patch like this and expect me to apply it when it's obvious you
> didn't even test it out? That's the most _basic_ test for any dynamic type of
> device (add it and remove it and make sure the kernel still works.) And
> obviously you didn't do it.
>
> I need a really good reason why to take you seriously now.
It's actually my fault.
I'm lack of release testability then.
Now I can do this exactly.
According to internal reviewers' comments, it should be RFC patch to see if any objections there in the community.
I'll complete the test and refresh the patches if you think the functionality is interesting and useful.
Please be noted the device tree guys can also get benefit from such tty target enumeration mechanism.
Thanks
-Lv
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
