Re: [PATCH] powerpc/powernv: add hdat attribute to sysfs

2017-02-22 Thread Andrew Donnellan
On 23/02/17 18:42, Andrew Donnellan wrote: I haven't checked the actual important bit of how HDAT is exposed in device tree but other than that things look fairly good. Boring stylistic comments below! And now that I have - it's probably worth noting that this patch depends on a corresponding

Re: [PATCH] powerpc/powernv: add hdat attribute to sysfs

2017-02-22 Thread Andrew Donnellan
On 23/02/17 13:29, Matt Brown wrote: From: Matt Brown Did you intend for this to be different from the email you're using to send + sign-off? The HDAT data area is consumed by skiboot and turned into a device-tree. In some cases we would like to look directly at

Re: [PATCH] powerpc/powernv: add hdat attribute to sysfs

2017-02-22 Thread Oliver O'Halloran
On Thu, Feb 23, 2017 at 1:29 PM, Matt Brown wrote: > From: Matt Brown > > The HDAT data area is consumed by skiboot and turned into a device-tree. > In some cases we would like to look directly at the HDAT, so this patch > adds a sysfs node

[PATCH] powerpc/powernv: add hdat attribute to sysfs

2017-02-22 Thread Matt Brown
From: Matt Brown The HDAT data area is consumed by skiboot and turned into a device-tree. In some cases we would like to look directly at the HDAT, so this patch adds a sysfs node to allow it to be viewed. This is not possible through /dev/mem as it is reserved memory