On Wed, Mar 21, 2018 at 11:40:07PM +0530, Harsh Shandilya wrote: > > > On 21 March 2018 9:54:37 PM IST, Greg KH <[email protected]> wrote: > >On Wed, Mar 21, 2018 at 09:09:57AM -0700, Vic Wei wrote: > >> QCA Rome controllers can do both LE scan and BR/EDR inquiry at once. > >> > >> Change-Id: I89e1412d635f4cd7b2500f7492f37430ea139f0c > > > >What is this line for? > > That's the Change ID required by everything that goes through Gerrit. CAF > (I'm assuming Vic is from CodeAurora Forums judging by the email) uses Gerrit > as their review platform for the Android for MSM project.
It was a rhetorical question, I know what it is, and the fact that it should not be in a kernel patch submission, I wanted the submitter to learn to read their own patches _before_ they send them in. Also, if they used checkpatch.pl, it would have told them to remove this. It's like saying to a kid, "what is this mess on the floor"... greg k-h
