Re: [PATCH 07/13] pinctrl: add a pincontrol driver for BCM6358

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

 




On Mon, Aug 22, 2016 at 3:57 PM, Jonas Gorski <jonas.gorski@xxxxxxxxx> wrote:
> On 22 August 2016 at 15:21, Linus Walleij <linus.walleij@xxxxxxxxxx> wrote:

>>> +       else
>>> +               mode = syscon_regmap_lookup_by_pdevname("syscon.fffe0098");
>>
>> That looks very very fragile.
>
> For !OF I need to use the pdevname, so I need to "guess" the right
> one, and using the scheme of "syscon.<address>" seemed to be the one
> most likely not conflicting with anything else.

OK from my side, I can live with that.

CC:ing the syscon pdev support author: Alexander what is your
thoughts about this? Is there a good convention to follow for
pdev lookups?

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux