Messages in this thread | | | Date | Sat, 20 Jan 2024 18:00:32 +0100 | From | Christian Marangi <> | Subject | Re: [PATCH 2/2] ARM: decompressor: add option to ignore MEM ATAGs |
| |
On Sat, Jan 20, 2024 at 12:52:33PM +0100, Linus Walleij wrote: > On Fri, Jan 19, 2024 at 9:14 PM Christian Marangi <ansuelsmth@gmail.com> wrote: > > > +#if defined(CONFIG_ARM_ATAG_DTB_COMPAT_IGNORE_MEM) > > +#define do_ignore_mem 1 > > +#else > > +#define do_ignore_mem 0 > > +#endif > > Is there a reason why you can't just use: > > if (IS_ENABLED(CONFIG_ARM_ATAG_DTB_COMPAT_IGNORE_MEM)) > in the code? >
Was following the pattern, yes I can totally do this change... Will send a v2 with this changed.
Since the first patch has to be regression tested, is it ok to add the Tag in v2 or I should wait that to send v2?
-- Ansuel
| |