On Fri, 7 Feb 2025, Kurt Borja wrote: > On Fri Feb 7, 2025 at 10:05 AM -05, Ilpo Järvinen wrote: > > On Fri, 7 Feb 2025, Kurt Borja wrote: > > > >> Split alienware-wmi WMI drivers into different files. This is done > >> seamlessly by copying and pasting, however some blocks are reordered. > >> > >> Reviewed-by: Armin Wolf <W_Armin@xxxxxx> > >> Reviewed-by: Mario Limonciello <mario.limonciello@xxxxxxx> > >> Signed-off-by: Kurt Borja <kuurtb@xxxxxxxxx> > > > > Hi, > > > > Can you please check there's no error in driver_data assignments as the > > numbers in removed & added lines do not match: > > Hi Ilpo, > > There was indeed a wrong assignment to Alienware m16 r1 AMD, I'm not > really sure how it happened but it's fixed now! > > I'll send a v10. I apologize for the noise. > > > > > $ git diff-tree -p 73224c076cf2fa2968d61584c62937f6180c8e71 | grep driver_data | rev | sort | rev | uniq -c > > Thanks for this amazing trick btw. Apparently the rev trick wouldn't have been even necessary in this case but writing those things are a second nature to me. When reviewing especially move changes, one has to have a bag full of tricks. :-) It is one of the reasons why I prefer to have move patches do as little extra work as possible because I can use pipelines to verify the pre and post content is identical. I usually starting by diffing - and + lines in the diff which is how I came across this one too. In the best case there are no code line changes at all but all changes are in the boilerplate, it gives very high confidence on the move being done correctly. When a rebase conflicts, everyone (me included), might introduce unintended changes to move-only patches so I tend to check even my own move patches in similar fashion to avoid making stupid mistakes. Even the diff of diffs wouldn't get me to 100%, it find that <5% that differs so I can focus on whether it is sound. > > ~ Kurt > > > 1 + awcc = id->driver_data; > > 1 - awcc = id->driver_data; > > 4 + .driver_data = &generic_quirks, > > 5 - .driver_data = &generic_quirks, > > 7 + .driver_data = &g_series_quirks, > > 6 - .driver_data = &g_series_quirks, > > > > (That commit id is from my staging tree, not available to you but it's > > this patch.) > -- i.
