All of lore.kernel.org
 help / color / mirror / Atom feed
From: Maxime Ripard <maxime.ripard@free-electrons.com>
To: Jean-Francois Moine <moinejf@free.fr>
Cc: Rob Herring <robh+dt@kernel.org>, Chen-Yu Tsai <wens@csie.org>,
	Mike Turquette <mturquette@baylibre.com>,
	Stephen Boyd <sboyd@codeaurora.org>,
	devicetree@vger.kernel.org,
	Andre Przywara <andre.przywara@arm.com>,
	linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 00/13] arm64: Allwinner A64 support based on sunxi-ng
Date: Thu, 28 Jul 2016 22:07:05 +0200	[thread overview]
Message-ID: <20160728200705.GG6682@lukather> (raw)
In-Reply-To: <20160727104623.e9ca7b708e5c6462f44a5d01@free.fr>

[-- Attachment #1: Type: text/plain, Size: 1315 bytes --]

On Wed, Jul 27, 2016 at 10:46:23AM +0200, Jean-Francois Moine wrote:
> On Tue, 26 Jul 2016 22:30:28 +0200
> Maxime Ripard <maxime.ripard@free-electrons.com> wrote:
> 
> > ere is the previous A64 patches made by Andre [1], reworked to use
> > the new sunxi-ng clock framework.
> > 
> > This uses the current H3 clock code, as both are really similar. The
> > first patches are just meant to rework slightly the H3 code, before
> > introducing the A64-related patches.
> > 
> > Some WiP stuff have been removed, such as the MMC part, but this serie
> > already has a decent amount of devices supported: uart, i2c, rsb, etc.
> > 
> > Let me know what you think,
> 
> I don't see the interest to have common code for 32bits and 64bits.
> The clock driver of a SoC will never evolve, so, it is simpler to
> copy the source common with the H3 into a clean A64 clock driver.

I'm not sure why 32 bits vs 64 bits matters here. We're going to share
a significant number of drivers already between armv7 and armv8, like
MMC, EMAC, I2C, and so on.

And I expect to share the data in other SoCs for the A10, A13 and A20
for example, or A23/A33, which have a lot of clocks in common too.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: maxime.ripard@free-electrons.com (Maxime Ripard)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 00/13] arm64: Allwinner A64 support based on sunxi-ng
Date: Thu, 28 Jul 2016 22:07:05 +0200	[thread overview]
Message-ID: <20160728200705.GG6682@lukather> (raw)
In-Reply-To: <20160727104623.e9ca7b708e5c6462f44a5d01@free.fr>

On Wed, Jul 27, 2016 at 10:46:23AM +0200, Jean-Francois Moine wrote:
> On Tue, 26 Jul 2016 22:30:28 +0200
> Maxime Ripard <maxime.ripard@free-electrons.com> wrote:
> 
> > ere is the previous A64 patches made by Andre [1], reworked to use
> > the new sunxi-ng clock framework.
> > 
> > This uses the current H3 clock code, as both are really similar. The
> > first patches are just meant to rework slightly the H3 code, before
> > introducing the A64-related patches.
> > 
> > Some WiP stuff have been removed, such as the MMC part, but this serie
> > already has a decent amount of devices supported: uart, i2c, rsb, etc.
> > 
> > Let me know what you think,
> 
> I don't see the interest to have common code for 32bits and 64bits.
> The clock driver of a SoC will never evolve, so, it is simpler to
> copy the source common with the H3 into a clean A64 clock driver.

I'm not sure why 32 bits vs 64 bits matters here. We're going to share
a significant number of drivers already between armv7 and armv8, like
MMC, EMAC, I2C, and so on.

And I expect to share the data in other SoCs for the A10, A13 and A20
for example, or A23/A33, which have a lot of clocks in common too.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160728/0246c7fb/attachment.sig>

  reply	other threads:[~2016-07-28 20:07 UTC|newest]

