From: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
To: linux-clk@vger.kernel.org, x86@kernel.org,
platform-driver-x86@vger.kernel.org,
Stephen Boyd <sboyd@codeaurora.org>,
Darren Hart <dvhart@infradead.org>,
Thomas Gleixner <tglx@linutronix.de>
Cc: alsa-devel@alsa-project.org,
Irina Tirdea <irina.tirdea@intel.com>,
Michael Turquette <mturquette@baylibre.com>,
"Rafael J . Wysocki" <rjw@rjwysocki.net>,
Takashi Iwai <tiwai@suse.com>,
linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org,
Ingo Molnar <mingo@redhat.com>, Mark Brown <broonie@kernel.org>,
"H . Peter Anvin" <hpa@zytor.com>, Len Brown <lenb@kernel.org>,
Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
Vinod Koul <vinod.koul@intel.com>,
Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Subject: [PATCH v8 0/5] Add platform clock for BayTrail platforms
Date: Mon, 23 Jan 2017 12:07:40 -0600 [thread overview]
Message-ID: <1485194865-10400-1-git-send-email-pierre-louis.bossart@linux.intel.com> (raw)
These patches specifically enable the audio MCLK required by Baytrail
CR devices. It is the remaining part of a bigger set of patches
(already merged in Mark Brown's tree) that enable sound for Baytrail CR
devices (especially Asus T100TAF) [1]. They include the clock driver
and clock enabling in the pmc_atom code (along with moving of the
non-architectural pmc_atom driver code into drivers/platform/x86 as
suggested by Thomas Gleixner [2]). This move includes a new header in
include/linux/platform_data/x86/. While there is an agreement that the
definitions for PMC clocks are not really platform data this location
is seen as a good-enough compromise with an agreement between Darren
Hart and Andy Shevchenko [3]
[1] http://mailman.alsa-project.org/pipermail/alsa-devel/2016-August/111704.html
[2] http://mailman.alsa-project.org/pipermail/alsa-devel/2016-October/113936.html
[3] http://mailman.alsa-project.org/pipermail/alsa-devel/2016-December/115892.html
Changes from v7:
- Replaced clk-readl/writel by regular readl/writel
- Fixed two nitpick comments (newline and driver name string)
- Fixed [5:0] to [0..5] in comment (missed in v6)
Changes from v6:
- Addressed comments from Stephen Boyd, Andy Shevchenko and Vinod Koul
- Renamed clk-byt-plt to clk-pmc-atom
- Rebase to 4.10-rc3 (broonie/for-next)
- Added helper functions to make error handling more compact
- Removed module-related code since it's not possible/supported
- Removed useless tests on remove
- PMC_CLK_OFFSET move to clk-pcm-atom (more logical)
- Split typo correction and makefile fix in separate patches
- small fixes (spaces, defaults in switches, alphabetical order, use of
clk_writel/readl and no test on write, simpler error handling, use of
while(i--), unsigned loop variables, kernel doc fields, format-patch -M
-C -D, use of PLATFORM_DEVID_NONE)
Changes from v5:
- fix build error reported by kbuild test robot
- split the clk driver code from x86 platform changes
Changes from v4:
- move the pmc_atom driver from arch/x86/platform/atom to
drivers/platform/x86
Changes from v3:
- replace devm_kzalloc with devm_kcalloc
- add x86 architecture maintainers
Changes from v2:
- move clk platform data structures to a separate include file
- store clk_hw pointer for the fixed rate clocks
Changes from v1:
- register the clk device as child of pmc device
- pass iomem pointer from pmc driver to clk driver to avoid using
pmc_atom_read()/write() and use readl/writel API instead
- use devm_clk_hw_register/clkdev_hw_create instead of
clk_register/clkdev_create
Irina Tirdea (3):
clk: x86: Add Atom PMC platform clocks
arch/x86/platform/atom: Move pmc_atom to drivers/platform/x86
platform/x86: Enable Atom PMC platform clocks
Pierre-Louis Bossart (2):
clk: Make x86/ conditional on CONFIG_COMMON_CLK
platform/x86: fix typo in comment
arch/x86/Kconfig | 4 -
arch/x86/platform/atom/Makefile | 1 -
drivers/acpi/acpi_lpss.c | 2 +-
drivers/clk/Makefile | 2 +
drivers/clk/x86/Makefile | 1 +
drivers/clk/x86/clk-pmc-atom.c | 371 +++++++++++++++++++++
drivers/platform/x86/Kconfig | 5 +
drivers/platform/x86/Makefile | 1 +
.../atom => drivers/platform/x86}/pmc_atom.c | 88 ++++-
include/linux/platform_data/x86/clk-pmc-atom.h | 44 +++
.../linux/platform_data/x86}/pmc_atom.h | 2 +-
11 files changed, 506 insertions(+), 15 deletions(-)
create mode 100644 drivers/clk/x86/clk-pmc-atom.c
rename {arch/x86/platform/atom => drivers/platform/x86}/pmc_atom.c (87%)
create mode 100644 include/linux/platform_data/x86/clk-pmc-atom.h
rename {arch/x86/include/asm => include/linux/platform_data/x86}/pmc_atom.h (98%)
--
2.7.4
next reply other threads:[~2017-01-23 18:07 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-23 18:07 Pierre-Louis Bossart [this message]
2017-01-23 18:07 ` [PATCH v8 1/5] clk: Make x86/ conditional on CONFIG_COMMON_CLK Pierre-Louis Bossart
2017-01-27 0:22 ` Stephen Boyd
2017-01-23 18:07 ` [PATCH v8 2/5] clk: x86: Add Atom PMC platform clocks Pierre-Louis Bossart
2017-01-27 0:22 ` Stephen Boyd
2017-01-23 18:07 ` [PATCH v8 3/5] arch/x86/platform/atom: Move pmc_atom to drivers/platform/x86 Pierre-Louis Bossart
2017-01-23 18:07 ` [PATCH v8 4/5] platform/x86: Enable Atom PMC platform clocks Pierre-Louis Bossart
2017-01-27 0:23 ` Stephen Boyd
2017-07-10 10:19 ` Enric Balletbo Serra
2017-01-23 18:07 ` [PATCH v8 5/5] platform/x86: fix typo in comment Pierre-Louis Bossart
2017-01-27 0:23 ` Stephen Boyd
2017-01-23 21:47 ` [PATCH v8 0/5] Add platform clock for BayTrail platforms Andy Shevchenko
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=1485194865-10400-1-git-send-email-pierre-louis.bossart@linux.intel.com \
--to=pierre-louis.bossart@linux.intel.com \
--cc=alsa-devel@alsa-project.org \
--cc=andriy.shevchenko@linux.intel.com \
--cc=broonie@kernel.org \
--cc=dvhart@infradead.org \
--cc=hpa@zytor.com \
--cc=irina.tirdea@intel.com \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@redhat.com \
--cc=mturquette@baylibre.com \
--cc=platform-driver-x86@vger.kernel.org \
--cc=rjw@rjwysocki.net \
--cc=sboyd@codeaurora.org \
--cc=tglx@linutronix.de \
--cc=tiwai@suse.com \
--cc=vinod.koul@intel.com \
--cc=x86@kernel.org \
/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).