<somlo>
daveshah: I like the idea of *not* having a pr_info -- it all depends on whether the log output generated by BUG_ON() is enough to make it clear to the driver developer that they called litex_[get|set]_reg with the wrong register size
<somlo>
I'll run some tests where I screw this up on purpose to see what the BUG_ON log output looks like
lf_ has quit [Ping timeout: 258 seconds]
lf has joined #litex
McJ92 has joined #litex
McJ92 has quit [Remote host closed the connection]
martinraison has joined #litex
martinraison has quit [Ping timeout: 272 seconds]
Dolu has quit [Ping timeout: 240 seconds]
martinraison has joined #litex
martinraison has quit [Ping timeout: 256 seconds]
Degi_ has joined #litex
Degi has quit [Ping timeout: 272 seconds]
Degi_ is now known as Degi
shenki has quit [Ping timeout: 264 seconds]
shenki has joined #litex
martinraison has joined #litex
martinraison has quit [Ping timeout: 260 seconds]
Bertl is now known as Bertl_zZ
martinraison has joined #litex
martinraison has quit [Ping timeout: 246 seconds]
martinraison has joined #litex
martinraison has quit [Remote host closed the connection]
martinraison has joined #litex
martinraison has quit [Remote host closed the connection]
Dolu has joined #litex
martinraison has joined #litex
martinraison has quit [Ping timeout: 264 seconds]
martinraison has joined #litex
martinraison has quit [Remote host closed the connection]
martinraison has joined #litex
martinraison has quit [Remote host closed the connection]
Bertl_zZ is now known as Bertl
<somlo>
shorne: on second thought, I can always update the unreleased drivers in https://github.com/litex-hub/linux/tree/litex-rebase to *not* use [_]litex_[get|set]_reg() and replace all call sites with the appropriate litex_[read|write][X]() -- and not have to deal with BUG() at all. There's really no situation in which I can see an accessor determine the width of its access at *run time*
<somlo>
* s/determine/need to determine/
ambro718 has joined #litex
somlo has quit [Ping timeout: 272 seconds]
somlo has joined #litex
peeps[zen] has joined #litex
peepsalot has quit [Ping timeout: 240 seconds]
futarisIRCcloud has quit [Quit: Connection closed for inactivity]
peeps[zen] has quit [Ping timeout: 240 seconds]
peepsalot has joined #litex
peeps[zen] has joined #litex
peepsalot has quit [Ping timeout: 240 seconds]
SpaceCoaster has quit [Quit: ZNC 1.7.2+deb3 - https://znc.in]
SpaceCoaster has joined #litex
peepsalot has joined #litex
peeps[zen] has quit [Ping timeout: 272 seconds]
ambro718 has quit [Quit: Konversation terminated!]
SpaceCoaster has quit [Quit: ZNC 1.7.2+deb3 - https://znc.in]
SpaceCoaster has joined #litex
martinraison has joined #litex
martinra_ has joined #litex
martinra_ has quit [Remote host closed the connection]
martinraison has quit [Ping timeout: 260 seconds]
martinraison has joined #litex
ambro718 has joined #litex
martinraison has quit [Remote host closed the connection]
Bertl is now known as Bertl_oO
lambda has quit [Ping timeout: 260 seconds]
lambda has joined #litex
ambro718 has quit [Quit: Konversation terminated!]