[kbuild] Re: [PATCH 1/1] of: unittest: overlay: ensure proper alignment of copied FDT

2021-04-09 Thread Dan Carpenter
Hi, url: https://github.com/0day-ci/linux/commits/frowand-list-gmail-com/of-unittest-overlay-ensure-proper-alignment-of-copied-FDT/20210408-045317 base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next config: i386-randconfig-m021-20210407 (attached as .config)

Re: [PATCH 1/1] of: unittest: overlay: ensure proper alignment of copied FDT

2021-04-08 Thread Frank Rowand
On 4/7/21 5:01 PM, Guenter Roeck wrote: > On 4/7/21 1:59 PM, Frank Rowand wrote: >> Hi Guenter, >> >> On 4/7/21 3:51 PM, frowand.l...@gmail.com wrote: >>> From: Frank Rowand >>> >>> The Devicetree standard specifies an 8 byte alignment of the FDT. >>> Code in libfdt expects this alignment for an

Re: [PATCH 1/1] of: unittest: overlay: ensure proper alignment of copied FDT

2021-04-08 Thread Frank Rowand
On 4/7/21 4:34 PM, Rob Herring wrote: > On Wed, Apr 7, 2021 at 3:51 PM wrote: >> >> From: Frank Rowand >> >> The Devicetree standard specifies an 8 byte alignment of the FDT. >> Code in libfdt expects this alignment for an FDT image in memory. >> kmemdup() returns 4 byte alignment on openrisc.

Re: [PATCH 1/1] of: unittest: overlay: ensure proper alignment of copied FDT

2021-04-07 Thread Guenter Roeck
On 4/7/21 1:59 PM, Frank Rowand wrote: > Hi Guenter, > > On 4/7/21 3:51 PM, frowand.l...@gmail.com wrote: >> From: Frank Rowand >> >> The Devicetree standard specifies an 8 byte alignment of the FDT. >> Code in libfdt expects this alignment for an FDT image in memory. >> kmemdup() returns 4 byte

Re: [PATCH 1/1] of: unittest: overlay: ensure proper alignment of copied FDT

2021-04-07 Thread Rob Herring
On Wed, Apr 7, 2021 at 3:51 PM wrote: > > From: Frank Rowand > > The Devicetree standard specifies an 8 byte alignment of the FDT. > Code in libfdt expects this alignment for an FDT image in memory. > kmemdup() returns 4 byte alignment on openrisc. Replace kmemdup() > with kmalloc(), align

Re: [PATCH 1/1] of: unittest: overlay: ensure proper alignment of copied FDT

2021-04-07 Thread Frank Rowand
Hi Guenter, On 4/7/21 3:51 PM, frowand.l...@gmail.com wrote: > From: Frank Rowand > > The Devicetree standard specifies an 8 byte alignment of the FDT. > Code in libfdt expects this alignment for an FDT image in memory. > kmemdup() returns 4 byte alignment on openrisc. Replace kmemdup() > with

[PATCH 1/1] of: unittest: overlay: ensure proper alignment of copied FDT

2021-04-07 Thread frowand . list
From: Frank Rowand The Devicetree standard specifies an 8 byte alignment of the FDT. Code in libfdt expects this alignment for an FDT image in memory. kmemdup() returns 4 byte alignment on openrisc. Replace kmemdup() with kmalloc(), align pointer, memcpy() to get proper alignment. The 4 byte