From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH 2/2] [media] ir-rx51: Fix build after multiarch changes broke it Date: Thu, 28 Apr 2016 13:55:57 -0700 Message-ID: <20160428205557.GH5995@atomide.com> References: <1461714709-10455-1-git-send-email-tony@atomide.com> <1461714709-10455-3-git-send-email-tony@atomide.com> <572266AF.9020601@gmail.com> <20160428202248.GG5995@atomide.com> <572275DB.8090300@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: <572275DB.8090300@gmail.com> Sender: linux-media-owner@vger.kernel.org To: Ivaylo Dimitrov Cc: linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Aaro Koskinen , Sebastian Reichel , Pavel Machel , Timo Kokkonen , Mauro Carvalho Chehab , Neil Armstrong , linux-media@vger.kernel.org List-Id: linux-omap@vger.kernel.org * Ivaylo Dimitrov [160428 13:44]: > =D0=A5=D0=B8, >=20 > On 28.04.2016 23:22, Tony Lindgren wrote: > >* Ivaylo Dimitrov [160428 12:39]: > >>On 27.04.2016 02:51, Tony Lindgren wrote: > >> > >>omap_dm_timer_request_specific always fails with message "Please us= e > >>omap_dm_timer_request_by_cap/node()" with DT boot. > >> > >>I hacked the code to use omap_dm_timer_request_by_cap(OMAP_TIMER_HA= S_PWM) > >>and it seems to use the correct timer (IR LED blinks, checked with = the > >>camera of Samsung S4 mini), but it doesn't actually control either = of the TV > >>sets here. The same SW(pierogi) controls them when device is booted= to stock > >>kernel. However, this seems another problem not related to the patc= h. > > > >OK thanks for testing, I'll apply the pdata patch then. > > > >I assume you'll post a separate fix for the request_by_cap > >driver change? > > >=20 > Well, it was a hack, it just happens that the first matched timer is = GPT9, I > think we should aim for a proper solution (request_by_node()). >=20 > Shall I prepare a patch that gets the timer from the DT? Sounds good to me! Tony