On 7/10/24 15: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")
(no need for an empty line between your Signed-off-by and the Fixes tags BTW).
Reviewed-by: Florian Fainelli <florian.fainelli@xxxxxxxxxxxx> -- Florian