<div dir="ltr">Thanks! After reading the various comments on the thread, I do understand the argument that we're missing some abstraction by exposing individual bits of HW registers to user space. While it's certainly MUCH better than /dev/mem and particularly useful during rapid development, I still somewhat feel that it would be beneficial in the long-run to use a <span style="font-size:small;text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">proper abstraction layer with /sys as interface to </span>explicitly expose the "dozens" of configuration options.<div><br></div><div>I'm that having an explicit driver that only exposes the HICRA (to configure UART routing) as abstract symbols would be beneficial. Do you think I'd find support for getting that submitted?</div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Aug 2, 2018 at 11:00 AM Avi Fishman <<a href="mailto:avifishman70@gmail.com">avifishman70@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Aug 2, 2018 at 6:34 AM Andrew Jeffery <<a href="mailto:andrew@aj.id.au" target="_blank">andrew@aj.id.au</a>> wrote:<br>
><br>
> On Thu, 2 Aug 2018, at 12:50, Oskar Senft wrote:<br>
> > Interesting suggestion, that looks promising, thank you?<br>
> ><br>
> > What's the state of that driver? I.e. where do we expect it to land?<br>
><br>
> If you think the bmc-misc-ctrl series is useful, please reply to the<br>
> upstream thread to outline all of your use-cases. There's a lot of push-<br>
> back on using the devicetree to describe these features, and I haven't<br>
> had a lot of feedback on the acceptability of the rest (driver itself,<br>
> userspace ABI).<br>
><br>
> The more evidence we have of this being necessary/useful the better.<br>
<br>
Oskar,<br>
Here is the link to put your support:<br>
<a href="https://lore.kernel.org/patchwork/patch/961584/" rel="noreferrer" target="_blank">https://lore.kernel.org/patchwork/patch/961584/</a><br>
<br>
><br>
> Cheers,<br>
><br>
> Andrew<br>
><br>
> ><br>
> > Thanks<br>
> > Oskar.<br>
> ><br>
> > On Wed, Aug 1, 2018, 10:12 PM Lei YU <<a href="mailto:mine260309@gmail.com" target="_blank">mine260309@gmail.com</a>> wrote:<br>
> ><br>
> > > On Thu, Aug 2, 2018 at 9:55 AM Oskar Senft <<a href="mailto:osk@google.com" target="_blank">osk@google.com</a>> wrote:<br>
> > > ><br>
> > > > Hi Andrew<br>
> > > ><br>
> > > > Thanks for the suggestion. I see that the register would map quite<br>
> > > nicely to pinmux. However, the ability to change the configuration from<br>
> > > user space is critical. How could that be done via pinmux?<br>
> > ><br>
> > > I believe Andrew Jeffery's [bmc-misc-ctr][1] driver is perfect for this<br>
> > > requirement:<br>
> > > 1. You can define related registers in dts and expose as sysfs attributes<br>
> > > 2. User space is able to read/write the sysfs attributes to do what you<br>
> > > need<br>
> > >    to configure.<br>
> > ><br>
> > > [1]: <a href="https://patchwork.ozlabs.org/cover/942322/" rel="noreferrer" target="_blank">https://patchwork.ozlabs.org/cover/942322/</a><br>
> > ><br>
<br>
<br>
<br>
-- <br>
Regards,<br>
Avi<br>
</blockquote></div>