Re: [GIT PULL] pwm fixes for 6.7-rc5

[Thierry Reding <thierry.reding@gmail.com>]

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

On Fri, Dec 08, 2023 at 07:15:03PM +0100, Uwe Kleine-König wrote:
> Hello Thierry,
> 
> On Fri, Dec 08, 2023 at 04:15:33PM +0100, Thierry Reding wrote:
> > On Fri, Dec 08, 2023 at 03:02:53PM +0100, Uwe Kleine-König wrote:
> > > [...]
> > > 
> > > Please pull this as a fix for the next -rc release.
> > 
> > I don't know why you didn't let me know about your plans to send this
> > out. We should've at least discussed this.
> 
> I tried to discuss that but didn't get any response. I pointed out in
> two mails[1] that I think this should go in before 6.7 and I announced
> this PR last week[2]. Also you usually ignore me on irc.

I'm no longer able to keep up with IRC, sorry. Email is the most
reliable way for me to communicate, although obviously that also doesn't
work all the time since I missed that last email from you where you
"announced" that you were going to send a PR.

> So I don't think "you didn't let me know" is a fair suggestion and I
> honestly don't know what I should do better next time.
> 
> Given your reply might confuse Linus about what to do with my PR: Do you
> agree this bcm2835-pwm fix should go in before 6.7?

Of course it should go into 6.7 and it would've gone in. I usually pick
patches up into linux-next first because that's what most people will
end up running and where they will first notice when it's been taken
care of. Then I will typically try to batch things up and will usually
wait until around -rc6 before sending this type of trivial fix to avoid
sending multiple PRs for simple one-line fixes like this.

Linus has already pulled this and that's fine. But we now also see this
patch duplicated in linux-next and it's all leading to extra work where
people need to report this and I have to rebase my tree just because we
didn't communicate properly.

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]