Re: [PATCH v6 7/8] media: i2c: add DS90UB913 driver

Tomi Valkeinen
2023-01-09T14:03:52Z

On 09/01/2023 15:58, Andy Shevchenko wrote:
> On Mon, Jan 09, 2023 at 03:55:39PM +0200, Andy Shevchenko wrote:
>> On Mon, Jan 09, 2023 at 02:59:01PM +0200, Tomi Valkeinen wrote:
>>> On 09/01/2023 13:07, Andy Shevchenko wrote:
>>>> On Sun, Jan 08, 2023 at 06:06:34AM +0200, Laurent Pinchart wrote:
>>>>> On Thu, Jan 05, 2023 at 04:03:06PM +0200, Tomi Valkeinen wrote:
>
> ...
>
>>>>>> + gc->of_node = priv->client->dev.of_node;
>>>>
>>>> We don't have of_node anymore in gc. And if the parent device is set, you can
>>>> drop this line (it will work with older and newer kernels. Otherwise, use
>>>> fwnode.
>>>
>>> What do you mean "we don't have of_node anymore"?
>
> Just to add that you missed "gc" in the quote which is crucial to get
> what I meant.

Well, the original text was just above...

>> There is no of_node member of struct gpio_chip anymore. This will fail
>> to compile.

It's in Linus' tree, but yes, now that I fetched linux-next I see it's gone.

Tomi