Re: [PATCH] dt-bindings: leds: add "usbport" trigger

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

 



On Wed, Mar 1, 2023 at 1:27 AM Rafał Miłecki <zajec5@xxxxxxxxx> wrote:
>
> On 1.03.2023 01:02, Rob Herring wrote:
> > On Tue, Feb 28, 2023 at 03:49:33PM +0100, Rafał Miłecki wrote:
> >> From: Rafał Miłecki <rafal@xxxxxxxxxx>
> >>
> >> It's a trigger used on many home routers that have LEDs to indicate
> >> specific USB port state.
> >>
> >> Signed-off-by: Rafał Miłecki <rafal@xxxxxxxxxx>
> >> ---
> >>   Documentation/devicetree/bindings/leds/common.yaml | 1 +
> >>   1 file changed, 1 insertion(+)
> >>
> >> diff --git a/Documentation/devicetree/bindings/leds/common.yaml b/Documentation/devicetree/bindings/leds/common.yaml
> >> index 15e3f6645682..95b316ee3146 100644
> >> --- a/Documentation/devicetree/bindings/leds/common.yaml
> >> +++ b/Documentation/devicetree/bindings/leds/common.yaml
> >> @@ -99,6 +99,7 @@ properties:
> >>             - pattern
> >>             - usb-gadget
> >>             - usb-host
> >> +          - usbport
> >
> > Can we stop adding entries which are clearly likely to have multiple
> > instances. We have a better binding to map the trigger source...
>
> I'm sorry, I really don't understand this.
> I'm not sure what do you mean by multuple "usbport" instances.
> Could you point me to that better place, please?

Suppose I have a device with 4 USB ports and 4 LEDs for each one. How
would one define the connection of LEDs to USB ports? Extend this to
usbport[0-9]? No.

> This is probably something obvious but I really can't figure it out
> since yesterday.

"trigger-sources"

Rob




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux