On Mon, 28 Jun 2010, Kulikov Vasiliy wrote: > Change sizeof(x) / sizeof(*x) to ARRAY_SIZE(x). > > Signed-off-by: Kulikov Vasiliy <segooon@xxxxxxxxx> > --- > drivers/gpu/drm/i915/intel_sdvo.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_sdvo.c b/drivers/gpu/drm/i915/intel_sdvo.c > index 76993ac..03c231b 100644 > --- a/drivers/gpu/drm/i915/intel_sdvo.c > +++ b/drivers/gpu/drm/i915/intel_sdvo.c > @@ -392,13 +392,13 @@ static void intel_sdvo_debug_write(struct intel_encoder *intel_encoder, u8 cmd, > DRM_LOG_KMS("%02X ", ((u8 *)args)[i]); > for (; i < 8; i++) > DRM_LOG_KMS(" "); > - for (i = 0; i < sizeof(sdvo_cmd_names) / sizeof(sdvo_cmd_names[0]); i++) { > + for (i = 0; i < ARRAY_SIZE(sdvo_cmd_names); i++) { > if (cmd == sdvo_cmd_names[i].cmd) { > DRM_LOG_KMS("(%s)", sdvo_cmd_names[i].name); > break; > } > } > - if (i == sizeof(sdvo_cmd_names)/ sizeof(sdvo_cmd_names[0])) > + if (i == ARRAY_SIZE(sdvo_cmd_names)) > DRM_LOG_KMS("(%02X)", cmd); > DRM_LOG_KMS("\n"); > } Applied to trivial queue, as linux-next as of today doesn't carry this patch. Thanks, -- Jiri Kosina SUSE Labs, Novell Inc. -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html