RE: [PATCH v4 8/9] memory: renesas-rpc-if: Add RZ/G3E xSPI support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Krzysztof,

> -----Original Message-----
> From: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
> Sent: 15 April 2025 10:40
> Subject: Re: [PATCH v4 8/9] memory: renesas-rpc-if: Add RZ/G3E xSPI support
> 
> 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.

OK. Then I will update the patch for new code.

Later will add for the rest of the functions.

Cheers,
Biju
> 
> > I will add a separate patch for fixing that as well.
> 
> 
> Best regards,
> Krzysztof




[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux