All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Heiko Stübner" <heiko@sntech.de>
To: Rob Herring <robh@kernel.org>
Cc: arm@kernel.org, soc@kernel.org,
	linux-rockchip@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [GIT PULL] Rockchip dts64 changes for 6.15 #3
Date: Mon, 07 Apr 2025 18:49:27 +0200	[thread overview]
Message-ID: <2000188.PYKUYFuaPT@diego> (raw)
In-Reply-To: <CAL_JsqJVi5tf8bh6AnZwNwiBe+EG2CHQRGQGLK6SdMzQiiSbWA@mail.gmail.com>

Am Montag, 7. April 2025, 18:43:54 Mitteleuropäische Sommerzeit schrieb Rob Herring:
> On Mon, Apr 7, 2025 at 11:40 AM Heiko Stübner <heiko@sntech.de> wrote:
> >
> > Hi Rob,
> >
> > Am Montag, 7. April 2025, 17:56:37 Mitteleuropäische Sommerzeit schrieb Rob Herring:
> > > On Wed, Mar 19, 2025 at 6:56 PM Heiko Stuebner <heiko@sntech.de> wrote:
> > > >
> > > > Hi soc maintainers,
> > > >
> > > > I made an error and accidentially applied a patch that was meant for
> > > > the mfd tree. Thankfully Stephen noticed that when the duplicate
> > > > commit appeared in linux-next.
> > >
> > > Both commits are in v6.15-rc1 now and the revert is not, so this
> > > should not get applied/pulled. Or you will need to revert the revert.
> >
> > yes, that was the intention.
> >
> > Back when I submitted this PR, I talked with Arnd on IRC the next day.
> >
> > As both commits are identical sans some Signed-off-by lines, he suggested
> > not trying to put a revert in, but instead let git solve it itself, because
> >
> > arnd on IRC:
> > > [...], but I worry that this would make things worse if 'git merge'
> > > ends up doing the revert on top of the original commit once it gets to
> > > torvalds. Not sure if that's still a problem in git these days, but
> > > I've seen it happen in the past.
> > > if two identical patches are in different branches, just leaving them
> > > there is usually easier
> >
> > So this PR was already marked as "superseeded" in patchwork back
> > on march 20th.
> >
> >
> > Nevertheless, thanks for making sure no funky revert happens now.
> 
> The commit is still in linux-next though. That's how I happened upon this.

Ah right. I've recreated my next branch earlier today, after the -rc1
release, so this should be gone with the next linux-next.

Heiko




WARNING: multiple messages have this Message-ID (diff)
From: "Heiko Stübner" <heiko@sntech.de>
To: Rob Herring <robh@kernel.org>
Cc: arm@kernel.org, soc@kernel.org,
	linux-rockchip@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [GIT PULL] Rockchip dts64 changes for 6.15 #3
Date: Mon, 07 Apr 2025 18:49:27 +0200	[thread overview]
Message-ID: <2000188.PYKUYFuaPT@diego> (raw)
In-Reply-To: <CAL_JsqJVi5tf8bh6AnZwNwiBe+EG2CHQRGQGLK6SdMzQiiSbWA@mail.gmail.com>

Am Montag, 7. April 2025, 18:43:54 Mitteleuropäische Sommerzeit schrieb Rob Herring:
> On Mon, Apr 7, 2025 at 11:40 AM Heiko Stübner <heiko@sntech.de> wrote:
> >
> > Hi Rob,
> >
> > Am Montag, 7. April 2025, 17:56:37 Mitteleuropäische Sommerzeit schrieb Rob Herring:
> > > On Wed, Mar 19, 2025 at 6:56 PM Heiko Stuebner <heiko@sntech.de> wrote:
> > > >
> > > > Hi soc maintainers,
> > > >
> > > > I made an error and accidentially applied a patch that was meant for
> > > > the mfd tree. Thankfully Stephen noticed that when the duplicate
> > > > commit appeared in linux-next.
> > >
> > > Both commits are in v6.15-rc1 now and the revert is not, so this
> > > should not get applied/pulled. Or you will need to revert the revert.
> >
> > yes, that was the intention.
> >
> > Back when I submitted this PR, I talked with Arnd on IRC the next day.
> >
> > As both commits are identical sans some Signed-off-by lines, he suggested
> > not trying to put a revert in, but instead let git solve it itself, because
> >
> > arnd on IRC:
> > > [...], but I worry that this would make things worse if 'git merge'
> > > ends up doing the revert on top of the original commit once it gets to
> > > torvalds. Not sure if that's still a problem in git these days, but
> > > I've seen it happen in the past.
> > > if two identical patches are in different branches, just leaving them
> > > there is usually easier
> >
> > So this PR was already marked as "superseeded" in patchwork back
> > on march 20th.
> >
> >
> > Nevertheless, thanks for making sure no funky revert happens now.
> 
> The commit is still in linux-next though. That's how I happened upon this.

Ah right. I've recreated my next branch earlier today, after the -rc1
release, so this should be gone with the next linux-next.

Heiko



_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

  reply	other threads:[~2025-04-07 18:51 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-03-19 23:54 [GIT PULL] Rockchip dts64 changes for 6.15 #3 Heiko Stuebner
2025-03-19 23:54 ` Heiko Stuebner
2025-04-07 15:56 ` Rob Herring
2025-04-07 15:56   ` Rob Herring
2025-04-07 16:40   ` Heiko Stübner
2025-04-07 16:40     ` Heiko Stübner
2025-04-07 16:43     ` Rob Herring
2025-04-07 16:43       ` Rob Herring
2025-04-07 16:49       ` Heiko Stübner [this message]
2025-04-07 16:49         ` Heiko Stübner
2025-04-08 14:01         ` Rob Herring
2025-04-08 14:01           ` Rob Herring
2025-04-08 14:19           ` Heiko Stübner
2025-04-08 14:19             ` Heiko Stübner

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=2000188.PYKUYFuaPT@diego \
    --to=heiko@sntech.de \
    --cc=arm@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-rockchip@lists.infradead.org \
    --cc=robh@kernel.org \
    --cc=soc@kernel.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.