On Wed, Oct 07, 2020 at 11:25:04AM +0000, Joel Stanley wrote: > On Wed, 30 Sep 2020 at 04:08, Ryan Chen <[email protected]> wrote: > > > > Add support for AST2600 SOC UHCI driver. > > > > Signed-off-by: Ryan Chen <[email protected]> > > Reviewed-by: Joel Stanley <[email protected]> > > USB maintainers, can you please take this patch (2/3)? I will take the > others in this series through the aspeed tree. > > Cheers, > > Joel > > > > --- > > drivers/usb/host/uhci-platform.c | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/usb/host/uhci-platform.c > > b/drivers/usb/host/uhci-platform.c > > index 70dbd95c3f06..fa40fe125c2a 100644 > > --- a/drivers/usb/host/uhci-platform.c > > +++ b/drivers/usb/host/uhci-platform.c > > @@ -113,7 +113,8 @@ static int uhci_hcd_platform_probe(struct > > platform_device *pdev) > > num_ports); > > } > > if (of_device_is_compatible(np, "aspeed,ast2400-uhci") || > > - of_device_is_compatible(np, "aspeed,ast2500-uhci")) { > > + of_device_is_compatible(np, "aspeed,ast2500-uhci") > > || > > + of_device_is_compatible(np, "aspeed,ast2600-uhci")) > > { > > uhci->is_aspeed = 1; > > dev_info(&pdev->dev, > > "Enabled Aspeed implementation > > workarounds\n"); > > -- > > 2.17.1 > >
The indentation of the continuation lines isn't good, because the continued parts are aligned with the lines in the conditional block. They should be visually distinct. Either align with the "of_device..." at the start or indent by two extra tab stops. Once this is fixed, you can add: Acked-by: Alan Stern <[email protected]> Alan Stern
