All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>
To: Marco Scardovi <scardracs@disroot.org>
Cc: corentin.chary@gmail.com, denis.benato@linux.dev,
	 Hans de Goede <hansg@kernel.org>,
	LKML <linux-kernel@vger.kernel.org>,
	 luke@ljones.dev, platform-driver-x86@vger.kernel.org
Subject: Re: [PATCH v2] platform/x86: asus-armoury: add support for G614PR
Date: Tue, 30 Jun 2026 17:18:14 +0300 (EEST)	[thread overview]
Message-ID: <886fad99-08de-3c74-bf07-a65777191d08@linux.intel.com> (raw)
In-Reply-To: <w5lQn2_tRS-_d31lk_IeWw@disroot.org>

[-- Attachment #1: Type: text/plain, Size: 2166 bytes --]

On Mon, 29 Jun 2026, Marco Scardovi wrote:

> Hi Ilpo,
> 
> In data lunedì 29 giugno 2026 13:04:51 Ora legale dell’Europa centrale, Ilpo Järvinen ha scritto:
> > On Sat, 27 Jun 2026, Marco Scardovi wrote:
> > 
> > > you around? I'd need this to be fixed to set the correct values.
> > 
> > I'm not sure what the context here is, whether you're just asking for the 
> > patch to be applied (which is entirely unnecessary, as long as the patch 
> > appears in the patchwork's queue, I've not lost it), or if you're saying 
> > there's a problem in v2 patch?
> >
> There was a problem with the v1 patch, where I wrote 4 values wrongly,
> so I posted the v2 with the correct values.

Hi,

I did take v1 patch and it has in the meantime been merged into Linus' 
tree, making the original change immutable.

Thus, please post a delta patch with a justification instead. Use a Fixes 
tag pointing to the original commit.

For simple patches, placing the patch changelog (patch history) in that 
patch itself under --- line e.g. as bullet points is more effective and 
easy to spot that from the coverletter.

Preferrably post new versions as own thread because sometimes tools may 
get confused when more than one version appears within the same thread.

> > Usually, the merge window is quiet time anyway when it comes to applying 
> > patches so don't expect much to happen for them until clearly after rc1 
> > (= not right after rc1 has been published, because the piled up patch 
> > queue will take some time to drain and there are other things to do around 
> > that timeframe as well).
> > 
> I see, sorry for bothering. I'm new to the kernel patching workflow and
> still learning how it works.

It's no problem, no need to be sorry at all, we value also patches from 
newcomers. I just wrote this to explain how the kernel development cycle 
works. 

Some subsystems accept no new submissions during merge window, in pdx86 
we've no such rule but things are still quieter also here (I usually won't 
apply anything than very very critical fixes during that 2 weeks window of 
time, it's "normal").

-- 
 i.

      reply	other threads:[~2026-06-30 14:18 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-10 15:21 [PATCH] platform/x86: asus-armoury: add support for G614PR Marco Scardovi
2026-06-10 17:54 ` Denis Benato
2026-06-11 15:02 ` Ilpo Järvinen
2026-06-11 15:53   ` Marco Scardovi
2026-06-14  8:44   ` [PATCH v2 0/1] " Marco Scardovi
2026-06-14  8:44     ` [PATCH v2] " Marco Scardovi
2026-06-14 12:33       ` Denis Benato
2026-06-15 15:07         ` Marco Scardovi
2026-06-27  6:18           ` Marco Scardovi
2026-06-29 11:04             ` Ilpo Järvinen
2026-06-29 11:27               ` Marco Scardovi
2026-06-30 14:18                 ` Ilpo Järvinen [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=886fad99-08de-3c74-bf07-a65777191d08@linux.intel.com \
    --to=ilpo.jarvinen@linux.intel.com \
    --cc=corentin.chary@gmail.com \
    --cc=denis.benato@linux.dev \
    --cc=hansg@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=luke@ljones.dev \
    --cc=platform-driver-x86@vger.kernel.org \
    --cc=scardracs@disroot.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.