public inbox for linux-m68k@lists.linux-m68k.org
 help / color / mirror / Atom feed
From: Greg Ungerer <gregungerer@westnet.com.au>
To: Daniel Palmer <danieruru@gmail.com>,
	Geert Uytterhoeven <geert@linux-m68k.org>
Cc: linux-m68k <linux-m68k@lists.linux-m68k.org>
Subject: Re: [PATCH 1/2] Add correct PLL settings for DragonBall VZ.
Date: Fri, 04 Apr 2014 23:42:52 +1000	[thread overview]
Message-ID: <533EB6DC.5030003@westnet.com.au> (raw)
In-Reply-To: <CAO64nm6L=p-hcH9NSFxhfQF9RNaid4fPLno9zKsAy4Jn2QnHjg@mail.gmail.com>

Hi Daniel,

On 04/04/14 13:18, Daniel Palmer wrote:
> Only slightly related to the patch itself;
>
> At the moment you can select to support multiple DragonBall chips in a
> single kernel but I don't think that'll actually work if the PLL
> register values are hardcoded like they are at the moment. So I think
> the kconfig needs to be changed to only allow support for one chip.
> Should I make this change and send a patch?

Sure, that sounds reasonable.


> I doubt there are many people that are attempting to run Linux on
> Palms or extremely rare development boards but it does seem like it's
> possible to detect what chip the kernel is booting on from the chip id
> register. So the kernel could pick the right PLL settings based on the
> chip it's booting on and then the same kernel image could be used for
> all DragonBall based machines.

The mutually exclusive board selection can be changed back if or when
code gets added to do this. No problem there.

Regards
Greg

  parent reply	other threads:[~2014-04-04 13:42 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-02 14:43 [PATCH 1/2] Add correct PLL settings for DragonBall VZ Daniel Palmer
2014-04-02 14:43 ` [PATCH 2/2] Fix mach_sched_init for EZ and VZ DragonBall chips Daniel Palmer
2014-04-03 12:45   ` Greg Ungerer
2014-04-03 12:41 ` [PATCH 1/2] Add correct PLL settings for DragonBall VZ Greg Ungerer
2014-04-03 12:48   ` Geert Uytterhoeven
2014-04-04  3:18     ` Daniel Palmer
2014-04-04  8:16       ` Finn Thain
2014-04-04 13:42       ` Greg Ungerer [this message]
2014-04-04  7:45     ` Finn Thain
2014-04-04 13:35       ` Greg Ungerer
2014-04-05  8:46         ` Finn Thain

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=533EB6DC.5030003@westnet.com.au \
    --to=gregungerer@westnet.com.au \
    --cc=danieruru@gmail.com \
    --cc=geert@linux-m68k.org \
    --cc=linux-m68k@lists.linux-m68k.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