linux-doc.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Rob Herring <robh@kernel.org>
To: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Cc: Mark Brown <broonie@kernel.org>,
	Linux Doc Mailing List <linux-doc@vger.kernel.org>,
	Jonathan Corbet <corbet@lwn.net>,
	Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>,
	Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>,
	Liam Girdwood <lgirdwood@gmail.com>,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-samsung-soc@vger.kernel.org
Subject: Re: [PATCH v3 21/23] regulator: dt-bindings: update samsung,s2mpa01.yaml reference
Date: Tue, 26 Oct 2021 21:59:56 -0500	[thread overview]
Message-ID: <YXjArKJ1nw3rkx97@robh.at.kernel.org> (raw)
In-Reply-To: <20211020073013.6d144c0d@sal.lan>

On Wed, Oct 20, 2021 at 07:30:13AM +0100, Mauro Carvalho Chehab wrote:
> Em Tue, 19 Oct 2021 13:04:59 +0100
> Mark Brown <broonie@kernel.org> escreveu:
> 
> > On Tue, Oct 19, 2021 at 09:04:20AM +0100, Mauro Carvalho Chehab wrote:
> > 
> > > To mailbombing on a large number of people, only mailing lists were C/C on the cover.
> > > See [PATCH v3 00/23] at: https://lore.kernel.org/all/cover.1634630485.git.mchehab+huawei@kernel.org/  
> > 
> > It'd be a bit easier to put a note in here about what the dependencies
> > are rather than forcing people to go out to a link to figure out what's
> > going on unless it's complicated. 
> >
> > For a case like this where there's no
> > dependencies or real relationship between the patches it's probably
> > better to just not thread everything and send the patches separately to
> > everyone, the threading is just adding noise and confusion.
> 
> It is not that easy, unfortunately. On some cases (specially due to
> DT binding renames) some patches change the context of a hunk, affecting
> a subsequent patch.

Those should be reduced now. I've been checking the renames since early 
August. July really, but MAINTAINERS was not getting checked initially.


> I tried a couple of times in the past to send the patches individually,
> but that was messier, as there was harder for people to apply them,
> as, instead of running b4 just once to get everything, maintainers
> would need to apply each patch individually. Also, there were cases
> where the patch order would be relevant, due to context changes.

Just spliting between in Linus' tree and only in next would help me. The 
former I know I can just apply.

> Btw, talking about what it would be easier, the best procedure to
> adopt is to run:
> 
> 	./scripts/documentation-file-ref-check 
> 
> Before sending/applying patches touching documents.

Good luck with that. :(

> That would avoid the need of such fixup patches ;-)
> 
> Unfortunately, in the specific case of dt-bindings, things are not
> that easy, as doc changes usually go via one tree, while references 
> to them come from other places.
> 
> Regards,
> Mauro
> 

  parent reply	other threads:[~2021-10-27  3:00 UTC|newest]

Thread overview: 60+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-19  8:03 [PATCH v3 00/23] Fix some issues at documentation Mauro Carvalho Chehab
2021-10-19  8:04 ` [PATCH v3 01/23] visorbus: fix a copyright symbol that was bad encoded Mauro Carvalho Chehab
2021-10-19  8:04 ` [PATCH v3 02/23] libbpf: update index.rst reference Mauro Carvalho Chehab
2021-10-19  8:04 ` [PATCH v3 03/23] docs: accounting: update delay-accounting.rst reference Mauro Carvalho Chehab
2021-10-19  8:04 ` [PATCH v3 04/23] MAINTAINERS: update arm,vic.yaml reference Mauro Carvalho Chehab
2021-10-24 22:08   ` Linus Walleij
2021-10-25 12:32     ` Rob Herring
2021-10-19  8:04 ` [PATCH v3 05/23] MAINTAINERS: update aspeed,i2c.yaml reference Mauro Carvalho Chehab
2021-10-19  8:04 ` [PATCH v3 06/23] MAINTAINERS: update faraday,ftrtc010.yaml reference Mauro Carvalho Chehab
2021-10-24 21:53   ` Linus Walleij
2021-10-19  8:04 ` [PATCH v3 07/23] MAINTAINERS: update ti,sci.yaml reference Mauro Carvalho Chehab
2021-10-19  8:04 ` [PATCH v3 08/23] MAINTAINERS: update intel,ixp46x-rng.yaml reference Mauro Carvalho Chehab
2021-10-24 21:52   ` Linus Walleij
2021-10-19  8:04 ` [PATCH v3 09/23] MAINTAINERS: update nxp,imx8-jpeg.yaml reference Mauro Carvalho Chehab
2021-10-19  8:04 ` [PATCH v3 10/23] MAINTAINERS: update gemini.yaml reference Mauro Carvalho Chehab
2021-10-24 21:52   ` Linus Walleij
2021-10-19  8:04 ` [PATCH v3 11/23] MAINTAINERS: update brcm,unimac-mdio.yaml reference Mauro Carvalho Chehab
2021-10-19 16:08   ` Florian Fainelli
2021-10-19  8:04 ` [PATCH v3 12/23] MAINTAINERS: update mtd-physmap.yaml reference Mauro Carvalho Chehab
2021-10-24 21:53   ` Linus Walleij
2021-10-19  8:04 ` [PATCH v3 13/23] Documentation: update vcpu-requests.rst reference Mauro Carvalho Chehab
2021-10-19  9:22   ` Anup Patel
2021-10-19  8:04 ` [PATCH v3 14/23] bpftool: update bpftool-cgroup.rst reference Mauro Carvalho Chehab
2021-10-19  9:35   ` Quentin Monnet
2021-10-19 22:31     ` Alexei Starovoitov
2021-10-20  5:46       ` Mauro Carvalho Chehab
2021-10-19  8:04 ` [PATCH v3 15/23] docs: translations: zn_CN: irq-affinity.rst: add a missing extension Mauro Carvalho Chehab
2021-10-19 21:20   ` Jonathan Corbet
2021-10-19  8:04 ` [PATCH v3 16/23] docs: translations: zh_CN: memory-hotplug.rst: fix a typo Mauro Carvalho Chehab
2021-10-19 21:21   ` Jonathan Corbet
     [not found]     ` <f0269915-4863-9ed6-dedd-592f2e308f46@redhat.com>
