On Thu, Feb 1, 2024 at 11:24 AM Théo Lebrun <theo.lebrun@xxxxxxxxxxx> wrote:
> > Can't you just use regmap MMIO to access the banks then...?
> >
> > Maybe it doesn't add much here. I'm not sure.
>
> Indeed, I went the minimalist route. You tell me if you'd prefer an MMIO
> regmap.
I'm not sure, because it might be that it adds more overhead than
it saves and then it is pointless.
> I've not seen any helper to get a regmap based on a resource, targeting
> by name. Is the expected procedure to acquire the resource then create
> a regmap config then call devm_regmap_init_mmio()?
No... haven't seen such a thing.
> > > + enum eq5p_bank bank, enum eq5p_regs reg, int bit)
> > > +{
> > > + u32 val = readl(pctrl->base + eq5p_regs[bank][reg]);
> > > +
> > > + return (val & BIT(bit)) != 0;
> > > +}
> >
> > Maybe add a check for bit > 31?
>
> Will do. I like that sort of defensive programming. What behavior would
> you expect?
> - WARN_ON(bit > 31) and return false?
> - Just return false?
> - Something else?
Your pick is as good as mine :D
I let the author choose what to do there.
Yours,
Linus Walleij
