[PATCH net-next 2/2] drivers: net: dsa: mt7530: MT7530 optional GPIO support

Linus Walleij linus.walleij at linaro.org
Fri Jan 22 05:06:30 EST 2021


On Tue, Jan 19, 2021 at 4:20 AM DENG Qingfang <dqfext at gmail.com> wrote:
> On Mon, Jan 18, 2021 at 10:55 PM Linus Walleij <linus.walleij at linaro.org> wrote:
> >
> > So for offset 0..14 this becomes bits
> > 0, 1, 2, 4, 5, 6, 8, 9, 10, 12  ... 18
> >
> > What is the logic in this and is it what you intend?
>
> Yes. Bit 0..2 are phy 0's LED 0..2, bit 4..6 are phy 1's LED 0..2, etc.

OK add a comment and explain how the bits relate
to each PHY and how the lines are arranged per-phy
so it is crystal clear for people reading the driver.

Thanks!
Linus Walleij



More information about the linux-arm-kernel mailing list