From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Tue, 25 Sep 2018 10:20:24 -0500 From: Rob Herring Subject: Re: [PATCH v3 1/9] partial revert of "[media] tvp5150: add HW input connectors support" Message-ID: <20180925152024.GA18900@bogus> References: <20180918131453.21031-1-m.felsch@pengutronix.de> <20180918131453.21031-2-m.felsch@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180918131453.21031-2-m.felsch@pengutronix.de> To: Marco Felsch Cc: mchehab@kernel.org, mark.rutland@arm.com, kernel@pengutronix.de, devicetree@vger.kernel.org, p.zabel@pengutronix.de, javierm@redhat.com, laurent.pinchart@ideasonboard.com, sakari.ailus@linux.intel.com, afshin.nasser@gmail.com, linux-media@vger.kernel.org List-ID: On Tue, Sep 18, 2018 at 03:14:45PM +0200, Marco Felsch wrote: > From: Javier Martinez Canillas > > Commit f7b4b54e6364 ("[media] tvp5150: add HW input connectors support") > added input signals support for the tvp5150, but the approach was found > to be incorrect so the corresponding DT binding commit 82c2ffeb217a > ("[media] tvp5150: document input connectors DT bindings") was reverted. > > This left the driver with an undocumented (and wrong) DT parsing logic, > so lets get rid of this code as well until the input connectors support > is implemented properly. > > It's a partial revert due other patches added on top of mentioned commit > not allowing the commit to be reverted cleanly anymore. But all the code > related to the DT parsing logic and input entities creation are removed. > > Suggested-by: Laurent Pinchart > Signed-off-by: Javier Martinez Canillas > Acked-by: Laurent Pinchart > [m.felsch@pengutronix.de: rm TVP5150_INPUT_NUM define] > Signed-off-by: Marco Felsch > Signed-off-by: Mauro Carvalho Chehab > --- > drivers/media/i2c/tvp5150.c | 141 ---------------------------- > include/dt-bindings/media/tvp5150.h | 2 - > 2 files changed, 143 deletions(-) Acked-by: Rob Herring