On 15/09/2023 15:42, Stephan Gerhold wrote:
On Fri, Sep 15, 2023 at 08:03:04PM +0530, Gaurav Kohli wrote:Fix the apps iommu local address space range as per data sheet. Fixes: 6a6729f38436 ("arm64: dts: qcom: msm8916: Add IOMMU support") Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx> Tested-by: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx> Signed-off-by: Gaurav Kohli <quic_gkohli@xxxxxxxxxxx>Aside from the minor things Konrad mentioned (v1 -> v2) FWIW: Reviewed-by: Stephan Gerhold <stephan@xxxxxxxxxxx> Although I can kind of understand that starting to count at 0 often feels more intuitive, especially for Linux kernel things. :-)
Just put OPTION BASE 0 at the start of your cover letter ;) --- bod
