Re: [PATCH 02/14] firmware: qcom: Add a generic PAS service

On 3/6/2026 2:00 PM, Jeff Johnson wrote:

On 3/6/2026 11:47 AM, Trilok Soni wrote:

quoted

On 3/6/2026 2:50 AM, Sumit Garg wrote:

quoted

+MODULE_LICENSE("GPL");
+MODULE_AUTHOR("Sumit Garg [off-list ref]");

What is the convention for Qualcomm authored drivers? In some drivers
I find that Qualcomm doesn't add MODULE_AUTHOR. Can Qualcomm community
clarify it here. I prefer consistency here for the Qualcomm submissions. 

WLAN team was told to not have MODULE_AUTHOR(), so ath10k was the last WLAN
driver that had a MODULE_AUTHOR() -- ath11k and ath12k do not have one.

And in reality it is very rare for a given module, over time, to only have a
single author. The git history contains the real authorship. So just for that
reason I'd drop it.

I agree and that is the same guideline I follow for the drivers we write
downstream. I just want to make sure that we discuss it here. We may not be able
to find one single solution, but I prefer to get consistency for Qualcomm
submissions. 

---Trilok Soni