staging: media: atomisp: remove outdated TODO comment

[PATCH] staging: media: atomisp: remove outdated TODO comment

Posted by Shyam Sunder Reddy Padira 2 months, 1 week ago

Remove a TODO comment suggesting a filename change for
sw_event_global.h. The header is already consistently
used across the driver, making the comment obsolete.

No functional changes.

Signed-off-by: Shyam Sunder Reddy Padira <shyamsunderreddypadira@gmail.com>
---
 .../media/atomisp/pci/runtime/event/interface/ia_css_event.h    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/media/atomisp/pci/runtime/event/interface/ia_css_event.h b/drivers/staging/media/atomisp/pci/runtime/event/interface/ia_css_event.h
index d0c3278b0fd9..874331d8a435 100644
--- a/drivers/staging/media/atomisp/pci/runtime/event/interface/ia_css_event.h
+++ b/drivers/staging/media/atomisp/pci/runtime/event/interface/ia_css_event.h
@@ -8,7 +8,7 @@
 #define _IA_CSS_EVENT_H
 
 #include <type_support.h>
-#include "sw_event_global.h"    /*event macros.TODO : Change File Name..???*/
+#include "sw_event_global.h"    /*event macros.*/
 
 bool ia_css_event_encode(
     u8	*in,
-- 
2.43.0

Re: [PATCH] staging: media: atomisp: remove outdated TODO comment

Posted by Andy Shevchenko 2 months, 1 week ago

On Sun, Apr 12, 2026 at 5:06 PM Shyam Sunder Reddy Padira
<shyamsunderreddypadira@gmail.com> wrote:
>
> Remove a TODO comment suggesting a filename change for
> sw_event_global.h. The header is already consistently
> used across the driver, making the comment obsolete.
>
> No functional changes.

Unneeded churn.

...

>  #include <type_support.h>
> -#include "sw_event_global.h"    /*event macros.TODO : Change File Name..???*/
> +#include "sw_event_global.h"    /*event macros.*/

And still the comment has style issues...

-- 
With Best Regards,
Andy Shevchenko