On 7/11/24 01:16, Jim Quinlan wrote: > o Move the clk_prepare_enable() below the resource allocations. > o Add a jump target (clk_out) so that a bit of exception handling can be > better reused at the end of this function implementation. > > Signed-off-by: Jim Quinlan <james.quinlan@xxxxxxxxxxxx> > > Fixes: bb610757fcd7 ("PCI: brcmstb: Use reset/rearm instead of deassert/assert") I don't think the current patch is related to bb610757fcd7, please drop Fixes tag. > --- > drivers/pci/controller/pcie-brcmstb.c | 29 +++++++++++++++------------ > 1 file changed, 16 insertions(+), 13 deletions(-) Reviewed-by: Stanimir Varbanov <svarbanov@xxxxxxx> ~Stan