.../bindings/nvmem/mediatek,efuse.yaml | 17 +++++++++++++++-- arch/arm64/boot/dts/mediatek/mt8188.dtsi | 2 +- 2 files changed, 16 insertions(+), 3 deletions(-)
Hi everyone,
This is v2 of the MT8188 efuse GPU speed bin post-processing enablement
patches. In v1 [1] the change was made to the driver. Angelo, the platform
maintainer believes the change should be made to the DT binding instead
[2]. v2 adopts Angelo's argument.
Patch 1 updates the efuse DT binding so that MT8186 is a base compatible
with no fallback, and MT8188 falls back to MT8186.
Patch 2 updates the MT8188 DT to follow the new binding.
If possible I would like to see both patches merged through the soc
tree once the DT binding maintainers give an ack. This avoids prolonged
waiting for the binding changes to land and uncertainty about whether
things have fully landed or not.
Thanks
ChenYu
[1] https://lore.kernel.org/all/20241223100648.2166754-1-wenst@chromium.org/
[2] https://lore.kernel.org/all/11028242-afe4-474a-9d76-cd1bd9208987@collabora.com/
Chen-Yu Tsai (2):
dt-bindings: nvmem: mediatek: efuse: split MT8186/MT8188 from base
version
arm64: dts: mediatek: mt8188: Change efuse fallback compatible to
mt8186
.../bindings/nvmem/mediatek,efuse.yaml | 17 +++++++++++++++--
arch/arm64/boot/dts/mediatek/mt8188.dtsi | 2 +-
2 files changed, 16 insertions(+), 3 deletions(-)
--
2.50.0.rc0.604.gd4ff7b7c86-goog
Hi Angelo, On Tue, Jun 10, 2025 at 2:34 PM Chen-Yu Tsai <wenst@chromium.org> wrote: > > Hi everyone, > > This is v2 of the MT8188 efuse GPU speed bin post-processing enablement > patches. In v1 [1] the change was made to the driver. Angelo, the platform > maintainer believes the change should be made to the DT binding instead > [2]. v2 adopts Angelo's argument. > > Patch 1 updates the efuse DT binding so that MT8186 is a base compatible > with no fallback, and MT8188 falls back to MT8186. > > Patch 2 updates the MT8188 DT to follow the new binding. > > If possible I would like to see both patches merged through the soc > tree once the DT binding maintainers give an ack. This avoids prolonged > waiting for the binding changes to land and uncertainty about whether > things have fully landed or not. > > > Thanks > ChenYu > > [1] https://lore.kernel.org/all/20241223100648.2166754-1-wenst@chromium.org/ > [2] https://lore.kernel.org/all/11028242-afe4-474a-9d76-cd1bd9208987@collabora.com/ > > Chen-Yu Tsai (2): > dt-bindings: nvmem: mediatek: efuse: split MT8186/MT8188 from base > version > arm64: dts: mediatek: mt8188: Change efuse fallback compatible to > mt8186 Friendly ping. Please take a look and see if this scheme is to your liking. Thanks ChenYu > .../bindings/nvmem/mediatek,efuse.yaml | 17 +++++++++++++++-- > arch/arm64/boot/dts/mediatek/mt8188.dtsi | 2 +- > 2 files changed, 16 insertions(+), 3 deletions(-) > > -- > 2.50.0.rc0.604.gd4ff7b7c86-goog >
Il 01/07/25 11:15, Chen-Yu Tsai ha scritto: > Hi Angelo, > > On Tue, Jun 10, 2025 at 2:34 PM Chen-Yu Tsai <wenst@chromium.org> wrote: >> >> Hi everyone, >> >> This is v2 of the MT8188 efuse GPU speed bin post-processing enablement >> patches. In v1 [1] the change was made to the driver. Angelo, the platform >> maintainer believes the change should be made to the DT binding instead >> [2]. v2 adopts Angelo's argument. >> >> Patch 1 updates the efuse DT binding so that MT8186 is a base compatible >> with no fallback, and MT8188 falls back to MT8186. >> >> Patch 2 updates the MT8188 DT to follow the new binding. >> >> If possible I would like to see both patches merged through the soc >> tree once the DT binding maintainers give an ack. This avoids prolonged >> waiting for the binding changes to land and uncertainty about whether >> things have fully landed or not. >> >> >> Thanks >> ChenYu >> >> [1] https://lore.kernel.org/all/20241223100648.2166754-1-wenst@chromium.org/ >> [2] https://lore.kernel.org/all/11028242-afe4-474a-9d76-cd1bd9208987@collabora.com/ >> >> Chen-Yu Tsai (2): >> dt-bindings: nvmem: mediatek: efuse: split MT8186/MT8188 from base >> version >> arm64: dts: mediatek: mt8188: Change efuse fallback compatible to >> mt8186 > > Friendly ping. Please take a look and see if this scheme is to your liking. > For the whole series Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> > > Thanks > ChenYu > >> .../bindings/nvmem/mediatek,efuse.yaml | 17 +++++++++++++++-- >> arch/arm64/boot/dts/mediatek/mt8188.dtsi | 2 +- >> 2 files changed, 16 insertions(+), 3 deletions(-) >> >> -- >> 2.50.0.rc0.604.gd4ff7b7c86-goog >>
Srinivas, On Thu, Jul 3, 2025 at 7:48 PM AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> wrote: > > Il 01/07/25 11:15, Chen-Yu Tsai ha scritto: > > Hi Angelo, > > > > On Tue, Jun 10, 2025 at 2:34 PM Chen-Yu Tsai <wenst@chromium.org> wrote: > >> > >> Hi everyone, > >> > >> This is v2 of the MT8188 efuse GPU speed bin post-processing enablement > >> patches. In v1 [1] the change was made to the driver. Angelo, the platform > >> maintainer believes the change should be made to the DT binding instead > >> [2]. v2 adopts Angelo's argument. > >> > >> Patch 1 updates the efuse DT binding so that MT8186 is a base compatible > >> with no fallback, and MT8188 falls back to MT8186. > >> > >> Patch 2 updates the MT8188 DT to follow the new binding. > >> > >> If possible I would like to see both patches merged through the soc > >> tree once the DT binding maintainers give an ack. This avoids prolonged > >> waiting for the binding changes to land and uncertainty about whether > >> things have fully landed or not. > >> > >> > >> Thanks > >> ChenYu > >> > >> [1] https://lore.kernel.org/all/20241223100648.2166754-1-wenst@chromium.org/ > >> [2] https://lore.kernel.org/all/11028242-afe4-474a-9d76-cd1bd9208987@collabora.com/ > >> > >> Chen-Yu Tsai (2): > >> dt-bindings: nvmem: mediatek: efuse: split MT8186/MT8188 from base > >> version > >> arm64: dts: mediatek: mt8188: Change efuse fallback compatible to > >> mt8186 > > > > Friendly ping. Please take a look and see if this scheme is to your liking. > > > > For the whole series > > Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> Could you merge the DT binding patch so that we can merge the DT change ahead of -rc6? Or alternatively could you give an Ack so both patches go through the soc tree? Thanks ChenYu
On Tue, 10 Jun 2025 14:34:28 +0800, Chen-Yu Tsai wrote:
> This is v2 of the MT8188 efuse GPU speed bin post-processing enablement
> patches. In v1 [1] the change was made to the driver. Angelo, the platform
> maintainer believes the change should be made to the DT binding instead
> [2]. v2 adopts Angelo's argument.
>
> Patch 1 updates the efuse DT binding so that MT8186 is a base compatible
> with no fallback, and MT8188 falls back to MT8186.
>
> [...]
Applied, thanks!
[1/2] dt-bindings: nvmem: mediatek: efuse: split MT8186/MT8188 from base version
commit: ce32f3cdfbc31dab626e20122e577f75ea46b940
Best regards,
--
Srinivas Kandagatla <srini@kernel.org>
© 2016 - 2025 Red Hat, Inc.