On 15/04/2025 10:18, Biju Das wrote: > Hi Krzysztof Kozlowski, > > Thanks for the feedback. > >> -----Original Message----- >> From: Krzysztof Kozlowski <krzk@xxxxxxxxxx> >> Sent: 15 April 2025 07:17 >> Subject: Re: [PATCH v4 8/9] memory: renesas-rpc-if: Add RZ/G3E xSPI support >> >> On 01/04/2025 16:35, Biju Das wrote: >>> int rpcif_manual_xfer(struct device *dev) { >>> struct rpcif_priv *rpc = dev_get_drvdata(dev); @@ -493,7 +769,7 @@ >>> int rpcif_manual_xfer(struct device *dev) >>> if (ret) >>> return ret; >>> >>> - ret = rpcif_manual_xfer_impl(rpc); >>> + ret = rpc->info->impl->manual_xfer(rpc); >>> >>> pm_runtime_put(dev); >>> >>> @@ -543,6 +819,61 @@ static void memcpy_fromio_readw(void *to, >>> } >>> } >>> >> >> Missing kerneldoc. Exported functions shoud have kerneldoc. > > Currently there is no exported function in this driver have kernel doc. I know. My comment was about new code. > I will add a separate patch for fixing that as well. Best regards, Krzysztof