All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Gortmaker <paul.gortmaker@windriver.com>
To: Phillip Potter <phil@philpotter.co.uk>
Cc: Jonathan Corbet <corbet@lwn.net>,
	linux-kernel@vger.kernel.org, Jens Axboe <axboe@kernel.dk>,
	Christoph Hellwig <hch@lst.de>,
	Randy Dunlap <rdunlap@infradead.org>
Subject: Re: [PATCH 3/3] block: remove last remaining traces of IDE documentation
Date: Fri, 6 May 2022 11:32:41 -0400	[thread overview]
Message-ID: <20220506153241.GH12977@windriver.com> (raw)
In-Reply-To: <YmsmnGb3JNjH54Xb@equinox>

[Re: [PATCH 3/3] block: remove last remaining traces of IDE documentation] On 29/04/2022 (Fri 00:43) Phillip Potter wrote:

> On Wed, Apr 27, 2022 at 12:59:17PM -0400, Paul Gortmaker wrote:
> > [Re: [PATCH 3/3] block: remove last remaining traces of IDE documentation] On 27/04/2022 (Wed 08:50) Jonathan Corbet wrote:
> > 
> > > The Documentation/ide part of this is already dealt with in docs-next;
> > > obviously there was more to do, though :)
> > 
> > Ah, I'd checked mainline master of today but not sfr's next.
> > 
> > Here is a delta diff against today's linux-next
> > 
> > Thanks,
> > Paul.
> > --
> > 
> > From ecb86eb357e5151ba5f7e7d172c65d07d88c4c39 Mon Sep 17 00:00:00 2001
> > From: Paul Gortmaker <paul.gortmaker@windriver.com>
> > Date: Wed, 27 Apr 2022 12:45:50 -0400
> > Subject: [PATCH -next] Documentation: remove last remaining traces of IDE
> >  information
> > 
> > The last traces of the IDE driver went away in commit b7fb14d3ac63
> > ("ide: remove the legacy ide driver") but it left behind some traces
> > of old documentation.
> > 
> > As luck would have it Randy and I would submit similar changes within
> > a week of each other to address this.  As Randy's commit is in the doc
> > tree already - this delta is just the stuff my removal contained that
> > was not in Randy's IDE doc removal.

[...]

> 
> Hi Paul,
> 
> Apologies if I'm missing something, but this updated diff still seems to
> conflict with Randy's earlier one. As cdrom doesn't get a lot of churn,
> I agreed with Jens that I would usually just send on all accepted
> patches at once to him and he would take them via his tree (I currently
> have four accepted patches, including your two others and Randy's patch,
> plus one other).
> 
> Anyhow, please could this be corrected? Or me shown the error of my ways
> (always possible I'm making a mistake) :-) Many thanks.

Are you working off linux-next?  If not, what is your baseline and what
are you running and what do you see?  For example, the commands below:

The ecb86 that I sent in this e-mail still applies on linux-next of
today which contains Jens next as you can seed:

$git show next-20220506~84^2~5
commit 9c83ebdb74780b2bf2f0cdf4964b33fab696cc44
Merge: 6694d6faf1e5 49c3b9266a71
Author: Jens Axboe <axboe@kernel.dk>
Date:   Thu May 5 17:18:31 2022 -0600

    Merge branch 'for-5.19/drivers' into for-next
        
$git checkout -b ide-next next/master
Branch 'ide-next' set up to track remote branch 'master' from 'next'.
Switched to a new branch 'ide-next'

$git cherry-pick ecb86eb357e5151ba5f7e7d172c65d07d88c4c39
Auto-merging Documentation/index.rst
Removing Documentation/cdrom/ide-cd.rst   
Auto-merging Documentation/admin-guide/kernel-parameters.txt
[ide-next 458628cda8fe] Documentation: remove last remaining traces of
IDE information
 Date: Wed Apr 27 12:45:50 2022 -0400
  6 files changed, 9 insertions(+), 646 deletions(-)
   delete mode 100644 Documentation/cdrom/ide-cd.rst

Paul.
--

> 
> Regards,
> Phil

  reply	other threads:[~2022-05-06 15:33 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-27 13:24 [PATCH 0/3] remove remaining IDE driver fragments Paul Gortmaker
2022-04-27 13:24 ` [PATCH 1/3] cdrom: remove the unused driver specific disc change ioctl Paul Gortmaker
2022-04-28 23:28   ` Phillip Potter
2022-04-27 13:24 ` [PATCH 2/3] cdrom: mark CDROMGETSPINDOWN/CDROMSETSPINDOWN obsolete Paul Gortmaker
2022-04-28 23:32   ` Phillip Potter
2022-04-27 13:24 ` [PATCH 3/3] block: remove last remaining traces of IDE documentation Paul Gortmaker
2022-04-27 14:50   ` Jonathan Corbet
2022-04-27 16:50     ` Randy Dunlap
2022-04-27 16:59     ` Paul Gortmaker
2022-04-28 23:43       ` Phillip Potter
2022-05-06 15:32         ` Paul Gortmaker [this message]
2022-05-06 17:54           ` Phillip Potter
2022-05-06 20:24             ` Paul Gortmaker
2022-05-06 20:27               ` Randy Dunlap
2022-05-07 20:57               ` Phillip Potter
2022-05-09 22:17             ` Jonathan Corbet
2022-05-09 23:03               ` Phillip Potter
2022-05-09 23:17                 ` Jens Axboe
2022-05-10  8:00                   ` Phillip Potter
2022-05-10  5:51               ` Christoph Hellwig
2022-05-10  8:09                 ` Phillip Potter

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=20220506153241.GH12977@windriver.com \
    --to=paul.gortmaker@windriver.com \
    --cc=axboe@kernel.dk \
    --cc=corbet@lwn.net \
    --cc=hch@lst.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=phil@philpotter.co.uk \
    --cc=rdunlap@infradead.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.