From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans Verkuil Subject: Re: [PATCH v6 06/11] cec: add HDMI CEC framework Date: Wed, 13 May 2015 10:28:07 +0200 Message-ID: <55530B17.4050606@xs4all.nl> References: <1430760785-1169-1-git-send-email-k.debski@samsung.com> <1430760785-1169-7-git-send-email-k.debski@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1430760785-1169-7-git-send-email-k.debski@samsung.com> Sender: linux-input-owner@vger.kernel.org To: Kamil Debski , dri-devel@lists.freedesktop.org, linux-media@vger.kernel.org Cc: m.szyprowski@samsung.com, mchehab@osg.samsung.com, kyungmin.park@samsung.com, thomas@tommie-lie.de, sean@mess.org, dmitry.torokhov@gmail.com, linux-input@vger.kernel.org, linux-samsung-soc@vger.kernel.org, lars@opdenkamp.eu, Hans Verkuil List-Id: linux-samsung-soc@vger.kernel.org Hi Kamil, Here is the first cec-compliance bug report: CEC_G_CAPS doesn't zero the reserved field! cec.c needs a memset there. I think this is missing in cec.c for all structs with a reserved field in them. Only G_EVENT looks to be OK. Regards, Hans