On Mon, Mar 31, 2025 at 02:21:50PM +0100, Sidhartha Kumar wrote: > On 3/21/25 4:24 PM, Daniel Gomez wrote: > > diff --git a/tools/testing/shared/linux/cleanup.h b/tools/testing/shared/linux/cleanup.h > > new file mode 100644 > > index 0000000000000000000000000000000000000000..6e1691f56e300b498c16647bb4b91d8c8be9c3eb > > --- /dev/null > > +++ b/tools/testing/shared/linux/cleanup.h > > @@ -0,0 +1,7 @@ > > +/* SPDX-License-Identifier: GPL-2.0 */ > > +#ifndef _TEST_CLEANUP_H > > +#define _TEST_CLEANUP_H > > are these header guards needed? I think that is correct, it's redundant as of now. Perhaps it could be removed when/if merged?