<!-- Some styling for better description lists --><style type='text/css'>dt { font-weight: bold;float: left;display:inline;margin-right: 1em} dd { display:block; margin-left: 2em}</style>

   ***: mrpops2ko has joined #linux-media
   <br> rauji____ has joined #linux-media
   <br> frieder has joined #linux-media
   <br> GBenji has joined #linux-media
   <br> GBenji has quit IRC (Ping timeout: 480 seconds)
   <br> ao2 has joined #linux-media
   <br> frieder has quit IRC (Ping timeout: 480 seconds)
   <br> frieder has joined #linux-media
   <br> djrscally has joined #linux-media
   <br> jmassot has joined #linux-media
   <br> frieder has quit IRC (Ping timeout: 480 seconds)
   <br> frieder has joined #linux-media
   <br> ten15723774324630506 has quit IRC (Remote host closed the connection)
   <br> ten15723774324630506 has joined #linux-media
   <br> larunbe has quit IRC ()
   <br> alarumbe has joined #linux-media
   <br> lexano has quit IRC (Ping timeout: 480 seconds)
   <br> lexano has joined #linux-media
   hverkuil: <u>jmondi</u>: ping
   ***: Mo has joined #linux-media
   jmondi: <u>hverkuil</u>: pong
   <br> hi Hans
   hverkuil: <u>jmondi</u>: question regarding https://patchwork.linuxtv.org/project/linux-media/patch/20240531080707.34568-4-jacopo.mondi@ideasonboard.com/
   <br> struct pisp_be_tiles_config suddenly contains an int field, but everywhere else you use __u8/16/32. Is there a reason for that? I would expect to see __u32 there as well.
   jmondi: <u>hverkuil</u>: this ? "int num_tiles;"
   hverkuil: y
   jmondi: it's probably been there from the very beginning and it should indeed have an explicit size
   <br> are you concerned an int might have different sizes depending on the architecture ?
   hverkuil: No, but it is just weird and inconsistent. Also it is signed instead of unsigned, and you expect unsigned for num_tiles.
   jmondi: would you like a patch for that to be collected for v6.11 as well ?
   hverkuil: <u>jmondi</u>: I think another version of the series will be needed anyway, I see vb2 issues as well in the driver. Nothing major, but still. v4l2-compliance didn't complain, but I am beginning to suspect a bug in the create_bufs test.
   <br> I'll just reply to the patches, I think I need to dig into v4l2-compliance on Wednesday to see why it didn't complain.
   jmondi: ok, thanks for looking into it
   ***: eelstrebor has joined #linux-media
   <br> andrzej_p3 has quit IRC (Quit: The Lounge - https://thelounge.chat)
   <br> detlev has quit IRC (Quit: The Lounge - https://thelounge.chat)
   <br> dwlsalmeida has quit IRC (Quit: The Lounge - https://thelounge.chat)
   <br> padovan85 has quit IRC ()
   <br> SebastianCollabora has quit IRC (Quit: The Lounge - https://thelounge.chat)
   <br> ndufresne has quit IRC (Quit: The Lounge - https://thelounge.chat)
   <br> ocrete has quit IRC (Quit: The Lounge - https://thelounge.chat)
   <br> andrzej_p3 has joined #linux-media
   <br> syoung has quit IRC (Remote host closed the connection)
   <br> syoung has joined #linux-media
   <br> frieder has quit IRC (Remote host closed the connection)
   <br> syoung has quit IRC (Remote host closed the connection)
   <br> syoung has joined #linux-media
   <br> Mo has quit IRC (Quit: Leaving)
   sailus: I'd think we could just as well merge these and jmondi could post fixes on top.
   ***: gouchi has joined #linux-media
   <br> gouchi has quit IRC (Remote host closed the connection)
   <br> linkmauve has left Error from remote client
   <br> cyrinux30 has quit IRC ()
   <br> linkmauve has joined #linux-media
   <br> linkmauve has left
   <br> linkmauve has joined #linux-media
   <br> cyrinux30 has joined #linux-media
   <br> djrscally has quit IRC (Ping timeout: 480 seconds)
   <br> eelstrebor has quit IRC (Quit: Ex-Chat)
   <br> ao2 has quit IRC (Quit: Leaving)
   <br> ndufresne has joined #linux-media
   <br> BrianG61UK has joined #linux-media