On Wednesday 26 November 2014 20:10:28 Tony Lindgren wrote: > * Pali Rohár <pali.rohar@xxxxxxxxx> [141126 10:59]: > > On Wednesday 26 November 2014 19:19:35 Tony Lindgren wrote: > > > Maybe Pali can try to restart that discussion? To me it > > > seems the /proc/cpuinfo should be the same as it's a user > > > interface. Sorry forgot the details of the previous > > > discussion.. > > > > Yes, two days ago I again wrote emails about this problem... > > > > E.g. one of them, see: https://lkml.org/lkml/2014/11/24/774 > > > > > And with which app was that? Sorry I forgot.. > > > > More applications/libraries for N900 which running on Maemo > > 5 system. Some of them are Nokia proprietary, some of them > > are open source and some are mine. > > > > Basically problem is that non DT boot provides this info in > > /proc/cpuinfo: > > > > Hardware : Nokia RX-51 board > > Revision : 0012 > > > > New DT boot provides this: > > > > Hardware : Generic OMAP3 (Flattened Device Tree) > > Revision : 0000 > > Oh you can easily fix that by adding a n900 specific > DT_MACHINE_START entry to mach-omap2/board-generic.c. > I would like to see some solution which does not depend on distributing addition patch which will not be in mainline kernel... For this problem I proposed patch (which was rejected): https://lkml.org/lkml/2014/6/18/853 Basically Hardware is used to check if application is running on Nokia N900 or not. Also entry from Hardware is appended to Web browser user agent and some internet services using it as identifier (N900 device). > The revision entry you can populate too in pdata-quirks.c, > or maybe add something generic to populate it based on the > cmdline or a dts entry as I believe that comes from the > legacy ATAGs. I think that's just the system_rev or some > other *_rev global in the kernel. > > Regards, > > Tony Revision comes from bootloader (via ATAG) and it is HW revision of N900 device. It cannot be hardcoded into kernel or DTS as it it depends on HW. -- Pali Rohár pali.rohar@xxxxxxxxx
Attachment:
signature.asc
Description: This is a digitally signed message part.
