All of lore.kernel.org
 help / color / mirror / Atom feed
From: ijc@hellion.org.uk (Ian Campbell)
To: linux-arm-kernel@lists.infradead.org
Subject: Bug#782364: linux-image-3.16.0-4-armmp: please configure drivers for both Cubox i4pro real time clocks
Date: Sat, 02 May 2015 17:21:35 +0100	[thread overview]
Message-ID: <1430583695.15640.190.camel@hellion.org.uk> (raw)
In-Reply-To: <20150502161226.GS12732@n2100.arm.linux.org.uk>

On Sat, 2015-05-02 at 17:12 +0100, Russell King - ARM Linux wrote:
> On Sat, May 02, 2015 at 05:01:04PM +0100, Ian Campbell wrote:
> > So is your advice for a multi platform kernel supporting all Cubox
> > devices to just enable both and to sort out any syncing/naming etc in
> > userspace?
> 
> I don't see a problem locally, because I have both built in to my kernel:
> 
> hub 2-0:1.0: USB hub found
> hub 2-0:1.0: 1 port detected
> mousedev: PS/2 mouse device common for all mice
> rtc-pcf8523 0-0068: rtc core: registered rtc-pcf8523 as rtc0
> snvs_rtc 20cc034.snvs-rtc-lp: rtc core: registered 20cc034.snvs-rtc-lp as rtc1
> i2c /dev entries driver
> 
> and so the battery-backed one gets the primary RTC device.

We typically build RTCs statically (for this sort of reason) so it seems
like the right thing for us to do here is to build both in.

Does the battery backed one automatically get synced to the time based
wake up one too, or is that in userspace?

What I wasn't sure about was if both were =y whether the init order was
guaranteed across kernel versions etc. It sounds like you are saying it
will Just Work if we build both in, which is what I was hoping for.

> I guess the problem comes when you have them as modules, and then systemd
> or udev, or whatever init daemon flavour you're using can load the modules
> in a random order (or even in parallel) so you've no idea which RTC is
> which.

Right. 

> Yes, I believe this is a userspace problem, because:

[...]

Yes, ack.

Thanks,
Ian.

  reply	other threads:[~2015-05-02 16:21 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20150410232854.23367.58862.reportbug@cube.rcthomas.org>
2015-04-11  0:08 ` Bug#782364: linux-image-3.16.0-4-armmp: please configure drivers for both Cubox i4pro real time clocks Ben Hutchings
2015-04-29  3:02   ` Fabio Estevam
2015-04-29  5:46     ` Rick Thomas
2015-04-29  5:48     ` Rick Thomas
2015-04-29  8:57   ` Russell King - ARM Linux
2015-05-02 16:01     ` Ian Campbell
2015-05-02 16:12       ` Russell King - ARM Linux
2015-05-02 16:21         ` Ian Campbell [this message]
2015-05-06  8:01           ` Ian Campbell
2015-05-06 10:00             ` Rick Thomas
2015-05-06 10:27               ` Ian Campbell
2015-05-06 10:35                 ` Rick Thomas
2015-05-12  8:33                   ` Rick Thomas
2015-05-12  9:11                     ` Ian Campbell
2015-05-13  4:54                       ` Rick Thomas

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=1430583695.15640.190.camel@hellion.org.uk \
    --to=ijc@hellion.org.uk \
    --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.