Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1577708

Re: [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm

From Sekhar Nori <nsekhar@ti.com>
Newsgroups linux.kernel
Subject Re: [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm
Date 2017-02-09 16:40 +0100
Message-ID <t8YZd-7Tl-47@gated-at.bofh.it> (permalink)
References <t8h7P-5ki-3@gated-at.bofh.it> <t8h7P-5ki-1@gated-at.bofh.it> <t8iGB-6nr-7@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Tuesday 07 February 2017 11:51 PM, Kevin Hilman wrote:
> Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
> 
>> When we enable vpif capture on the da850-evm we hit a BUG_ON() because
>> the i2c adapter can't be found. The board file boot uses i2c adapter 1
>> but in the DT mode it's actually adapter 0. Drop the problematic lines.
>>
>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>> ---
>>  arch/arm/mach-davinci/pdata-quirks.c | 4 ----
>>  1 file changed, 4 deletions(-)
>>
>> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
>> index 94948c1..09f62ac 100644
>> --- a/arch/arm/mach-davinci/pdata-quirks.c
>> +++ b/arch/arm/mach-davinci/pdata-quirks.c
>> @@ -116,10 +116,6 @@ static void __init da850_vpif_legacy_init(void)
>>  	if (of_machine_is_compatible("ti,da850-lcdk"))
>>  		da850_vpif_capture_config.subdev_count = 1;
>>  
>> -	/* EVM (UI card) uses i2c adapter 1 (not default: zero) */
>> -	if (of_machine_is_compatible("ti,da850-evm"))
>> -		da850_vpif_capture_config.i2c_adapter_id = 1;
>> -
> 
> oops, my bad.
> 
> Acked-by: Kevin Hilman <khilman@baylibre.com>

The offending code is not in my master branch. Since its almost certain
that VPIF platform support is going to wait for v4.12, can you or Kevin
please update Kevin's original patches with these fixes rolled in?

Thanks,
Sekhar

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2017-02-07 17:50 +0100
  Re: [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm Kevin Hilman <khilman@baylibre.com> - 2017-02-07 19:30 +0100
    Re: [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm Sekhar Nori <nsekhar@ti.com> - 2017-02-09 16:40 +0100
      Re: [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2017-02-09 17:50 +0100
      Re: [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm Kevin Hilman <khilman@baylibre.com> - 2017-02-13 19:40 +0100

csiph-web