From: andrew@lunn.ch (Andrew Lunn)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC PATCH] clk: mvebu: armada-xp: Support for MSYS SoC
Date: Thu, 20 Nov 2014 22:36:57 +0100 [thread overview]
Message-ID: <20141120213657.GO31782@lunn.ch> (raw)
In-Reply-To: <546E5938.7060501@alliedtelesis.co.nz>
> msys is the name Marvell use for the embedded dual core CPU across
> several product lines. I believe the CPU core is the same on all of
> them. It's also the name used in the LSP Marvell provide. This was the
> main reason I went with "msys" despite the possible confusion with
> MSYS/MINGW.
>
> > We need to be careful here,
> > because there could be other packet processors with embedded Armada-XP
> > cores with different tclk speeds. Rather than using msys, it might be
> > better to use the specific packet processor product ID.
>
> The specific chip I'm working with is the 98DX4251 but there are at
> least 4 variants in that product line. I could probably go with 98DX42xx
> to cover all those variants but there is a whole other product line
> (sorry don't know the model numbers) with the same embedded core.
In the DT world, you generally base the compatibility string on the
first device which gets added. So i would go with 98DX4251. The other
products in the family just need to say they are compatible with the
98DX4251.
> On a side note would people prefer I send my entire work in progress
> get-linux-working-on-this-board series or drip feed individual patches
> as I have been doing?
Release early, release often. It makes no sense to spend a lot of time
on polishing something, if we are going to rip it to shreds and tell
you to do it another way. Post something when it works, so that we can
review it and let you know if you are going in the right direction.
There is an interesting presentation by Thomas from ELC 2014
http://free-electrons.com/pub/conferences/2014/elc/petazzoni-soc-mainlining-lessons-learned/petazzoni-soc-mainlining-lessons-learned.pdf
See Lesson #0.
I also like Lesson #13 :-)
Andrew
next prev parent reply other threads:[~2014-11-20 21:36 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-20 5:01 [RFC PATCH] clk: mvebu: armada-xp: Support for MSYS SoC Chris Packham
2014-11-20 5:17 ` Andrew Lunn
2014-11-20 21:12 ` Chris Packham
2014-11-20 21:36 ` Andrew Lunn [this message]
2014-11-20 22:31 ` Chris Packham
2014-11-20 14:56 ` Thomas Petazzoni
2014-11-20 15:03 ` Andrew Lunn
2014-11-20 21:14 ` Chris Packham
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=20141120213657.GO31782@lunn.ch \
--to=andrew@lunn.ch \
--cc=linux-arm-kernel@lists.infradead.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.