All of lore.kernel.org
 help / color / mirror / Atom feed
From: contact@paulk.fr (Paul Kocialkowski)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 2/2] arch: arm: Show the serial number from devicetree in cpuinfo
Date: Wed, 06 May 2015 11:37:26 +0200	[thread overview]
Message-ID: <1430905046.2577.1.camel@collins> (raw)
In-Reply-To: <20150506090105.GC2067@n2100.arm.linux.org.uk>

Le mercredi 06 mai 2015 ? 10:01 +0100, Russell King - ARM Linux a
?crit :
> On Sat, May 02, 2015 at 05:46:22PM +0200, Paul Kocialkowski wrote:
> > Le mardi 28 avril 2015 ? 17:09 +0100, Russell King - ARM Linux a ?crit :
> > > On Mon, Apr 27, 2015 at 04:42:27PM +0200, Paul Kocialkowski wrote:
> > > > Well, of_property_read_string is only defined when CONFIG_OF is set
> > > > (base.c is always built in drivers/of but the directory is only included
> > > > when CONFIG_OF is set).
> > > > 
> > > > Of course, on ARM, we now expect that it is the case, but it seems like
> > > > good practice to check for it, since it could theoretically be disabled.
> > > 
> > > That is _not_ "is the case" - there are a number of non-OF platforms...
> > > including some which will probably never be converted.
> > 
> > Right, so I think v4 should be fine too, given the empty functions in
> > case CONFIG_OF is not set.
> 
> Yes, can you please put them in my patch system so we can get this
> applied?

What exactly do you mean? 

I sent v4 to linux-arm-kernel at lists.infradead.org and
devicetree at vger.kernel.org with you and Rob in the CC list, on the 28th
of April.

> Rob - does your ack apply to the v4 patch?
> 
> Thanks.
> 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150506/d301340a/attachment-0001.sig>

WARNING: multiple messages have this Message-ID (diff)
From: Paul Kocialkowski <contact-W9ppeneeCTY@public.gmane.org>
To: Russell King - ARM Linux <linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org>
Cc: Rob Herring <robherring2-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	"devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
	<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
	Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	Hans De Goede <hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
	Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org>,
	Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
	Ian Campbell
	<ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>,
	Stefan Agner <stefan-XLVq0VzYD2Y@public.gmane.org>,
	Kumar Gala <galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
Subject: Re: [PATCH v3 2/2] arch: arm: Show the serial number from devicetree in cpuinfo
Date: Wed, 06 May 2015 11:37:26 +0200	[thread overview]
Message-ID: <1430905046.2577.1.camel@collins> (raw)
In-Reply-To: <20150506090105.GC2067-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org>

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

Le mercredi 06 mai 2015 à 10:01 +0100, Russell King - ARM Linux a
écrit :
> On Sat, May 02, 2015 at 05:46:22PM +0200, Paul Kocialkowski wrote:
> > Le mardi 28 avril 2015 à 17:09 +0100, Russell King - ARM Linux a écrit :
> > > On Mon, Apr 27, 2015 at 04:42:27PM +0200, Paul Kocialkowski wrote:
> > > > Well, of_property_read_string is only defined when CONFIG_OF is set
> > > > (base.c is always built in drivers/of but the directory is only included
> > > > when CONFIG_OF is set).
> > > > 
> > > > Of course, on ARM, we now expect that it is the case, but it seems like
> > > > good practice to check for it, since it could theoretically be disabled.
> > > 
> > > That is _not_ "is the case" - there are a number of non-OF platforms...
> > > including some which will probably never be converted.
> > 
> > Right, so I think v4 should be fine too, given the empty functions in
> > case CONFIG_OF is not set.
> 
> Yes, can you please put them in my patch system so we can get this
> applied?

What exactly do you mean? 

I sent v4 to linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org and
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org with you and Rob in the CC list, on the 28th
of April.

> Rob - does your ack apply to the v4 patch?
> 
> Thanks.
> 


[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

  reply	other threads:[~2015-05-06  9:37 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-18  9:58 [PATCH v3 1/2] Documentation: devicetree: root node serial-number property documentation Paul Kocialkowski
2015-04-18  9:58 ` Paul Kocialkowski
2015-04-18  9:58 ` [PATCH v3 2/2] arch: arm: Show the serial number from devicetree in cpuinfo Paul Kocialkowski
2015-04-18  9:58   ` Paul Kocialkowski
2015-04-27  8:27   ` Paul Kocialkowski
2015-04-27  8:27     ` Paul Kocialkowski
2015-04-27 13:48   ` Rob Herring
2015-04-27 13:48     ` Rob Herring
2015-04-27 14:42     ` Paul Kocialkowski
2015-04-27 14:42       ` Paul Kocialkowski
2015-04-27 15:20       ` Rob Herring
2015-04-27 15:20         ` Rob Herring
2015-04-27 18:45         ` Paul Kocialkowski
2015-04-27 18:45           ` Paul Kocialkowski
2015-04-28 16:09       ` Russell King - ARM Linux
2015-04-28 16:09         ` Russell King - ARM Linux
2015-05-02 15:46         ` Paul Kocialkowski
2015-05-02 15:46           ` Paul Kocialkowski
2015-05-06  9:01           ` Russell King - ARM Linux
2015-05-06  9:01             ` Russell King - ARM Linux
2015-05-06  9:37             ` Paul Kocialkowski [this message]
2015-05-06  9:37               ` Paul Kocialkowski

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=1430905046.2577.1.camel@collins \
    --to=contact@paulk.fr \
    --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.