diff mbox series

[v6,17/20] ARM: dts: aspeed: Add IBM Huygens BMC system

Message ID 20240522192524.3286237-18-eajames@linux.ibm.com
State Not Applicable
Headers show
Series ARM: dts: aspeed: Add IBM P11 BMC systems | expand

Commit Message

Eddie James May 22, 2024, 7:25 p.m. UTC
The Huygens is a Rainier with modifed FSI wiring.

Signed-off-by: Eddie James <eajames@linux.ibm.com>
---
 arch/arm/boot/dts/aspeed/Makefile             |  1 +
 .../dts/aspeed/aspeed-bmc-ibm-huygens.dts     | 23 +++++++++++++++++++
 2 files changed, 24 insertions(+)
 create mode 100644 arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-huygens.dts

Comments

kernel test robot May 23, 2024, 12:48 p.m. UTC | #1
Hi Eddie,

kernel test robot noticed the following build errors:

[auto build test ERROR on andi-shyti/i2c/i2c-host]
[also build test ERROR on linus/master v6.9 next-20240523]
[cannot apply to robh/for-next broonie-spi/for-next]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Eddie-James/spi-dt-bindings-Document-the-IBM-FSI-attached-SPI-controller/20240523-033334
base:   git://git.kernel.org/pub/scm/linux/kernel/git/andi.shyti/linux.git i2c/i2c-host
patch link:    https://lore.kernel.org/r/20240522192524.3286237-18-eajames%40linux.ibm.com
patch subject: [PATCH v6 17/20] ARM: dts: aspeed: Add IBM Huygens BMC system
config: arm-randconfig-001-20240523 (https://download.01.org/0day-ci/archive/20240523/202405232008.olE9azVd-lkp@intel.com/config)
compiler: arm-linux-gnueabi-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240523/202405232008.olE9azVd-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202405232008.olE9azVd-lkp@intel.com/

All errors (new ones prefixed by >>):

>> Error: arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-huygens.dts:13.2-37 Properties must precede subnodes
   FATAL ERROR: Unable to parse input tree
Markus Elfring May 23, 2024, 6:45 p.m. UTC | #2
> The Huygens is a Rainier with modifed FSI wiring.

Will imperative wordings become helpful for a better commit message here?

Regards,
Markus
Eddie James May 23, 2024, 7 p.m. UTC | #3
On 5/23/24 13:45, Markus Elfring wrote:
>> The Huygens is a Rainier with modifed FSI wiring.
> Will imperative wordings become helpful for a better commit message here?


This statement is a description of hardware. I cannot word that 
imperatively. The commit message is imperative - "Add Huygens system".


>
> Regards,
> Markus
Eddie James May 23, 2024, 7:07 p.m. UTC | #4
On 5/23/24 07:48, kernel test robot wrote:
> Hi Eddie,
>
> kernel test robot noticed the following build errors:


Silly mistake. If this patch can be dropped from the series, I can send 
an updated Huygens DTS after this series is merged, unless I need to do 
a v7.


Thanks,

Eddie


>
> [auto build test ERROR on andi-shyti/i2c/i2c-host]
> [also build test ERROR on linus/master v6.9 next-20240523]
> [cannot apply to robh/for-next broonie-spi/for-next]
> [If your patch is applied to the wrong git tree, kindly drop us a note.
> And when submitting patch, we suggest to use '--base' as documented in
> https://git-scm.com/docs/git-format-patch#_base_tree_information]
>
> url:    https://github.com/intel-lab-lkp/linux/commits/Eddie-James/spi-dt-bindings-Document-the-IBM-FSI-attached-SPI-controller/20240523-033334
> base:   git://git.kernel.org/pub/scm/linux/kernel/git/andi.shyti/linux.git i2c/i2c-host
> patch link:    https://lore.kernel.org/r/20240522192524.3286237-18-eajames%40linux.ibm.com
> patch subject: [PATCH v6 17/20] ARM: dts: aspeed: Add IBM Huygens BMC system
> config: arm-randconfig-001-20240523 (https://download.01.org/0day-ci/archive/20240523/202405232008.olE9azVd-lkp@intel.com/config)
> compiler: arm-linux-gnueabi-gcc (GCC) 13.2.0
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240523/202405232008.olE9azVd-lkp@intel.com/reproduce)
>
> If you fix the issue in a separate patch/commit (i.e. not just a new version of
> the same patch/commit), kindly add following tags
> | Reported-by: kernel test robot <lkp@intel.com>
> | Closes: https://lore.kernel.org/oe-kbuild-all/202405232008.olE9azVd-lkp@intel.com/
>
> All errors (new ones prefixed by >>):
>
>>> Error: arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-huygens.dts:13.2-37 Properties must precede subnodes
>     FATAL ERROR: Unable to parse input tree
>
Markus Elfring May 23, 2024, 7:30 p.m. UTC | #5
>>> The Huygens is a Rainier with modifed FSI wiring.
>> Will imperative wordings become helpful for a better commit message here?
>
>
> This statement is a description of hardware. I cannot word that imperatively.

Please take another look at corresponding improvement possibilities.


> The commit message is imperative - "Add Huygens system".

This information fits to the summary phrase.

Would you like to mention in the changelog that a hardware description
should be extended anyhow?
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.9#n94

Regards,
Markus
Conor Dooley May 23, 2024, 7:33 p.m. UTC | #6
Markus,

On Thu, May 23, 2024 at 09:30:55PM +0200, Markus Elfring wrote:
> >>> The Huygens is a Rainier with modifed FSI wiring.
> >> Will imperative wordings become helpful for a better commit message here?
> >
> >
> > This statement is a description of hardware. I cannot word that imperatively.
> 
> Please take another look at corresponding improvement possibilities.
> 
> 
> > The commit message is imperative - "Add Huygens system".
> 
> This information fits to the summary phrase.
> 
> Would you like to mention in the changelog that a hardware description
> should be extended anyhow?
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.9#n94

You are talking absolute crap here. Stop harassing contributors with
your inane comments.

Thanks,
Conor.
Markus Elfring May 23, 2024, 7:46 p.m. UTC | #7
>> Would you like to mention in the changelog that a hardware description
>> should be extended anyhow?
>> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.9#n94
>
> You are talking absolute crap here. Stop harassing contributors with
> your inane comments.

Why do you interpret my patch review contributions in this direction
when the official Linux development documentation provides special advice
on affected wording details?

Regards,
Markus
Krzysztof Kozlowski May 23, 2024, 8:24 p.m. UTC | #8
On 23/05/2024 21:00, Eddie James wrote:
> 
> On 5/23/24 13:45, Markus Elfring wrote:
>>> The Huygens is a Rainier with modifed FSI wiring.
>> Will imperative wordings become helpful for a better commit message here?
> 
> 
> This statement is a description of hardware. I cannot word that 
> imperatively. The commit message is imperative - "Add Huygens system".

Feel free to ignore all comments coming from Markus (or implement,
entirely up to you).

Markus is banned from mailing lists and most of maintainers already
ignore him or already marked as spam.

Best regards,
Krzysztof
Markus Elfring May 23, 2024, 8:33 p.m. UTC | #9
> Markus is banned from mailing lists and most of maintainers already
> ignore him or already marked as spam.

The running patch review can eventually become more constructive
despite of recurring communication difficulties.

Regards,
Markus
Al Viro May 23, 2024, 8:33 p.m. UTC | #10
On Thu, May 23, 2024 at 09:46:48PM +0200, Markus Elfring wrote:
> >> Would you like to mention in the changelog that a hardware description
> >> should be extended anyhow?
> >> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.9#n94
> >
> > You are talking absolute crap here. Stop harassing contributors with
> > your inane comments.
> 
> Why do you interpret my patch review contributions in this direction
> when the official Linux development documentation provides special advice
> on affected wording details?

Your "contributions" are garbage in general, and this thread is not an exception.
More specifically, you are picking an advice that is inapplicable, transforming
it into a question and "contributing" the result.

And your entire modus operandi fits that pattern - you spew random garbage and
expect the contributors to spend their time and efforts on checking if your
(contents-free) "advice" happens to make any sense.

That.  Is.  Worthless.

According to people who'd met you in person you *are* a member of our species,
and I can't exclude the possibility that in some other environments you might
be capable of sentience.  Unfortunately, the kernel development is clearly
not among those.
Ninad Palsule May 23, 2024, 9:38 p.m. UTC | #11
Reviewed-by: Ninad Palsule <ninad@linux.ibm.com>
Markus Elfring May 24, 2024, 6:38 a.m. UTC | #12
>> Why do you interpret my patch review contributions in this direction
>> when the official Linux development documentation provides special advice
>> on affected wording details?
>
> Your "contributions" are garbage in general,

My contributions are also varying (as usual) through the years.


>                                              and this thread is not an exception.

It is just another example for involved communication challenges.


> More specifically, you are picking an advice

Some development activities are reminders according to known information sources.


>                                              that is inapplicable,
> transforming it into a question and "contributing" the result.
>
> And your entire modus operandi fits that pattern - you spew random garbage and
> expect the contributors to spend their time and efforts on checking if your
> (contents-free) "advice" happens to make any sense.

Do you express special concerns here which can be reconsidered because of
advices and requirements from software development guidelines?


…
>                           Unfortunately, the kernel development is clearly
> not among those.

How does such a view fit to an other data representation?
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/log/?qt=author&q=Elfring

Regards,
Markus
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/aspeed/Makefile b/arch/arm/boot/dts/aspeed/Makefile
index 5e3392621697a..ac2804c96554a 100644
--- a/arch/arm/boot/dts/aspeed/Makefile
+++ b/arch/arm/boot/dts/aspeed/Makefile
@@ -36,6 +36,7 @@  dtb-$(CONFIG_ARCH_ASPEED) += \
 	aspeed-bmc-ibm-bonnell.dtb \
 	aspeed-bmc-ibm-everest.dtb \
 	aspeed-bmc-ibm-fuji.dtb \
+	aspeed-bmc-ibm-huygens.dtb \
 	aspeed-bmc-ibm-rainier.dtb \
 	aspeed-bmc-ibm-rainier-1s4u.dtb \
 	aspeed-bmc-ibm-rainier-4u.dtb \
diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-huygens.dts b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-huygens.dts
new file mode 100644
index 0000000000000..4a731b772fc0b
--- /dev/null
+++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-huygens.dts
@@ -0,0 +1,23 @@ 
+// SPDX-License-Identifier: GPL-2.0-or-later
+// Copyright 2024 IBM Corp.
+/dts-v1/;
+
+#include "aspeed-bmc-ibm-rainier.dts"
+
+/ {
+	model = "Huygens";
+};
+
+&fsim0 {
+	/delete-node/ cfam@0,0;
+	/delete-property/ cfam-reset-gpios;
+
+	bus-frequency = <100000000>;
+};
+
+&fsim1 {
+	#address-cells = <2>;
+	#size-cells = <0>;
+	status = "okay";
+	bus-frequency = <100000000>;
+};