[0/3] Use scope based cleanup in drivers/soc/ti/

Message

Kousik Sanagavarapu May 10, 2024, 7:13 a.m. UTC

Do scope based "struct device_node" cleanup in drivers/soc/ti/*.  This
covers all the possible cases.

Kousik Sanagavarapu (3):
  soc: ti: pruss: do device_node auto cleanup
  soc: ti: knav_qmss_queue: do device_node auto cleanup
  soc: ti: pm33xx: do device_node auto cleanup

 drivers/soc/ti/knav_qmss_queue.c | 17 ++++++-----
 drivers/soc/ti/pm33xx.c          | 20 +++++--------
 drivers/soc/ti/pruss.c           | 48 +++++++++++---------------------
 3 files changed, 31 insertions(+), 54 deletions(-)

Comments

Kousik Sanagavarapu May 11, 2024, 9:06 a.m. UTC | #1

On Fri, May 10, 2024 at 08:17:54AM -0500, Nishanth Menon wrote:
> On 12:38-20240510, Kousik Sanagavarapu wrote:
> > Do scope based "struct device_node" cleanup in drivers/soc/ti/*.  This
> > series converts all the possible cases.
> 
> Thank you - also will be great if you can indicate how this was caught?

This is kind of a refactoring.  I grepped for them manually.

As for the motivation itself, I would say

	https://lwn.net/Articles/934679/

Me and the other LKMP mentees are working on doing this to various parts
of the source with the help of Julia, Shuah and Javier.

As mentioned in the commit messages, this refactoring has the advantage
of reducing the number of leaks in case we add new code in the future
introducing new OF stuff.

Thanks

PS: I sent out the cover letter once before without CCing the mailing
lists and Nishanth replied to that.  So that mail can't be seen here
independently.