[lkml]   [2021]   [Feb]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 03/14] cxl/mem: Find device capabilities
On Wed, Feb 03, 2021 at 01:23:31PM -0800, Dan Williams wrote:
> > I'd prefer to keep the helpers for now as I do find them helpful, and so far
> > nobody else who has touched the code has complained. If you feel strongly, I
> > will change it.
> After seeing the options, I think I'd prefer to not have to worry what
> extra magic is happening with cxl_read_mbox_reg32()
> cxl_read_mbox_reg32(cxlm, CXLDEV_MB_CAPS_OFFSET);
> readl(cxlm->mbox_regs + CXLDEV_MB_CAPS_OFFSET);
> The latter is both shorter and more idiomatic.

Same here. That being said I know some driver maintainers like
wrappers, my real main irk was the macro magic to generate them.

 \ /
  Last update: 2021-02-04 08:18    [W:0.125 / U:0.580 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site