Re: [PATCH v9 1/4] clk: qcom: gdsc: Release pm subdomains in reverse add order

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jan 06, 2025 at 04:55:18PM +0000, Bryan O'Donoghue wrote:
> On 06/01/2025 16:53, Bjorn Andersson wrote:
> > This sounds very reasonable to me, but what's the actual reason?
> > 
> > > Fixes: 1b771839de05 ("clk: qcom: gdsc: enable optional power domain support")
> > > Cc:stable@xxxxxxxxxxxxxxx
> > Without a reason it's hard to see why this needs to be backported.
> > 
> > Regards,
> > Bjorn
> 
> The reason is it makes the next patch much cleaner and makes backporting the
> Fixes in the next patch cleaner too.
> 

That makes sense, but let's state that in the commit message then.

> I could squash the two patches together as another option..

That would work too. Although in the unexpected case that the order has
any impact on outcome it would still be nice to have a comment about why
it was done - and why it was flagged for stable backporting.

Regards,
Bjorn




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux