Messages in this thread | | | From | Rob Herring <> | Date | Fri, 30 Jan 2015 15:03:36 -0600 | Subject | Re: [PATCH] ARM: /proc/cpuinfo: DT: Add support for Revision |
| |
lOn Fri, Jan 30, 2015 at 8:14 AM, Pali Rohár <pali.rohar@gmail.com> wrote: > On Monday 26 January 2015 21:22:27 Rob Herring wrote: >> On Mon, Jan 26, 2015 at 1:09 PM, Pali Rohár > <pali.rohar@gmail.com> wrote: >> > Ok, here is patch which set Revision field (global variable >> > system_rev) in /proc/cpuinfo from DT revision property:
[...]
>> > I do not know which DT property to use for storing HW >> > Revision, so if "/revision" is not correct one, let me know >> > what to use instead. Also you can add my Signed-off-by for >> > both patches. >> >> It is the correct property, but /revision in DT is a string. >> >> You need to add your own sign-off. >> >> Rob > > Any documentation which says that /revision is string?
Sorry, I was confusing this with serial-number, so I guess a number is fine here.
> If it is really string, how to store ATAG_REVISON (number) to > string? dec or hex?
However /proc/cpuinfo displays it would be fine if you stay with a string.
Rob
| |