2021-11-01  7:46       ` Alex Shi
2021-11-01 13:38         ` teng sterling
2021-11-01 13:59           ` David Hildenbrand
2021-10-19  8:04 ` [PATCH v3 17/23] docs: fs: locks.rst: update comment about mandatory file locking Mauro Carvalho Chehab
2021-10-19 10:47   ` Jeff Layton
2021-10-19  8:04 ` [PATCH v3 18/23] fs: remove a comment pointing to the removed mandatory-locking file Mauro Carvalho Chehab
2021-10-19 10:50   ` Jeff Layton
2021-10-19 14:14     ` J. Bruce Fields
2021-10-19 15:46       ` Jeff Layton
2021-10-19 16:16         ` J. Bruce Fields
2021-10-19 16:27           ` Jeff Layton
2021-10-19 17:38             ` [PATCH] locks: remove changelog comments J. Bruce Fields
2021-10-19 18:14               ` Jeff Layton
2021-10-20  5:50                 ` Mauro Carvalho Chehab
2021-10-19  8:04 ` [PATCH v3 19/23] Documentation/process: fix a cross reference Mauro Carvalho Chehab
2021-10-19  8:04 ` [PATCH v3 20/23] dt-bindings: mfd: update x-powers,axp152.yaml reference Mauro Carvalho Chehab
2021-10-19 16:04   ` Chen-Yu Tsai
2021-10-21 12:12   ` Lee Jones
2021-10-19  8:04 ` [PATCH v3 21/23] regulator: dt-bindings: update samsung,s2mpa01.yaml reference Mauro Carvalho Chehab
2021-10-19  8:24   ` Krzysztof Kozlowski
2021-10-19 12:04   ` Mark Brown
2021-10-20  6:30     ` Mauro Carvalho Chehab
2021-10-20 11:04       ` Mark Brown
2021-10-27  2:59       ` Rob Herring [this message]
2021-10-19  8:04 ` [PATCH v3 22/23] regulator: dt-bindings: update samsung,s5m8767.yaml reference Mauro Carvalho Chehab
2021-10-19  8:24   ` Krzysztof Kozlowski
2021-10-19  8:04 ` [PATCH v3 23/23] dt-bindings: reserved-memory: ramoops: update ramoops.yaml references Mauro Carvalho Chehab
2021-10-19  9:45   ` David Heidelberg
2021-10-19 16:45   ` Rob Herring
2021-10-19 16:53 ` [PATCH v3 00/23] Fix some issues at documentation Rob Herring

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=YXjArKJ1nw3rkx97@robh.at.kernel.org \
    --to=robh@kernel.org \
    --cc=b.zolnierkie@samsung.com \
    --cc=broonie@kernel.org \
    --cc=corbet@lwn.net \
    --cc=devicetree@vger.kernel.org \
    --cc=krzysztof.kozlowski@canonical.com \
    --cc=lgirdwood@gmail.com \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=mchehab+huawei@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).