Thread overview: 91+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-26 20:30 [PATCH 00/13] arm64: Allwinner A64 support based on sunxi-ng Maxime Ripard
2016-07-26 20:30 ` Maxime Ripard
2016-07-26 20:30 ` [PATCH 01/13] clk: sunxi-ng: mux: Rename mux macro to be consistent Maxime Ripard
2016-07-26 20:30   ` Maxime Ripard
2016-07-26 20:30 ` [PATCH 02/13] clk: sunxi-ng: mux: Add mux table support Maxime Ripard
2016-07-26 20:30   ` Maxime Ripard
2016-07-26 20:30 ` [PATCH 03/13] clk: sunxi-ng: sun8i: Rename DDR and video plls Maxime Ripard
2016-07-26 20:30   ` Maxime Ripard
2016-07-26 20:30   ` Maxime Ripard
2016-07-27  7:57   ` kbuild test robot
2016-07-27  7:57     ` kbuild test robot
2016-07-27  7:57     ` kbuild test robot
2016-07-26 20:30 ` [PATCH 04/13] clk: sunxi-ng: sun8i: Fix register offset Maxime Ripard
2016-07-26 20:30   ` Maxime Ripard
2016-07-26 20:30 ` [PATCH 05/13] clk: sunxi-ng: sun8i: Rename H3 only clocks Maxime Ripard
2016-07-26 20:30   ` Maxime Ripard
2016-07-26 20:30 ` [PATCH 06/13] clk: sunxi-ng: sun8i: Move fixed factors around Maxime Ripard
2016-07-26 20:30   ` Maxime Ripard
2016-07-26 20:30 ` [PATCH 07/13] clk: sunxi-ng: sun8i: Prefix clock defines by SoC Name Maxime Ripard
2016-07-26 20:30   ` Maxime Ripard
2016-07-26 20:30   ` Maxime Ripard
2016-07-26 20:30 ` [PATCH 08/13] clk: sunxi-ng: Add A64 clocks Maxime Ripard
2016-07-26 20:30   ` Maxime Ripard
2016-07-29 21:15   ` Rob Herring
2016-07-29 21:15     ` Rob Herring
2016-07-26 20:30 ` [PATCH 09/13] arm64: sunxi: Kconfig: add essential pinctrl driver Maxime Ripard
2016-07-26 20:30   ` Maxime Ripard
2016-07-26 20:30 ` [PATCH 10/13] arm64: Kconfig: sunxi: add PINCTRL Maxime Ripard
2016-07-26 20:30   ` Maxime Ripard
2016-07-26 20:30   ` Maxime Ripard
2016-07-26 20:30 ` [PATCH 11/13] Documentation: devicetree: add vendor prefix for Pine64 Maxime Ripard
2016-07-26 20:30   ` Maxime Ripard
2016-07-26 20:30 ` [PATCH 12/13] arm64: dts: add Allwinner A64 SoC .dtsi Maxime Ripard
2016-07-26 20:30   ` Maxime Ripard
2016-07-26 20:30   ` Maxime Ripard
2016-09-08  8:41   ` Andre Przywara
2016-09-08  8:41     ` Andre Przywara
2016-09-09 20:04     ` Maxime Ripard
2016-09-09 20:04       ` Maxime Ripard
2016-07-26 20:30 ` [PATCH 13/13] arm64: dts: add Pine64 support Maxime Ripard
2016-07-26 20:30   ` Maxime Ripard
2016-07-26 20:30   ` Maxime Ripard
2016-07-27  8:46 ` [PATCH 00/13] arm64: Allwinner A64 support based on sunxi-ng Jean-Francois Moine
2016-07-27  8:46   ` Jean-Francois Moine
2016-07-27  8:46   ` Jean-Francois Moine
2016-07-28 20:07   ` Maxime Ripard [this message]
2016-07-28 20:07     ` Maxime Ripard
2016-07-29  5:48     ` Jean-Francois Moine
2016-07-29  5:48       ` Jean-Francois Moine
2016-07-29  5:48       ` Jean-Francois Moine
2016-07-31  8:51       ` Maxime Ripard
2016-07-31  8:51         ` Maxime Ripard
2016-08-01  1:43 ` André Przywara
2016-08-01  1:43   ` André Przywara
2016-08-01  1:43   ` André Przywara
2016-08-01  8:30   ` Jean-Francois Moine
2016-08-01  8:30     ` Jean-Francois Moine
2016-08-01  8:30     ` Jean-Francois Moine
2016-08-01  8:30     ` Jean-Francois Moine
2016-08-01  9:13     ` Chen-Yu Tsai
2016-08-01  9:13       ` Chen-Yu Tsai
2016-08-01  9:13       ` Chen-Yu Tsai
2016-08-01  9:13       ` Chen-Yu Tsai
2016-08-01 12:00       ` Jean-Francois Moine
2016-08-01 12:00         ` Jean-Francois Moine
2016-08-01 12:00         ` Jean-Francois Moine
2016-08-01 12:00         ` Jean-Francois Moine
2016-08-01 12:01         ` Chen-Yu Tsai
2016-08-01 12:01           ` Chen-Yu Tsai
2016-08-01 12:01           ` Chen-Yu Tsai
2016-08-01 12:11           ` Jean-Francois Moine
2016-08-01 12:11             ` Jean-Francois Moine
2016-08-01 12:11             ` Jean-Francois Moine
2016-08-01 10:44     ` Andre Przywara
2016-08-01 10:44       ` Andre Przywara
2016-08-01 12:19       ` Icenowy Zheng
2016-08-01 12:19         ` Icenowy Zheng
2016-08-01 12:19         ` Icenowy Zheng
2016-08-01  9:11   ` Chen-Yu Tsai
2016-08-01  9:11     ` Chen-Yu Tsai
2016-08-01  9:11     ` Chen-Yu Tsai
2016-08-11  0:36     ` André Przywara
2016-08-11  0:36       ` André Przywara
2016-08-23 19:31   ` Maxime Ripard
2016-08-23 19:31     ` Maxime Ripard
2016-08-23 19:31     ` Maxime Ripard
2016-08-24 23:54     ` André Przywara
2016-08-24 23:54       ` André Przywara
2016-08-24 23:54       ` André Przywara
2016-08-26 22:18       ` Maxime Ripard
2016-08-26 22:18         ` Maxime Ripard

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=20160728200705.GG6682@lukather \
    --to=maxime.ripard@free-electrons.com \
    --cc=andre.przywara@arm.com \
    --cc=devicetree@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-clk@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=moinejf@free.fr \
    --cc=mturquette@baylibre.com \
    --cc=robh+dt@kernel.org \
    --cc=sboyd@codeaurora.org \
    --cc=wens@csie.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.