All of lore.kernel.org
 help / color / mirror / Atom feed
diff for duplicates of <1e15c452-f8be-85fa-e424-b7463cc703eb@intel.com>

diff --git a/a/1.txt b/N1/1.txt
index b1cffdd..5145b0b 100644
--- a/a/1.txt
+++ b/N1/1.txt
@@ -5,8 +5,8 @@ Thank Yee-Li for discovering this problem. I would suggest we (intel)
 will process a patch to address this problem. Can I put you kindly as 
 "Suggested-by"?
 > 
-> Neftin, Sasha <sasha.neftin at intel.com <mailto:sasha.neftin@intel.com>> 
-> ?2021?6?30??? ??2:13???
+> Neftin, Sasha <sasha.neftin@intel.com <mailto:sasha.neftin@intel.com>> 
+> 于2021年6月30日周三 下午2:13写道:
 > 
 >     On 6/29/2021 20:33, Yee Li wrote:
 >      >
@@ -26,7 +26,7 @@ will process a patch to address this problem. Can I put you kindly as
 > 
 >     Then:
 >     if (lat_enc_d > max_ltr_enc_d)
->      ? ? ? ? lat_enc = max_ltr_enc;
+>              lat_enc = max_ltr_enc;
 >     what do you think?
 > 
 >      >
@@ -36,106 +36,106 @@ will process a patch to address this problem. Can I put you kindly as
 >      > Neftin, Sasha <sasha.neftin@intel.com
 >     <mailto:sasha.neftin@intel.com> <mailto:sasha.neftin@intel.com
 >     <mailto:sasha.neftin@intel.com>>>
->      > ? 2021?6?29??? 22:49???
+>      > 于 2021年6月29日周二 22:49写道:
 >      >
->      >? ? ?On 6/29/2021 11:21, YeeLi wrote:
->      >? ? ?Yeeli,
->      >? ? ? > In e1000e driver, a PCIe-like device, the max
+>      >     On 6/29/2021 11:21, YeeLi wrote:
+>      >     Yeeli,
+>      >      > In e1000e driver, a PCIe-like device, the max
 >     snoop/no-snoop latency
->      >? ? ? > is the upper limit.So, directly compare the size of
+>      >      > is the upper limit.So, directly compare the size of
 >     lat_enc and
->      >? ? ? > max_ltr_enc is incorrect.
->      >? ? ? >
->      >? ? ?why?
->      >? ? ? >? ? ? In 1000Mbps, 0x8b9 < 0x1003, 189440 ns < 3145728 ns,
+>      >      > max_ltr_enc is incorrect.
+>      >      >
+>      >     why?
+>      >      >      In 1000Mbps, 0x8b9 < 0x1003, 189440 ns < 3145728 ns,
 >     correct.
->      >? ? ? >
->      >? ? ? >? ? ? In 100Mbps, 0xc3a < 0x1003, 1900544 ns < 3145728 ns,
+>      >      >
+>      >      >      In 100Mbps, 0xc3a < 0x1003, 1900544 ns < 3145728 ns,
 >     correct.
->      >? ? ? >
->      >? ? ? >? ? ? In 10Mbps, 0xe40 < 0x1003, 18874368 > 3145728, incorrect.
->      >? ? ? >
->      >? ? ?Platform LTR encoded is 0x1003 - right. It is meant 1048576ns
+>      >      >
+>      >      >      In 10Mbps, 0xe40 < 0x1003, 18874368 > 3145728, incorrect.
+>      >      >
+>      >     Platform LTR encoded is 0x1003 - right. It is meant 1048576ns
 >     x 3 =
->      >? ? ?3145738ns.
->      >? ? ?Now,
->      >? ? ?for 1000M: 0x08b9 => 185ns x 1024 = 189440ns (you are correct)
->      >? ? ?for 100M: 0x0c3a => 58ns x 32768 = 1900544ns (correct)
->      >? ? ?for 10M: 0x0e41 => 577ns x 32768 = 18907136ns (ok?)
->      >? ? ?18907136ns > 3145738ns, (latency encoded is great than
+>      >     3145738ns.
+>      >     Now,
+>      >     for 1000M: 0x08b9 => 185ns x 1024 = 189440ns (you are correct)
+>      >     for 100M: 0x0c3a => 58ns x 32768 = 1900544ns (correct)
+>      >     for 10M: 0x0e41 => 577ns x 32768 = 18907136ns (ok?)
+>      >     18907136ns > 3145738ns, (latency encoded is great than
 >     maximum LTR
->      >? ? ?encoded by platform) - so, there is no point to wait more
+>      >     encoded by platform) - so, there is no point to wait more
 >     than platform
->      >? ? ?required, and lat_enc=max_ltr_enc. It is expected and we sent
+>      >     required, and lat_enc=max_ltr_enc. It is expected and we sent
 >     right
->      >? ? ?value to the power management controller.
->      >? ? ?What is the problem you try solve?
+>      >     value to the power management controller.
+>      >     What is the problem you try solve?
 >      >
->      >? ? ? > Decoded the lat_enc and max_ltr_enc before compare them is
+>      >      > Decoded the lat_enc and max_ltr_enc before compare them is
 >     necessary.
->      >? ? ? >
->      >? ? ? > Signed-off-by: YeeLi <seven.yi.lee@gmail.com
+>      >      >
+>      >      > Signed-off-by: YeeLi <seven.yi.lee@gmail.com
 >     <mailto:seven.yi.lee@gmail.com>
->      >? ? ?<mailto:seven.yi.lee at gmail.com <mailto:seven.yi.lee@gmail.com>>>
->      >? ? ? > ---
->      >? ? ? >? ?drivers/net/ethernet/intel/e1000e/ich8lan.c | 23
->      >? ? ?++++++++++++++++++++-
->      >? ? ? >? ?1 file changed, 22 insertions(+), 1 deletion(-)
->      >? ? ? >
->      >? ? ? > diff --git a/drivers/net/ethernet/intel/e1000e/ich8lan.c
->      >? ? ?b/drivers/net/ethernet/intel/e1000e/ich8lan.c
->      >? ? ? > index 590ad110d383..3bff1b570b76 100644
->      >? ? ? > --- a/drivers/net/ethernet/intel/e1000e/ich8lan.c
->      >? ? ? > +++ b/drivers/net/ethernet/intel/e1000e/ich8lan.c
->      >? ? ? > @@ -986,6 +986,27 @@ static s32
+>      >     <mailto:seven.yi.lee@gmail.com <mailto:seven.yi.lee@gmail.com>>>
+>      >      > ---
+>      >      >   drivers/net/ethernet/intel/e1000e/ich8lan.c | 23
+>      >     ++++++++++++++++++++-
+>      >      >   1 file changed, 22 insertions(+), 1 deletion(-)
+>      >      >
+>      >      > diff --git a/drivers/net/ethernet/intel/e1000e/ich8lan.c
+>      >     b/drivers/net/ethernet/intel/e1000e/ich8lan.c
+>      >      > index 590ad110d383..3bff1b570b76 100644
+>      >      > --- a/drivers/net/ethernet/intel/e1000e/ich8lan.c
+>      >      > +++ b/drivers/net/ethernet/intel/e1000e/ich8lan.c
+>      >      > @@ -986,6 +986,27 @@ static s32
 >     e1000_k1_workaround_lpt_lp(struct
->      >? ? ?e1000_hw *hw, bool link)
->      >? ? ? >? ? ? ?return ret_val;
->      >? ? ? >? ?}
->      >? ? ? >
->      >? ? ? > +static u32 convert_e1000e_ltr_scale(u32 val)
->      >? ? ? > +{
->      >? ? ? > +? ? ?if (val > 5)
->      >? ? ? > +? ? ? ? ? ? ?return 0;
->      >? ? ? > +
->      >? ? ? > +? ? ?return 1U << (5 * val);
->      >? ? ? > +}
->      >? ? ? > +
->      >? ? ? > +static u64 decoded_ltr(u32 val)
->      >? ? ? > +{
->      >? ? ? > +? ? ?u64 decoded_latency;
->      >? ? ? > +? ? ?u32 value;
->      >? ? ? > +? ? ?u32 scale;
->      >? ? ? > +
->      >? ? ? > +? ? ?value = val & 0x03FF;
->      >? ? ? > +? ? ?scale = (val & 0x1C00) >> 10;
->      >? ? ? > +? ? ?decoded_latency = value *
+>      >     e1000_hw *hw, bool link)
+>      >      >       return ret_val;
+>      >      >   }
+>      >      >
+>      >      > +static u32 convert_e1000e_ltr_scale(u32 val)
+>      >      > +{
+>      >      > +     if (val > 5)
+>      >      > +             return 0;
+>      >      > +
+>      >      > +     return 1U << (5 * val);
+>      >      > +}
+>      >      > +
+>      >      > +static u64 decoded_ltr(u32 val)
+>      >      > +{
+>      >      > +     u64 decoded_latency;
+>      >      > +     u32 value;
+>      >      > +     u32 scale;
+>      >      > +
+>      >      > +     value = val & 0x03FF;
+>      >      > +     scale = (val & 0x1C00) >> 10;
+>      >      > +     decoded_latency = value *
 >     convert_e1000e_ltr_scale(scale);
->      >? ? ? > +
->      >? ? ? > +? ? ?return decoded_latency;
->      >? ? ? > +}
->      >? ? ? > +
->      >? ? ? >? ?/**
->      >? ? ? >? ? *? e1000_platform_pm_pch_lpt - Set platform power
+>      >      > +
+>      >      > +     return decoded_latency;
+>      >      > +}
+>      >      > +
+>      >      >   /**
+>      >      >    *  e1000_platform_pm_pch_lpt - Set platform power
 >     management
->      >? ? ?values
->      >? ? ? >? ? *? @hw: pointer to the HW structure
->      >? ? ? > @@ -1059,7 +1080,7 @@ static s32
+>      >     values
+>      >      >    *  @hw: pointer to the HW structure
+>      >      > @@ -1059,7 +1080,7 @@ static s32
 >     e1000_platform_pm_pch_lpt(struct
->      >? ? ?e1000_hw *hw, bool link)
->      >? ? ? >? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? E1000_PCI_LTR_CAP_LPT + 2,
->      >? ? ?&max_nosnoop);
->      >? ? ? >? ? ? ? ? ? ? ?max_ltr_enc = max_t(u16, max_snoop,
+>      >     e1000_hw *hw, bool link)
+>      >      >                                    E1000_PCI_LTR_CAP_LPT + 2,
+>      >     &max_nosnoop);
+>      >      >               max_ltr_enc = max_t(u16, max_snoop,
 >     max_nosnoop);
->      >? ? ? >
->      >? ? ? > -? ? ? ? ? ? ?if (lat_enc > max_ltr_enc)
->      >? ? ? > +? ? ? ? ? ? ?if (decoded_ltr(lat_enc) >
+>      >      >
+>      >      > -             if (lat_enc > max_ltr_enc)
+>      >      > +             if (decoded_ltr(lat_enc) >
 >     decoded_ltr(max_ltr_enc))
->      >? ? ? >? ? ? ? ? ? ? ? ? ? ? ?lat_enc = max_ltr_enc;
->      >? ? ? >? ? ? ?}
->      >? ? ? >
->      >? ? ? >
->      >? ? ?sasha
+>      >      >                       lat_enc = max_ltr_enc;
+>      >      >       }
+>      >      >
+>      >      >
+>      >     sasha
 >      >
 > 
 sasha
diff --git a/a/content_digest b/N1/content_digest
index 2a0dfd3..68f9ef5 100644
--- a/a/content_digest
+++ b/N1/content_digest
@@ -4,9 +4,23 @@
  "ref\00234b97a-f207-47b0-1545-582ee5282824@intel.com\0"
  "ref\0CALX8JfTOG7bXn7gP+4n1_3CfFFk6aL-RJDY03RCOeBSpUeTEPQ@mail.gmail.com\0"
  "From\0Neftin, Sasha <sasha.neftin@intel.com>\0"
- "Subject\0[Intel-wired-lan] [PATCH] driver core: fix e1000e ltr bug\0"
+ "Subject\0Re: [Intel-wired-lan] [PATCH] driver core: fix e1000e ltr bug\0"
  "Date\0Thu, 1 Jul 2021 11:34:49 +0300\0"
- "To\0intel-wired-lan@osuosl.org\0"
+ "To\0Yee Li <seven.yi.lee@gmail.com>\0"
+ "Cc\0jesse.brandeburg@intel.com"
+  anthony.l.nguyen@intel.com
+  davem@davemloft.net
+  kuba@kernel.org
+  intel-wired-lan@lists.osuosl.org
+  linux-kernel@vger.kernel.org
+  Ruinskiy
+  Dima <dima.ruinskiy@intel.com>
+  Edri
+  Michael <michael.edri@intel.com>
+  Efrati
+  Nir <nir.efrati@intel.com>
+  Neftin
+ " Sasha <sasha.neftin@intel.com>\0"
  "\00:1\0"
  "b\0"
  "On 6/30/2021 09:26, Yee Li wrote:\n"
@@ -16,8 +30,8 @@
  "will process a patch to address this problem. Can I put you kindly as \n"
  "\"Suggested-by\"?\n"
  "> \n"
- "> Neftin, Sasha <sasha.neftin at intel.com <mailto:sasha.neftin@intel.com>> \n"
- "> ?2021?6?30??? ??2:13???\n"
+ "> Neftin, Sasha <sasha.neftin@intel.com <mailto:sasha.neftin@intel.com>> \n"
+ "> \344\272\2162021\345\271\2646\346\234\21030\346\227\245\345\221\250\344\270\211 \344\270\213\345\215\2102:13\345\206\231\351\201\223\357\274\232\n"
  "> \n"
  ">     On 6/29/2021 20:33, Yee Li wrote:\n"
  ">      >\n"
@@ -37,7 +51,7 @@
  "> \n"
  ">     Then:\n"
  ">     if (lat_enc_d > max_ltr_enc_d)\n"
- ">      ? ? ? ? lat_enc = max_ltr_enc;\n"
+ ">      \302\240 \302\240 \302\240 \302\240 lat_enc = max_ltr_enc;\n"
  ">     what do you think?\n"
  "> \n"
  ">      >\n"
@@ -47,108 +61,108 @@
  ">      > Neftin, Sasha <sasha.neftin@intel.com\n"
  ">     <mailto:sasha.neftin@intel.com> <mailto:sasha.neftin@intel.com\n"
  ">     <mailto:sasha.neftin@intel.com>>>\n"
- ">      > ? 2021?6?29??? 22:49???\n"
+ ">      > \344\272\216 2021\345\271\2646\346\234\21029\346\227\245\345\221\250\344\272\214 22:49\345\206\231\351\201\223\357\274\232\n"
  ">      >\n"
- ">      >? ? ?On 6/29/2021 11:21, YeeLi wrote:\n"
- ">      >? ? ?Yeeli,\n"
- ">      >? ? ? > In e1000e driver, a PCIe-like device, the max\n"
+ ">      >\302\240 \302\240 \302\240On 6/29/2021 11:21, YeeLi wrote:\n"
+ ">      >\302\240 \302\240 \302\240Yeeli,\n"
+ ">      >\302\240 \302\240 \302\240 > In e1000e driver, a PCIe-like device, the max\n"
  ">     snoop/no-snoop latency\n"
- ">      >? ? ? > is the upper limit.So, directly compare the size of\n"
+ ">      >\302\240 \302\240 \302\240 > is the upper limit.So, directly compare the size of\n"
  ">     lat_enc and\n"
- ">      >? ? ? > max_ltr_enc is incorrect.\n"
- ">      >? ? ? >\n"
- ">      >? ? ?why?\n"
- ">      >? ? ? >? ? ? In 1000Mbps, 0x8b9 < 0x1003, 189440 ns < 3145728 ns,\n"
+ ">      >\302\240 \302\240 \302\240 > max_ltr_enc is incorrect.\n"
+ ">      >\302\240 \302\240 \302\240 >\n"
+ ">      >\302\240 \302\240 \302\240why?\n"
+ ">      >\302\240 \302\240 \302\240 >\302\240 \302\240 \302\240 In 1000Mbps, 0x8b9 < 0x1003, 189440 ns < 3145728 ns,\n"
  ">     correct.\n"
- ">      >? ? ? >\n"
- ">      >? ? ? >? ? ? In 100Mbps, 0xc3a < 0x1003, 1900544 ns < 3145728 ns,\n"
+ ">      >\302\240 \302\240 \302\240 >\n"
+ ">      >\302\240 \302\240 \302\240 >\302\240 \302\240 \302\240 In 100Mbps, 0xc3a < 0x1003, 1900544 ns < 3145728 ns,\n"
  ">     correct.\n"
- ">      >? ? ? >\n"
- ">      >? ? ? >? ? ? In 10Mbps, 0xe40 < 0x1003, 18874368 > 3145728, incorrect.\n"
- ">      >? ? ? >\n"
- ">      >? ? ?Platform LTR encoded is 0x1003 - right. It is meant 1048576ns\n"
+ ">      >\302\240 \302\240 \302\240 >\n"
+ ">      >\302\240 \302\240 \302\240 >\302\240 \302\240 \302\240 In 10Mbps, 0xe40 < 0x1003, 18874368 > 3145728, incorrect.\n"
+ ">      >\302\240 \302\240 \302\240 >\n"
+ ">      >\302\240 \302\240 \302\240Platform LTR encoded is 0x1003 - right. It is meant 1048576ns\n"
  ">     x 3 =\n"
- ">      >? ? ?3145738ns.\n"
- ">      >? ? ?Now,\n"
- ">      >? ? ?for 1000M: 0x08b9 => 185ns x 1024 = 189440ns (you are correct)\n"
- ">      >? ? ?for 100M: 0x0c3a => 58ns x 32768 = 1900544ns (correct)\n"
- ">      >? ? ?for 10M: 0x0e41 => 577ns x 32768 = 18907136ns (ok?)\n"
- ">      >? ? ?18907136ns > 3145738ns, (latency encoded is great than\n"
+ ">      >\302\240 \302\240 \302\2403145738ns.\n"
+ ">      >\302\240 \302\240 \302\240Now,\n"
+ ">      >\302\240 \302\240 \302\240for 1000M: 0x08b9 => 185ns x 1024 = 189440ns (you are correct)\n"
+ ">      >\302\240 \302\240 \302\240for 100M: 0x0c3a => 58ns x 32768 = 1900544ns (correct)\n"
+ ">      >\302\240 \302\240 \302\240for 10M: 0x0e41 => 577ns x 32768 = 18907136ns (ok?)\n"
+ ">      >\302\240 \302\240 \302\24018907136ns > 3145738ns, (latency encoded is great than\n"
  ">     maximum LTR\n"
- ">      >? ? ?encoded by platform) - so, there is no point to wait more\n"
+ ">      >\302\240 \302\240 \302\240encoded by platform) - so, there is no point to wait more\n"
  ">     than platform\n"
- ">      >? ? ?required, and lat_enc=max_ltr_enc. It is expected and we sent\n"
+ ">      >\302\240 \302\240 \302\240required, and lat_enc=max_ltr_enc. It is expected and we sent\n"
  ">     right\n"
- ">      >? ? ?value to the power management controller.\n"
- ">      >? ? ?What is the problem you try solve?\n"
+ ">      >\302\240 \302\240 \302\240value to the power management controller.\n"
+ ">      >\302\240 \302\240 \302\240What is the problem you try solve?\n"
  ">      >\n"
- ">      >? ? ? > Decoded the lat_enc and max_ltr_enc before compare them is\n"
+ ">      >\302\240 \302\240 \302\240 > Decoded the lat_enc and max_ltr_enc before compare them is\n"
  ">     necessary.\n"
- ">      >? ? ? >\n"
- ">      >? ? ? > Signed-off-by: YeeLi <seven.yi.lee@gmail.com\n"
+ ">      >\302\240 \302\240 \302\240 >\n"
+ ">      >\302\240 \302\240 \302\240 > Signed-off-by: YeeLi <seven.yi.lee@gmail.com\n"
  ">     <mailto:seven.yi.lee@gmail.com>\n"
- ">      >? ? ?<mailto:seven.yi.lee at gmail.com <mailto:seven.yi.lee@gmail.com>>>\n"
- ">      >? ? ? > ---\n"
- ">      >? ? ? >? ?drivers/net/ethernet/intel/e1000e/ich8lan.c | 23\n"
- ">      >? ? ?++++++++++++++++++++-\n"
- ">      >? ? ? >? ?1 file changed, 22 insertions(+), 1 deletion(-)\n"
- ">      >? ? ? >\n"
- ">      >? ? ? > diff --git a/drivers/net/ethernet/intel/e1000e/ich8lan.c\n"
- ">      >? ? ?b/drivers/net/ethernet/intel/e1000e/ich8lan.c\n"
- ">      >? ? ? > index 590ad110d383..3bff1b570b76 100644\n"
- ">      >? ? ? > --- a/drivers/net/ethernet/intel/e1000e/ich8lan.c\n"
- ">      >? ? ? > +++ b/drivers/net/ethernet/intel/e1000e/ich8lan.c\n"
- ">      >? ? ? > @@ -986,6 +986,27 @@ static s32\n"
+ ">      >\302\240 \302\240 \302\240<mailto:seven.yi.lee@gmail.com <mailto:seven.yi.lee@gmail.com>>>\n"
+ ">      >\302\240 \302\240 \302\240 > ---\n"
+ ">      >\302\240 \302\240 \302\240 >\302\240 \302\240drivers/net/ethernet/intel/e1000e/ich8lan.c | 23\n"
+ ">      >\302\240 \302\240 \302\240++++++++++++++++++++-\n"
+ ">      >\302\240 \302\240 \302\240 >\302\240 \302\2401 file changed, 22 insertions(+), 1 deletion(-)\n"
+ ">      >\302\240 \302\240 \302\240 >\n"
+ ">      >\302\240 \302\240 \302\240 > diff --git a/drivers/net/ethernet/intel/e1000e/ich8lan.c\n"
+ ">      >\302\240 \302\240 \302\240b/drivers/net/ethernet/intel/e1000e/ich8lan.c\n"
+ ">      >\302\240 \302\240 \302\240 > index 590ad110d383..3bff1b570b76 100644\n"
+ ">      >\302\240 \302\240 \302\240 > --- a/drivers/net/ethernet/intel/e1000e/ich8lan.c\n"
+ ">      >\302\240 \302\240 \302\240 > +++ b/drivers/net/ethernet/intel/e1000e/ich8lan.c\n"
+ ">      >\302\240 \302\240 \302\240 > @@ -986,6 +986,27 @@ static s32\n"
  ">     e1000_k1_workaround_lpt_lp(struct\n"
- ">      >? ? ?e1000_hw *hw, bool link)\n"
- ">      >? ? ? >? ? ? ?return ret_val;\n"
- ">      >? ? ? >? ?}\n"
- ">      >? ? ? >\n"
- ">      >? ? ? > +static u32 convert_e1000e_ltr_scale(u32 val)\n"
- ">      >? ? ? > +{\n"
- ">      >? ? ? > +? ? ?if (val > 5)\n"
- ">      >? ? ? > +? ? ? ? ? ? ?return 0;\n"
- ">      >? ? ? > +\n"
- ">      >? ? ? > +? ? ?return 1U << (5 * val);\n"
- ">      >? ? ? > +}\n"
- ">      >? ? ? > +\n"
- ">      >? ? ? > +static u64 decoded_ltr(u32 val)\n"
- ">      >? ? ? > +{\n"
- ">      >? ? ? > +? ? ?u64 decoded_latency;\n"
- ">      >? ? ? > +? ? ?u32 value;\n"
- ">      >? ? ? > +? ? ?u32 scale;\n"
- ">      >? ? ? > +\n"
- ">      >? ? ? > +? ? ?value = val & 0x03FF;\n"
- ">      >? ? ? > +? ? ?scale = (val & 0x1C00) >> 10;\n"
- ">      >? ? ? > +? ? ?decoded_latency = value *\n"
+ ">      >\302\240 \302\240 \302\240e1000_hw *hw, bool link)\n"
+ ">      >\302\240 \302\240 \302\240 >\302\240 \302\240 \302\240 \302\240return ret_val;\n"
+ ">      >\302\240 \302\240 \302\240 >\302\240 \302\240}\n"
+ ">      >\302\240 \302\240 \302\240 >\n"
+ ">      >\302\240 \302\240 \302\240 > +static u32 convert_e1000e_ltr_scale(u32 val)\n"
+ ">      >\302\240 \302\240 \302\240 > +{\n"
+ ">      >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240if (val > 5)\n"
+ ">      >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240return 0;\n"
+ ">      >\302\240 \302\240 \302\240 > +\n"
+ ">      >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240return 1U << (5 * val);\n"
+ ">      >\302\240 \302\240 \302\240 > +}\n"
+ ">      >\302\240 \302\240 \302\240 > +\n"
+ ">      >\302\240 \302\240 \302\240 > +static u64 decoded_ltr(u32 val)\n"
+ ">      >\302\240 \302\240 \302\240 > +{\n"
+ ">      >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240u64 decoded_latency;\n"
+ ">      >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240u32 value;\n"
+ ">      >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240u32 scale;\n"
+ ">      >\302\240 \302\240 \302\240 > +\n"
+ ">      >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240value = val & 0x03FF;\n"
+ ">      >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240scale = (val & 0x1C00) >> 10;\n"
+ ">      >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240decoded_latency = value *\n"
  ">     convert_e1000e_ltr_scale(scale);\n"
- ">      >? ? ? > +\n"
- ">      >? ? ? > +? ? ?return decoded_latency;\n"
- ">      >? ? ? > +}\n"
- ">      >? ? ? > +\n"
- ">      >? ? ? >? ?/**\n"
- ">      >? ? ? >? ? *? e1000_platform_pm_pch_lpt - Set platform power\n"
+ ">      >\302\240 \302\240 \302\240 > +\n"
+ ">      >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240return decoded_latency;\n"
+ ">      >\302\240 \302\240 \302\240 > +}\n"
+ ">      >\302\240 \302\240 \302\240 > +\n"
+ ">      >\302\240 \302\240 \302\240 >\302\240 \302\240/**\n"
+ ">      >\302\240 \302\240 \302\240 >\302\240 \302\240 *\302\240 e1000_platform_pm_pch_lpt - Set platform power\n"
  ">     management\n"
- ">      >? ? ?values\n"
- ">      >? ? ? >? ? *? @hw: pointer to the HW structure\n"
- ">      >? ? ? > @@ -1059,7 +1080,7 @@ static s32\n"
+ ">      >\302\240 \302\240 \302\240values\n"
+ ">      >\302\240 \302\240 \302\240 >\302\240 \302\240 *\302\240 @hw: pointer to the HW structure\n"
+ ">      >\302\240 \302\240 \302\240 > @@ -1059,7 +1080,7 @@ static s32\n"
  ">     e1000_platform_pm_pch_lpt(struct\n"
- ">      >? ? ?e1000_hw *hw, bool link)\n"
- ">      >? ? ? >? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? E1000_PCI_LTR_CAP_LPT + 2,\n"
- ">      >? ? ?&max_nosnoop);\n"
- ">      >? ? ? >? ? ? ? ? ? ? ?max_ltr_enc = max_t(u16, max_snoop,\n"
+ ">      >\302\240 \302\240 \302\240e1000_hw *hw, bool link)\n"
+ ">      >\302\240 \302\240 \302\240 >\302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 E1000_PCI_LTR_CAP_LPT + 2,\n"
+ ">      >\302\240 \302\240 \302\240&max_nosnoop);\n"
+ ">      >\302\240 \302\240 \302\240 >\302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240max_ltr_enc = max_t(u16, max_snoop,\n"
  ">     max_nosnoop);\n"
- ">      >? ? ? >\n"
- ">      >? ? ? > -? ? ? ? ? ? ?if (lat_enc > max_ltr_enc)\n"
- ">      >? ? ? > +? ? ? ? ? ? ?if (decoded_ltr(lat_enc) >\n"
+ ">      >\302\240 \302\240 \302\240 >\n"
+ ">      >\302\240 \302\240 \302\240 > -\302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240if (lat_enc > max_ltr_enc)\n"
+ ">      >\302\240 \302\240 \302\240 > +\302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240if (decoded_ltr(lat_enc) >\n"
  ">     decoded_ltr(max_ltr_enc))\n"
- ">      >? ? ? >? ? ? ? ? ? ? ? ? ? ? ?lat_enc = max_ltr_enc;\n"
- ">      >? ? ? >? ? ? ?}\n"
- ">      >? ? ? >\n"
- ">      >? ? ? >\n"
- ">      >? ? ?sasha\n"
+ ">      >\302\240 \302\240 \302\240 >\302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240lat_enc = max_ltr_enc;\n"
+ ">      >\302\240 \302\240 \302\240 >\302\240 \302\240 \302\240 \302\240}\n"
+ ">      >\302\240 \302\240 \302\240 >\n"
+ ">      >\302\240 \302\240 \302\240 >\n"
+ ">      >\302\240 \302\240 \302\240sasha\n"
  ">      >\n"
  "> \n"
  sasha
 
-e88c2d18f97db331b94f5353112722c2d1032e7a0477322dc88766346dee2d50
+8df0fe7f893162223c28d690a0c599c0d2801c87ac0a5a31472cb85750ccd773

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.