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