Re: [PATCH v6 00/13] mtd: rawnand: brcmnand: driver and doc updates

From: Miquel Raynal <miquel.raynal@bootlin.com>
To: Florian Fainelli <f.fainelli@gmail.com>
Cc: "Florian Fainelli" <florian.fainelli@broadcom.com>,
	"William Zhang" <william.zhang@broadcom.com>,
	"Linux MTD List" <linux-mtd@lists.infradead.org>,
	"Linux ARM List" <linux-arm-kernel@lists.infradead.org>,
	"Broadcom Kernel List" <bcm-kernel-feedback-list@broadcom.com>,
	kursad.oney@broadcom.com, joel.peshkin@broadcom.com,
	anand.gore@broadcom.com, dregan@mail.com,
	kamal.dasu@broadcom.com, tomer.yacoby@broadcom.com,
	dan.beygelman@broadcom.com,
	"Andre Przywara" <andre.przywara@arm.com>,
	"Rob Herring" <robh+dt@kernel.org>,
	"Kamal Dasu" <kdasu.kdev@gmail.com>,
	"Conor Dooley" <conor+dt@kernel.org>,
	"Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro.org>,
	"Rafał Miłecki" <rafal@milecki.pl>,
	"David Regan" <dregan@broadcom.com>,
	devicetree@vger.kernel.org,
	"Alexandre TORGUE" <alexandre.torgue@st.com>,
	"Vignesh Raghavendra" <vigneshr@ti.com>,
	"Brian Norris" <computersforpeace@gmail.com>,
	"Shawn Guo" <shawnguo@kernel.org>,
	linux-kernel@vger.kernel.org,
	"Richard Weinberger" <richard@nod.at>
Subject: Re: [PATCH v6 00/13] mtd: rawnand: brcmnand: driver and doc updates
Date: Fri, 15 Mar 2024 00:03:46 +0100	[thread overview]
Message-ID: <20240315000346.6fad883e@xps-13> (raw)
In-Reply-To: <d3c66783-b71f-431f-9028-45897f12f2a0@gmail.com>


> >> I'm sorry for not thinking about this ahead of time, I was also not
> >> Cced on the other patches, I noticed it (told Willliam) and just forgot
> >> about this when I applied the series.
> >>
> >> It is currently living in -next so if there is any problem I can still
> >> act.
> >>
> >> However for this kind of change I usually apply the bindings and .c
> >> changes independently from the DT patches. I believe there is no
> >> problem having one or the other being merged first, or do I overlook
> >> something?  
> > 
> > What the heck /o\ I just understand now my mistake, I am very truly
> > sorry for that...
> > 
> > You were telling me I should sync with you before taking DT changes,
> > and I was so convinced I _did_not_ take the DT, when I looked at the
> > branch I did not understand your point. But I am totally sorry I
> > actually did take the DTs by mistake and I truly did not notice it.
> > Confirmation bias I suppose. My very sincere apologies.
> > 
> > As mentioned previously, I was not CC'ed on the DT patches, but I
> > believe the linux-mtd list was, so the patches didn't appear in my
> > inbox, and once I was happy with the binding/driver changes I applied
> > it all without noticing the DT changes had sneaked in.
> > 
> > I'm finally preparing the PR for Linus and I see it now...
> > 
> > I believe the SoC tree is closed now so it's up to you what I should do
> > with them. Let me know if you want me to keep them in my tree and
> > forward them to Linus or if I should drop them and you'll take them for
> > the next cycle. Also, if I keep them, shall I add some tag of yours on
> > these 3 patches? For the record I did not review them.  
> 
> Yes please add my:
> 
> Acked-by: Florian Fainelli <florian.fainelli@broadcom.com>
> 
> tag, and it's fine I don't expect that we will get conflicts for those files.
> 
> > 
> > Thanks and again, I'm confused. I never apply DT patches like that,
> > your initial remark was more than legitimate.  
> 
> Not a problem!

Thanks :-)

Miquèl

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/