LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Tony Lindgren <tony@atomide.com>
To: Christina Quast <cquast@baylibre.com>
Cc: linux-omap@vger.kernel.org, devicetree@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, narmstrong@baylibre.com,
	Oleg Kokorin <ole2mail@mail.com>
Subject: Re: [PATCH 1/2] ARM: dts: am335x: Replace numeric pinmux address with macro defines
Date: Mon, 30 Apr 2018 10:31:46 -0700	[thread overview]
Message-ID: <20180430173146.GG5671@atomide.com> (raw)
In-Reply-To: <20180430112006.20861-2-cquast@baylibre.com>

Hi,

* Christina Quast <cquast@baylibre.com> [180430 11:23]:
> The values are extraced from the "AM335x SitaraTM Processors Technical
> Reference Manual", Section 9.3.1 CONTROL_MODULE Registers, based on the
> file autogenerated by TI PinMux.

This certainly makes things easier to mux :)

Have you verified that the registers are the same across all am335x
models and different revisions?

It used to be that different SoC revisions could have different
amount of registers and also different options in some cases.

> +#define AM335X_CONTROL_REVISION			0x0
> +#define AM335X_CONTROL_HWINFO			0x4
> +#define AM335X_CONTROL_SYSCONFIG		0x10
> +#define AM335X_CONTROL_STATUS			0x40

You should only list the padconf mux registers here, no need to
list any of the controller registers.

Regards,

Tony

  reply	other threads:[~2018-04-30 17:31 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-30 11:20 [PATCH 0/2] Add macro defines for AM335x Christina Quast
2018-04-30 11:20 ` [PATCH 1/2] ARM: dts: am335x: Replace numeric pinmux address with macro defines Christina Quast
2018-04-30 17:31   ` Tony Lindgren [this message]
2018-05-03 15:52     ` Grygorii Strashko
2018-05-03 16:19       ` Tony Lindgren
2018-05-03 18:25         ` Grygorii Strashko
2018-04-30 11:20 ` [PATCH 2/2] ARM: dts: am335x: Replaced register offsets with defines Christina Quast
2019-03-13 14:27 [PATCH 0/2] ARM: dts: am335x: Replace numeric pinmux address with macro defines Christina Quast
2019-03-13 14:27 ` [PATCH 1/2] " Christina Quast
2019-03-14 14:59   ` Tony Lindgren

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=20180430173146.GG5671@atomide.com \
    --to=tony@atomide.com \
    --cc=cquast@baylibre.com \
    --cc=devicetree@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=narmstrong@baylibre.com \
    --cc=ole2mail@mail.com \
    --subject='Re: [PATCH 1/2] ARM: dts: am335x: Replace numeric pinmux address with macro defines' \
    /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

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).