Re: [PATCH] net: dsa: Fix conditional handling of Wake-on-Lan configuration in dsa_user_set_wol

From: Mohammed Anees
Date: Mon Oct 07 2024 - 18:43:50 EST


Hi Vladimir,

Thank you for your thoughtful feedback; I really appreciate it. I want
to be honest, I’m quite new to this process and have been exploring the
the code to learn as much as I can, which was when i noticed this. I was
excited to contribute and took a more theoretical approach without fully
grasping the importance of testing to see if this is ever possible.

I completely understand your points now and apologize for not testing
this. I haven’t had the chance to experiment with it on any platform yet,
but I plan to do so to see if I encounter any issues. If I do run into
problems, I’ll definitely continue with this patch and work on resolving them.

Thanks once more for your guidance and for clarifying the intricacies
of MAC WoL and PHY WoL interaction.