dwc_read_reg32 is used nowhere throughout the code. One of dwc_read32 and dwc_read_reg32 should be removed IMO. There was once only dwc_read_reg32. In version 5 of your patchset I believe. Why did you add another function? AFAIK it is not correct to store pointers in u32 because they need 8 bytes on 64-bit archs. So it was ok with the old dwc_read_reg32. [Marri] If u32 is 8bytes isn't pointer type would be 8bytes as well. I had change the API to avoid type castings to register addresses. _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
- Re: [PATCH V8 03/10] USB/ppc4xx: Add Synopsys DWC OTG Co... Alexander Gordeev
- RE: [PATCH V8 03/10] USB/ppc4xx: Add Synopsys DWC O... David Laight
- Re: [PATCH V8 03/10] USB/ppc4xx: Add Synopsys D... Alexander Gordeev
- RE: [PATCH V8 03/10] USB/ppc4xx: Add Synopsys D... Tirumala Marri
- RE: [PATCH V8 03/10] USB/ppc4xx: Add Synopsys DWC O... Tirumala Marri
- Re: [PATCH V8 03/10] USB/ppc4xx: Add Synopsys D... Alexander Gordeev
- Re: [PATCH V8 03/10] USB/ppc4xx: Add Synops... Greg KH
- RE: [PATCH V8 03/10] USB/ppc4xx: Add Sy... Tirumala Marri
- RE: [PATCH V8 03/10] USB/ppc4xx: A... David Laight
- Re: [PATCH V8 03/10] USB/ppc4x... Alexander Gordeev
- Re: [PATCH V8 03/10] USB/ppc4x... Greg KH
- RE: [PATCH V8 03/10] USB/ppc4xx: Add Synopsys D... David Laight