On Sun, Dec 15, 2024 at 11:28:13AM +0900, Damien Le Moal wrote:
> When compiling the rockchip endpoint driver with -W=1, gcc output the
> following warnings:
>
> drivers/pci/controller/pcie-rockchip-ep.c:59: warning: Function parameter or struct member 'perst_irq' not described in 'rockchip_pcie_ep'
> drivers/pci/controller/pcie-rockchip-ep.c:59: warning: Function parameter or struct member 'perst_asserted' not described in 'rockchip_pcie_ep'
> drivers/pci/controller/pcie-rockchip-ep.c:59: warning: Function parameter or struct member 'link_up' not described in 'rockchip_pcie_ep'
> drivers/pci/controller/pcie-rockchip-ep.c:59: warning: Function parameter or struct member 'link_training' not described in 'rockchip_pcie_ep'
>
> Avoid these warnings by adding the missing field descriptions in
> struct rockchip_pcie_ep kdoc comment.
>
> Reported-by: Bjorn Helgaas <helgaas@xxxxxxxxxx>
> Fixes: a7137cbf6bd5 ("PCI: rockchip-ep: Handle PERST# signal in EP mode")
> Fixes: bd6e61df4b2e ("PCI: rockchip-ep: Improve link training")
> Signed-off-by: Damien Le Moal <dlemoal@xxxxxxxxxx>
> ---
> drivers/pci/controller/pcie-rockchip-ep.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/drivers/pci/controller/pcie-rockchip-ep.c b/drivers/pci/controller/pcie-rockchip-ep.c
> index 1064b7b06cef..36b868530769 100644
> --- a/drivers/pci/controller/pcie-rockchip-ep.c
> +++ b/drivers/pci/controller/pcie-rockchip-ep.c
> @@ -40,6 +40,10 @@
> * @irq_pci_fn: the latest PCI function that has updated the mapping of
> * the MSI/INTX IRQ dedicated outbound region.
> * @irq_pending: bitmask of asserted INTX IRQs.
> + * @perst_irq: IRQ used for perst# signal.
s/perst#/PERST#
here and below
> + * @perst_asserted: perst# signal state (true if perst# was asserted).
> + * @link_up: PCI link state (true if the link is up).
This is not holding PCIe link state, but just indication of whether link is up
or not. So how about:
@link_up: True if link is up
- Mani
--
மணிவண்ணன் சதாசிவம்
