On Tue, Feb 28, 2017 at 3:51 PM, Rafał Miłecki <zajec5@xxxxxxxxx> wrote: > On 02/28/2017 10:38 PM, Jacek Anaszewski wrote: >> >> I think that it would be simpler if we could initially see >> a complete sample dts implementation containing all required DT >> nodes. The example could contain timer trigger as well as usb-port >> trigger specific bindings. > > > Please take a look at attached patch. I used it on Tenda AC9 with: I'm not sure about this extra level of indirection. I don't see the need. > usb_trigger: usb-trigger { > trigger-type = "usbport"; Why do we need to know the type? The trigger device knows what type it is. All we should need to know here is what device(s) controls an LED. The rest the kernel can figure out. > ports = <&ohci_port1>, <&ehci_port1>; > }; > > usb { > label = "bcm53xx:blue:usb"; > gpios = <&chipcommon 1 GPIO_ACTIVE_HIGH>; > triggers = <&usb_trigger>; > };