From patchwork Wed Nov 27 12:29:44 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vladimir Zapolskiy X-Patchwork-Id: 845770 Received: from mail-lj1-f194.google.com (mail-lj1-f194.google.com [209.85.208.194]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B68A31FBC88 for ; Wed, 27 Nov 2024 12:29:56 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.194 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1732710598; cv=none; b=fyPotB8yRyFKw0JZq+CCXIxuTWwqoOACb3fd+Bovgw3RgTHpxFGslkVktln0NYyINxH03L1/Y9bszHqWBSoPhjx/ZErSR9DDEdCg0FPxmqRUInmttQK75ZwCn78XMRTV/7kRcktPT+ePnGYLjxnpV5ADqnIUrnuub+WDvn0OpYc= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1732710598; c=relaxed/simple; bh=eO8pZeLmCY07ZvONQ95F7bnDt5LzCJtb1w9NSPUfJh8=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=R2tW3yk8dE8jugMxvhfH1/8LHjSYvt8yoichpLbSv/bFlEWGDCYk3Nw2G/ukZxjvFAoIrsUbfajzEAOIxNI1bDF4l8XlNaeyhOjQSGRWCCk4r/T2xV3rWpGsTwa/LOZ2v3ZtbgOhP3Xjrxy2PJUnjtJfcSi4nK+Ne5xQBIYVUDA= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=GkdoMgmI; arc=none smtp.client-ip=209.85.208.194 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="GkdoMgmI" Received: by mail-lj1-f194.google.com with SMTP id 38308e7fff4ca-2ffd2026437so994001fa.1 for ; Wed, 27 Nov 2024 04:29:56 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1732710595; x=1733315395; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=KRcdSs7XMlGnAilpDhcNiYT3hFI3KAq1n7SCl+kMm14=; b=GkdoMgmIwaaGmz5I/jMOZc2DEuvqJYiAiw6yeY7DixADaY0Y+L7rIka7ebJktp3K2B tISsYEh1ACr+1dbu16T+snXfgu6HtL3qZw2uzql3dIiYwZFKcl6HK+/tam3oDPdByjvt 8kaRjngBkX9KEZgWV32oa3NYJrChUulq4hTHx9Gdmkrvy4L5feTi8f1/mn46aubCrPVr JHSm6v6ElKctF+qPY3QMQOUMvTvtGmJ4DlM7AAcVGo2sD7OL9IHMgz2513bOu42IaCa/ Otsvp8qxCwi4+nohGF7rjBdyAeHhoyyV6oqwQsgueOcGYMFgOhcrZcttGdKbuD+gLEWb rDOQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1732710595; x=1733315395; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=KRcdSs7XMlGnAilpDhcNiYT3hFI3KAq1n7SCl+kMm14=; b=BTWwWJeUgZpy8LCPtqjatYvpINDQCa9b0CmtXz9tQqkbY/YBfoJrEF5LjarmiR8z5P Kp0ofyWZvjrEYE0fx3OKE5/RAdG3Rmq4nw+YlxdYECU64Qpy+XfbBdD7R9VyaDIl1FtO JTBt4RmEWKS72Hs4hN5qsnKwTO2dgjbsH5ogZgXtXS/SPsSA+wLJ+QW4SWbeqpHWTpmN 0Z9nfej+D0JXKMOooFm+SBVuHPQZGP1QAbIPhRbJIirVrcK3e6OhzAhruoZcSx6bUn+m I0kF71kt+1wKUexKY6mwfnAsRt35MoahI7h5v8geP5UPcbQ7tHQaVyC84Zre/c/3ptj+ 5JMw== X-Forwarded-Encrypted: i=1; AJvYcCWG8A/7lThFxW+8g8osz4hkzJo6IwiRMWfFB9n8sqcc2/xUVYe82uFgSGVuerDKaN+QqIYAix1aoE/5VKGd@vger.kernel.org X-Gm-Message-State: AOJu0Yw+CL7R+l+YZdquBWV1CqR5DuvRi0Jdci/c7GgJhZ1FbX8xtzGF nGPCMWOdEXJy+lO4vbGiaya8q9dxS/gA8+Gr1HF8zbwdLxTcUEm7i/KpeSzXMKpnuJbVqNx7qSs cfbVmeg== X-Gm-Gg: ASbGncuL2pf7SHbXh4JHhA1Tke4AAptHhUGoBg9osOSEPtCpnkP0z0K5ZbhYCLn7TL4 wQLqVVfqP5rHN1JeFqaDprO86/logDGeafSHdyRJZYo5KLMhfOCh3TRYDNx4xju3+h68WpRQ5Mr XMygvosfKUOkAnMYE7GR9EYpN/PPlPAIpmJD4f/ji1nwugthcVnwHvBETbAv4yUBRLQGaWNkJkT 4qicLraLNWZLSRzMETjiE9XhDaQScuC4a+ngRcBzcMgqMbgkUVkPW3CchHVcU/t1BhksWiE+9AP Y10/HaBqgdXhhjx3wApy0ICHxMu+s770pmXXILc= X-Google-Smtp-Source: AGHT+IEmhY/XevE73lIO5R+T76MBj7DXNOF8lKpN8Jyne4r1vkv6Io8XVDa+jBnr77B+i8BS8pEL7A== X-Received: by 2002:a05:6512:3e23:b0:53d:ddc6:259e with SMTP id 2adb3069b0e04-53df0108cbcmr392654e87.11.1732710594838; Wed, 27 Nov 2024 04:29:54 -0800 (PST) Received: from localhost.localdomain (88-112-131-206.elisa-laajakaista.fi. [88.112.131.206]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-53dd24953a1sm2276621e87.228.2024.11.27.04.29.54 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 27 Nov 2024 04:29:54 -0800 (PST) From: Vladimir Zapolskiy To: Bjorn Andersson , Konrad Dybcio Cc: Robert Foss , Todor Tomov , Bryan O'Donoghue , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Mauro Carvalho Chehab , linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org Subject: [PATCH v3 0/6] dt-bindings: media: camss: Fix interrupt types Date: Wed, 27 Nov 2024 14:29:44 +0200 Message-ID: <20241127122950.885982-1-vladimir.zapolskiy@linaro.org> X-Mailer: git-send-email 2.45.2 Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 It was discovered that on a few Qualcomm platforms types of interrupts do not match the type given by the Qualcomm IP catalog, the type in downstream code and the type requested by the CAMSS driver itself. The mismatched interrupt type between firmware and the correspondent CAMSS driver leads to known problems, similar to the ones which were discussed previously: https://lore.kernel.org/lkml/20220530080842.37024-4-manivannan.sadhasivam@linaro.org/ Here the situation is essentially the same, namely a repeated bind of the CAMSS device is not possible due to a wrongly specified interrupt type, and it may lead to an issue in runtime manifested like this: irq: type mismatch, failed to map hwirq-509 for interrupt-controller@17a00000! Changes from v2 to v3: * gave clear commit messages addressing Qualcomm IP, thanks to Depeng Shao, * hence keep Krzysztof Acked-by tag, since the only expressed concern was a missing reference to the hardware specs, * added Reviewed-by tags from Bryan, Link to v2 of the changeset: https://lore.kernel.org/all/20240923072827.3772504-1-vladimir.zapolskiy@linaro.org/ Changes from v1 to v2: * added gained Acked-by, Tested-by and Reviewed-by tags, * per patch review requests from Krzysztof deduplicated "media:" from subjects. Link to v1 of the changeset: https://lore.kernel.org/all/20240905164142.3475873-1-vladimir.zapolskiy@linaro.org/ Vladimir Zapolskiy (6): dt-bindings: media: qcom,sc8280xp-camss: Fix interrupt types dt-bindings: media: qcom,sdm845-camss: Fix interrupt types dt-bindings: media: qcom,sm8250-camss: Fix interrupt types arm64: dts: qcom: sc8280xp: Fix interrupt type of camss interrupts arm64: dts: qcom: sdm845: Fix interrupt types of camss interrupts arm64: dts: qcom: sm8250: Fix interrupt types of camss interrupts .../bindings/media/qcom,sc8280xp-camss.yaml | 40 +++++++++---------- .../bindings/media/qcom,sdm845-camss.yaml | 20 +++++----- .../bindings/media/qcom,sm8250-camss.yaml | 28 ++++++------- arch/arm64/boot/dts/qcom/sc8280xp.dtsi | 40 +++++++++---------- arch/arm64/boot/dts/qcom/sdm845.dtsi | 20 +++++----- arch/arm64/boot/dts/qcom/sm8250.dtsi | 28 ++++++------- 6 files changed, 88 insertions(+), 88 deletions(-)