Re: [PATCH v4 02/14] arm64: dts: sc7180: Add minimal dts/dtsi files for SC7180 soc

From: Stephen Boyd
Date: Fri Nov 08 2019 - 14:12:28 EST


Quoting Rajendra Nayak (2019-11-07 19:48:57)
>
> On 11/7/2019 11:16 PM, Stephen Boyd wrote:
> > Quoting Rajendra Nayak (2019-11-05 22:50:05)
> >> + qup_uart8_default: qup-uart8-default {
> >> + pinmux {
> >> + pins = "gpio44", "gpio45";
> >> + function = "qup12";
> >
> > That looks weird to have qup12 function on uart8. It's right?
>
> So we have 2 qup instances each with 6 SEs on sc7180.
> So the i2c/uart/spi SE instances are numbered from 0 to 5 in the first qup
> and 6 to 11 in the next.
> The pinctrl functions however have it named qup0 to 5 for first and
> qup10 to 15 for the next which is weird. Now all data in the pinctrl
> driver is autogenerated using hw description so its coming from that.
>
> Just for comparison, on sdm845 we had 2 qup instances with 8 SE's
> and the function names were qup0 to 8 for first and 9 to 15 for the
> second.
>

Alright. Good to know the hardware description is all messed up.