Received-SPF: pass (google.com: domain of linux-kernel+bounces-202868-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Date: Wed, 5 Jun 2024 17:53:07 +0200
From: Borislav Petkov <bp@alien8.de>
To: Jeff Johnson <quic_jjohnson@quicinc.com>
Cc: Yazen Ghannam <Yazen.Ghannam@amd.com>, Tony Luck <tony.luck@intel.com>,
	linux-edac@vger.kernel.org, linux-kernel@vger.kernel.org,
	kernel-janitors@vger.kernel.org
Subject: Re: [PATCH] RAS/AMD/ATL: add missing MODULE_DESCRIPTION() macro
Message-ID: <20240605155307.GGZmCJ46VkY-hWn_z8@fat_crate.local>
References: <20240604-md-ras-amd-atl-v1-1-d4eb3cf3abe4@quicinc.com>
 <20240605094051.GDZmAyo1-j_uBri_YO@fat_crate.local>
 <1d9e1530-be63-4374-b952-e12b15cee9d3@quicinc.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <1d9e1530-be63-4374-b952-e12b15cee9d3@quicinc.com>

On Wed, Jun 05, 2024 at 07:10:14AM -0700, Jeff Johnson wrote:
> I actually did use a simple script to do a conversion locally, but the hard
> part is determining what text to use for the description. So my local
> conversion just added:
> MODULE_DESCRIPTION("TBD");

Ah, there's that. Can't script that. ;-\

> https://lore.kernel.org/all/20240604-md-hid-letsketch-v1-1-ff38ae7b4cb0@quicinc.com/
> https://lore.kernel.org/all/20240604-md-hid-lg-g15-v1-1-265b094db089@quicinc.com/
> https://lore.kernel.org/all/20240604-md-hid-logitech-dj-v1-1-560f6b3cb54b@quicinc.com/
> https://lore.kernel.org/all/20240603-md-hid-logitech-hidpp-v1-1-060f06e4529f@quicinc.com/

I would've merged them into one - it's not like it is really complicated
changes you're doing so that each maintainer must deal with it
separately.

> I then sent a large patch for all of the remaining drivers/hid patches:
> https://lore.kernel.org/all/20240604-md-hid-misc-v1-1-4f9560796f3c@quicinc.com/

Yap, exactly what I had in mind.

Thx.

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette