Re: [PATCH v5 6/8] arm64: dts: freescale: Add the GOcontroll Moduline Display baseboard

[Maud Spierings | GOcontroll <maudspierings@gocontroll.com>]

On 4/15/25 20:58, Frank Li wrote:
> On Tue, Apr 15, 2025 at 08:54:29AM +0200, Maud Spierings via B4 Relay wrote:
>> From: Maud Spierings <maudspierings@gocontroll.com>
>>
>> The Moduline Display platform is a part of the wider GOcontroll Moduline
>> ecosystem. These are embedded controllers that focus on modularity with
>> their swappable IO modules.
>>
>> The base Moduline Display board includes a board-to-board connector with
>> various busses to enable adding new display types required by the
>> application. It includes 2 Moduline IO module slots, a simple mono
>> codec/amplifier, a four channel adc, 2 CAN busses, an RTC and optional
>> wifi/bluetooth.
>>
>> busses to the display adapter include:
>> - 4 lane LVDS
>> - 4 lane MIPI-DSI
>> - 4 lane MIPI-CSI
>> - HDMI 2.0a
>> - USB 2.0
>> - I2S
>> - I2C
>> - SPI
>>
>> Also a couple of GPIO and PWM pins for controlling various ICs on the
>> display adapter board.
>>
>> Signed-off-by: Maud Spierings <maudspierings@gocontroll.com>
>> ---
[...]
>> +
>> +&ecspi1 {
>> +	cs-gpios = <&gpio2 12 GPIO_ACTIVE_LOW>,
>> +			   <&gpio1 11 GPIO_ACTIVE_LOW>,
>> +			   <&gpio1 10 GPIO_ACTIVE_LOW>;
> 
> indention mess!
I'm not really sure how to resolve this correctly, currently the next
lines have three tabs and three spaces, which lines up which tab = 4
spaces but that is not the case for everyone.

I just saw two other ways this was done:
4 tabs (doesn't line up at all)
2 tabs and 3 spaces (doesn't line up for tab = 4 spaces but does for tab
= 8 spaces

Another way I thought is 1 tab and 11 spaces which lines up for all
cases, but excessive amount of spaces.

> Frank
-- 
Met vriendelijke groeten/with kind regards,
Maud Spierings