[U-Boot] [PATCH 1/2 V4] misc: add "call" uclass op

Stephen Warren swarren at wwwdotorg.org
Fri Aug 12 00:17:33 CEST 2016


On 08/08/2016 02:33 PM, Tom Warren wrote:
> Simon,
>
>> -----Original Message-----
>> From: Stephen Warren [mailto:swarren at wwwdotorg.org]
>> Sent: Monday, August 08, 2016 12:54 PM
>> To: Simon Glass <sjg at chromium.org>
>> Cc: U-Boot Mailing List <u-boot at lists.denx.de>; Tom Warren
>> <TWarren at nvidia.com>; Stephen Warren <swarren at nvidia.com>; Tom Rini
>> <trini at konsulko.com>
>> Subject: Re: [PATCH 1/2 V4] misc: add "call" uclass op
>>
>> On 08/08/2016 01:47 PM, Simon Glass wrote:
>>> +Tom
>>>
>>> Hi Stephen,
>>>
>>> On 8 August 2016 at 10:41, Stephen Warren <swarren at wwwdotorg.org>
>> wrote:
>>>> On 08/08/2016 10:38 AM, Simon Glass wrote:
>>>>>
>>>>> Hi Stephen,
>>>>>
>>>>> On 8 August 2016 at 09:41, Stephen Warren <swarren at wwwdotorg.org>
>> wrote:
>>>>>>
>>>>>>
>>>>>> From: Stephen Warren <swarren at nvidia.com>
>>>>>>
>>>>>> The call op requests that the callee pass a message to the
>>>>>> underlying HW or device, wait for a response, and then pass back
>>>>>> the response error code and message to the callee. It is useful for
>>>>>> drivers that represent some kind of messaging or IPC channel to a
>>>>>> remote device.
>>>>>>
>>>>>> Signed-off-by: Stephen Warren <swarren at nvidia.com>
>>>>>> Acked-by: Simon Glass <sjg at chromium.org>
>>>>>> ---
>>>>>> v4: Adjust misc_call() to return the response msg size on success.
>>>>>> v3: New patch.
>>>>>> ---
>>>>>>  drivers/misc/misc-uclass.c | 11 +++++++++++
>>>>>>  include/misc.h             | 35 +++++++++++++++++++++++++++++++++++
>>>>>>  2 files changed, 46 insertions(+)
>>>>>
>>>>>
>>>>> Are you planning for the Tegra186 stuff to go into the upcoming release?
>>>>
>>>>
>>>> I'd like it to if at all possible; it's all pretty much
>>>> Tegra-specific drivers so shouldn't cause any fallout for other
>>>> platforms. The only exception is the fdt_translate_address() changes,
>>>> which are a dependency, which will be built into other platforms, but
>>>> there's a simple Boolean parameter that controls the new behaviour, so it
>> should be pretty safe.
>>>
>>> OK, sounds reasonable to me. I'll pick these up later in the week and
>>> send a pull request. Can you bring any other dependencies in via
>>> Tegra?
>>
>> Yes, I was assuming that TomW would take everything else; it's all Tegra
>> drivers.
 >
> Stephen and I are working together to get it all packaged up and in to u-boot-tegra ASAP.

Simon, any word on the u-boot-dm pull request? TomW is waiting for it 
before applying all the Tegra commits on top of it so he won't have to 
rebase, and he'd probably best send a Tegra pull request on Friday in 
order to make -rc2, which would be good to do. Thanks.


More information about the U-Boot mailing list