Messages in this thread | | | Subject | Re: [PATCH v12 4/5] clocksource/drivers: Add a goldfish-timer clocksource | From | Daniel Lezcano <> | Date | Wed, 26 Jan 2022 18:25:48 +0100 |
| |
On 26/01/2022 18:14, Laurent Vivier wrote: > Le 26/01/2022 à 12:55, Daniel Lezcano a écrit : >> >> Hi Laurent, >> > > Hi Daniel, > >> On 21/01/2022 21:07, Laurent Vivier wrote: >>> Add a clocksource based on the goldfish-rtc device. >> >> As a first submission, please provide a more detailed description of the >> timer. > > ok. > > The timer uses in fact the goldifsh RTC clock so I supposed refering to > drivers/rtc/rtc-goldfish.c could give the missing pieces of information
[ ... ]
> I agree for irq, but res is passed to request_resource() and I think it > is added to a linked tree, so I'm not sure it can be local to the function. > > Do you mean a "static" declaration in the function?
I missed request_resource() was adding the resource in a list, thus the parameter must not be freed. So discard the comment for 'res'.
[ ... ]
-- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog
| |