On Thu, Jun 16, 2022 at 5:48 PM Yann Dirson <ydirson@xxxxxxx> wrote: > > > > > > On 2022-06-15 10:34, Yann Dirson wrote: > > > Nice! > > > > > >> diff --git a/Documentation/gpu/amdgpu/apu-asic-info-table.csv > > >> b/Documentation/gpu/amdgpu/apu-asic-info-table.csv > > > ... > > >> +Ryzen 3000 series / AMD Ryzen Embedded V1*/R1* with Radeon Vega > > >> Gfx, > > >> RAVEN/PICASSO, DCN 1.0, 9.1.0, VCN 1.0, 4.1.0 > > > ... > > >> +Ryzen 3000 series / AMD Ryzen Embedded V1*/R1* with Radeon Vega > > >> Gfx, > > >> RAVEN2, DCN 1.0, 9.2.2, VCN 1.0.1, 4.1.1 > > > > > > Those similar-but-not-quite lines tend to confuse me :) > > > > I guess it will be easier to visualize this data after build the > > kernel doc since the csv file will became an html table. > > Possibly, though my surprise is rather that 2 different lines in the > table have the same label in their first cell. Do RAVEN/PICASSO and > RAVEN2 really share the same product name despite different hardware > blocks ? Sort of, the first column is marketing names. Various Ryzen 3000 variants are Raven or Picasso or Raven2. There are tons of parts in that series: Ryzen 3 3100, Ryzen 5 3500, Ryzen 7 3700X, etc. Same with the embedded parts: AMD Ryzen Embedded V1202B, AMD Ryzen Embedded V1807B, etc. I'm not sure how feasible it is to put every SKU in the table. Alex