diff for duplicates of <20161013090722.GI1743@imgtec.com> diff --git a/a/1.txt b/N1/1.txt index a4a659c..c225f16 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -2,7 +2,7 @@ On Thu, Oct 13, 2016 at 11:55:08AM +0300, Dan Carpenter wrote: > We want to read 3 bytes here, but because the parenthesis are in the > wrong place we instead read: > -> sizeof(intel_dp->edp_dpcd) = sizeof(intel_dp->edp_dpcd) +> sizeof(intel_dp->edp_dpcd) == sizeof(intel_dp->edp_dpcd) > > which is one byte. > @@ -24,9 +24,13 @@ Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com> > /* Read the eDP Display control capabilities registers */ > if ((intel_dp->dpcd[DP_EDP_CONFIGURATION_CAP] & DP_DPCD_DISPLAY_CONTROL_CAPABLE) && > drm_dp_dpcd_read(&intel_dp->aux, DP_EDP_DPCD_REV, -> - intel_dp->edp_dpcd, sizeof(intel_dp->edp_dpcd) = +> - intel_dp->edp_dpcd, sizeof(intel_dp->edp_dpcd) == > - sizeof(intel_dp->edp_dpcd))) -> + intel_dp->edp_dpcd, sizeof(intel_dp->edp_dpcd)) = +> + intel_dp->edp_dpcd, sizeof(intel_dp->edp_dpcd)) == > + sizeof(intel_dp->edp_dpcd)) > DRM_DEBUG_KMS("EDP DPCD : %*ph\n", (int) sizeof(intel_dp->edp_dpcd), > intel_dp->edp_dpcd); +_______________________________________________ +dri-devel mailing list +dri-devel@lists.freedesktop.org +https://lists.freedesktop.org/mailman/listinfo/dri-devel diff --git a/a/content_digest b/N1/content_digest index 9b127d8..46a299b 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,7 +1,7 @@ "ref\020161013085508.GJ16198@mwanda\0" "From\0Eric Engestrom <eric.engestrom@imgtec.com>\0" "Subject\0Re: [patch] drm/i915: fix a read size argument\0" - "Date\0Thu, 13 Oct 2016 09:07:22 +0000\0" + "Date\0Thu, 13 Oct 2016 10:07:22 +0100\0" "To\0Dan Carpenter <dan.carpenter@oracle.com>\0" "Cc\0intel-gfx@lists.freedesktop.org" kernel-janitors@vger.kernel.org @@ -14,7 +14,7 @@ "> We want to read 3 bytes here, but because the parenthesis are in the\n" "> wrong place we instead read:\n" "> \n" - "> \tsizeof(intel_dp->edp_dpcd) = sizeof(intel_dp->edp_dpcd)\n" + "> \tsizeof(intel_dp->edp_dpcd) == sizeof(intel_dp->edp_dpcd)\n" "> \n" "> which is one byte.\n" "> \n" @@ -36,11 +36,15 @@ "> \t/* Read the eDP Display control capabilities registers */\n" "> \tif ((intel_dp->dpcd[DP_EDP_CONFIGURATION_CAP] & DP_DPCD_DISPLAY_CONTROL_CAPABLE) &&\n" "> \t drm_dp_dpcd_read(&intel_dp->aux, DP_EDP_DPCD_REV,\n" - "> -\t\t\t intel_dp->edp_dpcd, sizeof(intel_dp->edp_dpcd) =\n" + "> -\t\t\t intel_dp->edp_dpcd, sizeof(intel_dp->edp_dpcd) ==\n" "> -\t\t\t sizeof(intel_dp->edp_dpcd)))\n" - "> +\t\t\t intel_dp->edp_dpcd, sizeof(intel_dp->edp_dpcd)) =\n" + "> +\t\t\t intel_dp->edp_dpcd, sizeof(intel_dp->edp_dpcd)) ==\n" "> +\t\t\t sizeof(intel_dp->edp_dpcd))\n" "> \t\tDRM_DEBUG_KMS(\"EDP DPCD : %*ph\\n\", (int) sizeof(intel_dp->edp_dpcd),\n" - "> \t\t\t intel_dp->edp_dpcd);" + "> \t\t\t intel_dp->edp_dpcd);\n" + "_______________________________________________\n" + "dri-devel mailing list\n" + "dri-devel@lists.freedesktop.org\n" + https://lists.freedesktop.org/mailman/listinfo/dri-devel -15f11326f0ded7c022110c9ca5e47da6de62c62f7aebe154b93b197bf2887244 +3c9634412eeb94aae8d370971a857ce24bfbe6b6cc161e12c60d64551c9095ab
diff --git a/a/1.txt b/N2/1.txt index a4a659c..8ede516 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -2,7 +2,7 @@ On Thu, Oct 13, 2016 at 11:55:08AM +0300, Dan Carpenter wrote: > We want to read 3 bytes here, but because the parenthesis are in the > wrong place we instead read: > -> sizeof(intel_dp->edp_dpcd) = sizeof(intel_dp->edp_dpcd) +> sizeof(intel_dp->edp_dpcd) == sizeof(intel_dp->edp_dpcd) > > which is one byte. > @@ -24,9 +24,9 @@ Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com> > /* Read the eDP Display control capabilities registers */ > if ((intel_dp->dpcd[DP_EDP_CONFIGURATION_CAP] & DP_DPCD_DISPLAY_CONTROL_CAPABLE) && > drm_dp_dpcd_read(&intel_dp->aux, DP_EDP_DPCD_REV, -> - intel_dp->edp_dpcd, sizeof(intel_dp->edp_dpcd) = +> - intel_dp->edp_dpcd, sizeof(intel_dp->edp_dpcd) == > - sizeof(intel_dp->edp_dpcd))) -> + intel_dp->edp_dpcd, sizeof(intel_dp->edp_dpcd)) = +> + intel_dp->edp_dpcd, sizeof(intel_dp->edp_dpcd)) == > + sizeof(intel_dp->edp_dpcd)) > DRM_DEBUG_KMS("EDP DPCD : %*ph\n", (int) sizeof(intel_dp->edp_dpcd), > intel_dp->edp_dpcd); diff --git a/a/content_digest b/N2/content_digest index 9b127d8..25a99f6 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,20 +1,21 @@ "ref\020161013085508.GJ16198@mwanda\0" "From\0Eric Engestrom <eric.engestrom@imgtec.com>\0" "Subject\0Re: [patch] drm/i915: fix a read size argument\0" - "Date\0Thu, 13 Oct 2016 09:07:22 +0000\0" + "Date\0Thu, 13 Oct 2016 10:07:22 +0100\0" "To\0Dan Carpenter <dan.carpenter@oracle.com>\0" - "Cc\0intel-gfx@lists.freedesktop.org" + "Cc\0Daniel Vetter <daniel.vetter@intel.com>" + " Ville Syrj\303\244l\303\244 <ville.syrjala@linux.intel.com>" + intel-gfx@lists.freedesktop.org kernel-janitors@vger.kernel.org linux-kernel@vger.kernel.org - dri-devel@lists.freedesktop.org - " Daniel Vetter <daniel.vetter@intel.com>\0" + " dri-devel@lists.freedesktop.org\0" "\00:1\0" "b\0" "On Thu, Oct 13, 2016 at 11:55:08AM +0300, Dan Carpenter wrote:\n" "> We want to read 3 bytes here, but because the parenthesis are in the\n" "> wrong place we instead read:\n" "> \n" - "> \tsizeof(intel_dp->edp_dpcd) = sizeof(intel_dp->edp_dpcd)\n" + "> \tsizeof(intel_dp->edp_dpcd) == sizeof(intel_dp->edp_dpcd)\n" "> \n" "> which is one byte.\n" "> \n" @@ -36,11 +37,11 @@ "> \t/* Read the eDP Display control capabilities registers */\n" "> \tif ((intel_dp->dpcd[DP_EDP_CONFIGURATION_CAP] & DP_DPCD_DISPLAY_CONTROL_CAPABLE) &&\n" "> \t drm_dp_dpcd_read(&intel_dp->aux, DP_EDP_DPCD_REV,\n" - "> -\t\t\t intel_dp->edp_dpcd, sizeof(intel_dp->edp_dpcd) =\n" + "> -\t\t\t intel_dp->edp_dpcd, sizeof(intel_dp->edp_dpcd) ==\n" "> -\t\t\t sizeof(intel_dp->edp_dpcd)))\n" - "> +\t\t\t intel_dp->edp_dpcd, sizeof(intel_dp->edp_dpcd)) =\n" + "> +\t\t\t intel_dp->edp_dpcd, sizeof(intel_dp->edp_dpcd)) ==\n" "> +\t\t\t sizeof(intel_dp->edp_dpcd))\n" "> \t\tDRM_DEBUG_KMS(\"EDP DPCD : %*ph\\n\", (int) sizeof(intel_dp->edp_dpcd),\n" "> \t\t\t intel_dp->edp_dpcd);" -15f11326f0ded7c022110c9ca5e47da6de62c62f7aebe154b93b197bf2887244 +5eb40f5ce120d280157595921f1817a142ffcf0bfc23ab15d8d26abd3295cb65
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.