[PATCH dev-5.0 v1 2/3] ARM: dts: nuvoton: Add Quanta GSJ BMC machine.

Fran Hsu (徐誌謙) Fran.Hsu at quantatw.com
Wed Apr 3 00:54:44 AEDT 2019


Hi Benjamin, Tomer,
	Tomer, thanks for your explanation.
Yes, if we need to define each pin configuration that exist it will be very very long file.
So I will update the nuvoton-npcm730-gsj-gpio.dtsi, remove the useless pin definition from this file.
And then send the patch v2 for reviewing.
Thanks all.

>Hi Fran,
>
>On Mon, 1 Apr 2019 at 18:20, Fran Hsu (徐誌謙) <Fran.Hsu at quantatw.com> wrote:
>Hi Tomer, 
>        I think the main idea is to provide a full and common pinctrl file (just like nuvoton-npcm7xx-gpio.dtsi) for each NPCM7xx BMC family.
>The nuvoton-npcm7xx-gpio define the pin configuration according the board that using it and not the pin controllers that defined in the 
>nuvoton-common-npcm7xx.dtsi.
>
>If we need to define each pin configuration that exist it will be very very long file. because each pin can 
>have several configurations and several pin configuration combinations.
>
>This is why each board need to define if own pin configuration (if it needed), this can be done also in the dts file 
>and not in a different dtsi file.
>
>About nuvoton-npcm750-gpio.dtsi as i said in the last mail it should be nuvoton-npcm750-gpio-evb.dtsi, and probably I will remove a lot of
>unused GPIO pin configuration in this file.
>
>Hope this helps
>
>The device tree maintainers can reference the GPIO configuration from this file what they needed.
>Once if the driver is changed, maybe the only change is the common pinctrl file.
>But if there is no common pinctrl file, each device tree maintainers have to modify the gpio-dtsi file.
>And as your opinion, there will be a lots of gpio-dtsi files for different boards or different vendors.
>And most of gpio-dtsi files will include same GPIO setting. 
>
>Thanks,
>Fran
> 
>>
>>Hi Benjamin,
>>
>>On Sat, 30 Mar 2019 at 02:31, Benjamin Fair <mailto:benjaminfair at google.com> wrote:
>>Hi Fran,
>>
>>Thanks for sending these patches!
>>
>>On Thu, Mar 28, 2019 at 8:06 AM <mailto:fran.hsu at quantatw.com> wrote:
>>>
>>> From: FranHsu <mailto:Fran.Hsu at quantatw.com>
>>>
>>> Add pinctrl definition file for quanta-gsj BMC device tree.
>>>
>>> Signed-off-by: FranHsu <mailto:Fran.Hsu at quantatw.com>
>>> ---
>>>  .../boot/dts/nuvoton-npcm730-gsj-gpio.dtsi    | 2591 +++++++++++++++++
>>>  1 file changed, 2591 insertions(+)
>>>  create mode 100644 arch/arm/boot/dts/nuvoton-npcm730-gsj-gpio.dtsi
>
>Would it make more sense to merge this into the existing
>nuvoton-npcm750-gpio.dtsi file? That seems like it would make it easier to
>maintain the definitions and these are a superset of the existing ones, so it
>shouldn't conflict.
> 
>the GPIO configuration depend on the server board that using it, we think it is better that each 
>server board or a family of server boards will have its own GPIO device tree definition.
>
>We know that the GPIO device tree name and placement is confusing, because is called nuvoton-npcm750-gpio.dtsi and not nuvoton-npcm750-gpio-evb.dtsi 
>and placed at nuvoton-npcm750.dtsi but should move the nuvoton-npcm750-evb.dts file.
>
>we will modify the gpio npcm750 device tree soon.
>
>Fran,
>
>Does the nuvoton-npcm730-gsj-gpio.dtsi set GPIO configuration you are using or will use in the future on Quanta server boards?
>For example I see you defined "gpio0o_pins" pin configuration, and you are not using it in your dts file. will you use it in the future or in other Quanta based NPCM7xx BMC boards?
>if not I think you should remove it. eqtully I think you should defined only the GPIO/PIN configuration you are using, (I attend to do it at nuvoton-npcm750-gpio.dtsi as well)
>
>
>>
>> diff --git a/arch/arm/boot/dts/nuvoton-npcm730-gsj-gpio.dtsi b/arch/arm/boot/dts/nuvoton-npcm730-gsj-gpio.dtsi
>> new file mode 100644
>> index 000000000000..b66ea5099e5e
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/nuvoton-npcm730-gsj-gpio.dtsi
>> @@ -0,0 +1,2591 @@
>> +// SPDX-License-Identifier: GPL-2.0
>> +// Copyright (c) 2018 Nuvoton Technology mailto:tomer.maimon at nuvoton.com
>> +
>> +/ {
>> +       pinctrl: pinctrl at f0800000 {
>> +               gpio0o_pins: gpio0o-pins {
>> +                       pins = "GPIO0/IOX1DI";
>> +                       bias-disable;
>> +                       output-high;
>> +               };
>> +               gpio0ol_pins: gpio0ol-pins {
>> +                       pins = "GPIO0/IOX1DI";
>> +                       bias-disable;
>> +                       output-low;
>> +               };
>> +               gpio0od_pins: gpio0od-pins {
>> +                       pins = "GPIO0/IOX1DI";
>> +                       bias-disable;
>> +                       drive-open-drain;
>> +               };
>> +               gpio0pp_pins: gpio0pp-pins {
>> +                       pins = "GPIO0/IOX1DI";
>> +                       bias-disable;
>> +                       drive-push-pull;
>> +               };
>> +               gpio1_pins: gpio1-pins {
>> +                       pins = "GPIO1/IOX1LD";
>> +                       bias-disable;
>> +                       input-enable;
>> +               };
>> +               gpio1o_pins: gpio1o-pins {
>> +                       pins = "GPIO1/IOX1LD";
>> +                       bias-disable;


More information about the openbmc mailing list