[lkml]   [2015]   [Feb]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] led/led-class: Handle LEDs with the same name
Hello Bryan

On Wed, Feb 18, 2015 at 1:52 AM, Bryan Wu <> wrote:
>> Lets say that we have a type of add-on card. Described by this DT
>> overlay (card.dtb):
> I think who write this card.dtb should understand this issue. And
> choose the right name.

card.dtb just describe the hardware in the card, and it is not be
aware of the rest of the system.

I dont think it is practical to have card_HOST0_PCI1.dtb,
card_HOST0_PCI2.dtb to HOST0_PCI16.dtb and then HOST1_, HOST2....

>> gpio_0: gpio_0 {
>What happen if you just use name 'gpio: gpio {' here.? Any conflicts
>or kernel oops?

No problem here, one will create the device


and the other:


Name is created with hierarnchy

/sys/class/gpio/ will also work fine, because the gpiochip id is
created dynamically

On the other hand all the leds are under,


Do not have any dynamic naming or hierarchical name.

> So you're supposed to get 2 card.dtb files for 2 PCI cards, right?
> They should be different and you need to choose different name for the
> hardware.

There is only one card.dtb because both cards are identical cards,
they are just connected to different ports

> From device tree point of view, I believe different device should got
> different name although they can match to same compatible string. Let
> me invite DT folks for help.

Another example of duplicated names could be partitions on an mtd. You
can have two devices with a partition called Golden.

If my memory is right, you can even have two partition with the same
name on the same device. The offset of the partition will be part of
the name in that case.

> Thanks,
> -Bryan


Ricardo Ribalda

 \ /
  Last update: 2015-02-18 02:21    [W:0.159 / U:1.696 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site