From: Tony Lindgren <tony@atomide.com>
To: Gireesh.Hiremath@in.bosch.com
Cc: linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
linux-omap@vger.kernel.org, robh+dt@kernel.org,
bcousson@baylibre.com, sjoerd.simons@collabora.co.uk,
VinayKumar.Shettar@in.bosch.com,
Govindaraji.Sivanantham@in.bosch.com,
anaclaudia.dias@de.bosch.com
Subject: Re: [PATCH 00/15] am335x, guardian: update board specific dts changes
Date: Thu, 21 Apr 2022 14:08:55 +0300 [thread overview]
Message-ID: <YmE7R3eVO6070X5H@atomide.com> (raw)
In-Reply-To: <20220325100613.1494-1-Gireesh.Hiremath@in.bosch.com>
* Gireesh.Hiremath@in.bosch.com <Gireesh.Hiremath@in.bosch.com> [220325 12:04]:
> From: Gireesh Hiremath <Gireesh.Hiremath@in.bosch.com>
>
> update guardian device tree
Thanks applying into omap-for-v5.19/dt.
Few ideas for future though.
Not sure how you handle the nand partition table changes.. Seems the partition
values in the devicetree should not change in general.
The node names in devicetree should not use "_" but "-" instead. And the node
names should be general like you mostly already have. Anyways, we have a lot
of these especially for pinctrl, but we may want to clean them up eventually.
Just FYI.
Regards,
Tony
prev parent reply other threads:[~2022-04-21 11:09 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-03-25 10:05 [PATCH 00/15] am335x, guardian: update board specific dts changes Gireesh.Hiremath
2022-03-25 10:05 ` [PATCH 01/15] ARM: dts: am335x: Guardian: Update NAND partition table Gireesh.Hiremath
2022-03-25 10:06 ` [PATCH 02/15] ARM: dts: am335x: Guardian: Rename power button label Gireesh.Hiremath
2022-03-25 10:06 ` [PATCH 03/15] ARM: dts: am335x: Guardian: Add keypad Gireesh.Hiremath
2022-03-25 10:06 ` [PATCH 04/15] ARM: dts: am335x: Guardian: Disable poweroff support from RTC Gireesh.Hiremath
2022-03-25 10:06 ` [PATCH 05/15] ARM: dts: am335x: Guardian: Remove mmc status led Gireesh.Hiremath
2022-03-25 10:06 ` [PATCH 06/15] ARM: dts: am335x: Guardian: Update life led Gireesh.Hiremath
2022-03-25 10:06 ` [PATCH 07/15] ARM: dts: am335x: Guardian: Update beeper label Gireesh.Hiremath
2022-03-25 10:06 ` [PATCH 08/15] ARM: dts: am335x: Guardian: Update regulator node name Gireesh.Hiremath
2022-03-25 10:06 ` [PATCH 09/15] ARM: dts: am335x: Guardian: Add lcd port Gireesh.Hiremath
2022-03-25 10:06 ` [PATCH 10/15] ARM: dts: am335x: Guardian: Update backlight parameter Gireesh.Hiremath
2022-03-25 10:06 ` [PATCH 11/15] ARM: dts: am335x: Guardian: Enable UART port two Gireesh.Hiremath
2022-03-25 10:06 ` [PATCH 12/15] ARM: dts: am335x: Guardian: Disable DMA property of USB1 Gireesh.Hiremath
2022-03-25 10:06 ` [PATCH 13/15] ARM: dts: am335x: Guardian: Update interface pinmux Gireesh.Hiremath
2022-03-25 10:06 ` [PATCH 14/15] ARM: dts: am335x: Guardian: Add gpio line manes Gireesh.Hiremath
2022-03-25 10:06 ` [PATCH 15/15] ARM: dts: am335x: Guardian: Update comments Gireesh.Hiremath
2022-04-21 11:08 ` Tony Lindgren [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=YmE7R3eVO6070X5H@atomide.com \
--to=tony@atomide.com \
--cc=Gireesh.Hiremath@in.bosch.com \
--cc=Govindaraji.Sivanantham@in.bosch.com \
--cc=VinayKumar.Shettar@in.bosch.com \
--cc=anaclaudia.dias@de.bosch.com \
--cc=bcousson@baylibre.com \
--cc=devicetree@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=robh+dt@kernel.org \
--cc=sjoerd.simons@collabora.co.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).