Re: [PATCH v2] media: uvcvideo: Use scope-based cleanup helper

From: Chen Changcheng
Date: Tue Jan 06 2026 - 03:14:57 EST


> > > This is an undesired syntax explicitly documented as one to avoid. You
> > > need here proper assignment, not NULL. Please don't use cleanup.h if you
> > > do not intend to follow it because it does not make the code simpler.
> >
> > Thank you for your correction and feedback. I made the change initially
> > because I saw similar patterns using "__free(...) = NULL" in patches from
> > other driver modules, which led me to overlook the core paradigm of
> > cleanup.h that requires a "declaration with a meaningful initializer."

> Do you plan to send a v3 ?

No, I don't plan to send a v3. Thank you.

Best regards,
Changcheng Chen