netdev - Re: [PATCH net-next] ptp: don't include ptp_clock

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <Ydh842WVE/QeNwtg@sirena.org.uk>
Date:   Fri, 7 Jan 2022 17:48:19 +0000
From:   Mark Brown <broonie@...nel.org>
To:     Jakub Kicinski <kuba@...nel.org>
Cc:     davem@...emloft.net, netdev@...r.kernel.org,
        vladimir.oltean@....com
Subject: Re: [PATCH net-next] ptp: don't include ptp_clock_kernel.h in spi.h

On Fri, Jan 07, 2022 at 08:51:27AM -0800, Jakub Kicinski wrote:
> On Fri, 7 Jan 2022 16:01:33 +0000 Mark Brown wrote:

> > Nack, this is a purely SPI patch and should go via SPI (and you've not
> > even bothered to fix the subject line).  

> Hold off unnecessary comments. If you want me to change something just
> tell me rather than making vague references to following "the style for
> the subsystem".

If you're unclear on feedback please do ask, in this case the issue is
that you've submitted a SPI patch with a subject line starting "ptp: "
instead of "spi: " or some close enough variant of that.  The message
I sent originally is a form letter that I send for any issue with
subjects, it's not a specific thing customised.  

> > It's already in my queue with that fixed.

> What does it mean that it's in "your queue" is, it does not appear in
> any branch of
> ttps://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git/

I've queued it to be applied but I still need to actually do so and test
it, if it makes it through testing then it'll get merged - there's a
bunch of last minute patches running at the minute.  It's far enough
along that everything applied but tests are still running.

> Will this patch make 5.17?

If there's no problem with testing it should do.  I'm not sure what the
rush is though?  It's just a cleanup and this is the Friday before the
merge window.

Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)