On Tue, Oct 17, 2017 at 11:08:53AM -0500, Rob Herring wrote: > On Mon, Oct 16, 2017 at 8:06 AM, Johan Hovold <johan@xxxxxxxxxx> wrote: > > Add missing error handling for tty-driver open() which may fail (e.g. if > > resource allocation fails or if a port is being disconnected). > > > > Note that close() must be called also in case of failed open() and that > > the operation sanity check is amended to catch buggy drivers. > > That's a very odd pattern. Wouldn't a must_check annotation be enough? An odd pattern indeed, but that's how the TTY drivers are implemented and the interface documented (i.e. that close() will be called also after failed open()). Not sure how a must_check could help here. Johan -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
