Hi, On 06/20/2017 09:42 AM, Russell King - ARM Linux wrote: > On Fri, May 19, 2017 at 12:57:08PM -0500, Dave Gerlach wrote: >> + .arm >> + .align 3 >> + >> +ENTRY(ti_emif_sram) > > Will you ever want to have any of this code as Thumb? I cannot see any requirement for that. I will say it is tested and inter-operates happily with CONFIG_THUMB_KERNEL, but because it's constrained to am335x and am437x I don't think there are unknown situations where every single instruction must be THUMB. > >> +extern inline void ti_emif_offsets(void) >> +{ > > "extern inline" is frowned upon in the kernel - any reason this > can't be "static inline" ? It should be static. > > Any reason not to provide a stub for when it's not configured, > and eliminate the ifdef in arch/arm/kernel/asm-offsets.c ? > No, I just was following the ifdef example set by others in the file, a stub would be cleaner I agree. Regards, Dave -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html